diff options
| author |  Googler <noreply@google.com> | 2017-07-27 13:42:07 +0200 |
|---|---|---|
| committer |  Jakob Buchgraber <buchgr@google.com> | 2017-07-27 16:26:22 +0200 |
| commit | 2c0583f3b7d7966b751c8db55d0a7aed520120af (patch) | |
| tree | bf00aff789859ddba6b8acf4a78490efbd2c1b9e /src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java | |
| parent | a95150a42a24f8a7af718514dce4d4a326839d1e (diff) | |
Exclude test/java/ in instrumentation_filter default
The computed default used in bazel coverage excludes both javatests/ and test/java. Having the stated default exclude only one of those is needlessly confusing. (Even though in practice that static default will rarely be used.)
Also updates the comment about the computed default, since that logic has moved.
PiperOrigin-RevId: 163325837
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java index 802ca947c7..a77a5242f1 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java @@ -596,11 +596,11 @@ public final class BuildConfiguration implements BuildEvent { public boolean stampBinaries; // This default value is always overwritten in the case of "bazel coverage" by - // CoverageCommand.setDefaultInstrumentationFilter(). + // a value returned by InstrumentationFilterSupport.computeInstrumentationFilter. @Option( name = "instrumentation_filter", converter = RegexFilter.RegexFilterConverter.class, - defaultValue = "-/javatests[/:]", + defaultValue = "-/javatests[/:],-/test/java[/:]", category = "semantics", documentationCategory = OptionDocumentationCategory.UNCATEGORIZED, effectTags = {OptionEffectTag.UNKNOWN}, |