diff options
author | Simon Warta <[email protected]> | 2017-04-05 17:38:12 +0200 |
---|---|---|
committer | Simon Warta <[email protected]> | 2017-04-07 09:10:14 +0200 |
commit | 4857ab9f3ffe5578c047f2a99baa2b9fbeca9af5 (patch) | |
tree | 39f38d4bc6d6cf73dc6daef76b042dd5e1fceb67 /configure.py | |
parent | 80ead8f098847943e519774a35858473edeca977 (diff) |
Do not enable and disable module at the same time
Diffstat (limited to 'configure.py')
-rwxr-xr-x | configure.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/configure.py b/configure.py index c726335f7..92b891a82 100755 --- a/configure.py +++ b/configure.py @@ -2004,8 +2004,7 @@ def choose_modules_to_use(modules, module_policy, archinfo, ccinfo, options): elif usable: if modname in options.enabled_modules: to_load.add(modname) # trust the user - - if module.load_on == 'never': + elif module.load_on == 'never': not_using_because['disabled as buggy'].add(modname) elif module.load_on == 'request': if options.with_everything: @@ -2092,9 +2091,10 @@ def choose_modules_to_use(modules, module_policy, archinfo, ccinfo, options): logging.info('Loading modules: %s', ' '.join(sorted_modules_to_load)) def validate_state(used_modules, unused_modules): - for _, unused_for_reason in unused_modules.items(): + for reason, unused_for_reason in unused_modules.items(): if not unused_for_reason.isdisjoint(used_modules): - raise InternalError("Disabled modules and modules to load have common elements") + raise InternalError( + "Disabled modules (%s) and modules to load have common elements" % reason) validate_state(to_load, not_using_because) |