commit 124bd0a7d32c28cc53ef8d8beab4e422d5ee9c42
author Ulf Adams <ulfjack@google.com> Wed Aug 10 13:10:46 2016 +0000
committer Yue Gan <yueg@google.com> Wed Aug 10 13:55:45 2016 +0000
tree e07f44003e204429656f911c0476e689b325597b
parent 7db73d3236f59029a8a59c2a46d342eb897012b6

Remove the outputPath parameter from BlazeModule.getFileSystem.

It's never used, and it's tricky to run with multiple different file systems
in the same process -> remove it to simplify.

--
MOS_MIGRATED_REVID=129858142

src/main/java/com/google/devtools/build/lib/runtime/BlazeRuntime.java

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/BlazeRuntime.java b/src/main/java/com/google/devtools/build/lib/runtime/BlazeRuntime.java
index 851f647..eb3e6a6 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/BlazeRuntime.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/BlazeRuntime.java
@@ -882,11 +882,9 @@
           "Bad --output_base option specified: '" + outputBase + "'");
     }
 
-    PathFragment outputPathFragment = BlazeDirectories.outputPathFromOutputBase(
-        outputBase, workspaceDirectory, startupOptions.deepExecRoot, productName);
     FileSystem fs = null;
     for (BlazeModule module : blazeModules) {
-      FileSystem moduleFs = module.getFileSystem(options, outputPathFragment);
+      FileSystem moduleFs = module.getFileSystem(options);
       if (moduleFs != null) {
         Preconditions.checkState(fs == null, "more than one module returns a file system");
         fs = moduleFs;