Skip to content

Commit

Permalink
build: use variable for simdutf path
Browse files Browse the repository at this point in the history
PR-URL: nodejs#56196
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Cheng Zhao <zcbenz@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
  • Loading branch information
codebytere authored Dec 11, 2024
1 parent 9ec8b9e commit a1d980c
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
6 changes: 5 additions & 1 deletion node.gni
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,15 @@ declare_args() {
# The location of Node.js in source code tree.
node_path = "//node"

# The location of V8, use the one from node's deps by default.
# The location of V8 - use the one from node's deps by default.
node_v8_path = "$node_path/deps/v8"

# The location of OpenSSL - use the one from node's deps by default.
node_openssl_path = "$node_path/deps/openssl"

# The location of simdutf - use the one from node's deps by default.
node_simdutf_path = "$node_path/deps/simdutf"

# The NODE_MODULE_VERSION defined in node_version.h.
node_module_version = exec_script("$node_path/tools/getmoduleversion.py", [], "value")

Expand Down
6 changes: 3 additions & 3 deletions unofficial.gni
Original file line number Diff line number Diff line change
Expand Up @@ -158,10 +158,10 @@ template("node_gn_build") {
"deps/nghttp2",
"deps/ngtcp2",
"deps/postject",
"deps/simdutf",
"deps/sqlite",
"deps/uvwasi",
"//third_party/zlib",
"$node_simdutf_path",
"$node_v8_path:v8_libplatform",
]

Expand Down Expand Up @@ -300,8 +300,8 @@ template("node_gn_build") {

executable("node_js2c") {
deps = [
"deps/simdutf",
"deps/uv",
"$node_simdutf_path",
]
sources = [
"tools/js2c.cc",
Expand Down Expand Up @@ -358,7 +358,7 @@ template("node_gn_build") {
"deps/googletest",
"deps/googletest:gtest_main",
"deps/nbytes",
"deps/simdutf",
"$node_simdutf_path",
]

sources = gypi_values.node_cctest_sources
Expand Down

0 comments on commit a1d980c

Please sign in to comment.