diff options
| author |  gregce <gregce@google.com> | 2018-05-25 12:18:27 -0700 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2018-05-25 12:19:51 -0700 |
| commit | 475d91a2b9314e6803aba5da83156712af63f837 (patch) | |
| tree | f7550719165ab75428f73bf420f6ade67510373e /src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupport.java | |
| parent | 2897a1dbbc83f1b81f16d2de25b299e459c3a90f (diff) | |
Remove attribute read checks expecting a DATA transition.
This is a precursor to removing the DATA transition outright.
While we could also have changed the Mode.DATA instances to
Mode.TARGET (which would declare that we expect the attribute not
to apply any transition), that would break existing definitions and
make depot cleanup more delicate. Plus, these checks weren't being
consistently applied across attributes anyway so they don't really
offer much. A lot of this logic is really just leftover legacy
from the pre-dynamic configuration days.
PiperOrigin-RevId: 198085059
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupport.java')
| -rw-r--r-- | src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupport.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupport.java b/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupport.java index 0c4f797160..c6cfedf21c 100644 --- a/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupport.java +++ b/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupport.java @@ -104,7 +104,7 @@ public final class RunfilesSupport { if (runUnder != null && runUnder.getLabel() != null && TargetUtils.isTestRule(ruleContext.getRule())) { TransitiveInfoCollection runUnderTarget = - ruleContext.getPrerequisite(":run_under", Mode.DATA); + ruleContext.getPrerequisite(":run_under", Mode.DONT_CHECK); runfiles = new Runfiles.Builder( ruleContext.getWorkspaceName(), ruleContext.getConfiguration().legacyExternalRunfiles()) .merge(getRunfiles(runUnderTarget, ruleContext.getWorkspaceName())) |