diff options
author | Simon Warta <[email protected]> | 2017-04-05 18:44:01 +0200 |
---|---|---|
committer | Simon Warta <[email protected]> | 2017-04-07 09:10:14 +0200 |
commit | f1ff37684ae01084e115bca6ac4fad88b3980788 (patch) | |
tree | 6121365de224d9b03d7e3b27635d0f78fc8177f5 | |
parent | 6a25acab798bae4e29b16caecce2eca4a98420bd (diff) |
Extract _validate_dependencies_exist
-rwxr-xr-x | configure.py | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/configure.py b/configure.py index e93b26ea5..05ce1a7e9 100755 --- a/configure.py +++ b/configure.py @@ -1955,6 +1955,8 @@ class ModulesChooser(object): # string to set mapping with reasons as key and modules as value self._not_using_because = collections.defaultdict(set) + ModulesChooser._validate_dependencies_exist(self._modules) + def _check_usable(self, module, modname): if not module.compatible_os(self._options.os): self._not_using_because['incompatible OS'].add(modname) @@ -1999,10 +2001,12 @@ class ModulesChooser(object): raise InternalError( "Disabled modules (%s) and modules to load have common elements" % reason) - def choose(self): - for mod in self._modules.values(): - mod.dependencies_exist(self._modules) + @staticmethod + def _validate_dependencies_exist(modules): + for module in modules.values(): + module.dependencies_exist(modules) + def choose(self): to_load = set() maybe_dep = [] |