From f6cccae5b6f9c0ad0e7d0bf7bd31ea1263449316 Mon Sep 17 00:00:00 2001 From: Googler Date: Fri, 28 Jan 2022 07:18:37 -0800 Subject: [PATCH] * add change to allow blaze info to skip Starlark build settings that start with --no prefix * add unit tests for both info and clean commands PiperOrigin-RevId: 424863510 --- .../devtools/build/lib/runtime/StarlarkOptionsParser.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/com/google/devtools/build/lib/runtime/StarlarkOptionsParser.java b/src/main/java/com/google/devtools/build/lib/runtime/StarlarkOptionsParser.java index 1d9eb89d490305..ca412a1864cd63 100644 --- a/src/main/java/com/google/devtools/build/lib/runtime/StarlarkOptionsParser.java +++ b/src/main/java/com/google/devtools/build/lib/runtime/StarlarkOptionsParser.java @@ -330,7 +330,7 @@ public static Pair, ImmutableList> removeStarlarkO String potentialStarlarkFlag = name.substring(2); // Check if the string uses the "no" prefix for setting boolean flags to false, trim // off "no" if so. - if (name.startsWith("no")) { + if (potentialStarlarkFlag.startsWith("no")) { potentialStarlarkFlag = potentialStarlarkFlag.substring(2); } // Check if the string contains a value, trim off the value if so.