Skip to content

build: use //third_party/simdutf by default in GN - #58115

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
codebytere:simdutf8-3
May 4, 2025
Merged

build: use //third_party/simdutf by default in GN#58115
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
codebytere:simdutf8-3

Conversation

@codebytere

Copy link
Copy Markdown
Member

Use //third_party/simdutf by default in the GN build instead of deps/simdutf. Similar to what's currently done for //third_party/zlib for the same reason.

Added in https://chromium-review.googlesource.com/c/v8/node-ci/+/6177642

@codebytere
codebytere requested a review from zcbenzMay 2, 2025 08:29
@anonriganonrig added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels May 2, 2025
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label May 4, 2025
@nodejs-github-bot
nodejs-github-bot merged commit 5aad0a7 into nodejs:mainMay 4, 2025
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 5aad0a7

aduh95 pushed a commit that referenced this pull request May 8, 2025
PR-URL: #58115
Reviewed-By: Cheng Zhao <zcbenz@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
aduh95 pushed a commit that referenced this pull request Jun 10, 2025
PR-URL: #58115
Reviewed-By: Cheng Zhao <zcbenz@gmail.com>
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
codebytere added a commit to electron/electron that referenced this pull request Jun 26, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 26, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 27, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 27, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 28, 2025
* chore: bump node in DEPS to v22.17.0
* build: use //third_party/simdutf by default in GN
nodejs/node#58115
* chore: adjust crypto specs:
- nodejs/node#58117
- nodejs/node#58387
* deps: update libuv to 1.51.0
nodejs/node#58124
* test: fix test-buffer-tostring-range on allocation failure
nodejs/node#58416
* build: use FILE_OFFSET_BITS=64 esp. on 32-bit arch
nodejs/node#58090
* build: use //third_party/simdutf by default in GN
nodejs/node#58115
* inspector: add protocol method Network.dataReceived
nodejs/node#58001
* test: force slow JSON.stringify path for overflow
nodejs/node#58181
* chore: fixup patch indices
* 6049967: Remove protocol::Maybe and roll inspector_protocol
https://chromium-review.googlesource.com/c/chromium/src/+/6049967
* chore: fixup crypto test patch
* src: fix module buffer allocation
nodejs/node#57738
* crypto: expose process.features.openssl_is_boringssl
nodejs/node#58387
* util: add internal assignFunctionName() function
nodejs/node#57916
* build: fix pointer compression builds
nodejs/node#58171
* chore: put back config options
* fixup! deps: update libuv to 1.51.0
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
* chore: bump node in DEPS to v22.17.0
* build: use //third_party/simdutf by default in GN
nodejs/node#58115
* chore: adjust crypto specs:
- nodejs/node#58117
- nodejs/node#58387
* deps: update libuv to 1.51.0
nodejs/node#58124
* test: fix test-buffer-tostring-range on allocation failure
nodejs/node#58416
* build: use FILE_OFFSET_BITS=64 esp. on 32-bit arch
nodejs/node#58090
* build: use //third_party/simdutf by default in GN
nodejs/node#58115
* inspector: add protocol method Network.dataReceived
nodejs/node#58001
* test: force slow JSON.stringify path for overflow
nodejs/node#58181
* chore: fixup patch indices
* 6049967: Remove protocol::Maybe and roll inspector_protocol
https://chromium-review.googlesource.com/c/chromium/src/+/6049967
* chore: fixup crypto test patch
* src: fix module buffer allocation
nodejs/node#57738
* crypto: expose process.features.openssl_is_boringssl
nodejs/node#58387
* util: add internal assignFunctionName() function
nodejs/node#57916
* build: fix pointer compression builds
nodejs/node#58171
* chore: put back config options
* fixup! deps: update libuv to 1.51.0
* chore: update patches
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
Co-authored-by: patchup[bot] <73610968+patchup[bot]@users.noreply.github.com>
codebytere added a commit to electron/electron that referenced this pull request Jun 30, 2025
* chore: bump node in DEPS to v22.17.0
* build: use //third_party/simdutf by default in GN
nodejs/node#58115
* chore: adjust crypto specs:
- nodejs/node#58117
- nodejs/node#58387
* deps: update libuv to 1.51.0
nodejs/node#58124
* test: fix test-buffer-tostring-range on allocation failure
nodejs/node#58416
* build: use FILE_OFFSET_BITS=64 esp. on 32-bit arch
nodejs/node#58090
* build: use //third_party/simdutf by default in GN
nodejs/node#58115
* inspector: add protocol method Network.dataReceived
nodejs/node#58001
* test: force slow JSON.stringify path for overflow
nodejs/node#58181
* chore: fixup patch indices
* 6049967: Remove protocol::Maybe and roll inspector_protocol
https://chromium-review.googlesource.com/c/chromium/src/+/6049967
* chore: fixup crypto test patch
* src: fix module buffer allocation
nodejs/node#57738
* crypto: expose process.features.openssl_is_boringssl
nodejs/node#58387
* util: add internal assignFunctionName() function
nodejs/node#57916
* build: fix pointer compression builds
nodejs/node#58171
* chore: put back config options
* fixup! deps: update libuv to 1.51.0
* chore: update patches
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
Co-authored-by: patchup[bot] <73610968+patchup[bot]@users.noreply.github.com>
codebytere added a commit to electron/electron that referenced this pull request Jul 2, 2025
codebytere added a commit to electron/electron that referenced this pull request Jul 2, 2025
codebytere added a commit to electron/electron that referenced this pull request Jul 2, 2025
* chore: bump node in DEPS to v22.17.0
* build: use //third_party/simdutf by default in GN
nodejs/node#58115
* chore: adjust crypto specs:
- nodejs/node#58117
- nodejs/node#58387
* deps: update libuv to 1.51.0
nodejs/node#58124
* test: fix test-buffer-tostring-range on allocation failure
nodejs/node#58416
* build: use FILE_OFFSET_BITS=64 esp. on 32-bit arch
nodejs/node#58090
* build: use //third_party/simdutf by default in GN
nodejs/node#58115
* inspector: add protocol method Network.dataReceived
nodejs/node#58001
* chore: fixup patch indices
* 6049967: Remove protocol::Maybe and roll inspector_protocol
https://chromium-review.googlesource.com/c/chromium/src/+/6049967
* chore: fixup crypto test patch
* src: fix module buffer allocation
nodejs/node#57738
* crypto: expose process.features.openssl_is_boringssl
nodejs/node#58387
* util: add internal assignFunctionName() function
nodejs/node#57916
* build: fix pointer compression builds
nodejs/node#58171
* chore: put back config options
* fixup! deps: update libuv to 1.51.0
* chore: update patches
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
Co-authored-by: patchup[bot] <73610968+patchup[bot]@users.noreply.github.com>
kigh-ota pushed a commit to kigh-ota/electron that referenced this pull request Sep 30, 2025
* chore: bump node in DEPS to v22.17.0
* build: use //third_party/simdutf by default in GN
nodejs/node#58115
* chore: adjust crypto specs:
- nodejs/node#58117
- nodejs/node#58387
* deps: update libuv to 1.51.0
nodejs/node#58124
* test: fix test-buffer-tostring-range on allocation failure
nodejs/node#58416
* build: use FILE_OFFSET_BITS=64 esp. on 32-bit arch
nodejs/node#58090
* build: use //third_party/simdutf by default in GN
nodejs/node#58115
* inspector: add protocol method Network.dataReceived
nodejs/node#58001
* test: force slow JSON.stringify path for overflow
nodejs/node#58181
* chore: fixup patch indices
* 6049967: Remove protocol::Maybe and roll inspector_protocol
https://chromium-review.googlesource.com/c/chromium/src/+/6049967
* chore: fixup crypto test patch
* src: fix module buffer allocation
nodejs/node#57738
* crypto: expose process.features.openssl_is_boringssl
nodejs/node#58387
* util: add internal assignFunctionName() function
nodejs/node#57916
* build: fix pointer compression builds
nodejs/node#58171
* chore: put back config options
* fixup! deps: update libuv to 1.51.0
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@codebytere@nodejs-github-bot@zcbenz@anonrig@legendecas