On Wed, Jan 20, 2016 at 12:23:38PM -0800, Junio C Hamano wrote: > Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> writes: > > > This will be important later when diff_opt_parse() accepts paths as > > arguments. Paths must be prefixed before access because setup code > > moves cwd but does not (and cannot) update command line options. > > The above sounds like a sensible thing to do (note: I didn't read > the patch or remainder of the series), but makes me wonder how the > existing --orderfile option works without this support. Perhaps it > is not working and needs to be updated to take advantage of this > change, too? Yeah, I think it simply does not work. $ >main-order $ mkdir subdir && >subdir/sub-order $ cd subdir $ git show -Osub-order fatal: failed to read orderfile 'sub-order': No such file or directory $ git show -Omain-order [shows diff] -Peff -- 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