Hi Junio, On Fri, 6 Jul 2018, Junio C Hamano wrote: > "Johannes Schindelin via GitGitGadget" <gitgitgadget@xxxxxxxxx> > writes: > > > From: Johannes Schindelin <johannes.schindelin@xxxxxx> > > > > The problem solved by the code introduced in this commit goes like this: > > given two sets of items, and a cost matrix which says how much it > > "costs" to assign any given item of the first set to any given item of > > the second, assign all items (except when the sets have different size) > > in the cheapest way. > > > > We use the Jonker-Volgenant algorithm to solve the assignment problem to > > answer questions such as: given two different versions of a topic branch > > (or iterations of a patch series), what is the best pairing of > > commits/patches between the different versions? > > > > Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx> > > --- > > Does the "gitgitgadget" thing lie on the Date: e-mail header? No, GitGitGadget takes the literal output from `git format-patch`, as far as I can tell. So if at all, it is `format-patch` that is lying. You can compare the mail's date to the commit date: https://public-inbox.org/git/39272eefcfe66de3ca1aa2ee43d6626ce558caae.1530617166.git.gitgitgadget@xxxxxxxxx/ https://github.com/dscho/git/commit/39272eefcfe66de3ca1aa2ee43d6626ce558caae (the nice thing about GitGitGadget is that you can rely on its mails to reflect *precisely* what the commit is like, the user does not have any opportunity to interfere with the code that generates the mails: https://github.com/gitgitgadget/gitgitgadget/blob/c4805370f/lib/patch-series.ts#L605-L611). > Postdating the patch with in-body header is fine, but mailbox tools > often use and trust the Date: timestamp when sorting and finding > messages etc. so sending a new patch to add linear-assignment.c that > is different from what was added 9 weeks ago with "Date: Mon, 30 Apr > 2018" header can easily cause me to miss that message when I look > for things that happened within the past few weeks, for example. Well, isn't it too bad that we use emails to transport commits, then. Seriously, I have very little sympathy here, as all I am doing is to automate *the suggested usage* of `git format-patch` and `git send-email` (the latter of which I cannot even use due to its limitations). So if you want to see this "fixed", you should think how you want to see `git format-patch` fixed. Or maybe you want to write a script that re-orders the patches on top of the cover letter according to the `[PATCH M/N]` order, to reinstate the order of the original commits that got somewhat lost via emailing them. Of course, you could also save yourself a lot of trouble and use Git: git fetch https://github.com/gitgitgadget/git \ pr-1/dscho/branch-diff-v3 git cherry-pick -s ..FETCH_HEAD (This is assuming that you insist, as you did in the past, on changing the base commit from what the original author chose. If you are fine with my choice, which is the current `master`, then you could save yourself *even more* trouble by just pulling my branch, and merely signing off on the merge commit. Which would be totes okay with me.) Ciao, Dscho
