On Thu, Mar 27, 2014 at 09:58:41AM -0700, Junio C Hamano wrote: > "Michael S. Tsirkin" <mst@xxxxxxxxxx> writes: > > > Patch id changes if you reorder hunks in a diff. > > If you reorder hunks, the patch should no longer apply [*1*], so a > feature to make patch-id stable across such move would have no > practical use ;-), but I am guessing you meant something else. > > Perhaps this is about using "-O <orderfile>" option, even though you > happened to have implemented the id mixing at per-hunk level? Yes. > > [Footnote] > > *1* It has been a long time since I looked at the code, and I do not > know offhand if "git apply" has such a bug not to diagnose a hunk > for a file for an earlier part that comes later in its input stream > after seeing another hunk for the same file as a bug. If it does > not, perhaps we should. Hmm you are right. For some reason I thought that it does work. I'll drop this part then, less code this way. Thanks! Any more comments before I spin v2? -- MST -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html