Bazel: Don't use assets parameter in polygerrit_plugin rule
All assets parameter does is add its content to the output. We don't
really need it, as we add the assets into static directory of plugin
artifact in genrule anyway.
Given that this plugin is the only caller site that is using assets
parameter, we can kill it now in gerrit core and simplify
polygerrit_plugin rule.
Change-Id: I54d4a910631c9410883b7fd64ba0d88638f9d579
diff --git a/BUILD b/BUILD
index 8b11bd5..7e2c5c6 100644
--- a/BUILD
+++ b/BUILD
@@ -15,11 +15,14 @@
genrule2(
name = "cm-static",
- srcs = [":codemirror_editor"],
+ srcs = [
+ ":codemirror-element",
+ ":codemirror_editor",
+ ],
outs = ["cm-static.jar"],
cmd = " && ".join([
"mkdir $$TMP/static",
- "cp -r $(locations :codemirror_editor) $$TMP/static",
+ "cp $(SRCS) $$TMP/static",
"cd $$TMP",
"zip -Drq $$ROOT/$@ -g .",
]),
@@ -40,7 +43,4 @@
polygerrit_plugin(
name = "codemirror_editor",
app = "gr-editor/gr-editor.js",
- assets = [
- ":codemirror-element",
- ],
)
