diff options
author | 2017-11-28 05:24:51 -0800 | |
---|---|---|
committer | 2017-11-28 05:26:55 -0800 | |
commit | 683bdc7a069a8328676740798d2fb6abc2119a2f (patch) | |
tree | aa8f8961ef1b2dbd2cbcce39b3fabd075d3ee239 /src/test/java/com/google/devtools | |
parent | 55d746c618214bb10ea81696c4e5811e30701a0e (diff) |
refactor: add LocationExpander factories
Stop using LocationExpander.Options in
LocationExpander constructors, because the Options
semantics are confusing.
I also need the refactoring in order to extend the
expansion semantics: to support expanding to
absolute paths on Windows, where $(location)
should not expand to the (non-existent) runfiles
path, but to the absolute path the runfiles
symlink would point to.
See https://github.com/bazelbuild/bazel/issues/4171
Change-Id: Ie4d47ec3807bc3c6e39156efa1746b666f69f99c
PiperOrigin-RevId: 177147372
Diffstat (limited to 'src/test/java/com/google/devtools')
-rw-r--r-- | src/test/java/com/google/devtools/build/lib/analysis/LocationExpanderIntegrationTest.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/test/java/com/google/devtools/build/lib/analysis/LocationExpanderIntegrationTest.java b/src/test/java/com/google/devtools/build/lib/analysis/LocationExpanderIntegrationTest.java index a1137cd02d..1db06a8883 100644 --- a/src/test/java/com/google/devtools/build/lib/analysis/LocationExpanderIntegrationTest.java +++ b/src/test/java/com/google/devtools/build/lib/analysis/LocationExpanderIntegrationTest.java @@ -43,7 +43,7 @@ public class LocationExpanderIntegrationTest extends BuildViewTestCase { private LocationExpander makeExpander(String label) throws Exception { ConfiguredTarget target = getConfiguredTarget(label); RuleContext ruleContext = getRuleContext(target); - return new LocationExpander(ruleContext); + return LocationExpander.withRunfilesPaths(ruleContext); } @Test |