Update --incompatible_* error messages

RELNOTES: None PiperOrigin-RevId: 159954456
sigaev · Jun 26, 2017 · f5b8d6f · f5b8d6f
1 parent 0f5b720
commit f5b8d6f
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 6 deletions.
diff --git a/src/main/java/com/google/devtools/build/lib/syntax/LoadStatement.java b/src/main/java/com/google/devtools/build/lib/syntax/LoadStatement.java
@@ -66,7 +66,7 @@ void doExec(Environment env) throws EvalException, InterruptedException {
         throw new EvalException(
             getLocation(),
             "First argument of 'load' must be a label and start with either '//' or ':'. "
-                + "Use --incompatibleLoadArgumentIsLabel to temporarily disable this check.");
+                + "Use --incompatibleLoadArgumentIsLabel=false to temporarily disable this check.");
       }
     }
 

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/MethodLibrary.java b/src/main/java/com/google/devtools/build/lib/syntax/MethodLibrary.java
@@ -1617,8 +1617,9 @@ public Integer invoke(Object x, Location loc, Environment env) throws EvalExcept
             throw new EvalException(
                 loc,
                 EvalUtils.getDataTypeName(x)
-                    + " is not iterable. Use --incompatible_depset_is_not_iterable=false to "
-                    + "temporarily disable this check.");
+                    + " is not iterable. You may use `len(<depset>.to_list())` instead. Use "
+                    + "--incompatible_depset_is_not_iterable=false to temporarily disable this "
+                    + "check.");
           }
           int l = EvalUtils.size(x);
           if (l == -1) {

diff --git a/src/main/java/com/google/devtools/build/lib/syntax/ValidationEnvironment.java b/src/main/java/com/google/devtools/build/lib/syntax/ValidationEnvironment.java
@@ -155,8 +155,8 @@ private static void checkLoadAfterStatement(List<Statement> statements) throws E
             "load() statements must be called before any other statement. "
                 + "First non-load() statement appears at "
                 + firstStatement
-                + ". Use --incompatible_bzl_disallow_load_after_statement to temporarily disable "
-                + "this check.");
+                + ". Use --incompatible_bzl_disallow_load_after_statement=false to temporarily "
+                + "disable this check.");
       }
 
       if (firstStatement == null) {
@@ -173,7 +173,7 @@ private static void checkToplevelIfStatement(List<Statement> statements) throws
             statement.getLocation(),
             "if statements are not allowed at the top level. You may move it inside a function "
                 + "or use an if expression (x if condition else y). "
-                + "Use --incompatible_disallow_toplevel_if_statement to temporarily disable "
+                + "Use --incompatible_disallow_toplevel_if_statement=false to temporarily disable "
                 + "this check.");
       }
     }