build: do not depend on V8 deps on --without-bundled-v8 builds

Author: aduh95

node.gyp

@@ -895,7 +895,6 @@
         '<(SHARED_INTERMEDIATE_DIR)' # for node_natives.h
       ],
       'dependencies': [
-        'tools/v8_gypfiles/abseil.gyp:abseil',
         'node_js2c#host',
       ],
 
@@ -956,6 +955,9 @@
             'src/node_snapshot_stub.cc',
           ]
         }],
+        [ 'node_use_bundled_v8!="false"', {
+          'dependencies': [ 'tools/v8_gypfiles/abseil.gyp:abseil' ],
+        }],
         [ 'node_shared_gtest=="false"', {
           'dependencies': [
             'deps/googletest/googletest.gyp:gtest_prod',
@@ -1276,7 +1278,6 @@
 
       'dependencies': [
         '<(node_lib_target_name)',
-        'tools/v8_gypfiles/abseil.gyp:abseil',
       ],
 
       'includes': [
@@ -1310,6 +1311,9 @@
         [ 'node_shared_gtest=="true"', {
           'libraries': [ '-lgtest_main' ],
         }],
+        [ 'node_use_bundled_v8!="false"', {
+          'dependencies': [ 'tools/v8_gypfiles/abseil.gyp:abseil' ],
+        }],
         [ 'node_shared_hdr_histogram=="false"', {
           'dependencies': [
             'deps/histogram/histogram.gyp:histogram',
@@ -1548,7 +1552,7 @@
         [ 'OS=="mac"', {
           'libraries': [ '-framework CoreFoundation -framework Security' ],
         }],
-        [ 'node_shared_simdutf=="false"', {
+        [ 'node_shared_simdutf=="false" and node_use_bundled_v8!="false"', {
           'dependencies': [ 'tools/v8_gypfiles/v8.gyp:simdutf#host' ],
         }],
         [ 'node_shared_libuv=="false"', {

node.gypi

@@ -234,7 +234,7 @@
         'dependencies': [ 'deps/simdjson/simdjson.gyp:simdjson' ],
     }],
 
-    [ 'node_shared_simdutf=="false"', {
+    [ 'node_shared_simdutf=="false" and node_use_bundled_v8!="false"', {
         'dependencies': [ 'tools/v8_gypfiles/v8.gyp:simdutf' ],
     }],
 

tools/nix/v8.nix

@@ -8,6 +8,7 @@
 
 let
   nodejs = pkgs.nodejs-slim_latest;
+  v8Dir = ../../deps/v8;
 
   version =
     let
@@ -16,7 +17,7 @@ let
         + "#define V8_MINOR_VERSION ([0-9]+).*"
         + "#define V8_BUILD_NUMBER ([0-9]+).*"
         + "#define V8_PATCH_LEVEL ([0-9]+).*"
-      ) (builtins.readFile ../../deps/v8/include/v8-version.h);
+      ) (builtins.readFile "${v8Dir}/include/v8-version.h");
       v8_embedder_string = builtins.match ".*'v8_embedder_string': '-(node.[0-9]+)'.*" (
         builtins.readFile ../../common.gypi
       );
@@ -41,7 +42,7 @@ pkgs.stdenv.mkDerivation (finalAttrs: {
         ../../configure.py
         ../../deps/inspector_protocol/inspector_protocol.gyp
         ../../deps/ncrypto/ncrypto.gyp
-        ../../deps/v8
+        v8Dir
         ../../node.gyp
         ../../node.gypi
         ../../src/inspector/node_inspector.gypi
@@ -89,7 +90,7 @@ pkgs.stdenv.mkDerivation (finalAttrs: {
   '';
   installPhase = ''
     ${
-      if pkgs.stdenv.buildPlatform.isDarwin then
+      if pkgs.stdenv.hostPlatform.isDarwin then
         # Darwin is excluded from creating thin archive in tools/gyp/pylib/gyp/generator/ninja.py:2488
         "install -Dm644 out/Release/lib* -t $out/lib"
       else
@@ -109,22 +110,17 @@ pkgs.stdenv.mkDerivation (finalAttrs: {
           done
         ''
     }
-    libs=$(for f in $out/lib/lib*.a; do
-      b=$(basename "$f" .a)
-      printf " -l%s" "''${b#lib}"
-    done)
-
-    # copy v8 headers
-    cp -r deps/v8/include $out/
 
-    # create a pkgconfig file for v8
     mkdir -p $out/lib/pkgconfig
     cat -> $out/lib/pkgconfig/v8.pc << EOF
     Name: v8
     Description: V8 JavaScript Engine build for Node.js CI
     Version: ${version}
-    Libs: -L$out/lib $libs
-    Cflags: -I$out/include
+    Libs: -L$out/lib $(for f in $out/lib/lib*.a; do
+      b=$(basename "$f" .a)
+      printf " -l%s" "''${b#lib}"
+    done)
+    Cflags: -I${v8Dir}/include -I${v8Dir}/third_party/abseil-cpp -I${v8Dir}/third_party/simdutf
     EOF
   '';
 })