Re: [PATCH] largefile: add dependencies to this module

[Paul Eggert]

On 07/25/11 18:25, Bruno Haible wrote:
> New module                          Used by
> 
> creat                               fcntl-safer
> fgetpos                             -
> fsetpos                             -
> fstat                               acl, chdir-safer, chown, copy-file, 
> fchdir,
>                                     fdopendir, fopen, ftruncate, fts, getcwd,
>                                     getloadavg, isapipe, linkat, lseek,
>                                     mkdir-p, open, openat, read-file,
>                                     [relocatable-prog, 
> relocatable-prog-wrapper,]
>                                     renameat, utimens
> opendir                             backupfile, dirent-safer, fchdir, 
> fdopendir,
>                                     fts, getcwd, glob, mountlist, savedir,
>                                     scandir
> readdir                             backupfile, getcwd, glob, mountlist, 
> savedir
> rewinddir                           getcwd
> seekdir                             -
> telldir                             -

In looking through these I see that I don't have much to add to my
previous email.  The fgetpos and fsetpos modules are not important
(nobody uses those functions, and for good reason).  The other modules
are a more-complicated way of establishing dependencies on largefile
than I'd like (as per my previous email).  These modules don't break
anything, so I suppose they're OK; but I wish it were simpler, as
complexity imposes its own costs.

Perhaps someone else can chime in.