[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] PATH: use double quote
From: |
Eric Blake |
Subject: |
Re: [PATCH] PATH: use double quote |
Date: |
Mon, 22 Sep 2014 21:32:44 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.1.0 |
On 09/22/2014 08:06 PM, KO Myung-Hun wrote:
>
>> Why are you changing this? I see no difference in the resulting
>> shell syntax, except that backslashes in $(abs_builddir) are now
>> handled differently. You need a lot more justification for WHY
>> this patch is necessary.
>
>
> On OS/2, PATH_SEPARATOR is ';'. If is is not quoted, ';' works as a
> mark of end of sentence. And how about this for $(abs_builddir) ? At
> least, it works here.
>
> - { PATH='$(abs_builddir)/t/wrap'$(PATH_SEPARATOR)$$PATH && export
> PATH; }
> + { PATH='$(abs_builddir)/t/wrap'"$(PATH_SEPARATOR)$$PATH" && export
> PATH; }
Ah, that's the sentence you were missing from your proposed change.
Note that $$PATH doesn't need to be quoted, but the makefile expansion
of $(PATH_SEPARATOR) does; but it would also be sufficient to use:
{ PATH='$(abs_builddir)/t/wrap$(PATH_SEPARATOR)'$$PATH && export PATH; }
if we're trying to go for the shortest line.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature