aboutsummaryrefslogtreecommitdiffstats
path: root/configure.py
diff options
context:
space:
mode:
authorSimon Warta <[email protected]>2017-04-05 17:38:12 +0200
committerSimon Warta <[email protected]>2017-04-07 09:10:14 +0200
commit4857ab9f3ffe5578c047f2a99baa2b9fbeca9af5 (patch)
tree39f38d4bc6d6cf73dc6daef76b042dd5e1fceb67 /configure.py
parent80ead8f098847943e519774a35858473edeca977 (diff)
Do not enable and disable module at the same time
Diffstat (limited to 'configure.py')
-rwxr-xr-xconfigure.py8
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)