[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: tests: temporary files may be left behind
From: |
Ralf Wildenhues |
Subject: |
Re: tests: temporary files may be left behind |
Date: |
Sat, 14 Nov 2009 13:19:18 +0100 |
User-agent: |
Mutt/1.5.20 (2009-08-09) |
Hi Jim,
* Jim Meyering wrote on Sat, Nov 14, 2009 at 01:11:38PM CET:
> I propose to fix it by adding a trap ... 0.
> A welcome side-effect is that with this change, you remove
> temporary files from only one place: the trap, rather
> than just prior to every other exit point.
>
> The only tricky part about the trap-0 is that you
> have to be sure to preserve the exit status.
Not only that, you also have to ensure to get the right exit status into
the zero trap, by using something like
(exit $N); exit $N
throughout, see '(autoconf.info)Limitations of Builtins'.
> Here's the patch for that particular file:
It is lacking a few of the above.
Cheers,
Ralf
- tests: temporary files may be left behind, Jim Meyering, 2009/11/14
- Re: tests: temporary files may be left behind,
Ralf Wildenhues <=
- Re: tests: temporary files may be left behind, Jim Meyering, 2009/11/14
- Re: tests: temporary files may be left behind, Ralf Wildenhues, 2009/11/14
- Re: tests: temporary files may be left behind, Jim Meyering, 2009/11/14
- Re: tests: temporary files may be left behind, Ralf Wildenhues, 2009/11/14
- Re: tests: temporary files may be left behind, Jim Meyering, 2009/11/14
- Re: tests: temporary files may be left behind, Eric Blake, 2009/11/14