diff options
author | laurentlb <laurentlb@google.com> | 2018-05-16 10:49:15 -0700 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-05-16 10:51:08 -0700 |
commit | 09431dab8ab63d296ef518e20d3e4ef9610aa56a (patch) | |
tree | 774beac4b827d8833f9ec9b80c720b7af4c58e71 /src/main | |
parent | 188b578c0e9995c864d284f22de4e48c220b0596 (diff) |
Remove --incompatible_disable_glob_tracking flag.
RELNOTES:
Flag `--incompatible_disable_glob_tracking` is removed.
PiperOrigin-RevId: 196852718
Diffstat (limited to 'src/main')
4 files changed, 4 insertions, 27 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/packages/PackageFactory.java b/src/main/java/com/google/devtools/build/lib/packages/PackageFactory.java index ff021167ba..c65b4aa8e4 100644 --- a/src/main/java/com/google/devtools/build/lib/packages/PackageFactory.java +++ b/src/main/java/com/google/devtools/build/lib/packages/PackageFactory.java @@ -579,12 +579,10 @@ public final class PackageFactory { } GlobList<String> globList = GlobList.captureResults(includes, excludes, matches); - if (env.getSemantics().incompatibleDisableGlobTracking()) { - // Converting to ImmutableList will remove glob information from the list. - return MutableList.copyOf(env, ImmutableList.copyOf(globList)); - } else { - return MutableList.copyOf(env, globList); - } + + // TODO(laurentlb): Get rid of globList type. + // Converting to ImmutableList will remove glob information from the list. + return MutableList.copyOf(env, ImmutableList.copyOf(globList)); } /** diff --git a/src/main/java/com/google/devtools/build/lib/packages/SkylarkSemanticsCodec.java b/src/main/java/com/google/devtools/build/lib/packages/SkylarkSemanticsCodec.java index af06231ffe..28b3c6747a 100644 --- a/src/main/java/com/google/devtools/build/lib/packages/SkylarkSemanticsCodec.java +++ b/src/main/java/com/google/devtools/build/lib/packages/SkylarkSemanticsCodec.java @@ -46,7 +46,6 @@ public final class SkylarkSemanticsCodec implements ObjectCodec<SkylarkSemantics codedOut.writeBoolNoTag(semantics.incompatibleBzlDisallowLoadAfterStatement()); codedOut.writeBoolNoTag(semantics.incompatibleDepsetIsNotIterable()); codedOut.writeBoolNoTag(semantics.incompatibleDepsetUnion()); - codedOut.writeBoolNoTag(semantics.incompatibleDisableGlobTracking()); codedOut.writeBoolNoTag(semantics.incompatibleDisableObjcProviderResources()); codedOut.writeBoolNoTag(semantics.incompatibleDisallowDictPlus()); codedOut.writeBoolNoTag(semantics.incompatibleDisallowFileType()); @@ -70,7 +69,6 @@ public final class SkylarkSemanticsCodec implements ObjectCodec<SkylarkSemantics builder.incompatibleBzlDisallowLoadAfterStatement(codedIn.readBool()); builder.incompatibleDepsetIsNotIterable(codedIn.readBool()); builder.incompatibleDepsetUnion(codedIn.readBool()); - builder.incompatibleDisableGlobTracking(codedIn.readBool()); builder.incompatibleDisableObjcProviderResources(codedIn.readBool()); builder.incompatibleDisallowDictPlus(codedIn.readBool()); builder.incompatibleDisallowFileType(codedIn.readBool()); diff --git a/src/main/java/com/google/devtools/build/lib/packages/SkylarkSemanticsOptions.java b/src/main/java/com/google/devtools/build/lib/packages/SkylarkSemanticsOptions.java index e62f837088..82c89a025a 100644 --- a/src/main/java/com/google/devtools/build/lib/packages/SkylarkSemanticsOptions.java +++ b/src/main/java/com/google/devtools/build/lib/packages/SkylarkSemanticsOptions.java @@ -106,19 +106,6 @@ public class SkylarkSemanticsOptions extends OptionsBase implements Serializable public boolean incompatibleDepsetIsNotIterable; @Option( - name = "incompatible_disable_glob_tracking", - defaultValue = "true", - documentationCategory = OptionDocumentationCategory.UNCATEGORIZED, - effectTags = {OptionEffectTag.UNKNOWN}, - metadataTags = { - OptionMetadataTag.INCOMPATIBLE_CHANGE, - OptionMetadataTag.TRIGGERED_BY_ALL_INCOMPATIBLE_CHANGES - }, - help = "If set to true, do not track the values of globs (this is used by rare specific cases" - ) - public boolean incompatibleDisableGlobTracking; - - @Option( name = "incompatible_disable_objc_provider_resources", defaultValue = "false", documentationCategory = OptionDocumentationCategory.UNCATEGORIZED, @@ -288,7 +275,6 @@ public class SkylarkSemanticsOptions extends OptionsBase implements Serializable .incompatibleBzlDisallowLoadAfterStatement(incompatibleBzlDisallowLoadAfterStatement) .incompatibleDepsetIsNotIterable(incompatibleDepsetIsNotIterable) .incompatibleDepsetUnion(incompatibleDepsetUnion) - .incompatibleDisableGlobTracking(incompatibleDisableGlobTracking) .incompatibleDisableObjcProviderResources(incompatibleDisableObjcProviderResources) .incompatibleDisallowDictPlus(incompatibleDisallowDictPlus) .incompatibleDisallowFileType(incompatibleDisallowFileType) diff --git a/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java b/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java index b4b23880c2..d19b2db21c 100644 --- a/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java +++ b/src/main/java/com/google/devtools/build/lib/syntax/SkylarkSemantics.java @@ -45,8 +45,6 @@ public abstract class SkylarkSemantics { public abstract boolean incompatibleDepsetUnion(); - public abstract boolean incompatibleDisableGlobTracking(); - public abstract boolean incompatibleDisableObjcProviderResources(); public abstract boolean incompatibleDisallowDictPlus(); @@ -89,7 +87,6 @@ public abstract class SkylarkSemantics { .incompatibleBzlDisallowLoadAfterStatement(false) .incompatibleDepsetIsNotIterable(false) .incompatibleDepsetUnion(false) - .incompatibleDisableGlobTracking(true) .incompatibleDisableObjcProviderResources(false) .incompatibleDisallowDictPlus(false) .incompatibleDisallowFileType(false) @@ -115,8 +112,6 @@ public abstract class SkylarkSemantics { public abstract Builder incompatibleDepsetUnion(boolean value); - public abstract Builder incompatibleDisableGlobTracking(boolean value); - public abstract Builder incompatibleDisableObjcProviderResources(boolean value); public abstract Builder incompatibleDisallowDictPlus(boolean value); |