pacakge/qt5/qt5base: fix build with ccache

PATCH Superseded Benoît Thébaudeau 2 participants 5 messages Aug 28, 2015

lore

Share & tools

lore

Benoît Thébaudeau

author

2015-08-28 09:46 UTC #0 lore raw

PATCH v2 1/1 pacakge/qt5/qt5base: fix build with ccache

Building with ccache failed with:

Running configuration tests...
Failed to process makespec for platform 'devices/linux-buildroot-g++'
Project ERROR: Compiler <path_to_output_dir>/host/usr/bin/ccache <path_to_output_dir>/host/usr/bin/<cross_compile>-g++ not found. Check the value of CROSS_COMPILE -device-option Could not read qmake configuration file <path_to_output_dir>/build/qt5base-5.5.0/mkspecs/devices/linux-buildroot-g++/qmake.conf.
Error processing project file: /dev/null

This was caused by Buildroot setting this in qt5base-5.5.0/mkspecs/devices/linux-buildroot-g++/qmake.conf:

    QMAKE_CXX               = $${BR_CCACHE} $${CROSS_COMPILE}g++

But qt5base-5.5.0/mkspecs/features/device_config.prf expects QMAKE_CXX to be a single valid (absolute or QMAKE_PATH_ENV-relative) path to an existing file, which is not possible if using ccache as above.

Add a patch fixing this by testing only the first value in QMAKE_CXX.

Signed-off-by Beno?t Th?baudeau <[email protected]>

---
Changes v1 -> v2:
- Use the first value in QMAKE_CXX instead of removing the whole test.

---
 .../qt5/qt5base/0009-fix-build-with-ccache.patch   | 49 ++++++++++++++++++++++
 1 file changed, 49 insertions(+)
 create mode 100644 package/qt5/qt5base/0009-fix-build-with-ccache.patch

package/qt5/qt5base/0009-fix-build-with-ccache.patch new file +49 -0

new file mode 100644
index 0000000..896c0ef
@@ -0,0 +1,49 @@
+Fix QMAKE_CXX/CROSS_COMPILE verification with ccache
+
+The use of ccache leads to QMAKE_CXX definitions of the form:
+
+    QMAKE_CXX = $${CCACHE} $${CROSS_COMPILE}g++
+
+The previous test required QMAKE_CXX to be a single valid (absolute or
+QMAKE_PATH_ENV-relative) path to an existing file, which was not
+compatible with definitions of QMAKE_CXX like the one above.
+
+Fix this by using only the first value in QMAKE_CXX, which usually
+points to the compiler executable, or to the ccache executable in the
+above case.
+
+Signed-off-by: Beno?t Th?baudeau <[email protected]>
+---
+ mkspecs/features/device_config.prf | 9 +++++++--
+ 1 file changed, 7 insertions(+), 2 deletions(-)
+
+diff --git a/mkspecs/features/device_config.prf b/mkspecs/features/device_config.prf
+index cd3a0cf..eee4ac6 100644
+--- a/mkspecs/features/device_config.prf
++++ b/mkspecs/features/device_config.prf
+@@ -19,10 +19,15 @@ defineTest(deviceSanityCheckCompiler) {
+     else: \
+         sfx =
+ 
++    # Build the compiler filename using the first value in QMAKE_CXX in order to
++    # support tools like ccache, which give QMAKE_CXX values of the form:
++    #     ccache <path_to_compiler>
++    compiler = $$first(QMAKE_CXX)$$sfx
++
+     # Check if the binary exists with an absolute path. Do this check
+     # before the CROSS_COMPILE empty check below to allow the mkspec
+     # to derive the compiler path from other device options.
+-    exists($$QMAKE_CXX$$sfx):return()
++    exists($$compiler):return()
+ 
+     # Check for possible reasons of failure
+     # check if CROSS_COMPILE device-option is set
+@@ -31,7 +36,7 @@ defineTest(deviceSanityCheckCompiler) {
+     # Check if QMAKE_CXX points to an executable.
+     ensurePathEnv()
+     for (dir, QMAKE_PATH_ENV) {
+-        exists($$dir/$${QMAKE_CXX}$$sfx): \
++        exists($$dir/$${compiler}): \
+             return()
+     }
+

Signature · 1 line

2.1.4

Thomas Petazzoni

2015-08-28 09:57 UTC #1 lore raw

PATCH v2 1/1 pacakge/qt5/qt5base: fix build with ccache

Dear Beno?t Th?baudeau,

package/qt5/qt5base/0009-fix-build-with-ccache.patch line 31

Quoted 12 lines · Beno?t Th?baudeau

package/qt5/qt5base/0009-fix-build-with-ccache.patch new file +12 -0

+diff --git a/mkspecs/features/device_config.prf b/mkspecs/features/device_config.prf
+index cd3a0cf..eee4ac6 100644
+--- a/mkspecs/features/device_config.prf
++++ b/mkspecs/features/device_config.prf
+@@ -19,10 +19,15 @@ defineTest(deviceSanityCheckCompiler) {
+     else: \
+         sfx =
+ 
++    # Build the compiler filename using the first value in QMAKE_CXX in order to
++    # support tools like ccache, which give QMAKE_CXX values of the form:
++    #     ccache <path_to_compiler>
++    compiler = $$first(QMAKE_CXX)$$sfx

Then wouldn't it make more sense to use the last() function, to use the last value rather than the first, and actual check the compiler rather than ccache?

compiler = $$last(QMAKE_CXX)$$sfx

Thanks,

Thomas

Signature · 3 lines

Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Benoît Thébaudeau

author

2015-08-28 10:02 UTC #2 lore raw

PATCH v2 1/1 pacakge/qt5/qt5base: fix build with ccache

Quoted 19 lines · Thomas Petazzoni

package/qt5/qt5base/0009-fix-build-with-ccache.patch line 31
Quoted 12 lines · Beno?t Th?baudeau
package/qt5/qt5base/0009-fix-build-with-ccache.patch new file +12 -0
+diff --git a/mkspecs/features/device_config.prf b/mkspecs/features/device_config.prf
+index cd3a0cf..eee4ac6 100644
+--- a/mkspecs/features/device_config.prf
++++ b/mkspecs/features/device_config.prf
+@@ -19,10 +19,15 @@ defineTest(deviceSanityCheckCompiler) {
+     else: \
+         sfx =
+ 
++    # Build the compiler filename using the first value in QMAKE_CXX in order to
++    # support tools like ccache, which give QMAKE_CXX values of the form:
++    #     ccache <path_to_compiler>
++    compiler = $$first(QMAKE_CXX)$$sfx
Then wouldn't it make more sense to use the last() function, to use the last value rather than the first, and actual check the compiler rather than ccache?

Quoted 1 line

compiler = $$last(QMAKE_CXX)$$sfx

The last() function could be less portable because there might be some other use cases out there that we're not thinking about. On the contrary, we're certain that the first value has to be an executable.

Best regards,
Beno?t

Thomas Petazzoni

2015-08-28 10:05 UTC #3 lore raw

PATCH v2 1/1 pacakge/qt5/qt5base: fix build with ccache

Dear Beno?t Th?baudeau,

Quoted 9 lines · Beno?t Th?baudeau

Quoted 5 lines
Then wouldn't it make more sense to use the last() function, to use the last value rather than the first, and actual check the compiler rather than ccache?
compiler = $$last(QMAKE_CXX)$$sfx
The last() function could be less portable because there might be some other use cases out there that we're not thinking about. On the contrary, we're certain that the first value has to be an executable.

Correct. It makes the test a bit useless though. But oh well that test is not very useful in the first place. If upstream is happy with a $$first() solution, I'm fine :)

Thanks!

Thomas

Signature · 3 lines

Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

Thomas Petazzoni

Applied

2015-08-28 13:39 UTC #4 lore raw

PATCH v2 1/1 pacakge/qt5/qt5base: fix build with ccache

In reply to Benoît Thébaudeau

Dear Beno?t Th?baudeau,

Quoted 19 lines · Beno?t Th?baudeau

Building with ccache failed with:
Running configuration tests...
Failed to process makespec for platform 'devices/linux-buildroot-g++'
Project ERROR: Compiler <path_to_output_dir>/host/usr/bin/ccache <path_to_output_dir>/host/usr/bin/<cross_compile>-g++ not found. Check the value of CROSS_COMPILE -device-option Could not read qmake configuration file <path_to_output_dir>/build/qt5base-5.5.0/mkspecs/devices/linux-buildroot-g++/qmake.conf.
Error processing project file: /dev/null
This was caused by Buildroot setting this in qt5base-5.5.0/mkspecs/devices/linux-buildroot-g++/qmake.conf:
    QMAKE_CXX               = $${BR_CCACHE} $${CROSS_COMPILE}g++
But qt5base-5.5.0/mkspecs/features/device_config.prf expects QMAKE_CXX to be a single valid (absolute or QMAKE_PATH_ENV-relative) path to an existing file, which is not possible if using ccache as above.
Add a patch fixing this by testing only the first value in QMAKE_CXX.

Quoted 8 lines

Signed-off-by Beno?t Th?baudeau <[email protected]>
---
Changes v1 -> v2:
- Use the first value in QMAKE_CXX instead of removing the whole test.
---
 .../qt5/qt5base/0009-fix-build-with-ccache.patch   | 49 ++++++++++++++++++++++
 1 file changed, 49 insertions(+)
 create mode 100644 package/qt5/qt5base/0009-fix-build-with-ccache.patch

Applied to master after fixing the typo in the commit title. Thanks!

Thomas

Signature · 3 lines

Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com