>>> Pádraig, I assume this warrants a NEWS entry under "Bug Fixes"? >> >> It's just a performance improvement, no? And not such a big >> improvement that it warrants being called aa "bug fix". > >I was conflicted on that since it might be considered a bug in fts. I >guess it worked fine though, just much slower. > >I pushed the attached patch to mention the improvement in NEWS.
It might be considered a performance regression. Although I haven't tested old versions of coreutils to be sure or looked to see where this sorting was originally introduced. >By the way Tim, the fts_* functions are a part of glibc as well. >However, the inode sorting code is not contained there. I assume it was >a gnulib invention among some other extensions we have, e.g., using >openat() instead of chdir(). So no need for any changes there. I forgot >to mention that in my previous mail. Thanks for the heads up and thanks for looking into this.
