diff options
author | Richard Maw <richard.maw@codethink.co.uk> | 2013-08-07 12:00:49 +0000 |
---|---|---|
committer | Richard Maw <richard.maw@codethink.co.uk> | 2013-08-07 13:41:40 +0100 |
commit | 40be81b74d63e1ba7bd3e10a78c79d3829673172 (patch) | |
tree | 1c88237b7431c734ec515df2067ad15e06327c80 /morphlib/builder2.py | |
parent | 66291811c81ed21f5005ba35e68898c73edd88bd (diff) | |
download | morph-40be81b74d63e1ba7bd3e10a78c79d3829673172.tar.gz |
morphologyfactory: move empty stratum check here
It is better to test whether a straum is empty here, since it will be
noticed earlier in the build, as soon as the morphologies are loaded,
rather than after they have all been parsed.
It is also conceptually nicer to put it here, since the morphologyfactory
was written to perform this kind of validation.
On a more practical note, the validation is moved here so that the test
for this error isn't masked by the test for no build dependencies.
To ensure tests still pass, we alter the stratum morphology used by
other unit tests to no longer be empty, and add an empty one to test.
Diffstat (limited to 'morphlib/builder2.py')
-rw-r--r-- | morphlib/builder2.py | 9 |
1 files changed, 0 insertions, 9 deletions
diff --git a/morphlib/builder2.py b/morphlib/builder2.py index daf50b56..e1eaaa86 100644 --- a/morphlib/builder2.py +++ b/morphlib/builder2.py @@ -469,8 +469,6 @@ class StratumBuilder(BuilderBase): with self.build_watch('overall-build'): constituents = [d for d in self.artifact.dependencies if self.is_constituent(d)] - if len(constituents) == 0: - raise EmptyStratumError(self.artifact.name) # the only reason the StratumBuilder has to download chunks is to # check for overlap now that strata are lists of chunks @@ -707,10 +705,3 @@ class Builder(object): # pragma: no cover built_artifacts = o.build_and_cache() logging.debug('Builder.build: done') return built_artifacts - - -class EmptyStratumError(cliapp.AppException): - - def __init__(self, stratum_name): # pragma: no cover - cliapp.AppException.__init__(self, - "Stratum %s is empty (has no dependencies)" % stratum_name) |