diff options
author | Kristina Chodorow <kchodorow@google.com> | 2016-01-29 15:04:31 +0000 |
---|---|---|
committer | Kristina Chodorow <kchodorow@google.com> | 2016-01-29 15:36:36 +0000 |
commit | 734e7f7b63c9c00a6aaa60769481a11bc4f76346 (patch) | |
tree | d51ce64080b580a98540ef7fd16b5ec5369d9e22 /src/main/java/com/google/devtools/build/lib/skyframe/GraphBackedRecursivePackageProvider.java | |
parent | f9f2e10dcbbf0efade6a9c9cf21dd3dde8e55f6b (diff) |
Parse the workspace name when a repository is loaded
Moved RepositoryValue to RepositoryDirectoryValue so that it could be cached
(and not re-downloaded) even if the WorkspaceAST caused a Skyframe restart
(as mentioned in https://github.com/bazelbuild/bazel/blob/master/src/main/java/com/google/devtools/build/lib/rules/repository/RepositoryDelegatorFunction.java#L130-L133).
--
MOS_MIGRATED_REVID=113358489
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/GraphBackedRecursivePackageProvider.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/skyframe/GraphBackedRecursivePackageProvider.java | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/GraphBackedRecursivePackageProvider.java b/src/main/java/com/google/devtools/build/lib/skyframe/GraphBackedRecursivePackageProvider.java index 60221ad319..8c0e56cf0e 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/GraphBackedRecursivePackageProvider.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/GraphBackedRecursivePackageProvider.java @@ -37,6 +37,7 @@ import com.google.devtools.build.lib.packages.Package; import com.google.devtools.build.lib.packages.Target; import com.google.devtools.build.lib.pkgcache.PathPackageLocator; import com.google.devtools.build.lib.pkgcache.RecursivePackageProvider; +import com.google.devtools.build.lib.rules.repository.RepositoryDirectoryValue; import com.google.devtools.build.lib.skyframe.TargetPatternValue.TargetPatternKey; import com.google.devtools.build.lib.util.Preconditions; import com.google.devtools.build.lib.vfs.Path; @@ -172,8 +173,8 @@ public final class GraphBackedRecursivePackageProvider implements RecursivePacka if (repository.isDefault()) { roots.addAll(pkgPath.getPathEntries()); } else { - RepositoryValue repositoryValue = - (RepositoryValue) graph.getValue(RepositoryValue.key(repository)); + RepositoryDirectoryValue repositoryValue = + (RepositoryDirectoryValue) graph.getValue(RepositoryDirectoryValue.key(repository)); if (repositoryValue == null) { // If this key doesn't exist, the repository is outside the universe, so we return // "nothing". |