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(a)redhat.com +1-919-301-3266
Libvirt virtualization library
http://libvirt.org