commit edd531d9fb9ff6b47ac2ea7a530fea8e5deef2bd
author Ulf Adams <ulfjack@google.com> Mon Mar 20 15:01:45 2017 +0000
committer Yue Gan <yueg@google.com> Tue Mar 21 12:47:59 2017 +0000
tree 226be86d2a2ad4c9c98ff70837a2c0bc9908cafc
parent 4475bf14d7963274f8cc48be629684c05a484c44

Use a relative path for the runfiles tree

Add preconditions to enforce this and remove some now unnecessary code.

A small step towards #1593.

--
PiperOrigin-RevId: 150625693
MOS_MIGRATED_REVID=150625693

src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupplierImpl.java

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupplierImpl.java b/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupplierImpl.java
index 79ecce1..a863234 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupplierImpl.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/RunfilesSupplierImpl.java
@@ -65,6 +65,7 @@
       PathFragment runfilesDir,
       Runfiles runfiles,
       @Nullable Artifact manifest) {
+    Preconditions.checkArgument(!runfilesDir.isAbsolute());
     this.runfilesDir = Preconditions.checkNotNull(runfilesDir);
     this.runfiles = Preconditions.checkNotNull(runfiles);
     this.manifest = manifest;