Patrick Williams | 03907ee | 2022-05-01 06:28:52 -0500 | [diff] [blame] | 1 | From 01f5ab0edf3dd8f76d37fd36c2dcd3108be33a7b Mon Sep 17 00:00:00 2001 |
| 2 | From: Alexander Kanavin <alex@linutronix.de> |
| 3 | Date: Mon, 4 Apr 2022 21:20:05 +0200 |
| 4 | Subject: [PATCH] When building introspection files, do not quote CFLAGS. |
| 5 | |
| 6 | This does not seem to be propagated to the compiler correctly: |
| 7 | |
| 8 | In file included from /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot-native/usr/lib/x86_64-poky-linux/gcc/x86_64-poky-linux/11.2.0/include-fixed/syslimits.h:7, |
| 9 | from /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot-native/usr/lib/x86_64-poky-linux/gcc/x86_64-poky-linux/11.2.0/include-fixed/limits.h:34, |
| 10 | from /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot/usr/lib/glib-2.0/include/glibconfig.h:11, |
| 11 | from /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot/usr/include/glib-2.0/glib/gtypes.h:32, |
| 12 | from /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot/usr/include/glib-2.0/glib/galloca.h:32, |
| 13 | from /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot/usr/include/glib-2.0/glib.h:30, |
| 14 | from /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/build/Source/JavaScriptCore/tmp-introspectb51ks33n/JavaScriptCore-4.0.c:2: |
| 15 | /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot-native/usr/lib/x86_64-poky-linux/gcc/x86_64-poky-linux/11.2.0/include-fixed/limits.h:203:75: error: no include path in which to search for limits.h |
| 16 | 203 | #include_next <limits.h> /* recurse down to the real one */ |
| 17 | | ^ |
| 18 | In file included from /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot/usr/include/glib-2.0/glib/galloca.h:32, |
| 19 | from /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot/usr/include/glib-2.0/glib.h:30, |
| 20 | from /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/build/Source/JavaScriptCore/tmp-introspectb51ks33n/JavaScriptCore-4.0.c:2: |
| 21 | /srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot/usr/include/glib-2.0/glib/gtypes.h:35:10: fatal error: time.h: No such file or directory |
| 22 | 35 | #include <time.h> |
| 23 | | ^~~~~~~~ |
| 24 | compilation terminated. |
| 25 | Traceback (most recent call last): |
| 26 | File "/srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot-native/usr/lib/python3.10/distutils/unixccompiler.py", line 117, in _compile |
| 27 | self.spawn(compiler_so + cc_args + [src, '-o', obj] + |
| 28 | File "/srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot-native/usr/lib/python3.10/distutils/ccompiler.py", line 910, in spawn |
| 29 | spawn(cmd, dry_run=self.dry_run) |
| 30 | File "/srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot-native/usr/lib/python3.10/distutils/spawn.py", line 91, in spawn |
| 31 | raise DistutilsExecError( |
| 32 | distutils.errors.DistutilsExecError: command '/srv/work/alex/poky/build-64-alt/tmp/work/core2-64-poky-linux/webkitgtk/2.36.0-r0/recipe-sysroot-native/usr/bin/x86_64-poky-linux/x86_64-poky-linux-gcc' failed with exit code 1 |
| 33 | |
| 34 | Upstream-Status: Inappropriate [upstream has rewritten the whole thing as of https://github.com/WebKit/WebKit/commit/b0ae032850bb6b2672051bab8032fc9f9ef5eb97] |
| 35 | Signed-off-by: Alexander Kanavin <alex@linutronix.de> |
| 36 | --- |
| 37 | Source/JavaScriptCore/PlatformGTK.cmake | 2 +- |
| 38 | Source/WebKit/PlatformGTK.cmake | 4 ++-- |
| 39 | 2 files changed, 3 insertions(+), 3 deletions(-) |
| 40 | |
| 41 | diff --git a/Source/JavaScriptCore/PlatformGTK.cmake b/Source/JavaScriptCore/PlatformGTK.cmake |
| 42 | index 1c3c8fb7..efd18807 100644 |
| 43 | --- a/Source/JavaScriptCore/PlatformGTK.cmake |
| 44 | +++ b/Source/JavaScriptCore/PlatformGTK.cmake |
| 45 | @@ -71,7 +71,7 @@ if (ENABLE_INTROSPECTION) |
| 46 | add_custom_command( |
| 47 | OUTPUT ${CMAKE_BINARY_DIR}/JavaScriptCore-${WEBKITGTK_API_VERSION}.gir |
| 48 | DEPENDS JavaScriptCore |
| 49 | - COMMAND CC=${CMAKE_C_COMPILER} CFLAGS="-Wno-deprecated-declarations ${CMAKE_C_FLAGS}" LDFLAGS= |
| 50 | + COMMAND CC=${CMAKE_C_COMPILER} CFLAGS=-Wno-deprecated-declarations\ ${CMAKE_C_FLAGS} LDFLAGS= |
| 51 | ${LOADER_LIBRARY_PATH_VAR}="${INTROSPECTION_ADDITIONAL_LIBRARY_PATH}" |
| 52 | ${INTROSPECTION_SCANNER} |
| 53 | --quiet |
| 54 | diff --git a/Source/WebKit/PlatformGTK.cmake b/Source/WebKit/PlatformGTK.cmake |
| 55 | index 48d1820d..ded05035 100644 |
| 56 | --- a/Source/WebKit/PlatformGTK.cmake |
| 57 | +++ b/Source/WebKit/PlatformGTK.cmake |
| 58 | @@ -675,7 +675,7 @@ if (ENABLE_INTROSPECTION) |
| 59 | add_custom_target(WebKit2-${WEBKITGTK_API_VERSION}-gir |
| 60 | DEPENDS WebKit |
| 61 | DEPENDS ${CMAKE_BINARY_DIR}/JavaScriptCore-${WEBKITGTK_API_VERSION}.gir |
| 62 | - COMMAND CC=${CMAKE_C_COMPILER} CFLAGS="-Wno-deprecated-declarations ${CMAKE_C_FLAGS}" LDFLAGS= |
| 63 | + COMMAND CC=${CMAKE_C_COMPILER} CFLAGS=-Wno-deprecated-declarations\ ${CMAKE_C_FLAGS} LDFLAGS= |
| 64 | ${LOADER_LIBRARY_PATH_VAR}="${INTROSPECTION_ADDITIONAL_LIBRARY_PATH}" |
| 65 | ${INTROSPECTION_SCANNER} |
| 66 | --quiet |
| 67 | @@ -721,7 +721,7 @@ if (ENABLE_INTROSPECTION) |
| 68 | OUTPUT ${CMAKE_BINARY_DIR}/WebKit2WebExtension-${WEBKITGTK_API_VERSION}.gir |
| 69 | DEPENDS ${CMAKE_BINARY_DIR}/JavaScriptCore-${WEBKITGTK_API_VERSION}.gir |
| 70 | DEPENDS WebKit2-${WEBKITGTK_API_VERSION}-gir |
| 71 | - COMMAND CC=${CMAKE_C_COMPILER} CFLAGS="-Wno-deprecated-declarations ${CMAKE_C_FLAGS}" |
| 72 | + COMMAND CC=${CMAKE_C_COMPILER} CFLAGS=-Wno-deprecated-declarations\ ${CMAKE_C_FLAGS} |
| 73 | LDFLAGS="${INTROSPECTION_ADDITIONAL_LDFLAGS}" |
| 74 | ${LOADER_LIBRARY_PATH_VAR}="${INTROSPECTION_ADDITIONAL_LIBRARY_PATH}" |
| 75 | ${INTROSPECTION_SCANNER} |
| 76 | -- |
| 77 | 2.30.2 |
| 78 | |