Re: [PATCH] stdlib: work around MirBSD WEXITSTATUS bug

2010-09-15 Thread Bruno Haible
Hi Eric, > +...@item > +Some platforms fail to provide @code{WEXITSTATUS} and friends: > +MirBSD 10. I find this problem description a bit ambiguous, therefore this follow-up patch, to improve the wording. 2010-09-15 Bruno Haible stdlib: clarify MirBSD WEXITSTATUS bug * lib/

Re: bootstrap: a new option --skip-git, prohibiting the use git in the bootstrap process.

2010-09-15 Thread Eric Blake
On 08/08/2010 03:48 PM, Aleksey Avdeev wrote: Hello. I propose to add to bootstrap key --skip-git (analog --skip-po), completely eliminates the use git in the bootstrap process. Now this behavior may occur if the directory specified by --gnulib-srcdir (or specified in the $ GNULIB_SRCDIR) not

Re: bug#6816: df bug on 64-bit Solaris (need to use getextmntent)

2010-09-15 Thread Eric Blake
[digging through older mail, and sorry for the delayed reply] On 08/06/2010 01:56 PM, Wood, David wrote: Hello, From mnttab(4) on Solaris 10: Applications requiring device number information for mounted file systems should use the getextmntent(3C) interface, which func

[PATCH] stdlib: work around MirBSD WEXITSTATUS bug

2010-09-15 Thread Eric Blake
* lib/stdlib.in.h (includes): Guarantee WEXITSTATUS. * modules/stdlib (Depends-on): Add sys_wait. * tests/test-sys_wait.c (main): Enhance test. * tests/test-stdlib.c (main): Likewise. * doc/posix-headers/stdlib.texi (stdlib.h): Document the bug. Signed-off-by: Eric Blake --- While debating about

Re: IPv6 structs on MacOS X

2010-09-15 Thread Justin Clift
On 09/16/2010 04:52 AM, Bruno Haible wrote: This message talks about the unportability of a field 's6_addr32' in 'struct in6_addr'. Sure this is unportable: POSIX [1] specifies only the presence of 's6_addr'. I don't see why gnulib should support this: An application can perfectly access the 16

Re: IPv6 structs on MacOS X

2010-09-15 Thread Justin Clift
On 09/16/2010 06:32 AM, Eric Blake wrote: I agree with Bruno - the portability problem stems from using a field member not specified by POSIX in the first place. It should be sufficiently easy to rewrite the offending libvirt code to use only the portable interfaces, without burdening gnulib wit

Re: IPv6 structs on MacOS X

2010-09-15 Thread Eric Blake
On 09/15/2010 02:29 PM, Justin Clift wrote: On 09/16/2010 04:52 AM, Bruno Haible wrote: This message talks about the unportability of a field 's6_addr32' in 'struct in6_addr'. Sure this is unportable: POSIX [1] specifies only the presence of 's6_addr'. I don't see why gnulib should support thi

Re: [libvirt] [PATCH] virsh: change wexitstatus order to allow compilation on mac osx

2010-09-15 Thread Eric Blake
On 09/15/2010 02:17 PM, Justin Clift wrote: On 09/16/2010 05:49 AM, Eric Blake wrote: +MacOS 10.5. Is "10.5" the right choice for this? Wondering if maybe it should be a range indicator, as "10.6"(.4)" here has this issue, and it sounds like pretty much all of the previous versions will too.

Re: [libvirt] [PATCH] virsh: change wexitstatus order to allow compilation on mac osx

2010-09-15 Thread Justin Clift
On 09/16/2010 05:49 AM, Eric Blake wrote: +MacOS 10.5. Is "10.5" the right choice for this? Wondering if maybe it should be a range indicator, as "10.6"(.4)" here has this issue, and it sounds like pretty much all of the previous versions will too. ?

[PATCH] strnlen: add tests

2010-09-15 Thread Eric Blake
* modules/strnlen-tests: New file. * tests/test-strnlen.c: Likewise. Signed-off-by: Eric Blake --- Always worth testsuite additions. ChangeLog |6 modules/strnlen-tests | 20 ++ tests/test-strnlen.c | 67 + 3

Re: [libvirt] [PATCH] virsh: change wexitstatus order to allow compilation on mac osx

2010-09-15 Thread Eric Blake
On 09/15/2010 12:41 PM, Bruno Haible wrote: Since WEXITSTATUS should be usable on constants Who says that? POSIX [1] says that the argument of WEXITSTATUS has to be an integer value returned by wait() or waitpid() function in the *stat_loc location. Interesting argument - and I can agree that

IPv6 structs on MacOS X

2010-09-15 Thread Bruno Haible
Justin Clift wrote: > If you can figure a way to address this OSX specific thing in gnulib at > the same time, that would be nifty: > >http://www.redhat.com/archives/libvir-list/2010-September/msg00248.html This message talks about the unportability of a field 's6_addr32' in 'struct in6_addr'

Re: module libposix (was: module gethostname)

2010-09-15 Thread Bruce Korb
Hi Bruno, On Wed, Sep 15, 2010 at 11:18 AM, Bruno Haible wrote: > > Have you tried to follow the approach of >  ? > I haven't, but I would like to hear about success or problems that > you encounter. That should

Re: [libvirt] [PATCH] virsh: change wexitstatus order to allow compilation on mac osx

2010-09-15 Thread Bruno Haible
Hi Eric, > #define _W_INT(w) (*(int *)&(w)) /* convert union wait to int */ > ... > Obviously, the Darwin folks are (mistakenly) assuming that you would > only ever use WEXITSTATUS with a 'union wait' lvalue No, it is assuming that the WEXITSTATUS argument is an lvalue, of type 'int' or '

Re: [libvirt] [PATCH] autogen: added glibtool and pkg-config checks

2010-09-15 Thread Justin Clift
On 09/16/2010 03:02 AM, Eric Blake wrote: [adding bug-gnulib] On 09/15/2010 06:36 AM, Justin Clift wrote: OSX MacPorts has libtool named as glibtool, with libtoolize named as glibtoolize. This patch adds support for this, and also adds a check for pkg-config, to warn when it is missing. Hmm -

Re: [libvirt] [PATCH] virsh: change wexitstatus order to allow compilation on mac osx

2010-09-15 Thread Eric Blake
[adding bug-gnulib; replies can drop libvir-list] - if (command_ret != WEXITSTATUS (0)) { + if (WEXITSTATUS(command_ret) != 0) { ACK. By the way, what was the compilation failure? Thanks, pushed. The compilation failure was: virsh.c:8605: error: lvalue required as unary '&' operand Which s

Re: module gethostname

2010-09-15 Thread Bruno Haible
Bruce, > Once upon a time, long ago and far away, I proposed a new > library, "libcompat" that more-or-less would make all target > platforms look pretty much identical. Lo and behold, over the > past decade, gnulib has gradually grown up to provide all > that infrastructure. The only problem wi

Re: module gethostname

2010-09-15 Thread Bruno Haible
Sam Steingold wrote: > The only problem which gethostname module fixes is: > > On mingw, this function has a prototype that differs from that > specified by POSIX, and it is defined only in the ws2_32 library. > > why does it have such extensive dependencies? (sockets! errno!) Look at the source

RE: MinGW-w64 Compatibility

2010-09-15 Thread Hoyt, David
> This is one of the reasons why I ask for a reproducible sample. If that sample > fails for you but works for other people, it would be an indication of > something peculiar with your setup. If that sample fails also for someone > who has experience with gnulib, that person can investigate it. I

Re: MinGW-w64 Compatibility

2010-09-15 Thread Bruno Haible
David Hoyt wrote: > I've had the same issue compiling gnutls ... > Could be there's something wrong w/ my setup (but I don't think so). This is one of the reasons why I ask for a reproducible sample. If that sample fails for you but works for other people, it would be an indication of something pe

RE: MinGW-w64 Compatibility

2010-09-15 Thread Hoyt, David
Bruno Haible wrote: > Can you please provide a minimal reproducible sample? That is, can you > give an input file that, when compiled with gnulib's , yields > an error? Also, please show the compiler command-line, and the error that > you get. Everyone's blaming someone else, so I honestly don't k

Re: module gethostname

2010-09-15 Thread Dustin J. Mitchell
On Wed, Sep 15, 2010 at 12:01 PM, Bruce Korb wrote: > So, let's start that discussion again:  let's make an installable > gnulib that basically does all the configure tests for the current > target and cover 99% of all configure runs.  If someone has > a strange configuration with extra libraries

Re: module gethostname

2010-09-15 Thread Bruce Korb
On Wed, Sep 15, 2010 at 10:04 AM, Ralf Wildenhues wrote: > Hi Bruce, > > * Bruce Korb wrote on Wed, Sep 15, 2010 at 07:01:11PM CEST: >> So, let's start that discussion again:  let's make an installable >> gnulib that basically does all the configure tests for the current >> target and cover 99% of

Re: [libvirt] [PATCH] autogen.sh: added glibtool and pkg-config checks

2010-09-15 Thread Bruce Korb
Please don't use my project name when talking about bootstrap process. "autogen.sh" has become a popular misnomer, so at least keep the ".sh" suffix. Thanks. On Wed, Sep 15, 2010 at 10:02 AM, Eric Blake wrote: > [adding bug-gnulib] > > On 09/15/2010 06:36 AM, Justin Clift wrote: >> >> OSX MacP

Re: module gethostname

2010-09-15 Thread Ralf Wildenhues
Hi Bruce, * Bruce Korb wrote on Wed, Sep 15, 2010 at 07:01:11PM CEST: > So, let's start that discussion again: let's make an installable > gnulib that basically does all the configure tests for the current > target and cover 99% of all configure runs. How about a config.site file? Cheers, Ralf

Re: [libvirt] [PATCH] autogen: added glibtool and pkg-config checks

2010-09-15 Thread Eric Blake
[adding bug-gnulib] On 09/15/2010 06:36 AM, Justin Clift wrote: OSX MacPorts has libtool named as glibtool, with libtoolize named as glibtoolize. This patch adds support for this, and also adds a check for pkg-config, to warn when it is missing. Hmm - would it be easier to make bootstrap.conf

Re: module gethostname

2010-09-15 Thread Bruce Korb
On Wed, Sep 15, 2010 at 9:34 AM, Paul Eggert wrote: > On 09/15/2010 08:41 AM, Sam Steingold wrote: >> I have to reiterate my complaint that the gnulib dependencies mean that one >> has >> to include (almost) the whole of gnulib if one includes (almost) any part >> thereof. > > I agree that gnuli

Re: module gethostname

2010-09-15 Thread Paul Eggert
On 09/15/2010 08:41 AM, Sam Steingold wrote: > I have to reiterate my complaint that the gnulib dependencies mean that one > has > to include (almost) the whole of gnulib if one includes (almost) any part > thereof. I agree that gnulib has too many dependencies and I try to remove them when I ca

module gethostname

2010-09-15 Thread Sam Steingold
The only problem which gethostname module fixes is: On mingw, this function has a prototype that differs from that specified by POSIX, and it is defined only in the ws2_32 library. why does it have such extensive dependencies? (sockets! errno!) I have to reiterate my complaint that the gnulib d

Re: MirBSD mbtowc bug? failure on test-wcrtomb

2010-09-15 Thread Bruno Haible
Hi Eric, > Bruno, this one probably needs your help for a gnulib workaround. On > MirBSD 10 (20100815), wcrtomb is found but not replaced; leading to > failure in the unit test: > > ac_cv_func_wcrtomb=yes > ac_cv_have_decl_wcrtomb=yes > gl_cv_func_wcrtomb_retval=yes > gl_cv_have_raw_decl_wcrto