From: Johannes Schindelin <Johannes.Schindelin@gmx.de>
To: Philip Oakley <philipoakley@iee.org>
Cc: Git List <git@vger.kernel.org>
Subject: Re: avoid duplicate patches from git log ?
Date: Wed, 4 May 2016 13:44:17 +0200 (CEST) [thread overview]
Message-ID: <alpine.DEB.2.20.1605041329040.9313@virtualbox> (raw)
In-Reply-To: <712E44CFAD1A41A982AEF1540C1F9F80@PhilipOakley>
Hi Philip,
On Tue, 3 May 2016, Philip Oakley wrote:
> I was trying to search the Git for Windows (G4W) history for commits that
> touched MSVC.
>
> I've used 'git log -SMSVC --pretty='tformat:%h (%s, %ad)' --date=short
> --reverse' to get a nice list of those commits.
>
> However, as the G4W project (https://github.com/git-for-windows/git/)
> follows the main git repo and its releases, it needs to rebase it's
> fixup patches, while retaining their original series, so has repeated
> copies of those fix patches on the second parent path (a technique Dscho
> called rebasing merges).
Actually, I no longer use rebasing merges, but instead merging rebases.
The difference is a little subtle:
Rebasing merge:
- upstream ----- rebased-A - rebased-B - rebasing-merge (-s ours)
/
- old-A - old-B -----------------------
Merging rebase:
- upstream ----- merging-rebase (-s ours) - rebased-A - rebased-B
/
- old-A - old-B
Of course both diagrams are drastically simplified, as I do not only
rebase mere patches, but topic branches, including merge structure
(currently 44 IIRC), to make it easier to break out the topic branches for
easy submission later.
It turned out that the rebasing-merge strategy actually made things
harder, as it was not quite as easy to figure out *which* commits needed
rebasing (remember, new commits were added after rebasing-merges).
With merging-rebases, at least, one knows that the tree of the merge
commit starting the whole shebang is identical to upstream's tree. All of
the patches that come on top of that merge commit need to be rebased the
next time round, including new patches.
Fittingly, the commit message of said merge commit begins with the
following text: "Start the merging-rebase ..."
BTW The reason for this rather unwieldy setup is that we historically had
a hard time getting our patches into upstream Git (there was some degree
of resistance in the past, and also a quite limiting lack of time on my
part).
> for example:
> > bf1a7ff (MinGW: disable CRT command line globbing, 2011-01-07)
> > a05e9a8 (MinGW: disable CRT command line globbing, 2011-01-07)
> > 45cfa35 (MinGW: disable CRT command line globbing, 2011-01-07)
> > 1d35390 (MinGW: disable CRT command line globbing, 2011-01-07)
> > 022e029 (MinGW: disable CRT command line globbing, 2011-01-07)
>
>
> How can I filter out all the duplicate patches which are identical other
> than the commit date?
I would go about it in a completely different manner. Remember that the
merging rebase starts with a merge that integrates the previous history,
but also resets the tree to upstream's. Therefore all the commits merged
at the start of the merging-rebase are the ones in which you are *not*
interested.
In other words, this command-line will yield the output you desire:
git log -SMSVC --pretty='tformat:%h (%s, %ad)' --date=short \
HEAD^{/Start.the.merging-rebase}^2..
Ciao,
Dscho
prev parent reply other threads:[~2016-05-04 11:44 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-05-03 20:11 avoid duplicate patches from git log ? Philip Oakley
2016-05-03 22:00 ` Jeff King
2016-05-03 22:12 ` Junio C Hamano
2016-05-03 22:36 ` Philip Oakley
2016-05-04 11:58 ` Johannes Schindelin
2016-05-04 19:15 ` Junio C Hamano
2016-05-04 11:44 ` Johannes Schindelin [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.DEB.2.20.1605041329040.9313@virtualbox \
--to=johannes.schindelin@gmx.de \
--cc=git@vger.kernel.org \
--cc=philipoakley@iee.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).