Skip to content

Commit 4959775

Browse files
authored
[qtwebengine] Add more diagnostic logging and reduce concurrency on Windows. (#53435)
1 parent 2e6b923 commit 4959775

5 files changed

Lines changed: 42 additions & 6 deletions

File tree

ports/qtwebengine/node-wrapper-diagnostics.diff

Lines changed: 28 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,33 @@
1+
diff --git a/src/3rdparty/chromium/third_party/devtools-frontend/src/scripts/build/typescript/ts_library.py b/src/3rdparty/chromium/third_party/devtools-frontend/src/scripts/build/typescript/ts_library.py
2+
index c4a2a65..ea6df74 100644
3+
--- a/src/3rdparty/chromium/third_party/devtools-frontend/src/scripts/build/typescript/ts_library.py
4+
+++ b/src/3rdparty/chromium/third_party/devtools-frontend/src/scripts/build/typescript/ts_library.py
5+
@@ -59,7 +59,19 @@ def runTsc(tsconfig_location):
6+
universal_newlines=True)
7+
stdout, stderr = process.communicate()
8+
# TypeScript does not correctly write to stderr because of https://github.com/microsoft/TypeScript/issues/33849
9+
- return process.returncode, stdout + stderr
10+
+ output = stdout + stderr
11+
+ if process.returncode != 0:
12+
+ details = [
13+
+ 'Command \'%s\' failed with exit code %d' %
14+
+ (' '.join(cmd), process.returncode),
15+
+ 'Working directory: %s' % os.getcwd(),
16+
+ 'stdout:',
17+
+ stdout,
18+
+ 'stderr:',
19+
+ stderr,
20+
+ ]
21+
+ output = '\n'.join(details)
22+
+ return process.returncode, output
23+
24+
25+
# To ensure that Ninja only rebuilds dependents when the actual content/public API of a TypeScript target changes,
126
diff --git a/src/3rdparty/chromium/third_party/node/node.py b/src/3rdparty/chromium/third_party/node/node.py
2-
index 3af0f1d9ee..20516742f5 100644
27+
index 067a2c6..0e7815f 100644
328
--- a/src/3rdparty/chromium/third_party/node/node.py
429
+++ b/src/3rdparty/chromium/third_party/node/node.py
5-
@@ -32,9 +32,15 @@ def RunNode(cmd_parts, stdout=None):
30+
@@ -36,10 +36,16 @@ def RunNode(cmd_parts, stdout=None):
631
stdout, stderr = process.communicate()
732

833
if process.returncode != 0:
@@ -22,3 +47,4 @@ index 3af0f1d9ee..20516742f5 100644
2247
+ raise RuntimeError('\n'.join(details))
2348

2449
return stdout
50+

ports/qtwebengine/portfile.cmake

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -199,6 +199,11 @@ if(buildtree_length GREATER 22 AND VCPKG_TARGET_IS_WINDOWS AND NOT VCPKG_TARGET_
199199
file(MAKE_DIRECTORY "${CURRENT_BUILDTREES_DIR}")
200200
endif()
201201

202+
set(qtwebengine_install_options ADD_BIN_TO_PATH)
203+
if(VCPKG_TARGET_IS_WINDOWS)
204+
# The outer CMake build invokes a parallel Chromium Ninja build.
205+
list(APPEND qtwebengine_install_options DISABLE_PARALLEL)
206+
endif()
202207
set(ENV{QTWEBENGINE_GN_THREADS} "${VCPKG_CONCURRENCY}")
203208
set(ENV{NINJAFLAGS} "-j${VCPKG_CONCURRENCY} $ENV{NINJAFLAGS}")
204209

@@ -252,7 +257,7 @@ if(NOT VCPKG_BUILD_TYPE)
252257
file(APPEND "${CURRENT_BUILDTREES_DIR}/${TARGET_TRIPLET}-dbg/src/core/Debug/${target_args_gn}" "\ngcc_target_rpath=\"\\\${ORIGIN}:${CURRENT_INSTALLED_DIR}/debug/lib\"\n")
253258
endif()
254259
vcpkg_host_path_list(PREPEND ENV{PKG_CONFIG_PATH} "${CURRENT_INSTALLED_DIR}/debug/lib/pkgconfig" "${CURRENT_INSTALLED_DIR}/share/pkgconfig")
255-
vcpkg_cmake_install(ADD_BIN_TO_PATH)
260+
vcpkg_cmake_install(${qtwebengine_install_options})
256261
endblock()
257262
endif()
258263
vcpkg_restore_env_variables(VARS PKG_CONFIG_PATH)
@@ -262,7 +267,7 @@ if(VCPKG_TARGET_IS_LINUX AND EXISTS "${CURRENT_BUILDTREES_DIR}/${TARGET_TRIPLET}
262267
file(APPEND "${CURRENT_BUILDTREES_DIR}/${TARGET_TRIPLET}-rel/src/core/Release/${target_args_gn}" "\ngcc_target_rpath=\"\\\${ORIGIN}:${CURRENT_INSTALLED_DIR}/lib\"\n")
263268
endif()
264269
vcpkg_host_path_list(PREPEND ENV{PKG_CONFIG_PATH} "${CURRENT_INSTALLED_DIR}/lib/pkgconfig" "${CURRENT_INSTALLED_DIR}/share/pkgconfig")
265-
vcpkg_cmake_install(ADD_BIN_TO_PATH)
270+
vcpkg_cmake_install(${qtwebengine_install_options})
266271
endblock()
267272
vcpkg_restore_env_variables(VARS PKG_CONFIG_PATH)
268273

ports/qtwebengine/vcpkg.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"$comment": "x86-windows is not within the upstream support matrix of Qt6",
33
"name": "qtwebengine",
44
"version": "6.11.1",
5-
"port-version": 1,
5+
"port-version": 2,
66
"description": "Qt modules for rendering web and PDF content.",
77
"homepage": "https://www.qt.io/",
88
"license": null,

versions/baseline.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8622,7 +8622,7 @@
86228622
},
86238623
"qtwebengine": {
86248624
"baseline": "6.11.1",
8625-
"port-version": 1
8625+
"port-version": 2
86268626
},
86278627
"qtwebsockets": {
86288628
"baseline": "6.11.1",

versions/q-/qtwebengine.json

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
{
22
"versions": [
3+
{
4+
"git-tree": "8e942b62c9538f2da5545cc095145124b9b9ee76",
5+
"version": "6.11.1",
6+
"port-version": 2
7+
},
38
{
49
"git-tree": "aa9371eee2255a22028bb2706483742f33d95149",
510
"version": "6.11.1",

0 commit comments

Comments
 (0)