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
