Skip to content

Case-insensitive CLI options #5101

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
vvolkl wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Case-insensitive CLI options #5101

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
30e27b1
Case-insensitive CLI options
vvolkl Jan 30, 2026
20f75cd
fix whitespace
vvolkl Jan 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions easybuild/base/generaloption.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,26 @@ def check_str_list_tuple(option, opt, value): # pylint: disable=unused-argument
return klass(split)


def check_choice_case_insensitive(option, opt, value):
"""
Case-insensitive check function for choice type.
Performs case-insensitive matching against available choices.
Returns the properly-cased choice value from the choices list.
"""
# Create a mapping from lowercase choices to original choices
choices_map = {choice.lower(): choice for choice in option.choices}

value_lower = value.lower()
if value_lower in choices_map:
# Return the properly-cased choice from the original choices list
return choices_map[value_lower]
else:
choices = ", ".join(map(repr, option.choices))
raise OptionValueError(
_gettext("option %s: invalid choice: %r (choose from %s)")
% (opt, value, choices))
Comment on lines +155 to +156
Copy link
Contributor

@smoors smoors Feb 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
_gettext("option %s: invalid choice: %r (choose from %s)")
% (opt, value, choices))
_gettext(f"option {opt}: invalid choice: {value} (choose from {choices})")



def get_empty_add_flex(allvalues, self=None):
"""Return the empty element for add_flex action for allvalues"""
empty = None
Expand Down Expand Up @@ -215,6 +235,8 @@ class ExtOption(CompleterOption):
TYPE_CHECKER.update(Option.TYPE_CHECKER)
TYPES = tuple(TYPE_STRLIST + list(Option.TYPES))
BOOLEAN_ACTIONS = ('store_true', 'store_false',) + EXTOPTION_LOG
# Override the choice checker to be case-insensitive
TYPE_CHECKER['choice'] = check_choice_case_insensitive

def __init__(self, *args, **kwargs):
"""Add logger to init"""
Expand Down