Comments on "Rename/copy detection fix."
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:41:58
I talked about patch reordering in the commit log without giving
specifics, because I did not think it is good to cast specifics
of uncoded ideas into stone in a commit log message.
But to throw it out open in the public, here are two of the
obvious ones.
(1) Give users a way to specify patches about which files
should come before which other files. Things like "*.h
files before *.c files", or "README and Documentation first
before anything else".
(2) Currently when pickaxe is used, it shows only diffs about
the files involved in the change we looked for. The user
may sometimes want to see such a diff in the context of
whole patches and I plan to add an option for doing so. It
may be useful in such a case to show the patch about the
file that triggered the commit to be selected by the
pickaxe first before patches for all other files.
These may or may not be useful, but this kind of reordering
would make it pretty much pointless to record which is a rename
and which is a copy in the diff_filepair structure in the
diff_queue. It all depends on which one comes first and which
one comes later in the final ordering, which is something
diffcore-rename cannot (and should not) know about.