diff options
author | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2018-04-16 21:45:39 +0900 |
---|---|---|
committer | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2018-04-16 21:52:04 +0900 |
commit | 827a12947bd8323fdfb7ec16233db963df965526 (patch) | |
tree | c4026ff61360e6f91700a0c741a2d76728cdc169 | |
parent | e48f2f820e7577f2c13537543f22aa9a7cb84460 (diff) | |
download | buildstream-827a12947bd8323fdfb7ec16233db963df965526.tar.gz |
_pipeline.py: Stop printing warning when a build plan does not use some existing workspaces
Fixes issue #360
-rw-r--r-- | buildstream/_pipeline.py | 17 |
1 files changed, 0 insertions, 17 deletions
diff --git a/buildstream/_pipeline.py b/buildstream/_pipeline.py index e99b4a540..80b0e01ac 100644 --- a/buildstream/_pipeline.py +++ b/buildstream/_pipeline.py @@ -303,10 +303,6 @@ class Pipeline(): # which are required to build the target. # def build(self, scheduler, *, build_all=False): - unused_workspaces = self._collect_unused_workspaces() - if unused_workspaces: - self._message(MessageType.WARN, "Unused workspaces", - detail="\n".join([el for el in unused_workspaces])) if build_all: plan = self.dependencies(Scope.ALL) @@ -634,19 +630,6 @@ class Pipeline(): for element in self.dependencies(Scope.ALL): element._preflight() - # _collect_unused_workspaces() - # - # Collect a list of any workspaces which are unused by the pipeline - # - def _collect_unused_workspaces(self): - unused_workspaces = [] - for element_name, _ in self.project.workspaces.list(): - for target in self.targets: - element = target.search(Scope.ALL, element_name) - if element is None: - unused_workspaces.append(element_name) - return unused_workspaces - # _initialize_remote_caches() # # Initialize remote artifact caches, checking what |