Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 077b553

Browse files
committedNov 16, 2024
build: compile bundled simdjson conditionally
The --shared-simdjson flag was introduced in nodejs#52924, but the implementation was incomplete. Resolves nodejs#52914
1 parent 78bd0ab commit 077b553

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed
 

‎node.gyp

-3
Original file line numberDiff line numberDiff line change
@@ -849,7 +849,6 @@
849849
'dependencies': [
850850
'deps/googletest/googletest.gyp:gtest_prod',
851851
'deps/histogram/histogram.gyp:histogram',
852-
'deps/simdjson/simdjson.gyp:simdjson',
853852
'deps/simdutf/simdutf.gyp:simdutf',
854853
'deps/nbytes/nbytes.gyp:nbytes',
855854
'node_js2c#host',
@@ -1170,7 +1169,6 @@
11701169
'deps/googletest/googletest.gyp:gtest',
11711170
'deps/googletest/googletest.gyp:gtest_main',
11721171
'deps/histogram/histogram.gyp:histogram',
1173-
'deps/simdjson/simdjson.gyp:simdjson',
11741172
'deps/simdutf/simdutf.gyp:simdutf',
11751173
'deps/nbytes/nbytes.gyp:nbytes',
11761174
],
@@ -1363,7 +1361,6 @@
13631361
'<(node_lib_target_name)',
13641362
'deps/histogram/histogram.gyp:histogram',
13651363
'deps/nbytes/nbytes.gyp:nbytes',
1366-
'deps/simdjson/simdjson.gyp:simdjson',
13671364
'deps/simdutf/simdutf.gyp:simdutf',
13681365
],
13691366

‎node.gypi

+4
Original file line numberDiff line numberDiff line change
@@ -216,6 +216,10 @@
216216
'dependencies': [ 'deps/ada/ada.gyp:ada' ],
217217
}],
218218

219+
[ 'node_shared_simdjson=="false"', {
220+
'dependencies': [ 'deps/simdjson/simdjson.gyp:simdjson' ],
221+
}],
222+
219223
[ 'node_shared_brotli=="false"', {
220224
'dependencies': [ 'deps/brotli/brotli.gyp:brotli' ],
221225
}],

0 commit comments

Comments
 (0)