Re: Bug in latest install-sh

2005-11-08 Thread Paul Eggert
Stepan Kasal <[EMAIL PROTECTED]> writes: >> Both do, but trap '' 0 seems to be the most portable. > > And it is also the clener one. > Proposed patch attached. Thanks. I installed that (replacing "trap - 0" with "trap '' 0" in install-sh) into Automake's main line. Perhaps it should be put into

Re: Bug in latest install-sh

2005-11-02 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Stepan Kasal on 10/31/2005 11:14 PM: > > If "trap - 0" doesn't work on your system, would "trap '' 0" work? > Or perhaps "trap : 0"? Be aware that in general, replacing - with '' or : is wrong: POSIX requires "trap '' 0" to ignore the si

Re: Bug in latest install-sh

2005-11-01 Thread Kean Johnston
Stepan Kasal wrote: Hello, thank you for your bug report. On Mon, Oct 31, 2005 at 04:21:31PM -0800, Kean Johnston wrote: There is a bug in teh latest install-sh that is in m4 1.4.4. This is the latest install-sh in the Automake CVS. So this bug report belongs to bug-automake, where I'm fo

Re: Bug in latest install-sh

2005-11-01 Thread Gary V. Vaughan
Hallo Stepan! Forgive the nit... Stepan Kasal wrote: 2005-11-01 Stepan Kasal <[EMAIL PROTECTED]> * doc/autoconf.texi (Limitations of Builtins): Document that "trap - SIG" is not yet portable. [[snip]] +But even though Posix defines that the trap can be unset by [EMAIL PRO

Re: Bug in latest install-sh

2005-11-01 Thread Stepan Kasal
Hello, On Mon, Oct 31, 2005 at 11:53:23PM -0800, Kean Johnston wrote: > >I looked at POSIX, and 'trap - 0' is standardized, while 'trap 0' is not. > Ok thats a fair comment. But then if you want to use POSIX > shell scripts shouldn't you ensure you are running in a > POSIX shell? I agree that th

Re: Bug in latest install-sh

2005-10-31 Thread Stepan Kasal
Hello, thank you for your bug report. On Mon, Oct 31, 2005 at 04:21:31PM -0800, Kean Johnston wrote: > There is a bug in teh latest install-sh that is in m4 1.4.4. This is the latest install-sh in the Automake CVS. So this bug report belongs to bug-automake, where I'm forwarding it. > It uses