From ceed7f1a72634ae63e1d63ca038bbddab604b113 Mon Sep 17 00:00:00 2001 From: Ryan Beasley <39353016+beasleyr-vmw@users.noreply.github.com> Date: Tue, 13 Sep 2022 12:00:39 -0400 Subject: [PATCH] * add change to allow blaze info to skip Starlark build settings that start with --no prefix (#16258) * add unit tests for both info and clean commands PiperOrigin-RevId: 424863510 Co-authored-by: Googler --- .../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 0c60ee576e0541..9a0685c6ac30ae 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 @@ -333,7 +333,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.