Add support for --disable-robot

It might be necessary to disable dependency resolution from the commandline.
E.g. when it is enabled in the configfile and we want to upload a single
file via `--new-pr`.
Add new cmdline action `store_or_False` to generically add support for
`--disable-<name>` for such cases.

Author: Flamefire

easybuild/base/generaloption.py

@@ -182,6 +182,8 @@ class ExtOption(CompleterOption):
            - set default to None if no option passed,
            - set to default if option without value passed,
            - set to value if option with value passed
+         - store_or_False: Same as store_or_None, but set to False instead of None
+           - Additionally supports --disable-
 
         Types:
           - strlist, strtuple : convert comma-separated string in a list resp. tuple of strings
@@ -193,14 +195,15 @@ class ExtOption(CompleterOption):
 
     ENABLE = 'enable'  # do nothing
     DISABLE = 'disable'  # inverse action
+    STORE_OR_FALSE = 'store_or_False'
 
     EXTOPTION_EXTRA_OPTIONS = ('date', 'datetime', 'regex', 'add', 'add_first', 'add_flex',)
-    EXTOPTION_STORE_OR = ('store_or_None', 'help')  # callback type
+    EXTOPTION_STORE_OR = ('store_or_None', STORE_OR_FALSE, 'help')  # callback type
     EXTOPTION_LOG = ('store_debuglog', 'store_infolog', 'store_warninglog',)
     EXTOPTION_HELP = ('shorthelp', 'confighelp', 'help')
 
     ACTIONS = Option.ACTIONS + EXTOPTION_EXTRA_OPTIONS + EXTOPTION_STORE_OR + EXTOPTION_LOG + EXTOPTION_HELP
-    STORE_ACTIONS = Option.STORE_ACTIONS + EXTOPTION_EXTRA_OPTIONS + EXTOPTION_LOG + ('store_or_None',)
+    STORE_ACTIONS = Option.STORE_ACTIONS + EXTOPTION_EXTRA_OPTIONS + EXTOPTION_LOG + ('store_or_None', STORE_OR_FALSE)
     TYPED_ACTIONS = Option.TYPED_ACTIONS + EXTOPTION_EXTRA_OPTIONS + EXTOPTION_STORE_OR
     ALWAYS_TYPED_ACTIONS = Option.ALWAYS_TYPED_ACTIONS + EXTOPTION_EXTRA_OPTIONS
 
@@ -229,7 +232,9 @@ def store_or(option, opt_str, value, parser, *args, **kwargs):  # pylint: disabl
                 """Callback for supporting options with optional values."""
                 # see http://stackoverflow.com/questions/1229146/parsing-empty-options-in-python
                 # ugly code, optparse is crap
-                if parser.rargs and not parser.rargs[0].startswith('-'):
+                if option.store_or == self.STORE_OR_FALSE and opt_str.startswith("--%s-" % self.DISABLE):
+                    val = False
+                elif parser.rargs and not parser.rargs[0].startswith('-'):
                     val = option.check_value(opt_str, parser.rargs.pop(0))
                 else:
                     val = kwargs.get('orig_default', None)
@@ -247,7 +252,7 @@ def store_or(option, opt_str, value, parser, *args, **kwargs):  # pylint: disabl
                 'orig_default': copy.deepcopy(self.default),
             }
             self.action = 'callback'  # act as callback
-            self.default = None
+            self.default = False if self.action == self.STORE_OR_FALSE else None
 
     def process(self, opt, value, values, parser):
         """Handle option-as-value issues before actually processing option."""
@@ -1212,6 +1217,8 @@ def add_group_parser(self, opt_dict, description, prefix=None, otherdefaults=Non
                 if action in self.parser.option_class.BOOLEAN_ACTIONS:
                     args.append("--%s-%s" % (self.parser.option_class.ENABLE, opt_name))
                     args.append("--%s-%s" % (self.parser.option_class.DISABLE, opt_name))
+                elif action == self.parser.option_class.STORE_OR_FALSE:
+                    args.append("--%s-%s" % (self.parser.option_class.DISABLE, opt_name))
 
             # force passed_kwargs as final nameds
             nameds.update(passed_kwargs)

easybuild/tools/options.py

@@ -278,7 +278,7 @@ def basic_options(self):
             'rebuild': ("Rebuild software, even if module already exists (don't skip OS dependencies checks)",
                         None, 'store_true', False),
             'robot': ("Enable dependency resolution, optionally consider additional paths to search for easyconfigs",
-                      'pathlist', 'store_or_None', [], 'r', {'metavar': '[PATH[%sPATH]]' % os.pathsep}),
+                      'pathlist', 'store_or_False', [], 'r', {'metavar': '[PATH[%sPATH]]' % os.pathsep}),
             'robot-paths': ("Additional paths to consider by robot for easyconfigs (--robot paths get priority)",
                             'pathlist', 'add_flex', self.default_robot_paths, {'metavar': 'PATH[%sPATH]' % os.pathsep}),
             'search-paths': ("Additional locations to consider in --search (next to --robot and --robot-paths paths)",
@@ -1181,7 +1181,9 @@ def _postprocess_config(self):
         if self.options.pretend:
             self.options.installpath = get_pretend_installpath()
 
-        if self.options.robot is not None:
+        if self.options.robot is False:
+            self.options.robot = None  # Set to None as-if not specified
+        elif self.options.robot is not None:
             # if a single path is specified to --robot/-r, it must be an existing directory;
             # this is required since an argument to --robot is optional,
             # which makes it susceptible to 'eating' the following argument/option;

test/framework/options.py

@@ -3326,6 +3326,11 @@ def test_robot(self):
             ec_regex = re.compile(r'^\s\*\s\[[xF ]\]\s%s' % os.path.join(test_ecs_path, ecfile), re.M)
             self.assertTrue(ec_regex.search(outtxt), "Pattern %s found in %s" % (ec_regex.pattern, outtxt))
 
+        # Check for disabling --robot
+        args.append('--disable-robot')
+        self.assertErrorRegex(EasyBuildError, 'Missing dependencies', self.eb_main, args, raise_error=True)
+        self.assertNotExists(tmpdir)
+
     def test_robot_path_check(self):
         """Test path check for --robot"""
         empty_file = os.path.join(self.test_prefix, 'empty')