Merge github.com:archlinux/archinstall into mypy-workflow
This commit is contained in:
commit
ef1d475fd0
|
|
@ -83,16 +83,18 @@ def get_password(prompt="Enter a password: "):
|
|||
def print_large_list(options, padding=5, margin_bottom=0, separator=': '):
|
||||
highest_index_number_length = len(str(len(options)))
|
||||
longest_line = highest_index_number_length + len(separator) + get_longest_option(options) + padding
|
||||
spaces_without_option = longest_line - (len(separator) + highest_index_number_length)
|
||||
max_num_of_columns = get_terminal_width() // longest_line
|
||||
max_options_in_cells = max_num_of_columns * (get_terminal_height()-margin_bottom)
|
||||
|
||||
if (len(options) > max_options_in_cells):
|
||||
for index, option in enumerate(options):
|
||||
print(f"{index}: {option}")
|
||||
return 1, index
|
||||
else:
|
||||
for row in range(0, (get_terminal_height()-margin_bottom)):
|
||||
for column in range(row, len(options), (get_terminal_height()-margin_bottom)):
|
||||
spaces = " "*(longest_line - len(options[column]))
|
||||
spaces = " "*(spaces_without_option - len(options[column]))
|
||||
print(f"{str(column): >{highest_index_number_length}}{separator}{options[column]}", end = spaces)
|
||||
print()
|
||||
|
||||
|
|
@ -100,6 +102,18 @@ def print_large_list(options, padding=5, margin_bottom=0, separator=': '):
|
|||
|
||||
|
||||
def generic_multi_select(options, text="Select one or more of the options above (leave blank to continue): ", sort=True, default=None, allow_empty=False):
|
||||
# Checking if the options are different from `list` or `dict` or if they are empty
|
||||
if type(options) not in [list, dict]:
|
||||
log(f" * Generic multi-select doesn't support ({type(options)}) as type of options * ", fg='red')
|
||||
log(" * If problem persists, please create an issue on https://github.com/archlinux/archinstall/issues * ", fg='yellow')
|
||||
raise RequirementError("generic_multi_select() requires list or dictionary as options.")
|
||||
if not options:
|
||||
log(f" * Generic multi-select didn't find any options to choose from * ", fg='red')
|
||||
log(" * If problem persists, please create an issue on https://github.com/archlinux/archinstall/issues * ", fg='yellow')
|
||||
raise RequirementError('generic_multi_select() requires at least one option to proceed.')
|
||||
# After passing the checks, function continues to work
|
||||
if type(options) == dict:
|
||||
options = list(options.values())
|
||||
if sort:
|
||||
options = sorted(options)
|
||||
|
||||
|
|
@ -108,7 +122,7 @@ def generic_multi_select(options, text="Select one or more of the options above
|
|||
selected_options = []
|
||||
|
||||
while True:
|
||||
if len(selected_options) <= 0 and default and default in options:
|
||||
if not selected_options and default in options:
|
||||
selected_options.append(default)
|
||||
|
||||
printed_options = []
|
||||
|
|
@ -119,32 +133,42 @@ def generic_multi_select(options, text="Select one or more of the options above
|
|||
printed_options.append(f'{option}')
|
||||
|
||||
section.clear(0, get_terminal_height()-section._cursor_y-1)
|
||||
x, y = print_large_list(printed_options, margin_bottom=2)
|
||||
print_large_list(printed_options, margin_bottom=2)
|
||||
section._cursor_y = len(printed_options)
|
||||
section._cursor_x = 0
|
||||
section.write_line(text)
|
||||
section.input_pos = section._cursor_x
|
||||
selected_option = section.get_keyboard_input(end=None)
|
||||
|
||||
if selected_option is None:
|
||||
if len(selected_options) <= 0 and default:
|
||||
selected_options = [default]
|
||||
|
||||
if len(selected_options) or allow_empty is True:
|
||||
break
|
||||
# This string check is necessary to correct work with it
|
||||
# Without this, Python will raise AttributeError because of stripping `None`
|
||||
# It also allows to remove empty spaces if the user accidentally entered them.
|
||||
if isinstance(selected_option, str):
|
||||
selected_option = selected_option.strip()
|
||||
try:
|
||||
if not selected_option:
|
||||
if not selected_options and default:
|
||||
selected_options = [default]
|
||||
elif selected_options or allow_empty:
|
||||
break
|
||||
else:
|
||||
raise RequirementError('Please select at least one option to continue')
|
||||
elif selected_option.isnumeric():
|
||||
if (selected_option := int(selected_option)) >= len(options):
|
||||
raise RequirementError(f'Selected option "{selected_option}" is out of range')
|
||||
selected_option = options[selected_option]
|
||||
if selected_option in selected_options:
|
||||
selected_options.remove(selected_option)
|
||||
else:
|
||||
selected_options.append(selected_option)
|
||||
elif selected_option in options:
|
||||
if selected_option in selected_options:
|
||||
selected_options.remove(selected_option)
|
||||
else:
|
||||
selected_options.append(selected_option)
|
||||
else:
|
||||
log('* Need to select at least one option!', fg='red')
|
||||
continue
|
||||
|
||||
elif selected_option.isdigit():
|
||||
if (selected_option := int(selected_option)) >= len(options):
|
||||
log('* Option is out of range, please select another one!', fg='red')
|
||||
continue
|
||||
selected_option = options[selected_option]
|
||||
if selected_option in selected_options:
|
||||
selected_options.remove(selected_option)
|
||||
else:
|
||||
selected_options.append(selected_option)
|
||||
raise RequirementError(f'Selected option "{selected_option}" does not exist in available options')
|
||||
except RequirementError as e:
|
||||
log(f" * {e} * ", fg='red')
|
||||
|
||||
return selected_options
|
||||
|
||||
|
|
@ -435,20 +459,24 @@ def generic_select(options, input_text="Select one of the above by index or abso
|
|||
this function returns an item from list, a string, or None
|
||||
"""
|
||||
|
||||
# Checking if options are different from `list` or `dict`
|
||||
# Checking if the options are different from `list` or `dict` or if they are empty
|
||||
if type(options) not in [list, dict]:
|
||||
log(f" * Generic select doesn't support ({type(options)}) as type of options * ", fg='red')
|
||||
log(" * If problem persists, please create an issue on https://github.com/archlinux/archinstall/issues * ", fg='yellow')
|
||||
raise RequirementError("generic_select() requires list or dictionary as options.")
|
||||
# To allow only `list` and `dict`, converting values of options here.
|
||||
# Therefore, now we can only provide the dictionary itself
|
||||
if type(options) == dict: options = list(options.values())
|
||||
if sort: options = sorted(options) # As we pass only list and dict (converted to list), we can skip converting to list
|
||||
if len(options) == 0:
|
||||
if not options:
|
||||
log(f" * Generic select didn't find any options to choose from * ", fg='red')
|
||||
log(" * If problem persists, please create an issue on https://github.com/archlinux/archinstall/issues * ", fg='yellow')
|
||||
raise RequirementError('generic_select() requires at least one option to proceed.')
|
||||
|
||||
# After passing the checks, function continues to work
|
||||
if type(options) == dict:
|
||||
# To allow only `list` and `dict`, converting values of options here.
|
||||
# Therefore, now we can only provide the dictionary itself
|
||||
options = list(options.values())
|
||||
if sort:
|
||||
# As we pass only list and dict (converted to list), we can skip converting to list
|
||||
options = sorted(options)
|
||||
|
||||
|
||||
# Added ability to disable the output of options items,
|
||||
# if another function displays something different from this
|
||||
|
|
@ -460,8 +488,8 @@ def generic_select(options, input_text="Select one of the above by index or abso
|
|||
# Now the try...except block handles validation for invalid input from the user
|
||||
while True:
|
||||
try:
|
||||
selected_option = input(input_text)
|
||||
if len(selected_option.strip()) == 0:
|
||||
selected_option = input(input_text).strip()
|
||||
if not selected_option:
|
||||
# `allow_empty_input` parameter handles return of None on empty input, if necessary
|
||||
# Otherwise raise `RequirementError`
|
||||
if allow_empty_input:
|
||||
|
|
@ -469,8 +497,7 @@ def generic_select(options, input_text="Select one of the above by index or abso
|
|||
raise RequirementError('Please select an option to continue')
|
||||
# Replaced `isdigit` with` isnumeric` to discard all negative numbers
|
||||
elif selected_option.isnumeric():
|
||||
selected_option = int(selected_option)
|
||||
if selected_option >= len(options):
|
||||
if (selected_option := int(selected_option)) >= len(options):
|
||||
raise RequirementError(f'Selected option "{selected_option}" is out of range')
|
||||
selected_option = options[selected_option]
|
||||
break
|
||||
|
|
@ -480,7 +507,6 @@ def generic_select(options, input_text="Select one of the above by index or abso
|
|||
raise RequirementError(f'Selected option "{selected_option}" does not exist in available options')
|
||||
except RequirementError as err:
|
||||
log(f" * {err} * ", fg='red')
|
||||
continue
|
||||
|
||||
return selected_option
|
||||
|
||||
|
|
@ -649,6 +675,7 @@ def select_driver(options=AVAILABLE_GFX_DRIVERS):
|
|||
"""
|
||||
|
||||
drivers = sorted(list(options))
|
||||
default_option = options["All open-source (default)"]
|
||||
|
||||
if drivers:
|
||||
lspci = sys_command(f'/usr/bin/lspci')
|
||||
|
|
@ -660,6 +687,10 @@ def select_driver(options=AVAILABLE_GFX_DRIVERS):
|
|||
print(' ** AMD card detected, suggested driver: AMD / ATI **')
|
||||
|
||||
initial_option = generic_select(drivers, input_text="Select your graphics card driver: ")
|
||||
|
||||
if not initial_option:
|
||||
return default_option
|
||||
|
||||
selected_driver = options[initial_option]
|
||||
|
||||
if type(selected_driver) == dict:
|
||||
|
|
@ -691,6 +722,6 @@ def select_kernel(options):
|
|||
kernels = sorted(list(options))
|
||||
|
||||
if kernels:
|
||||
return generic_multi_select(kernels, f"Choose which kernel to use (leave blank for default: {DEFAULT_KERNEL}): ", default=DEFAULT_KERNEL)
|
||||
return generic_multi_select(kernels, f"Choose which kernels to use (leave blank for default: {DEFAULT_KERNEL}): ", default=DEFAULT_KERNEL, sort=False)
|
||||
|
||||
raise RequirementError("Selecting kernels require a least one kernel to be given as an option.")
|
||||
|
|
|
|||
Loading…
Reference in New Issue