diff options
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkActionFactory.java | 5 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkActionFactoryApi.java | 11 |
2 files changed, 9 insertions, 7 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkActionFactory.java b/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkActionFactory.java index 72e2a5f802..755a909214 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkActionFactory.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/skylark/SkylarkActionFactory.java @@ -290,7 +290,10 @@ public class SkylarkActionFactory implements SkylarkActionFactoryApi { } } else if (commandUnchecked instanceof SkylarkList) { SkylarkList commandList = (SkylarkList) commandUnchecked; - + if (argumentList.size() > 0) { + throw new EvalException(location, + "'arguments' must be empty if 'command' is a sequence of strings"); + } @SuppressWarnings("unchecked") List<String> command = commandList.getContents(String.class, "command"); builder.setShellCommand(command); diff --git a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkActionFactoryApi.java b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkActionFactoryApi.java index 15d940ea74..282890de33 100644 --- a/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkActionFactoryApi.java +++ b/src/main/java/com/google/devtools/build/lib/skylarkbuildapi/SkylarkActionFactoryApi.java @@ -369,12 +369,11 @@ public interface SkylarkActionFactoryApi extends SkylarkValue { positional = false, doc = "Shell command to execute.<br><br>" - + "<b>Passing a sequence of strings to this attribute is deprecated and Blaze" - + "may " - + "stop accepting such values in the future.</b><br><br>" - + "The command can access the elements of the <code>arguments</code> object " - + "via " - + "<code>$1</code>, <code>$2</code>, etc.<br>" + + "<b>Passing a sequence of strings to this attribute is deprecated and Blaze " + + "may stop accepting such values in the future.</b> If a sequence of strings " + + "is passed, then <code>arguments</code> must not be used.<br><br> " + + "The command string can access the elements of the <code>arguments</code> " + + "object via <code>$1</code>, <code>$2</code>, etc.<br> " + "When this argument is a string, it must be a valid shell command. For " + "example: " + "\"<code>echo foo > $1</code>\". Blaze uses the same shell to execute the " |