On 17/05/15 23:16, Jim Meyering wrote: > On Sun, May 17, 2015 at 7:48 AM, Pádraig Brady <[email protected]> wrote: >> -sc_case_insensitive_file_names: >> - @git ls-files | sort -f | $(srcdir)/src/uniq -Di | grep . && \ >> +sc_case_insensitive_file_names: src/uniq >> + @git ls-files | sort -f | src/uniq -Di | grep . && \ > > Looks good. Thanks! > Is there a reason not to prefix "sort" as "src/sort", as is done with uniq?
Well sort -f is standardized, and using the systems sort avoids the need for an explicit dependency on src/sort. There are issues with the specific orders of some versions of sort, though uniq only cares that matching items are together, so doesn't matter in this case. cheers, Pádraig.
