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