Use && for internal_copied_filegroup.

So that the rule fails if one or more files cannot be copied.

Use && for internal_copied_filegroup.
So that the rule fails if one or more files cannot be copied.
6ddcae22 · Jisi Liu · 66e3a6d0 · 6ddcae22
Commit 6ddcae22 authored Oct 21, 2015 by Jisi Liu
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 3 deletions

protobuf.bzl protobuf.bzl +3 -3

No files found.
--- a/protobuf.bzl
+++ b/protobuf.bzl
@@ -180,9 +180,9 @@ def internal_copied_filegroup(
      name=name+"_genrule",
      srcs=srcs,
      outs=outs,
-      cmd=";".join(["cp $(location %s) $(location %s)" % \
-                    (s, _RelativeOutputPath(s, include)) \
-                    for s in srcs]))
+      cmd=" && ".join(["cp $(location %s) $(location %s)" %
+                       (s, _RelativeOutputPath(s, include))
+                       for s in srcs]))

  native.filegroup(
      name=name,