[libvirt] [PATCH2 2/1] build: fix testing of augeas files in VPATH builds

Eric Blake eblake at redhat.com
Wed May 30 15:34:53 UTC 2012


On 05/30/2012 08:39 AM, Martin Kletzander wrote:
> On 05/30/2012 12:03 AM, Eric Blake wrote:
>> Without this fix, a VPATH build (such as used by ./autobuild.sh)
>> fails with messages like:
>>
>> make[3]: Entering directory `/home/remote/eblake/libvirt-tmp2/build/daemon'
>> ../../build-aux/augeas-gentest.pl libvirtd.conf ../../daemon/test_libvirtd.aug.in test_libvirtd.aug
>> cannot read libvirtd.conf: No such file or directory at ../../build-aux/augeas-gentest.pl line 38.
>>

>> -check-augeas-sanlock: locking/test_libvirt_sanlock.aug
>> +check-augeas-sanlock: test_libvirt_sanlock.aug
>>  	$(AM_V_GEN)if test -x '$(AUGPARSE)'; then \
>> -	    '$(AUGPARSE)' -I $(srcdir)/locking \
>> -	    $(srcdir)/locking/test_libvirt_sanlock.aug; \
>> +	    '$(AUGPARSE)' -I $(srcdir)/locking test_libvirt_sanlock.aug; \
>>  	fi
>>
>>  #
> 
> I'm not sure why you use "&& \" to join the lines when there is nothing
> to share between the processes, but it doesn't hurt. Otherwise it looks
> like nice cleanup, so ACK from me (if that's enough for you =) ).

v1 used '&& \' because it did a mkdir in the first command, and the
second command would fail if the first didn't succeed.  Using '; \' to
join lines ignores earlier failures.

v2 doesn't use '&& \' for any line joining, so I'm not quite sure where
your comment came from.

At any rate, I've now pushed the series, with the configure.ac change
floated into the second patch per your request.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120530/009d3fd0/attachment-0001.sig>


More information about the libvir-list mailing list