Re: [PATCH 1/4] msi: use libvirt conditionally

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

 



On Wed, 2014-12-17 at 05:44 -0500, Marc-André Lureau wrote:
> 
> ----- Original Message -----
> > Only include libvirt as dep when it's explicitly done in the configure.
> > ---
> >  data/Makefile.am        | 1 +
> >  data/virt-viewer.wxs.in | 8 ++++++--
> >  2 files changed, 7 insertions(+), 2 deletions(-)
> > 
> > diff --git a/data/Makefile.am b/data/Makefile.am
> > index c80b970..e3d127e 100644
> > --- a/data/Makefile.am
> > +++ b/data/Makefile.am
> > @@ -40,6 +40,7 @@ virt-viewer-$(WIXL_ARCH)-$(VERSION).msi: virt-viewer.wxs
> > deps.txt
> >  	MANUFACTURER="$(MANUFACTURER)" wixl -D SourceDir=$(prefix)	\
> >               -D DESTDIR=$$DESTDIR$(prefix)				\
> >               -D GtkVersion=${GTK_API_VERSION}				\
> > +             -D HaveLibvirt=${HAVE_LIBVIRT}				\
> 
> 
> ack series
> 
> Btw, why do you use the ${} notation?

Hmmm. Good question, Marc-André. I could simply use $VAR instead of
${VAR}, for instance, but it's absolutely mechanical for me.

Do you have a preference for $VAR instead of ${VAR}?

Best Regards,
--
Fabiano Fidêncio

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list





[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux