bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-27 Thread Jim Meyering
Pádraig Brady wrote: > On 16/07/11 01:51, Paul Eggert wrote: >> On 07/15/11 03:28, Pádraig Brady wrote: >>> What I was getting was that it's probably better to leave >>> the following to the app too: >>> >>> #ifndef SA_RESETHAND >>> # define SA_RESETHAND 0 >>> /* Now the app writer knows they need

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-20 Thread Pádraig Brady
On 16/07/11 01:51, Paul Eggert wrote: > On 07/15/11 03:28, Pádraig Brady wrote: >> What I was getting was that it's probably better to leave >> the following to the app too: >> >> #ifndef SA_RESETHAND >> # define SA_RESETHAND 0 >> /* Now the app writer knows they need to handle this case */ >> #end

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-19 Thread Joachim Schmitz
> -Original Message- > From: Paul Eggert [mailto:egg...@cs.ucla.edu] > Sent: Saturday, July 16, 2011 3:12 AM Sorry for the delay, got distracted with $DAYJOB > To: Pádraig Brady > Cc: Joachim Schmitz; 9...@debbugs.gnu.org > Subject: Re: bug#9076: coreutils-8.12 use

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-16 Thread Bastien ROUCARIES
2011/7/15 Eric Blake : > On 07/15/2011 04:28 AM, Pádraig Brady wrote: >> On 15/07/11 08:50, Paul Eggert wrote: >>> On 07/14/11 17:25, Pádraig Brady wrote: I'm not sure about defining these to 0 in gnulib. That will silently ignore the intent of a program on certain platforms. > >> Absolut

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-15 Thread Paul Eggert
Here are three proposed patches to coreutils to address the SA_RESETHAND and SA_RESTART issues. I hope that if you combine them with the patch I already installed: http://git.savannah.gnu.org/cgit/coreutils.git/commit/?id=837e1f55196f826b92d660808f594fde36651655 these should fix all the issues m

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-15 Thread Paul Eggert
On 07/15/11 03:28, Pádraig Brady wrote: > What I was getting was that it's probably better to leave > the following to the app too: > > #ifndef SA_RESETHAND > # define SA_RESETHAND 0 > /* Now the app writer knows they need to handle this case */ > #endif Yes, you're probably right. I'll send som

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-15 Thread Eric Blake
On 07/15/2011 04:28 AM, Pádraig Brady wrote: > On 15/07/11 08:50, Paul Eggert wrote: >> On 07/14/11 17:25, Pádraig Brady wrote: >>> I'm not sure about defining these to 0 in gnulib. >>> That will silently ignore the intent of a program on certain platforms. > Absolutely. What I was getting was tha

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-15 Thread Bruno Haible
Paul Eggert wrote: > --- a/doc/posix-functions/sigaction.texi > +++ b/doc/posix-functions/sigaction.texi A documentation update for the header file, in doc/posix-headers/signal.texi, would also be useful. Bruno -- In memoriam Natalya Estemirova

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-15 Thread Pádraig Brady
On 15/07/11 08:50, Paul Eggert wrote: > On 07/14/11 17:25, Pádraig Brady wrote: >> I'm not sure about defining these to 0 in gnulib. >> That will silently ignore the intent of a program on certain platforms. >> Wouldn't it be better to fail to compile so that each program then does: >> >> #ifndef S

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-15 Thread Paul Eggert
On 07/14/11 17:25, Pádraig Brady wrote: > I'm not sure about defining these to 0 in gnulib. > That will silently ignore the intent of a program on certain platforms. > Wouldn't it be better to fail to compile so that each program then does: > > #ifndef SA_RESETHAND > /* do something else */ > #end

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-14 Thread Pádraig Brady
On 14/07/11 23:33, Paul Eggert wrote: > [CC'ing bug-gnulib re .] > > On 07/13/11 23:54, Joachim Schmitz wrote: > >> coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally and on HP >> NonStop these don't exist. > > Thanks for reporting

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-14 Thread Paul Eggert
[CC'ing bug-gnulib re .] On 07/13/11 23:54, Joachim Schmitz wrote: > coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally and on HP > NonStop these don't exist. Thanks for reporting this. I think coreutils used to be portable to host

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-14 Thread Joachim Schmitz
From: Pádraig Brady [mailto:p...@draigbrady.com] Sent: Thursday, July 14, 2011 1:22 PM To: Joachim Schmitz Cc: 9...@debbugs.gnu.org Subject: Re: bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally On 14/07/11 07:54, Joachim Schmitz wrote: > Hi folks > > > >

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-14 Thread Pádraig Brady
On 14/07/11 07:54, Joachim Schmitz wrote: > Hi folks > > > > coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally and on HP > NonStop these don't exist. Ok we'll need to add the above info to: http://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=blob;f=doc/posix-functions/sigaction.te

bug#9076: coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally

2011-07-13 Thread Joachim Schmitz
Hi folks coreutils-8.12 uses SA_RESETHAND and SA_RESTART unconditionally and on HP NonStop these don't exist. SA_RESETHAND is used in csplit.c and dd.c SA_RESTART is used in ls.c and timeout.c Bye, Jojo