Forbid **kwargs and *args in BUILD files.

Rationale: it makes BUILD files less declarative and makes harder to do
automated changes on BUILD files. It is however still allowed in .bzl
files.

--
MOS_MIGRATED_REVID=94577442

1 files changed, 14 insertions, 0 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/Parser.java b/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
index ae267e52e3..67e13e537c 100644
--- a/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
+++ b/src/main/java/com/google/devtools/build/lib/syntax/Parser.java
@@ -407,16 +407,30 @@ class Parser {
 
   // arg ::= IDENTIFIER '=' nontupleexpr
   //       | expr
+  //       | *args       (only in Skylark mode)
+  //       | **kwargs    (only in Skylark mode)
+  // To keep BUILD files declarative and easy to process, *args and **kwargs
+  // arguments are allowed only in Skylark mode.
   private Argument.Passed parseFuncallArgument() {
     final int start = token.left;
     // parse **expr
     if (token.kind == TokenKind.STAR_STAR) {
+      if (!skylarkMode) {
+        reportError(
+            lexer.createLocation(token.left, token.right),
+            "**kwargs arguments are not allowed in BUILD files");
+      }
       nextToken();
       Expression expr = parseNonTupleExpression();
       return setLocation(new Argument.StarStar(expr), start, expr);
     }
     // parse *expr
     if (token.kind == TokenKind.STAR) {
+      if (!skylarkMode) {
+        reportError(
+            lexer.createLocation(token.left, token.right),
+            "*args arguments are not allowed in BUILD files");
+      }
       nextToken();
       Expression expr = parseNonTupleExpression();
       return setLocation(new Argument.Star(expr), start, expr);