diff --git a/src/main/java/com/google/devtools/build/lib/packages/semantics/BuildLanguageOptions.java b/src/main/java/com/google/devtools/build/lib/packages/semantics/BuildLanguageOptions.java index 1b1b2e0b41fe3d..f7e2056842578a 100644 --- a/src/main/java/com/google/devtools/build/lib/packages/semantics/BuildLanguageOptions.java +++ b/src/main/java/com/google/devtools/build/lib/packages/semantics/BuildLanguageOptions.java @@ -287,7 +287,7 @@ public final class BuildLanguageOptions extends OptionsBase { @Option( name = "incompatible_no_implicit_watch_label", - defaultValue = "false", + defaultValue = "true", documentationCategory = OptionDocumentationCategory.STARLARK_SEMANTICS, metadataTags = OptionMetadataTag.INCOMPATIBLE_CHANGE, effectTags = OptionEffectTag.LOADING_AND_ANALYSIS, @@ -963,7 +963,7 @@ public StarlarkSemantics toStarlarkSemantics() { public static final String EXPERIMENTAL_ISOLATED_EXTENSION_USAGES = "-experimental_isolated_extension_usages"; public static final String INCOMPATIBLE_NO_IMPLICIT_WATCH_LABEL = - "-incompatible_no_implicit_watch_label"; + "+incompatible_no_implicit_watch_label"; public static final String EXPERIMENTAL_GOOGLE_LEGACY_API = "-experimental_google_legacy_api"; public static final String EXPERIMENTAL_PLATFORMS_API = "-experimental_platforms_api"; public static final String EXPERIMENTAL_REPO_REMOTE_EXEC = "-experimental_repo_remote_exec"; diff --git a/src/test/java/com/google/devtools/build/lib/bazel/repository/starlark/StarlarkRepositoryContextTest.java b/src/test/java/com/google/devtools/build/lib/bazel/repository/starlark/StarlarkRepositoryContextTest.java index 8530a03f5d9f67..49b1fd674ad769 100644 --- a/src/test/java/com/google/devtools/build/lib/bazel/repository/starlark/StarlarkRepositoryContextTest.java +++ b/src/test/java/com/google/devtools/build/lib/bazel/repository/starlark/StarlarkRepositoryContextTest.java @@ -516,7 +516,11 @@ public void testWorkspaceRoot() throws Exception { @Test public void testNoIncompatibleNoImplicitWatchLabel() throws Exception { - setUpContextForRule("test"); + setUpContextForRule( + "test", + StarlarkSemantics.DEFAULT.toBuilder() + .setBool(BuildLanguageOptions.INCOMPATIBLE_NO_IMPLICIT_WATCH_LABEL, false) + .build()); scratch.file(root.getRelative("foo").getPathString()); StarlarkPath unusedPath = context.getPath(fakeFileLabel); String unusedRead = context.readFile(fakeFileLabel, "no", thread);