Re: [PATCH 1/2] spec: Use power64 macro

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

 



On 07/18/2014 11:28 AM, Cole Robinson wrote:
> From: Peter Robinson <pbrobinson@xxxxxxxxxxxxxxxxx>
> 
> Covers the whole ppc64 family. Example bug for binutils:
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=834651
> ---
>  libvirt.spec.in | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)

> 
> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index 76e57aa..cac7d99 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -60,15 +60,16 @@
>  %if 0%{?fedora}
>      %if 0%{?fedora} < 16
>          # Fedora doesn't have any QEMU on ppc64 until FC16 - only ppc
> +        # I think F17 is the first release with the power64 macro
>          %ifarch ppc64
>              %define with_qemu_tcg 0
>          %endif

Of course, we may delete this hunk in a later patch, but it works for
now. I did not try to fire up my F16 VM to run 'make dist' - we may have
bit-rotted for that build in the meantime, but it doesn't stop my from
saying:

ACK.


-- 
Eric Blake   eblake redhat com    +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

[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]