[PATCH] * lib/strtod.c (HAVE_RAW_DECL_STRTOD): Remove; no longer used.

2013-02-19 Thread Paul Eggert
--- ChangeLog| 1 + lib/strtod.c | 3 --- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/ChangeLog b/ChangeLog index 4f3a198..a6c50ff 100644 --- a/ChangeLog +++ b/ChangeLog @@ -5,6 +5,7 @@ HAVE_RAW_DECL_STRTOD might not be correct in coreutils, which disables the

Re: XALLOC_INLINE static on HP-UX?

2013-02-19 Thread Paul Eggert
> Date: Tue, 19 Feb 2013 15:56:41 + (GMT) > From: Richard Lloyd > cc -O -I/usr/local/include -Wl,+b -Wl,/usr/local/lib/hpux32 > -L/usr/local/lib/hpux32 -o makedoc makedoc.o ../gnulib/lib/libgnu.a -lncurses > /usr/local/lib/hpux32/libintl.so /usr/local/lib/hpux32/libiconv.so -Wl,+b > -Wl,

Re: [Win32] putenv modifications not inherited by child processed (Was: Heap corruption in putenv)

2013-02-19 Thread John W. Eaton
On 02/19/2013 05:46 PM, Paul Eggert wrote: On 02/19/13 14:04, John W. Eaton wrote: and also to use _putenv in the normal case when the argument string to putenv contains "VAR=VAL". Ouch, thanks for catching that; it suggests further fixes. Could you please try this patch to gnulib instead?

Re: full_read depends on incoming errno

2013-02-19 Thread Eli Zaretskii
> Date: Tue, 19 Feb 2013 12:19:38 -0800 > From: Paul Eggert > CC: bug-gnulib , Eli Zaretskii > > On 02/19/13 11:05, Andy Wingo wrote: > > We use full_read in Guile and just got a bug report that full_read was > > depending on the incoming errno. > > Sorry, I don't see the bug here. The calling

Re: XALLOC_INLINE static on HP-UX?

2013-02-19 Thread Karl Berry
Hi Paul, Richard Lloyd (cc'd again) followed up with the info below (on bug-texinfo). (Richard: Paul/et al. are not on bug-texinfo, so please keep discussion on the gnulib/lib/xalloc.h change on this list, bug-gnulib.) Thanks, k Date: Tue, 19 Feb 2013 15:56:41 + (GMT) From: Richard Lloyd T

Re: [Win32] putenv modifications not inherited by child processed (Was: Heap corruption in putenv)

2013-02-19 Thread Paul Eggert
On 02/19/13 14:04, John W. Eaton wrote: > and also to use _putenv in the normal case when the argument string to putenv > contains "VAR=VAL". Ouch, thanks for catching that; it suggests further fixes. Could you please try this patch to gnulib instead? >From ae287e04d310465ed2e8bd7df61b65094b66b

[PATCH] strtod: support coreutils better

2013-02-19 Thread Paul Eggert
--- ChangeLog| 6 ++ lib/strtod.c | 19 +++ 2 files changed, 9 insertions(+), 16 deletions(-) diff --git a/ChangeLog b/ChangeLog index 79bda70..4f3a198 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,11 @@ 2013-02-19 Paul Eggert + strtod: support coreutils b

Re: [Win32] putenv modifications not inherited by child processed (Was: Heap corruption in putenv)

2013-02-19 Thread John W. Eaton
On 02/18/2013 10:40 PM, Paul Eggert wrote: Thanks, how about this change instead? It's a bit more conservative about always using _putenv and setting errno. And it uses WIN32_LEAN_AND_MEAN. I need the attached additional changes to provide declarations for putenv_result and putenv_error in th

Re: full_read depends on incoming errno

2013-02-19 Thread Andy Wingo
Hi, On Tue 19 Feb 2013 22:25, Paul Eggert writes: > On 02/19/13 13:15, Eli Zaretskii wrote: >> if (full_read (fd, cookie, sizeof cookie) != sizeof cookie >> || full_read (fd, SCM_BYTEVECTOR_CONTENTS (bv), >>SCM_BYTEVECTOR_LENGTH (bv)) != SCM_BYTEVECTOR_LENGTH (bv)) >

RE: usage of hasmntopt() in gnulib

2013-02-19 Thread Joachim Schmitz
> From: Paul Eggert [mailto:egg...@cs.ucla.edu] > Sent: Tuesday, February 19, 2013 10:07 PM > To: Joachim Schmitz > Cc: bug-gnulib@gnu.org > Subject: Re: usage of hasmntopt() in gnulib > > Thanks, I pushed a slightly different patch that > should do the same thing, here: > > http://git.savannah.g

Re: full_read depends on incoming errno

2013-02-19 Thread Paul Eggert
On 02/19/13 13:15, Eli Zaretskii wrote: > if (full_read (fd, cookie, sizeof cookie) != sizeof cookie > || full_read (fd, SCM_BYTEVECTOR_CONTENTS (bv), > SCM_BYTEVECTOR_LENGTH (bv)) != SCM_BYTEVECTOR_LENGTH (bv)) > { > int errno_save = errno; > (vo

Re: usage of hasmntopt() in gnulib

2013-02-19 Thread Paul Eggert
Thanks, I pushed a slightly different patch that should do the same thing, here: http://git.savannah.gnu.org/cgit/gnulib.git/commit/?id=62bb7a8bf95807d6339e1e17fc0d21c319b280a2

Re: full_read depends on incoming errno

2013-02-19 Thread Paul Eggert
On 02/19/13 11:05, Andy Wingo wrote: > We use full_read in Guile and just got a bug report that full_read was > depending on the incoming errno. Sorry, I don't see the bug here. The calling code should look like this: size_t r = full_read (fd, buf, n); if (r != n) { if (errno

full_read depends on incoming errno

2013-02-19 Thread Andy Wingo
Hi, We use full_read in Guile and just got a bug report that full_read was depending on the incoming errno. Eli Zaretskii proposed that the fix be like this: > +errno = 0; > if (full_read (fd, cookie, sizeof cookie) != sizeof cookie [...] To recall, safe_read (called by full_read) look

RE: coreutils-8.21, some changes needed

2013-02-19 Thread Joachim Schmitz
> -Original Message- > From: Joachim Schmitz [mailto:j...@schmitz-digital.de] > Sent: Tuesday, February 19, 2013 9:18 AM > To: '10...@debbugs.gnu.org'; 'bug-gnulib@gnu.org' > Subject: coreutils-8.21, some changes needed > > Hi folks > > In an attempt to port coreutils 8.21 to HP-NonStop I

usage of hasmntopt() in gnulib

2013-02-19 Thread Joachim Schmitz
Hi folks In mountlist.c hasmntopt() is used in two places, once properly protected by a HAVE_HASMNTOPT and a second time without that protection (inside a ifdef MOUNTED_GETMNTENT1). The following seems to fix it properly. diff --git a/lib/mountlist.c b/lib/mountlist.c index 8fb7e9a..7

RE: bug#10305: coreutils-8.21, some changes needed

2013-02-19 Thread Joachim Schmitz
> From: Pádraig Brady [mailto:p...@draigbrady.com] > Sent: Tuesday, February 19, 2013 11:29 AM > To: Joachim Schmitz > Cc: 10...@debbugs.gnu.org; bug-gnulib@gnu.org > Subject: Re: bug#10305: coreutils-8.21, some changes needed > > On 02/19/2013 09:08 AM, Joachim Schmitz wrote: > >> -Original M

Re: bug#10305: coreutils-8.21, some changes needed

2013-02-19 Thread Pádraig Brady
On 02/19/2013 09:08 AM, Joachim Schmitz wrote: -Original Message- From: Joachim Schmitz [mailto:j...@schmitz-digital.de] Sent: Tuesday, February 19, 2013 9:18 AM To: '10...@debbugs.gnu.org'; 'bug-gnulib@gnu.org' Subject: coreutils-8.21, some changes needed Hi folks In an attempt to port

RE: bug#10305: coreutils-8.14, "rm -r" fails with EBADF

2013-02-19 Thread Joachim Schmitz
> From: Joachim Schmitz [mailto:j...@schmitz-digital.de] > Sent: Saturday, July 21, 2012 12:42 PM Yes, it's ben a while... > To: 'Paul Eggert' > Cc: '10...@debbugs.gnu.org'; 'bug-gnulib@gnu.org'; 'Eric Blake'; 'Jim > Meyering'; 'Schmitz, Joachim'; 'nagendra...@hp.com' > Subject: RE: bug#10305: c

coreutils-8.21, some changes needed

2013-02-19 Thread Joachim Schmitz
Hi folks In an attempt to port coreutils 8.21 to HP-NonStop I stumbled accross a couple problems, here are my fixes to them: Here another file needs to get adjusted to using the new root-uid.h diff -EBbu ./src/copy.c.orig ./src/copy.c --- ./src/copy.c.orig 2013-02-07 03:37:05 -0600 +++ ./src/

Re: [Win32] putenv modifications not inherited by child processed (Was: Heap corruption in putenv)

2013-02-19 Thread Bastien ROUCARIES
On Fri, Feb 15, 2013 at 11:24 PM, Eric Blake wrote: > On 02/15/2013 08:10 AM, Michael Goffioul wrote: >> The problem is that the "environ" variable is just a mirror of the >> environment that is manipulated through GetEnvironmentVariable and >> SetEnvironmentVariable. Manipulating the "environ" va