Skip to content

Commit 8817b08

Browse files
devversionthePunderWoman
authored andcommitted
build: update postinstall patch to account for benchmark macro changes (angular#44830)
Updates the postinstall patch for the benchmark macro rule from dev-infra. We moved the ZoneJS setup to the bundler. This was necessary in order to switch away from the Go-based (windows-incompatible, m1-incompatible) concatjs devserver to a rather basic HTTP server (also provided by dev-infra now). PR Close angular#44830
1 parent 23732bf commit 8817b08

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed

packages/zone.js/dist/BUILD.bazel

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
1-
package(default_visibility = ["//visibility:public"])
2-
1+
load("@build_bazel_rules_nodejs//:index.bzl", "js_library")
32
load("//packages/zone.js/dist:tools.bzl", "copy_es2015_to_dist", "copy_es5_to_dist")
43

4+
package(default_visibility = ["//visibility:public"])
5+
56
copy_es2015_to_dist()
67

78
copy_es5_to_dist()
@@ -34,6 +35,13 @@ genrule(
3435
cmd = "cp $< $@",
3536
)
3637

38+
js_library(
39+
name = "zone",
40+
srcs = [":zone.js"],
41+
package_name = "zone.js/dist",
42+
visibility = ["//modules/benchmarks:__subpackages__"],
43+
)
44+
3745
filegroup(
3846
name = "dist_bundle_group",
3947
srcs = [

tools/postinstall-patches.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,7 @@ captureNgDevPatches(
8181
[
8282
['@npm//@angular/platform-browser', '@angular//packages/platform-browser'],
8383
['@npm//@angular/core', '@angular//packages/core'],
84-
['@npm//:node_modules/zone.js/bundles/zone.umd.js', '//packages/zone.js/bundles:zone.umd.js'],
84+
['@npm//zone.js', '//packages/zone.js/dist:zone'],
8585
[
8686
'load\\("@npm//@angular/bazel:index.bzl", "ng_module"\\)',
8787
'load\("@angular//tools:defaults.bzl", "ng_module"\)'

0 commit comments

Comments
 (0)