Re: c-stack: use libsigsegv when possible

2008-07-20 Thread Bruno Haible
> Yes, with one nit: Applied, with this nit fixed. Bruno

Re: c-stack: use libsigsegv when possible

2008-07-20 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Bruno Haible on 7/20/2008 11:21 AM: | | OK to apply? (Note: One needs the '(exit 77); exit 77' idiom only when | a trap handler is installed which refers to $?. In most gnulib tests it | is not needed.) Yes, with one nit: | 77) if g

Re: c-stack: use libsigsegv when possible

2008-07-20 Thread Bruno Haible
Eric Blake wrote: > this is what exposed the libsigsegv on OpenBSD 4.0 Currently the test-c-stack2.sh gets a SKIP with a confusing error message: it says "consider installing libsigsegv" although libsigsegv is already being used. You noticed that it's a bug in libsigsegv; normal testers may overlo

Re: c-stack: use libsigsegv when possible

2008-07-17 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Eric Blake on 7/16/2008 9:11 PM: | According to Bruno Haible on 7/16/2008 7:29 PM: | | If you want to save linking with libsigsegv, just do the test | | '#if HAVE_LIBSIGSEGV && ! HAVE_XSI_STACK_OVERFLOW_HEURISTIC' at configure | | time ra

Re: c-stack: use libsigsegv when possible

2008-07-16 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Bruno Haible on 7/16/2008 7:29 PM: | If you want to save linking with libsigsegv, just do the test | '#if HAVE_LIBSIGSEGV && ! HAVE_XSI_STACK_OVERFLOW_HEURISTIC' at configure | time rather than at compile-time, create two new variables LI

Re: c-stack: use libsigsegv when possible

2008-07-16 Thread Bruno Haible
Hi Eric, Thanks for this patch. > When compiling for platforms like Solaris where c-stack has determined > HAVE_XSI_STACK_OVERFLOW_HEURISTIC, the use of libsigsegv just bloats the > application, since c-stack was already sufficient. Agreed. > Should I also add lib-ignore to the > dependencies o

Re: c-stack: use libsigsegv when possible

2008-07-16 Thread Paul Eggert
Eric Blake <[EMAIL PROTECTED]> writes: > Should I also add lib-ignore to the > dependencies of c-stack, so that Solaris need not waste space if > libsigsegv is not used independently of c-stack? Yes, please. > In writing the patch, I noticed that it is probably not portable to > longjmp out of a

c-stack: use libsigsegv when possible

2008-07-16 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Is this patch okay to apply? So far, I've tested it on: cygwin with libsigsegv mingw without libsigsegv mingw with libsigsegv Linux without libsigsegv Linux with libsigsegv When compiling for platforms like Solaris where c-stack has determined HAVE