RESOLVED FIXED 276098
[WPE][cross-toolchain-helper] Fix issue with the icu installation and long paths
https://bugs.webkit.org/show_bug.cgi?id=276098
Summary [WPE][cross-toolchain-helper] Fix issue with the icu installation and long paths
Carlos Alberto Lopez Perez
Reported 2024-07-01 15:40:00 PDT
There is a bug libicu that causes libicu to be installed incorrectly when the build system uses long paths (around 500 or more chars). This was reported to icu here https://unicode-org.atlassian.net/browse/ICU-22813 and also summitted a PR here https://github.com/unicode-org/icu/pull/3058 The issue ended causing that nodejs couldn't run properly on the build system because icu was unproperly installed, making the chromium build fail which was introduced in 280438@main
Attachments
Carlos Alberto Lopez Perez
Comment 1 2024-07-01 15:51:25 PDT
The buildbots have a basepath larger than any developer machine. For example "/home/buildbot/worker/WPE-Linux-RPi4-64bits-Mesa-Release-Perf-Build/build" and that explains why this wasn't caught before.
Carlos Alberto Lopez Perez
Comment 2 2024-07-01 15:54:16 PDT
Example of failed build: https://build.webkit.org/#/builders/894/builds/12478 There you can see nodejs throwing an error related to ICU | RuntimeError: Command '/home/buildbot/worker/WPE-Linux-RPi4-64bits-Mesa-Release-Perf-Build/build/WebKitBuild/CrossToolChains/rpi4-64bits-mesa/build/tmp/work/cortexa72-poky-linux/chromium-ozone-wayland/124.0.6367.207/chromium-124.0.6367.207/out/Release/../../third_party/node/linux | /home/buildbot/worker/WPE-Linux-RPi4-64bits-Mesa-Release-Perf-Build/build/WebKitBuild/CrossToolChains/rpi4-64bits-mesa/build/tmp/work/cortexa72-poky-linux/chromium-ozone-wayland/124.0.6367.207/chromium-124.0.6367.207/third_party/devtools-frontend/src/node_modules/glob/common.js:1 | return a.localeCompare(b, 'en') | ^ | | RangeError: Internal error. Icu error.
Carlos Alberto Lopez Perez
Comment 3 2024-07-01 15:55:36 PDT
EWS
Comment 4 2024-07-01 16:15:15 PDT
Committed 280556@main (745d34f6850a): <https://commits.webkit.org/280556@main> Reviewed commits have been landed. Closing PR #30364 and removing active labels.
Radar WebKit Bug Importer
Comment 5 2024-07-01 16:16:17 PDT
Note You need to log in before you can comment on or make changes to this bug.