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@xxxxxxxxxx +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list