Re: Disabling build options for SSE on non-capable arches ?

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

 



On Thu, 2018-09-20 at 07:58 +1000, David Timms wrote:
> Hi, (this is a 4th retry, now trying devel as first try to packaging a
> week ago says held by moderator, but doesn't seem to have been posted to
> the list, and I don't know why...)
> 
> Audacity has had the following as part of rpm spec in the %configure
> section:
> 
> %configure \
>     --disable-dynamic-loading \
> ...
> %ifnarch %{ix86} x86_64
>     --disable-sse \
> %else
>     %{nil}
> %endif
> 
> Which fails on: non x86 arches:
> https://koji.fedoraproject.org/koji/taskinfo?taskID=29513664
> 
> If I remove the conditional, I get successful build, but not optimized
> on x86 with SSE.
> 
> What's wrong with this conditional ?

I think sticking a conditional in the middle of a line-wrapped command
like that may not work.

I'd try instead creating a variable for the args first, something like
this:

%ifnarch %{ix86} x86_64
    %global sseargs --disable-sse
%else
    %global sseargs

...

%configure %{sseargs} \
...
-- 
Adam Williamson
Fedora QA Community Monkey
IRC: adamw | Twitter: AdamW_Fedora | XMPP: adamw AT happyassassin . net
http://www.happyassassin.net
_______________________________________________
devel mailing list -- devel@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://getfedora.org/code-of-conduct.html
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/devel@xxxxxxxxxxxxxxxxxxxxxxx




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Fedora Announce]     [Fedora Users]     [Fedora Kernel]     [Fedora Testing]     [Fedora Formulas]     [Fedora PHP Devel]     [Kernel Development]     [Fedora Legacy]     [Fedora Maintainers]     [Fedora Desktop]     [PAM]     [Red Hat Development]     [Gimp]     [Yosemite News]

  Powered by Linux