Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit bab750d

Browse filesBrowse files
committed
build: do not depend on V8 deps on --without-bundled-v8 builds
PR-URL: #62033 Reviewed-By: Richard Lau <richard.lau@ibm.com> Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
1 parent 1d91a72 commit bab750d
Copy full SHA for bab750d

3 files changed

+17-17Lines changed: 17 additions & 17 deletions

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎node.gyp‎

Copy file name to clipboardExpand all lines: node.gyp
+7-3Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -895,7 +895,6 @@
895895
'<(SHARED_INTERMEDIATE_DIR)' # for node_natives.h
896896
],
897897
'dependencies': [
898-
'tools/v8_gypfiles/abseil.gyp:abseil',
899898
'node_js2c#host',
900899
],
901900

@@ -956,6 +955,9 @@
956955
'src/node_snapshot_stub.cc',
957956
]
958957
}],
958+
[ 'node_use_bundled_v8!="false"', {
959+
'dependencies': [ 'tools/v8_gypfiles/abseil.gyp:abseil' ],
960+
}],
959961
[ 'node_shared_gtest=="false"', {
960962
'dependencies': [
961963
'deps/googletest/googletest.gyp:gtest_prod',
@@ -1276,7 +1278,6 @@
12761278

12771279
'dependencies': [
12781280
'<(node_lib_target_name)',
1279-
'tools/v8_gypfiles/abseil.gyp:abseil',
12801281
],
12811282

12821283
'includes': [
@@ -1310,6 +1311,9 @@
13101311
[ 'node_shared_gtest=="true"', {
13111312
'libraries': [ '-lgtest_main' ],
13121313
}],
1314+
[ 'node_use_bundled_v8!="false"', {
1315+
'dependencies': [ 'tools/v8_gypfiles/abseil.gyp:abseil' ],
1316+
}],
13131317
[ 'node_shared_hdr_histogram=="false"', {
13141318
'dependencies': [
13151319
'deps/histogram/histogram.gyp:histogram',
@@ -1548,7 +1552,7 @@
15481552
[ 'OS=="mac"', {
15491553
'libraries': [ '-framework CoreFoundation -framework Security' ],
15501554
}],
1551-
[ 'node_shared_simdutf=="false"', {
1555+
[ 'node_shared_simdutf=="false" and node_use_bundled_v8!="false"', {
15521556
'dependencies': [ 'tools/v8_gypfiles/v8.gyp:simdutf#host' ],
15531557
}],
15541558
[ 'node_shared_libuv=="false"', {
Collapse file

‎node.gypi‎

Copy file name to clipboardExpand all lines: node.gypi
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,7 @@
234234
'dependencies': [ 'deps/simdjson/simdjson.gyp:simdjson' ],
235235
}],
236236

237-
[ 'node_shared_simdutf=="false"', {
237+
[ 'node_shared_simdutf=="false" and node_use_bundled_v8!="false"', {
238238
'dependencies': [ 'tools/v8_gypfiles/v8.gyp:simdutf' ],
239239
}],
240240

Collapse file

‎tools/nix/v8.nix‎

Copy file name to clipboardExpand all lines: tools/nix/v8.nix
+9-13Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88

99
let
1010
nodejs = pkgs.nodejs-slim_latest;
11+
v8Dir = ../../deps/v8;
1112

1213
version =
1314
let
@@ -16,7 +17,7 @@ let
1617
+ "#define V8_MINOR_VERSION ([0-9]+).*"
1718
+ "#define V8_BUILD_NUMBER ([0-9]+).*"
1819
+ "#define V8_PATCH_LEVEL ([0-9]+).*"
19-
) (builtins.readFile ../../deps/v8/include/v8-version.h);
20+
) (builtins.readFile "${v8Dir}/include/v8-version.h");
2021
v8_embedder_string = builtins.match ".*'v8_embedder_string': '-(node.[0-9]+)'.*" (
2122
builtins.readFile ../../common.gypi
2223
);
@@ -41,7 +42,7 @@ pkgs.stdenv.mkDerivation (finalAttrs: {
4142
../../configure.py
4243
../../deps/inspector_protocol/inspector_protocol.gyp
4344
../../deps/ncrypto/ncrypto.gyp
44-
../../deps/v8
45+
v8Dir
4546
../../node.gyp
4647
../../node.gypi
4748
../../src/inspector/node_inspector.gypi
@@ -86,7 +87,7 @@ pkgs.stdenv.mkDerivation (finalAttrs: {
8687
'';
8788
installPhase = ''
8889
${
89-
if pkgs.stdenv.buildPlatform.isDarwin then
90+
if pkgs.stdenv.hostPlatform.isDarwin then
9091
# Darwin is excluded from creating thin archive in tools/gyp/pylib/gyp/generator/ninja.py:2488
9192
"install -Dm644 out/Release/lib* -t $out/lib"
9293
else
@@ -106,22 +107,17 @@ pkgs.stdenv.mkDerivation (finalAttrs: {
106107
done
107108
''
108109
}
109-
libs=$(for f in $out/lib/lib*.a; do
110-
b=$(basename "$f" .a)
111-
printf " -l%s" "''${b#lib}"
112-
done)
113-
114-
# copy v8 headers
115-
cp -r deps/v8/include $out/
116110
117-
# create a pkgconfig file for v8
118111
mkdir -p $out/lib/pkgconfig
119112
cat -> $out/lib/pkgconfig/v8.pc << EOF
120113
Name: v8
121114
Description: V8 JavaScript Engine build for Node.js CI
122115
Version: ${version}
123-
Libs: -L$out/lib $libs
124-
Cflags: -I$out/include
116+
Libs: -L$out/lib $(for f in $out/lib/lib*.a; do
117+
b=$(basename "$f" .a)
118+
printf " -l%s" "''${b#lib}"
119+
done) -lstdc++
120+
Cflags: -I${v8Dir}/include -I${v8Dir}/third_party/abseil-cpp -I${v8Dir}/third_party/simdutf
125121
EOF
126122
'';
127123
})

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.