Re: euidaccess

2010-03-06 Thread Simon Josefsson
Jim Meyering writes: > Bruno Haible wrote: >>> > Thanks -- I've pushed the modified patch. Now the only license issue >>> > within gnulib is 'euidaccess'... >> >>> + euidaccess: relax license to LGPLv2+ > > Hi Bruno, > >> This won't help much, because 'euidaccess' depends on 'group-member', wh

new module 'iconv-h'

2010-03-06 Thread Bruno Haible
Another irregularity is that there is no module for the iconv.h include file. So far it has been part of the 'iconv_open' module. This is inconsistent. 2010-03-06 Bruno Haible New module 'iconv-h'. * modules/iconv-h: New file. * modules/iconv_open (Files): Remove lib/i

consistent naming of *.m4 files

2010-03-06 Thread Bruno Haible
Hi, Most .m4 files that correspond to a replacement header foo.in.h are called foo_h.m4 - for good reasons: the foo.m4 file sometimes is only a test whether foo exists, or a replacement for function foo. Only 2 files are not following this convention, for historical reasons. I'm renaming them:

Re: euidaccess

2010-03-06 Thread Jim Meyering
Bruno Haible wrote: > Jim Meyering wrote: >> For some of the same reasons, the euidaccess function/module should be >> avoided, simply because it cannot be used robustly -- by its very nature. >> Use faccessat instead. > > Here's a proposed doc update and additional warnings for the use of > 'acces

Re: euidaccess

2010-03-06 Thread Bruno Haible
Jim Meyering wrote: > For some of the same reasons, the euidaccess function/module should be > avoided, simply because it cannot be used robustly -- by its very nature. > Use faccessat instead. Here's a proposed doc update and additional warnings for the use of 'access' and 'euidaccess' (guarded b

Re: new module 'c++defs'

2010-03-06 Thread Bruno Haible
John W. Eaton wrote: > How can one easily find all the places where the GNULIB_NAMESPACE tag > is needed? Is there some way we can get the compiler to help with > this job? Good point. I'm adding a macro _GL_CXXALIASWARN, so that a warning will be emitted: foo.cc: In function ‘int main()’: f

Re: euidaccess

2010-03-06 Thread Jim Meyering
Bruno Haible wrote: >> > Thanks -- I've pushed the modified patch. Now the only license issue >> > within gnulib is 'euidaccess'... > >> +euidaccess: relax license to LGPLv2+ Hi Bruno, > This won't help much, because 'euidaccess' depends on 'group-member', which > depends on 'xalloc', which

Re: euidaccess

2010-03-06 Thread Bruno Haible
Hi Jim, > > Thanks -- I've pushed the modified patch. Now the only license issue > > within gnulib is 'euidaccess'... > + euidaccess: relax license to LGPLv2+ This won't help much, because 'euidaccess' depends on 'group-member', which depends on 'xalloc', which is GPL. 'xalloc' is GPL and n

Re: relax licenses for both exitfail and exit

2010-03-06 Thread Jim Meyering
Simon Josefsson wrote: ... > Thanks -- I've pushed the modified patch. Now the only license issue > within gnulib is 'euidaccess'... glibc-derived, so I've adjusted it: >From 032088a1c918358d4832c9ea524a757ea2739f8d Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Sat, 6 Mar 2010 11:46:04 +010

Re: exitfail

2010-03-06 Thread Bruno Haible
Simon Josefsson wrote: > Depends-on: > exit > > configure.ac: > -gl_EXITFAIL > +AC_LIBOBJ([exitfail]) > > Makefile.am: > Additionally, I'm applying this: Prefer lib_SOURCES over unconditional AC_LIBOBJ. It's a no-op for most projects that use gnulib, but gets rid of a .diff override in G