build: remove rules_nodejs legacy rollup_bundle patch (#33915)

We no longer depend on the legacy rollup_bundle rule so we don't require this patch

PR Close #33915
This commit is contained in:
Greg Magolan 2019-11-18 17:48:11 -08:00 committed by Alex Rickabaugh
parent 29b8666b10
commit 288009d6fb
2 changed files with 0 additions and 26 deletions

View File

@ -5,22 +5,9 @@ workspace(
load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")
# Uncomment for local bazel rules development
#local_repository(
# name = "build_bazel_rules_nodejs",
# path = "../rules_nodejs",
#)
#local_repository(
# name = "npm_bazel_typescript",
# path = "../rules_typescript",
#)
# Fetch rules_nodejs so we can install our npm dependencies
http_archive(
name = "build_bazel_rules_nodejs",
patch_args = ["-p1"],
# Patch https://github.com/bazelbuild/rules_nodejs/pull/903
patches = ["//tools:rollup_bundle_commonjs_ignoreGlobal.patch"],
sha256 = "9901bc17138a79135048fb0c107ee7a56e91815ec6594c08cb9a17b80276d62b",
urls = ["https://github.com/bazelbuild/rules_nodejs/releases/download/0.40.0/rules_nodejs-0.40.0.tar.gz"],
)

View File

@ -1,13 +0,0 @@
diff --git a/internal/rollup/rollup.config.js b/internal/rollup/rollup.config.js
index 5e4e0aa..fe2848c 100644
--- a/internal/rollup/rollup.config.js
+++ b/internal/rollup/rollup.config.js
@@ -173,7 +173,7 @@ const config = {
// with the amd plugin.
include: /\.ngfactory\.js$/i,
}),
- commonjs(), {
+ commonjs({ignoreGlobal: true}), {
name: 'notResolved',
resolveId: notResolved,
},