Hi, On Mon, 21 Oct 2019, Denton Liu wrote:
> Hi Johannes, > > On Mon, Oct 21, 2019 at 08:44:40PM +0200, Johannes Schindelin wrote: > > Hi Junio, > > > > On Fri, 18 Oct 2019, Junio C Hamano wrote: > > > > > Denton Liu <liu.den...@gmail.com> writes: > > > > > > > There are many += lists in the Makefile and, over time, they have gotten > > > > slightly out of order, alphabetically. Alphabetically sort all += lists > > > > to bring them back in order. > > > > ... > > > > > > Hmm. I like the general thrust, but ... > > > > > > > LIB_OBJS += combine-diff.o > > > > -LIB_OBJS += commit.o > > > > LIB_OBJS += commit-graph.o > > > > LIB_OBJS += commit-reach.o > > > > +LIB_OBJS += commit.o > > > > > > ... I do not particularly see this change (there may be similar > > > ones) desirable. I'd find it it be much more natural to sort > > > "commit-anything" after "commit", and that is true with or without > > > the common extension ".o" added to these entries. > > > > > > In short, flipping these entries because '.' sorts later than '-' is > > > making the result look "less sorted", at least to me. > > > > The problem with this argument is that it disagrees with ASCII, as `-` > > has code 0x2d while `.` has code 0x2e, i.e. it is lexicographically > > _larger_. > > > > So Denton's patch does the correct thing. > > I actually agree with Junio on this one. Without the prefixes, "commit" > would go before "commit-graph" so I think it would make more sense to > order with the prefixes removed instead of taking the naive ordering by > just sorting each block. That will make it harder on other contributors like me, who prefer to mark the lines in `vim` and then call `:sort` on them, and then not care about it any further. Any decision that makes automating tedious tasks harder puts more burden on human beings. I don't like that. Ciao, Dscho > > Thanks, > > Denton > > > > > Ciao, > > Dscho >
