Bump icu component version for package nodejs#17358
Conversation
98aed76 to
a6b7d8d
Compare
9df9962 to
6330316
Compare
|
Verified the ICU version mismatch from #17161 is resolved with this build : https://dev.azure.com/mariner-org/mariner/_build/results?buildId=1130209&view=results Reproduced the issue's check against the rebuilt nodejs-24.14.1-4.azl3 + nodejs-full-i18n-24.14.1-4.azl3 RPMs on linux/amd64:
Intl.Segmenter now initializes cleanly — no more U_FILE_ACCESS_ERROR: ... contains icudt78l.dat. Binary (ICU 78) and -full-i18n data file (icudt78l.dat) are consistent on amd64, which was the root cause called out in #17161.
|
|
Also verified the segfault repro from #17161 no longer triggers with this build. Ran the issue's exact TypeSpec reproducer in the OneBranch build image with the rebuilt nodejs-24.14.1-4.azl3 RPMs preinstalled:
Result: tsp compile runs to completion and docker exit: 0 — no more Segmentation fault / exit 139 that the stock image produces on amd64. Repeated 5 runs back-to-back, all clean.
|
|
BB - https://dev.azure.com/mariner-org/mariner/_build/results?buildId=1130209&view=results |
kgodara912
left a comment
There was a problem hiding this comment.
Component version upgrade for nodejs upgrade to 24 to prevent segfaults in particular scenarios. Buddy build is successful. LGTM.


Merge Checklist
All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)
*-staticsubpackages, etc.) have had theirReleasetag incremented../cgmanifest.json,./toolkit/scripts/toolchain/cgmanifest.json,.github/workflows/cgmanifest.json)./LICENSES-AND-NOTICES/SPECS/data/licenses.json,./LICENSES-AND-NOTICES/SPECS/LICENSES-MAP.md,./LICENSES-AND-NOTICES/SPECS/LICENSE-EXCEPTIONS.PHOTON)*.signatures.jsonfilessudo make go-tidy-allandsudo make go-test-coveragepassSummary
What does the PR accomplish, why was it needed?
This PR is to bump the icu component version of 78.2
Change Log
Does this affect the toolchain?
NO
Associated issues
Links to CVEs
Test Methodology