[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/2] config.status: minimize use of $tmp
From: |
Paul Eggert |
Subject: |
Re: [PATCH 2/2] config.status: minimize use of $tmp |
Date: |
Mon, 30 Aug 2010 11:29:25 -0700 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.11) Gecko/20100713 Thunderbird/3.0.6 |
On 08/30/10 09:41, Eric Blake wrote:
> - { test ! -d "${ac_tmp:=$tmp}" || rm -fr "$ac_tmp"; } && exit $exit_status
>
> + : "${ac_tmp:=$tmp}"
> + { test ! -d "$ac_tmp" || rm -fr "$ac_tmp"; } && exit $exit_status
I agree with Ralf on this point. That is,
I don't know of any shells where this change is needed, and
in POSIX-conforming shells this change is not needed. I wouldn't
bother with changes like this unless there is a known and
significant shell with the bug in question.
- Re: [PATCHv2] docs: mention another issue with variable expansion, (continued)
- Re: [PATCHv2] docs: mention another issue with variable expansion, Eric Blake, 2010/08/27
- [PATCH 0/2] cleanup $tmp usage, Eric Blake, 2010/08/27
- Re: [PATCH 0/2] cleanup $tmp usage, Ralf Wildenhues, 2010/08/27
- Re: [PATCH 0/2] cleanup $tmp usage, Eric Blake, 2010/08/30
- Re: [PATCH 0/2] cleanup $tmp usage, Ralf Wildenhues, 2010/08/30
- Re: [PATCH 0/2] cleanup $tmp usage, Eric Blake, 2010/08/30
- [PATCH 2/2] config.status: minimize use of $tmp, Eric Blake, 2010/08/27
- Re: [PATCH 2/2] config.status: minimize use of $tmp, Eric Blake, 2010/08/30
- Re: [PATCH 2/2] config.status: minimize use of $tmp, Ralf Wildenhues, 2010/08/30
- Re: [PATCH 2/2] config.status: minimize use of $tmp, Eric Blake, 2010/08/30
- Re: [PATCH 2/2] config.status: minimize use of $tmp,
Paul Eggert <=
- [PATCH 1/2] docs: avoid use of $tmp outside of config.status use, Eric Blake, 2010/08/27