Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 275710 - x11-libs/qt*-4.5.1 ignores CC/CXX
Summary: x11-libs/qt*-4.5.1 ignores CC/CXX
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard: [qting-edge]
Keywords: InOverlay
: 282984 (view as bug list)
Depends on:
Blocks: qt46
  Show dependency tree
 
Reported: 2009-06-28 16:43 UTC by Jeroen Roovers (RETIRED)
Modified: 2009-12-17 18:07 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
ps waux | grep [g]++ (qt-core.pswaux.grep.gpp,15.21 KB, text/plain)
2009-06-28 16:43 UTC, Jeroen Roovers (RETIRED)
Details
respect CC/CXXflags (qt4-build.patch,409 bytes, patch)
2009-07-02 14:47 UTC, Markos Chandras (RETIRED)
Details | Diff
respect CC, CXX, LINK (qt4-build.eclass.patch,903 bytes, patch)
2009-07-31 22:49 UTC, Davide Pesavento (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jeroen Roovers (RETIRED) gentoo-dev 2009-06-28 16:43:07 UTC
distcc cross compiling fails because the build system calls g++ directly. It also appears to add -pipe all by itself, which could be a problem for systems with little RAM. Leaving Severity at "normal" instead of setting "QA" because this makes compilation slow, which is particularly bad because of the size of the packages and the already long compile times. 

elmer ~ # ps waux|grep [h]ppa
root     28046  0.8  0.0   4680  1188 pts/3    SN+  18:37   0:00 /usr/hppa2.0-unknown-linux-gnu/gcc-bin/4.4.0/g++ -c -o qscriptvalue.o -O2 -pipe -mschedule=8000 -march=2.0 -g -ggdb -Wall -pipe -DQMAKE_OPENSOURCE_EDITION -I. -Igenerators -Igenerators/unix -Igenerators/win32 -Igenerators/mac -I/mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/include -I/mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/include/QtCore -I/mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/src/corelib/global -I/mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/src/script -DQT_NO_PCRE -DQT_BUILD_QMAKE -DQT_BOOTSTRAPPED -DQT_NO_TEXTCODEC -DQT_NO_UNICODETABLES -DQT_NO_COMPONENT -DQT_NO_STL -DQT_NO_COMPRESS -I/mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/mkspecs/linux-g++ -DHAVE_QCONFIG_CPP -DQT_NO_THREAD -DQT_NO_QOBJECT -DQT_NO_GEOM_VARIANT /mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/src/script/qscriptvalue.cpp
root     28047 95.7  2.3  55664 49440 pts/3    RN+  18:37   0:06 /usr/libexec/gcc/hppa2.0-unknown-linux-gnu/4.4.0/cc1plus -quiet -I. -Igenerators -Igenerators/unix -Igenerators/win32 -Igenerators/mac -I/mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/include -I/mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/include/QtCore -I/mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/src/corelib/global -I/mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/src/script -I/mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/mkspecs/linux-g++ -D_GNU_SOURCE -DQMAKE_OPENSOURCE_EDITION -DQT_NO_PCRE -DQT_BUILD_QMAKE -DQT_BOOTSTRAPPED -DQT_NO_TEXTCODEC -DQT_NO_UNICODETABLES -DQT_NO_COMPONENT -DQT_NO_STL -DQT_NO_COMPRESS -DHAVE_QCONFIG_CPP -DQT_NO_THREAD -DQT_NO_QOBJECT -DQT_NO_GEOM_VARIANT /mnt/alt/portage/x11-libs/qt-core-4.5.1/work/qt-x11-opensource-src-4.5.1/src/script/qscriptvalue.cpp -D_FORTIFY_SOURCE=2 -quiet -dumpbase qscriptvalue.cpp -mschedule=8000 -march=2.0 -auxbase-strip qscriptvalue.o -g -ggdb -O2 -Wall -o -
root     28048  1.1  0.1   5612  3688 pts/3    SN+  18:37   0:00 /usr/lib/gcc/hppa2.0-unknown-linux-gnu/4.4.0/../../../../hppa2.0-unknown-linux-gnu/bin/as -o qscriptvalue.o -
Comment 1 Jeroen Roovers (RETIRED) gentoo-dev 2009-06-28 16:43:56 UTC
Created attachment 195957 [details]
ps waux | grep [g]++
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2009-06-28 16:46:16 UTC
Oh, wait, it inserts CXXFLAGS twice...
Comment 3 Davide Pesavento (RETIRED) gentoo-dev 2009-06-28 17:40:29 UTC
I guess that inside build_directories() in qt4-build.eclass, qmake could be called with more arguments, for example QMAKE_CC, etc... have a look at what eqmake4 does.
Comment 4 Markos Chandras (RETIRED) gentoo-dev 2009-06-30 19:56:31 UTC
Håvard commited a patch on qt4-build-edge eclass. 

@Qt members, if you use -9999 qt-modules, please try to rebuild them and verify that this patch fixes this issue

Thanks :)
Comment 5 Markos Chandras (RETIRED) gentoo-dev 2009-07-02 14:07:55 UTC
Ok as I understand there is nothing wrong with CC,CXX and cross-compilation. At least it works for me :)

I am adjusting the title to be more specific about the actual error
Comment 6 Jeroen Roovers (RETIRED) gentoo-dev 2009-07-02 14:38:46 UTC
No, sorry. It doesn't use hppa2.0-unknown-linux-gnu-g++, but g++ instead. The C*FLAGS duplication is a minor problem.
Comment 7 Markos Chandras (RETIRED) gentoo-dev 2009-07-02 14:47:52 UTC
Created attachment 196372 [details, diff]
respect CC/CXXflags

Does the attached patch solves your issue?
Comment 8 Davide Pesavento (RETIRED) gentoo-dev 2009-07-31 20:23:19 UTC
jer, any updates?
Comment 9 Davide Pesavento (RETIRED) gentoo-dev 2009-07-31 22:49:34 UTC
Created attachment 199765 [details, diff]
respect CC, CXX, LINK

This patch implements a better approach: the relevant variables are passed to qmake, rather than being changed when emake is called.

I just committed a similar patch to qting-edge overlay:
http://github.com/gentoo-qt/qting-edge/commit/62d25aa94f9bd878717032f918a3c6d4766b520e
Comment 10 Markos Chandras (RETIRED) gentoo-dev 2009-08-01 09:13:51 UTC
(In reply to comment #9)
> Created an attachment (id=199765) [edit]
> respect CC, CXX, LINK
> 
> This patch implements a better approach: the relevant variables are passed to
> qmake, rather than being changed when emake is called.
> 
> I just committed a similar patch to qting-edge overlay:
> http://github.com/gentoo-qt/qting-edge/commit/62d25aa94f9bd878717032f918a3c6d4766b520e
> 

Indeed this patch seems much more better than the current approach. I will test it with live ebuilds

Thanks Davide
Comment 11 Davide Pesavento (RETIRED) gentoo-dev 2009-08-04 01:38:25 UTC
Unfortunately my patch seems to introduce many regressions, please ignore it.
Comment 12 Dominik Kapusta (RETIRED) gentoo-dev 2009-11-09 19:43:20 UTC
jer, can you please take a look at it now?

I applied some patches in the meantime and now I'm able to cross-compile successfully from amd64 to x86, so I guess that your issue is fixed as well. 

Thanks much.
Comment 13 Jeroen Roovers (RETIRED) gentoo-dev 2009-11-09 19:46:42 UTC
(In reply to comment #12)
> jer, can you please take a look at it now?
> 
> I applied some patches in the meantime and now I'm able to cross-compile
> successfully from amd64 to x86, so I guess that your issue is fixed as well. 

To which repo did you apply those patches?
Comment 14 Dominik Kapusta (RETIRED) gentoo-dev 2009-11-09 19:57:43 UTC
Ah, sorry, to gentoo-x86 :)
Comment 15 Ben de Groot (RETIRED) gentoo-dev 2009-12-17 15:49:25 UTC
*** Bug 282984 has been marked as a duplicate of this bug. ***
Comment 16 Dominik Kapusta (RETIRED) gentoo-dev 2009-12-17 18:07:25 UTC
Jeroen, ping?

I'm marking this as fixed, based on comment #12 and no issues being reported since I pushed my fix. Reopen if the compilation still fails for you.