Re: split off module fstatat from module openat

2011-11-09 Thread Bruno Haible
Jim Meyering wrote: > Did you find that by using some new tool to check > for unnecessary dependencies, or "just" by inspection? Just by inspection. The first approximation was done by 'grep'ing for the function calls, e.g. grep "fstat (". The second inspection was line by line. This particular

Re: split off module fstatat from module openat

2011-11-08 Thread Jim Meyering
Bruno Haible wrote: >> * modules/fstatat: New file, extracted from modules/openat. > > Oops, there was a small mistake here: fstatat() does not call fstat(). > > > 2011-11-08 Bruno Haible > > fstatat: Remove unused dependency. > * modules/fstatat (Depends-on): Remove fstat. >

Re: split off module fstatat from module openat

2011-11-08 Thread Bruno Haible
> * modules/fstatat: New file, extracted from modules/openat. Oops, there was a small mistake here: fstatat() does not call fstat(). 2011-11-08 Bruno Haible fstatat: Remove unused dependency. * modules/fstatat (Depends-on): Remove fstat. --- modules/fstatat.orig

Re: split off module fstatat from module openat

2011-11-04 Thread Bruno Haible
> 2011-11-03 Bruno Haible > > New module 'fstatat', split off from module 'openat'. Oops, this patch was incomplete. A megatestdir of the various *at modules revealed a link error in test-unlinkat on Solaris 9. This should fix it. 2011-11-04 Bruno Haible fchownat, renameat,

Re: split off module fstatat from module openat

2011-11-04 Thread Bruno Haible
Jim Meyering wrote: > I've reviewed this change-set in isolation, and it looks fine. I've pushed it now. Bruno -- In memoriam Yitzhak Rabin

Re: split off module fstatat from module openat

2011-11-04 Thread Jim Meyering
Bruno Haible wrote: > The next split-off is 'fstatat'. Here's the proposed patch. Tested on several > platforms. > > 2011-11-03 Bruno Haible > > New module 'fstatat', split off from module 'openat'. > * lib/openat.h (statat, lstatat): Enable only if GNULIB_FSTATAT is > defined.

split off module fstatat from module openat

2011-11-03 Thread Bruno Haible
Hi Jim, The next split-off is 'fstatat'. Here's the proposed patch. Tested on several platforms. 2011-11-03 Bruno Haible New module 'fstatat', split off from module 'openat'. * lib/openat.h (statat, lstatat): Enable only if GNULIB_FSTATAT is defined. * m4/fsta