opendev/gerrit
Code Issues Proposed changes

Add app to srcs polygerrit_plugin only if needed

Browse Source 
Change-Id: I381e4a3d24b1c70f781acda6e3e6803645fd8e88
This commit is contained in:
Viktar Donich 2018-07-12 10:00:22 -07:00
parent 7b5929068b
commit d3adb10813
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
tools/bzl/js.bzl
View File

@ -443,7 +443,13 @@ def polygerrit_plugin(name, app, srcs = [], assets = None, **kwargs):
"""
# Combines all .js and .html files into foo_combined.js and foo_combined.html
_vulcanize_rule(name = name + "_combined", app = app, srcs = srcs + [app], pkg = PACKAGE_NAME, **kwargs)
_vulcanize_rule(
name = name + "_combined",
app = app,
srcs = srcs if app in srcs else srcs + [app],
pkg = PACKAGE_NAME,
**kwargs
)
closure_js_binary(
name = name + "_bin",