Andreas Schwab wrote:
> Jim Meyering <[email protected]> writes:
>
>> Stefano Lattarini wrote:
>>> * tests/init.cfg (stty_reversible_init_): Quote '$abs_top_srcdir'
>>> properly.
>>> (fiemap_capable_): Quote '$abs_srcdir' properly.
>>> (require_dirent_d_type_): Likewise.
>>> ---
>>>  init.cfg | 6 +++---
>>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/init.cfg b/init.cfg
>>> index 7edfd0f..13cac74 100644
>>> --- a/init.cfg
>>> +++ b/init.cfg
>>> @@ -266,7 +266,7 @@ stty_reversible_init_()
>>>  {
>>>    # Pad start with one space for the first option to match in query 
>>> function.
>>>    stty_reversible_=' '$(perl -lne '/^ *{"(.*?)",.*\bREV\b/ and print $1' \
>>> -    $abs_top_srcdir/src/stty.c | tr '\n' ' ')
>>> +    "$abs_top_srcdir"/src/stty.c | tr '\n' ' ')
>> ...
>>
>> Thanks, but this c-set is unnecessary, since configure itself verifies
>> the sanity of $srcdir and $abs_top_srcdir:
>
> Except that $abs_top_srcdir may contain spaces.

So it does!  Good catch.
I'll reinsert that patch.  Thank you.

Reply via email to