Bug 432733

Summary: uses the wrong optimization flags on ppc
Product: [Fedora] Fedora Reporter: Michel Lind <michel>
Component: sonic-visualiserAssignee: Michel Lind <michel>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: 8CC: rdieter
Target Milestone: ---   
Target Release: ---   
Hardware: powerpc   
OS: Linux   
URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=425369
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-02-25 01:14:18 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Michel Lind 2008-02-14 02:22:40 UTC
Description of problem:
sonic-visualiser (in the Fedora packages CVS) uses qmake-qt4 to generate its
Makefile; the generated file is good for building on all platforms (including
ppc64) with the exception of ppc.

On ppc, for some reason the optimization flags for x86 is used

Version-Release number of selected component (if applicable):
The latest version as used by koji/mock

How reproducible:
Always

Steps to Reproduce:
1. checkout sonic-visualiser
2. make build

  
Actual results:
The ppc builder will fail

Expected results:
Should not fail: see URL included for the build result if ppc is ExcludeArch-ed.

Additional info:

Comment 1 Rex Dieter 2008-02-14 03:57:17 UTC
A scratch build including the failed ppc arch would be much more helpful.

Comment 2 Rex Dieter 2008-02-14 13:06:06 UTC
Tried a scratch build, it would appear rawhide has broken deps atm:
DEBUG util.py:261:  Error: Missing Dependency: xpeex-devel is needed by package
libfishsound-devel
http://koji.fedoraproject.org/koji/taskinfo?taskID=426336

Comment 3 Rex Dieter 2008-02-14 13:24:59 UTC
OK, tried again, building for dist-f8:
http://koji.fedoraproject.org/koji/taskinfo?taskID=426402

Comment 4 Rex Dieter 2008-02-14 13:26:06 UTC
No clue where these are coming from:
cc1plus: error: unrecognized command line option "-march=pentium3"
cc1plus: error: unrecognized command line option "-mfpmath=sse"

definitely aren't our default $RPM_OPT_FLAGS

Comment 5 Rex Dieter 2008-02-14 13:28:55 UTC
sonic-visualiser's bug, not ours:
sv.prf:linux-g++:QMAKE_CXXFLAGS_RELEASE += -DNDEBUG -DNO_TIMING -O2
-march=pentium3 -mfpmath=sse

omit/patch-out those arch-specific compiler optimizations, and you'll be better off.

Comment 6 Michel Lind 2008-02-15 04:02:38 UTC
Aha! Thanks.

Comment 7 Michel Lind 2008-02-25 01:14:18 UTC
Fixed in Rawhide; stable builds awaiting libfishsound fix entering the updates tree