[PATCH] io: Use gnulib fts implementation (BZ 22944, BZ 20331)

Adhemerval Zanella Netto adhemerval.zanella@linaro.org
Tue Feb 24 12:06:24 GMT 2026



On 24/02/26 08:58, Florian Weimer wrote:
> * Bruno Haible:
> 
>> Adhemerval Zanella wrote:
>>>  create mode 100644 io/cycle-check.c
>>>  create mode 100644 io/cycle-check.h
>>>  create mode 100644 io/dev-ino.h
>>>  create mode 100644 io/fts-cycle.c
>>>  create mode 100644 io/hash.c
>>>  create mode 100644 io/hash.h
>>>  create mode 100644 io/i-ring.c
>>>  create mode 100644 io/next-prime.c
>>>  create mode 100644 io/next-prime.h
>>>  create mode 100644 io/same-inode.h
>>
>> That's 10 new files, in a directory that already has 104 C source code files
>> (excluding tests), and which exist only for use by the fts.c, fts64.c,
>> fts64-time64.c compilation units.
>>
>> For maintainability, I would suggest to put all these 10 files, and possibly
>> also fts.c and fts.h, into a subdirectory of io/.
>>
>> Like the directory resolv/nss_dns/ under resolv/.
>> Or like the directories nss/nss_compat/, nss/nss_db/, nss/nss_files/
>> under nss/.
> 
> That requires using vpath.  The glibc build system is not really
> compatible with that.
> 
> We can argue with the choice of io for hash table functionality.
> That should probably go into misc.

I can move both hash and next-prime to misc; there are a couple of
instance on next-prime functions usage in other files.


More information about the Libc-alpha mailing list