Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 595574 - dev-qt/qtwebengine checks GCC version based on what dev-qt/qtcore was built with
Summary: dev-qt/qtwebengine checks GCC version based on what dev-qt/qtcore was built with
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Qt Bug Alias
URL:
Whiteboard:
Keywords:
: 596808 598292 603468 611066 (view as bug list)
Depends on: 649420
Blocks:
  Show dependency tree
 
Reported: 2016-09-29 19:30 UTC by Michael Palimaka (kensington)
Modified: 2022-03-18 03:33 UTC (History)
5 users (show)

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


Attachments
build.log (build.log,2.55 KB, text/plain)
2016-09-29 19:31 UTC, Michael Palimaka (kensington)
Details
qconfig.pri (qconfig.pri,1.30 KB, text/plain)
2016-09-29 19:31 UTC, Michael Palimaka (kensington)
Details
bogus_gcc_check.patch (bogus_gcc_check.patch,631 bytes, patch)
2017-03-02 00:38 UTC, cyrillic
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Palimaka (kensington) gentoo-dev 2016-09-29 19:30:39 UTC
From qtwebengine/tools/qmake/mkspecs/features/functions.prf:

> defineTest(isGCCVersionSupported) {
>   # The below will work for gcc 4.7 and up and also match gcc 5
>   greaterThan(QT_GCC_MINOR_VERSION, 6):return(true)
>   greaterThan(QT_GCC_MAJOR_VERSION, 4):return(true)
>   skipBuild("Using gcc version "$$QT_GCC_MAJOR_VERSION"."$$QT_GCC_MINOR_VERSION", but at least gcc version 4.7 is required to build Qt WebEngine.")
>   return(false)
> }

It appears the value of QT_GCC_*_VERSION is whatever was used to build qtcore and is sourced from /usr/lib64/qt5/mkspecs/qconfig.pri.

Apart from the obvious potential for version mismatches, QT_GCC_*_VERSION will not even be defined at all if qtcore was built with clang.

Further, if this magic GCC check fails, build silently fails in such a way that portage does not detect so the user ends up with a zero-file qtwebengine installed.
Comment 1 Michael Palimaka (kensington) gentoo-dev 2016-09-29 19:31:07 UTC
Created attachment 448470 [details]
build.log
Comment 2 Michael Palimaka (kensington) gentoo-dev 2016-09-29 19:31:59 UTC
Created attachment 448472 [details]
qconfig.pri
Comment 3 Michael Palimaka (kensington) gentoo-dev 2016-10-28 11:51:10 UTC
*** Bug 596808 has been marked as a duplicate of this bug. ***
Comment 4 Sergey 'L29Ah' Alirzaev 2016-10-28 18:24:28 UTC
*** Bug 598292 has been marked as a duplicate of this bug. ***
Comment 5 Michael Palimaka (kensington) gentoo-dev 2016-12-22 14:33:53 UTC
*** Bug 603468 has been marked as a duplicate of this bug. ***
Comment 6 Michael Palimaka (kensington) gentoo-dev 2017-02-27 09:55:44 UTC
*** Bug 611066 has been marked as a duplicate of this bug. ***
Comment 7 cyrillic 2017-03-02 00:38:46 UTC
Created attachment 465680 [details, diff]
bogus_gcc_check.patch

Yeah, I normally compile dev-qt/* using clang, so this GCC check was really not helpful.  qtwebengine has other issues that prevent it from compiling with clang, but it does work with gcc-6.3 now.