[libvirt] [PATCH] spec: Consolidate with_qemu* definitions

[Jiri Denemark]

On Thu, Jul 17, 2014 at 09:12:31 -0600, Eric Blake wrote:
> On 07/17/2014 08:22 AM, Jiri Denemark wrote:
> > Decisions whether qemu driver and libvirt-daemon-{qemu,kvm} packages
> > should be built on various OS/arch combinations were scattered around
> > the spec file. Let's make it easier to see where qemu driver is going to
> > be built.
> > 
> > Signed-off-by: Jiri Denemark <jdenemar at redhat.com>
> > ---
> >  libvirt.spec.in | 52 +++++++++++++++++++++++++---------------------------
> >  1 file changed, 25 insertions(+), 27 deletions(-)
> > 
> > diff --git a/libvirt.spec.in b/libvirt.spec.in
> > index 9c7b241..472fa4b 100644
> > --- a/libvirt.spec.in
> > +++ b/libvirt.spec.in
> > @@ -54,15 +54,27 @@
> >  %define with_vbox          0%{!?_without_vbox:%{server_drivers}}
> >  
> >  %define with_qemu_tcg      %{with_qemu}
> > -# Change if we ever provide qemu-kvm binaries on non-x86 hosts
> > -%if 0%{?fedora} >= 18
> > +
> > +%define qemu_kvm_arches %{ix86} x86_64
> > +
> > +%if 0%{?fedora}
> > +    %if 0%{?fedora} < 16
> > +        # Fedora doesn't have any QEMU on ppc64 until FC16 - only ppc
> > +        %ifarch ppc64
> > +            %define with_qemu_tcg 0
> > +        %endif
> > +    %endif
> 
> This may be removed if we come to a conclusion in the other thread about
> how much we want to support older Fedora.  But we can clean it up later,
> this patch is just moving it.
> 
> ACK.

Pushed, thanks.

Jirka