Merge pull request #17 from pubref/fix_issue_16

pcj · web-flow · commit bd14a465063d · 2017-05-30T07:18:24.000-06:00
Include 'script' file in ctx.action input list
diff --git a/node/internal/node_library.bzl b/node/internal/node_library.bzl
@@ -109,7 +109,7 @@ def node_library_impl(ctx):
 
     ctx.action(
         mnemonic = "NpmInstallLocal",
-        inputs = [node, npm, package_json_file] + srcs,
+        inputs = [node, npm, package_json_file, script] + srcs,
         outputs = [npm_package_json_file],
         command = " && ".join(cmds),
     )

Original file line number	Diff line number	Diff line change
`@@ -109,7 +109,7 @@ def node_library_impl(ctx):`
`109`	`109`
`110`	`110`	`ctx.action(`
`111`	`111`	`mnemonic = "NpmInstallLocal",`
`112`		`- inputs = [node, npm, package_json_file] + srcs,`
	`112`	`+ inputs = [node, npm, package_json_file, script] + srcs,`
`113`	`113`	`outputs = [npm_package_json_file],`
`114`	`114`	`command = " && ".join(cmds),`
`115`	`115`	`)`