On 2012-10-19 07:39, Pavel Raiskup wrote:
> Hi,
> 
> On Thu, 2012-10-18 at 22:40 +0200, Stefano Lattarini wrote:
>> * lib/depcomp (pgcc): Here.  Probably not needed, but since the
>> rest of the script seems to employ proper quoting, better to be
>> consistent.
>>
>> Signed-off-by: Stefano Lattarini <stefano.lattar...@gmail.com>
>> ---
>>  lib/depcomp | 8 ++++----
>>  1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/lib/depcomp b/lib/depcomp
>> index dbcacd0..2f0c6c5 100755
>> --- a/lib/depcomp
>> +++ b/lib/depcomp
>> @@ -358,24 +358,24 @@ pgcc)
>>    lockdir=$base.d-lock
>>    trap "
>>      echo '$0: caught signal, cleaning up...' >&2
>> -    rmdir $lockdir
>> +    rmdir '$lockdir'
> 
> here seems to be problem,
No, because the whole thing is inside double quotes, like this...

$ bash
$ lockdir="foo   foo"
$ trap "echo '$lockdir'" 0
$ exit
exit
foo   foo
$ 

...which looks much better than the preexisting version...

$ bash
$ lockdir="foo   foo"
$ trap "echo $lockdir" 0
$ exit
exit
foo foo
$ 

...but '$lockdir' sure looks odd, maybe this pattern is better?

$ bash
$ lockdir="foo   foo"
$ trap "echo \"$lockdir\""
$ exit
exit
foo   foo
$ 

Admittedly it requires escapes, but it doesn't look as
odd on first glance and therefore isn't as susceptible to
some later unwelcome "fixups" (classified as trivial).

On the other hand, there's the preexisting $0 also inside
single in that very same trap. All in all, I'm happy with
this patch as is. It's a definite improvement.

Cheers,
Peter


Reply via email to