Skip to content

Commit f8427fa

Browse files
author
Michal Klocek
committed
CMake: Utilize CMAKE_VERBOSE_MAKEFILE setting of CMake
Define CMAKE_VERBOSE_MAKEFILE also affects ninja generator by adding -v if called through cmake interface ie. 'cmake --build .' Kitware/CMake@ce935eb Use the value of CMAKE_VERBOSE_MAKEFILE to determine if gn or chromium part must be built verbosely, in which case append -v to the invocation of ninja. Unify ninja flags setup for gn and chromium part compile calls, so now NINJAFLAGS environment variable applies when building gn itself. Note this change has side effect compared to cmake patch meaning the verbose mode is applied also when calling ninja directly. Done-With: Fab Stz <[email protected]> Pick-to: 6.9 6.8 Fixes: QTBUG-124007 Change-Id: Id30ffc45ffa8ea4b7995d13905c855cbb4bf6407 Reviewed-by: Alexey Edelev <[email protected]>
1 parent 76d317c commit f8427fa

File tree

2 files changed

+12
-3
lines changed

2 files changed

+12
-3
lines changed

cmake/Functions.cmake

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -308,7 +308,10 @@ function(add_ninja_command)
308308
)
309309
_qt_internal_validate_all_args_are_parsed(arg)
310310

311-
string(REPLACE " " ";" NINJAFLAGS "$ENV{NINJAFLAGS}")
311+
string(REPLACE " " ";" ninja_flags "$ENV{NINJAFLAGS}")
312+
if(CMAKE_VERBOSE_MAKEFILE)
313+
list(APPEND ninja_flags -v)
314+
endif()
312315
list(TRANSFORM arg_OUTPUT PREPEND "${arg_BUILDDIR}/")
313316
list(TRANSFORM arg_BYPRODUCTS PREPEND "${arg_BUILDDIR}/")
314317
add_custom_command(
@@ -318,7 +321,7 @@ function(add_ninja_command)
318321
BYPRODUCTS ${arg_BYPRODUCTS}
319322
COMMENT "Running ninja for ${arg_TARGET} in ${arg_BUILDDIR}"
320323
COMMAND Ninja::ninja
321-
${NINJAFLAGS}
324+
${ninja_flags}
322325
-C ${arg_BUILDDIR}
323326
${arg_TARGET}
324327
USES_TERMINAL
@@ -541,6 +544,7 @@ macro(qt_webengine_externalproject_add)
541544
-DCMAKE_INSTALL_PREFIX:PATH=<INSTALL_DIR>
542545
-DCMAKE_PREFIX_PATH:PATH=<INSTALL_DIR>
543546
-DCMAKE_OSX_ARCHITECTURES=${OSX_ARCH_STR}
547+
-DCMAKE_VERBOSE_MAKEFILE=${CMAKE_VERBOSE_MAKEFILE}
544548
-DWEBENGINE_ROOT_BUILD_DIR=${PROJECT_BINARY_DIR}
545549
-DQT_ALLOW_SYMLINK_IN_PATHS=${QT_ALLOW_SYMLINK_IN_PATHS}
546550
-DPython3_EXECUTABLE=${Python3_EXECUTABLE}

src/gn/CMakeLists.txt

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,11 @@ if(GCC_LEGACY_SUPPORT)
5959
set(GN_GCC_LEGACY_SUPPORT --gcc-legacy-support)
6060
endif()
6161

62+
string(REPLACE " " ";" ninja_flags "$ENV{NINJAFLAGS}")
63+
if(CMAKE_VERBOSE_MAKEFILE)
64+
list(APPEND ninja_flags -v)
65+
endif()
66+
6267
list(JOIN CMAKE_OSX_ARCHITECTURES "$<COMMA>" OSX_ARCH)
6368
add_custom_command(
6469
OUTPUT ${GN_EXECUTABLE}
@@ -79,7 +84,7 @@ add_custom_command(
7984
$<$<AND:$<PLATFORM_ID:Darwin>,$<BOOL:${OSX_ARCH}>>:--osx-architectures>
8085
$<$<AND:$<PLATFORM_ID:Darwin>,$<BOOL:${OSX_ARCH}>>:${OSX_ARCH}>
8186
${GN_GCC_LEGACY_SUPPORT}
82-
COMMAND Ninja::ninja -C ${GN_BINARY_DIR}/$<CONFIG> ${GN_EXECUTABLE}
87+
COMMAND Ninja::ninja ${ninja_flags} -C ${GN_BINARY_DIR}/$<CONFIG> ${GN_EXECUTABLE}
8388
VERBATIM
8489
USES_TERMINAL
8590
COMMAND_EXPAND_LISTS

0 commit comments

Comments
 (0)