Junio C Hamano <junkio@xxxxxxx> wrote: > Linus Torvalds <torvalds@xxxxxxxx> writes: > > > (a) git-rev-list --pretty=oneline "$upstream"..ORIG_HEAD > rev-list > > > > (b) edit the rev-list, moving the single lines around, deleting them, etc > > > > (c) cat rev-list | > > git-format-patch -k --stdout --stdin --full_index | > > git-am > > [snip] > What this means is: > > git-format-patch $revargs > > is not equivalent to > > git-rev-list $revargs | git-format-patch --stdin > > but is equivalent to > > git-rev-list $revargs | tac | git-format-patch --stdin > > > Thoughts from the list? Aside from not knowing what tac is I agree with the latter. It is what the user would expect to happen. Except maybe add a --ancestors-first option to git-rev-list to do what tac does before outputting the revisions? git-rev-list --ancestors-first $revargs | git-format-patch --stdin -- Shawn. - : 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