On 02/07/17 20:20, Eric Blake wrote: > On 12/02/2016 03:01 PM, Laszlo Ersek wrote: >> When passed to git-diff (and to every other git command producing diffs >> and/or diffstats) with "-O" or "diff.orderFile", this list of patterns >> will place the more declarative / abstract hunks first, while changes to >> imperative code / details will be near the end of the patches. This saves >> on scrolling / searching and makes for easier reviewing. >> >> We intend to advise contributors in the Wiki to run >> >> git config diff.orderFile scripts/git.orderfile >> >> once, as part of their initial setup, before formatting their first (or, >> for repeat contributors, next) patches. > > Presumably, we'd also want this to be mentioned in HACKING?
Presumably. :) > >> >> See the "-O" option and the "diff.orderFile" configuration variable in >> git-diff(1) and git-config(1). >> >> Cc: "Michael S. Tsirkin" <[email protected]> >> Cc: Eric Blake <[email protected]> >> Cc: Fam Zheng <[email protected]> >> Cc: Gerd Hoffmann <[email protected]> >> Cc: John Snow <[email protected]> >> Cc: Max Reitz <[email protected]> >> Cc: Stefan Hajnoczi <[email protected]> >> Signed-off-by: Laszlo Ersek <[email protected]> >> --- > > Is it time to revive this patch? Time for whom? :) Cheers, Laszlo
