[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Fix testsuite when `pwd` contains spaces

From: Ralf Wildenhues
Subject: Re: Fix testsuite when `pwd` contains spaces
Date: Tue, 25 Mar 2008 20:46:43 +0100
User-agent: Mutt/1.5.17+20080114 (2008-01-14)

Hi Eric,

* Eric Blake wrote on Tue, Mar 25, 2008 at 01:02:38PM CET:
> According to Ralf Wildenhues on 3/25/2008 12:41 AM:
> | Applied as obvious.
> |
> | -abs_builddir=`pwd`
> | +abs_builddir='`pwd`'
> Huh?  Variable assignment does not need quotes if the syntax forms a
> single shell word, since it does not undergo word splitting.  Or did you
> really mean to add single quotes rather than double quotes to defer
> evaluation?

Hmm, maybe it wasn't as obvious as I thought  ;-)

The changes take place inside a here document.  If you meant for 
the command substitution to take place at micro-suite run time only,
then alternatively, the here-document (or the backticks) could be
quote-escaped.  I don't care, whatever you prefer.  I just used
what matched atconfig: expand `pwd` at atconfig creation time.
I don't care much that it may fail if `pwd` contains a single quote.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]