Arch Linux

Please read this before reporting a bug:
https://wiki.archlinux.org/title/Bug_reporting_guidelines

Do NOT report bugs when a package is just outdated, or it is in the AUR. Use the 'flag out of date' link on the package page, or the Mailing List.

REPEAT: Do NOT report bugs for outdated packages!
Tasklist

FS#13731 - [qt-4.5.0-2] Fix QMAKE CXX flags

Attached to Project: Arch Linux
Opened by Gerardo Exequiel Pozzi (djgera) - Tuesday, 10 March 2009, 05:27 GMT
Last edited by Pierre Schmitz (Pierre) - Wednesday, 18 March 2009, 06:54 GMT
Task Type Bug Report
Category Packages: Testing
Status Closed
Assigned To Pierre Schmitz (Pierre)
Andrea Scarpino (BaSh)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

The /usr/share/qt/mkspecs/common/g++.conf isn't modified in the PKGBUILD to set the correct QMAKE_CFLAGS_RELEASE

Please add a line like in PKGBUILD for qt-4.4.N to fix this

sed -i "s|-O2|$CXXFLAGS|" mkspecs/common/g++.conf
This task depends upon

Closed by  Pierre Schmitz (Pierre)
Wednesday, 18 March 2009, 06:54 GMT
Reason for closing:  Fixed
Comment by Pierre Schmitz (Pierre) - Tuesday, 10 March 2009, 11:30 GMT
Maybe I don't get it right, but what would be changed by this? The flags would be added only at buildtime of Qt; so no changes you made to CXXFLAGS will be taken into account for future builds of Qt apps.
Comment by Gerardo Exequiel Pozzi (djgera) - Tuesday, 10 March 2009, 15:26 GMT
Building QT-apps without this change, all are build without $CXXFLAGS. Then all PKGCONFIG that uses qmake, need to change to qmake QMAKE_CXXFLAGS="$CXXFLAGS" ...

If this new behavior is the expected, then need to notice about it. This change was done in the current QT3 and QT4 package.
Comment by Pierre Schmitz (Pierre) - Wednesday, 11 March 2009, 14:40 GMT
Why should this be needed? The g++.conf file in /usr/share/qt/mkspecs/common allready contains: "QMAKE_CFLAGS_RELEASE += $$CXXFLAGS"
Comment by Gerardo Exequiel Pozzi (djgera) - Wednesday, 11 March 2009, 16:54 GMT
Yes in extra/qt-4.4.3, but here, is about testing/qt-4.5.0-2

$ pacman -Q qt
qt 4.5.0-2
$ grep RELEASE /usr/share/qt/mkspecs/common/g++.conf
QMAKE_CFLAGS_RELEASE += -O2
QMAKE_CXXFLAGS_RELEASE += $$QMAKE_CFLAGS_RELEASE
QMAKE_LFLAGS_RELEASE += -Wl,-O1
Comment by Pierre Schmitz (Pierre) - Wednesday, 11 March 2009, 18:04 GMT
Ah, I see it now. Somehow my local Qt was right, while the package was not.

Loading...