Re: [PATCH] build: fix build --without-remote

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 28, 2011 at 09:31:50PM -0600, Eric Blake wrote:
> When configured --without-remote, the files remote_protocol.[ch]
> are not built by src/Makefile, but are still used by daemon/Makefile.
> The solution is to copy the implicit rule.
> 
> * daemon/Makefile.am (%protocol.c, %protocol.h): Copy from src.
> ---
> 
> Not quite trivial enough to claim the build-breaker rule.
> 
>  daemon/Makefile.am |    8 ++++++++
>  1 files changed, 8 insertions(+), 0 deletions(-)
> 
> diff --git a/daemon/Makefile.am b/daemon/Makefile.am
> index 8ed29b8..cb3caa2 100644
> --- a/daemon/Makefile.am
> +++ b/daemon/Makefile.am
> @@ -52,6 +52,14 @@ $(srcdir)/qemu_dispatch.h: $(srcdir)/../src/rpc/gendispatch.pl \
>  	$(AM_V_GEN)perl -w $(srcdir)/../src/rpc/gendispatch.pl -b qemu \
>  	  $(QEMU_PROTOCOL) > $@
> 
> +%protocol.c: %protocol.x %protocol.h $(srcdir)/../src/rpc/genprotocol.pl
> +	$(AM_V_GEN)perl -w $(srcdir)/../src/rpc/genprotocol.pl $(RPCGEN) -c \
> +	       $< $@
> +
> +%protocol.h: %protocol.x $(srcdir)/../src/rpc/genprotocol.pl
> +	$(AM_V_GEN)perl -w $(srcdir)/../src/rpc/genprotocol.pl $(RPCGEN) -h \
> +	       $< $@
> +
>  if WITH_LIBVIRTD
> 
>  man8_MANS = libvirtd.8

Does it make sense to allow building of the daemon, when the
client is disabled ?


I can see value in building client, but not daemon, since you
might want a remote only build. But I'm not sure I see any
point in the reverse. So we could make configure force the
daemon disabled, if the client is disabled instead. Or make
it raise an error if the user requests that combo

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]