Skip to content

Commit bf80e01

Browse files
committed
Revert "Fix emsdk patch to exclude nodejs dependency"
This reverts commit 38a0332. Emscripten depends on nodejs, so this was wrong. Root cause is likely some sort of CI-env fetch issue. Signed-off-by: Martijn Stevenson <[email protected]>
1 parent aee8ad9 commit bf80e01

File tree

1 file changed

+2
-14
lines changed

1 file changed

+2
-14
lines changed

bazel/emsdk.patch

Lines changed: 2 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,8 @@
11
diff --git a/bazel/emscripten_deps.bzl b/bazel/emscripten_deps.bzl
2-
index 219cd93..4842085 100644
2+
index 219cd93..c127d49 100644
33
--- a/bazel/emscripten_deps.bzl
44
+++ b/bazel/emscripten_deps.bzl
5-
@@ -111,11 +111,6 @@ def emscripten_deps(emscripten_version = "latest"):
6-
# This could potentially backfire for projects with multiple emscripten
7-
# dependencies that use different emscripten versions
8-
excludes = native.existing_rules().keys()
9-
- if "nodejs_toolchains" not in excludes:
10-
- # Node 16 is the first version that supports darwin_arm64
11-
- node_repositories(
12-
- node_version = "16.6.2",
13-
- )
14-
15-
if "emscripten_bin_linux" not in excludes:
16-
http_archive(
17-
@@ -167,38 +162,3 @@ def emscripten_deps(emscripten_version = "latest"):
5+
@@ -167,38 +167,3 @@ def emscripten_deps(emscripten_version = "latest"):
186
build_file_content = BUILD_FILE_CONTENT_TEMPLATE.format(bin_extension = ".exe"),
197
type = "zip",
208
)

0 commit comments

Comments
 (0)