Bug 432733 - uses the wrong optimization flags on ppc
Summary: uses the wrong optimization flags on ppc
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: sonic-visualiser
Version: 8
Hardware: powerpc
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Michel Lind
QA Contact: Fedora Extras Quality Assurance
URL: http://koji.fedoraproject.org/koji/ta...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-02-14 02:22 UTC by Michel Lind
Modified: 2008-02-25 01:14 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-02-25 01:14:18 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

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


Note You need to log in before you can comment on or make changes to this bug.