Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 717680 - qt5-build.eclass does not respect CXX (WAS: dev-qt/qtxml calls directly g++)
Summary: qt5-build.eclass does not respect CXX (WAS: dev-qt/qtxml calls directly g++)
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords:
: 725956 734860 (view as bug list)
Depends on:
Blocks: tc-directly
  Show dependency tree
 
Reported: 2020-04-16 11:19 UTC by Agostino Sarubbo
Modified: 2020-07-31 16:33 UTC (History)
4 users (show)

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


Attachments
build.log (build.log,10.55 KB, text/plain)
2020-04-16 11:20 UTC, Agostino Sarubbo
Details
qtbase-by-qt5-build.eclass.diff (qtbase-by-qt5-build.eclass.diff,2.97 KB, patch)
2020-06-03 18:06 UTC, Andreas Sturmlechner
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Agostino Sarubbo gentoo-dev 2020-04-16 11:19:55 UTC
This is an auto-filled bug because dev-qt/qtxml calls directly g++. Attached build log and emerge --info
Comment 1 Agostino Sarubbo gentoo-dev 2020-04-16 11:20:02 UTC
Created attachment 633174 [details]
build.log

build log and emerge --info
Comment 2 Agostino Sarubbo gentoo-dev 2020-04-20 10:20:19 UTC
this is confirmed for all dev-qt/qt* packages not just qtxml
Comment 3 Agostino Sarubbo gentoo-dev 2020-04-27 07:18:08 UTC
I guess it is an eclass bug.

in qt5-build_src_prepare() there is:
# Respect CC, CXX, *FLAGS, MAKEOPTS and EXTRA_EMAKE when bootstrapping qmake
                sed -i -e "/outpath\/qmake\".*\"\$MAKE\")/ s|)| \
                        ${MAKEOPTS} ${EXTRA_EMAKE} 'CC=$(tc-getCC)' 'CXX=$(tc-getCXX)' \
                        'QMAKE_CFLAGS=${CFLAGS}' 'QMAKE_CXXFLAGS=${CXXFLAGS}' 'QMAKE_LFLAGS=${LDFLAGS}'&|" \
                        -e 's/\(setBootstrapVariable\s\+\|EXTRA_C\(XX\)\?FLAGS=.*\)QMAKE_C\(XX\)\?FLAGS_\(DEBUG\|RELEASE\).*/:/' \
                        configure || die "sed failed (respect env for qmake build)"


But in the practice it does not work at least for CXX/g++
Comment 4 Davide Pesavento gentoo-dev 2020-05-29 06:24:23 UTC
*** Bug 725956 has been marked as a duplicate of this bug. ***
Comment 5 Andreas Sturmlechner gentoo-dev 2020-06-03 18:06:36 UTC
Created attachment 643294 [details, diff]
qtbase-by-qt5-build.eclass.diff

The above sed at least is still doing something in configure file - attaching diff that is the result of qt5-build_src_prepare on my system.

However, this one is a no-op:
> # Respect CXX in bsymbolic_functions, fvisibility, precomp, and a few other tests
> sed -i -e "/^QMAKE_CONF_COMPILER=/ s:=.*:=\"$(tc-getCXX)\":" \
> 	configure || die "sed failed (QMAKE_CONF_COMPILER)"

QMAKE_CONF_COMPILER var is gone since this commit:
https://github.com/qt/qtbase/commit/90eee08b3e672573e3ad93470d844e1d498651dd
Comment 6 Kent Fredric (IRC: kent\n) gentoo-dev 2020-06-11 02:27:48 UTC
(In reply to Agostino Sarubbo from comment #2)
> this is confirmed for all dev-qt/qt* packages not just qtxml

It affects far more than just that too, and is currently the leading cause of tc-direct based failures on my rig.

Samples include:
* app-editors/tea
* app-mobilephone/qtadb
* media-video/baka-mplayer
* app-text/qpdfview
Comment 7 Davide Pesavento gentoo-dev 2020-07-31 16:33:45 UTC
*** Bug 734860 has been marked as a duplicate of this bug. ***