On Wed, Mar 21, 2018 at 7:44 AM, Kamil Dudka <kdu...@redhat.com> wrote: > ... because there is no such module in gnulib > --- > lib/fts.c | 26 +++++--------------------- > 1 file changed, 5 insertions(+), 21 deletions(-) > > diff --git a/lib/fts.c b/lib/fts.c > index bfa73e31e..4195f6170 100644 > --- a/lib/fts.c > +++ b/lib/fts.c > @@ -253,7 +253,6 @@ static int fts_safe_changedir (FTS *, FTSENT *, int, > const char *) > # include <inttypes.h> > # include <stdint.h> > # include <stdio.h> > -# include "getcwdat.h"
Yikes! Hi Kamil, Thanks for noticing and fixing that. It took some digging, but I found the code that I once used (back in 2006!) when I was building/testing that with -DFTS_DEBUG. It was in a CVS-checked-out directory I had renamed to gnulib-corrupt. I'll include those files here, for reference, in case someone wants to modernize them and to make a module out of them -- they probably predate gnulib's modules directory. Given that you're the first to mention this in nearly 12 years, I suspect it's better just to forget about these two files and use your patch, though I do recall preferring to see actual directory names (rather than nearly useless FDs) in debugging output. Another problem: getcwdat.c was essentially copied from some old getcwd.c-related file; if we were to use this new module, we'd probably require that the .c file be mechanically derived, to avoid the duplication.
getcwdat.c
Description: Binary data
getcwdat.h
Description: Binary data