From 3c4be75a45c5872fd065ea10d368fc7191fed574 Mon Sep 17 00:00:00 2001 From: Dieter Plaetinck Date: Mon, 17 Nov 2008 18:54:28 +0100 Subject: menu options should start from 1, not 0 + finetuned NEXTITEM logic a bit --- src/core/procedures/interactive | 53 ++++++++++++++++++++--------------------- 1 file changed, 26 insertions(+), 27 deletions(-) (limited to 'src/core') diff --git a/src/core/procedures/interactive b/src/core/procedures/interactive index a430662..1fb3378 100644 --- a/src/core/procedures/interactive +++ b/src/core/procedures/interactive @@ -45,7 +45,7 @@ start_process () ALT-F1 will bring you back here." # menu item tracker- autoselect the next item - NEXTITEM="" + NEXTITEM="1" while true do @@ -59,37 +59,36 @@ mainmenu() default=no [ -n "$NEXTITEM" ] && default="$NEXTITEM" ask_option $default "MAIN MENU" \ - "0" "$worker_select_source_title" \ - "1" "$worker_set_clock_title" \ - "2" "$worker_prepare_disks_title" \ - "3" "$worker_package_list_title" \ - "4" "$worker_install_packages_title" \ - "5" "$worker_configure_system_title" \ - "6" "$worker_install_bootloader_title" \ - "7" "Exit Install" - NEXTITEM="$ANSWER_OPTION" #TODO: set next item always current+1 after succesfull finish of each phase/worker, if relevant + "1" "$worker_select_source_title" \ + "2" "$worker_set_clock_title" \ + "3" "$worker_prepare_disks_title" \ + "4" "$worker_package_list_title" \ + "5" "$worker_install_packages_title" \ + "6" "$worker_configure_system_title" \ + "7" "$worker_install_bootloader_title" \ + "8" "Exit Install" case $ANSWER_OPTION in - "0") + "1") execute worker select_source; ret=$?; [ $ret -eq 0 -a "$var_PKG_SOURCE_TYPE" = ftp ] && select_source_extras_menu - [ $ret -eq 0 ] && execute worker runtime_packages && NEXTITEM=1 ;; + [ $ret -eq 0 ] && execute worker runtime_packages && NEXTITEM=2 ;; - "1") - execute worker set_clock && NEXTITEM=2 ;; "2") - execute worker prepare_disks && NEXTITEM=3 ;; + execute worker set_clock && NEXTITEM=3 ;; "3") - check_depend worker prepare_disks && execute worker package_list && NEXTITEM=4 ;; + execute worker prepare_disks && NEXTITEM=4 ;; "4") + check_depend worker prepare_disks && execute worker package_list && NEXTITEM=5 ;; + "5") check_depend worker package_list && \ check_depend worker select_source && execute worker install_packages && { execute worker auto_fstab ; \ ended_ok worker runtime_network && execute worker auto_network ; \ - execute worker auto_locale ; } && NEXTITEM=5 ;; - "5") - check_depend worker install_packages && execute worker configure_system && { execute worker mkinitcpio ; \ - execute worker locales ; } && NEXTITEM=6 ;; #TODO: why is next item 4 if $?=0?. maybe fixed now + execute worker auto_locale ; } && NEXTITEM=6 ;; "6") - check_depend worker configure_system && execute worker install_bootloader && NEXTITEM=7 ;; + check_depend worker install_packages && execute worker configure_system && { execute worker mkinitcpio ; \ + execute worker locales ; } && NEXTITEM=7 ;; #TODO: why is next item 4 if $?=0?. maybe fixed now "7") + check_depend worker configure_system && execute worker install_bootloader && NEXTITEM=8 ;; + "8") notify "If the install finished successfully, you can now type 'reboot' to restart the system." && stop_installer ;; *) ask_yesno "Abort Installation?" && stop_installer ;; @@ -101,12 +100,12 @@ select_source_extras_menu () { while true; do ask_option no "FTP Installation" \ - "0" "$worker_runtime_network_title" \ - "1" "$worker_select_mirror_title" \ - "2" "Return to Main Menu" - [ "$ANSWER_OPTION" = 0 ] && execute worker runtime_network - [ "$ANSWER_OPTION" = 1 ] && execute worker select_mirror - [ "$ANSWER_OPTION" = 2 ] && break + "1" "$worker_runtime_network_title" \ + "2" "$worker_select_mirror_title" \ + "3" "Return to Main Menu" + [ "$ANSWER_OPTION" = 1 ] && execute worker runtime_network + [ "$ANSWER_OPTION" = 2 ] && execute worker select_mirror + [ "$ANSWER_OPTION" = 3 ] && break done } -- cgit v1.2.3-54-g00ecf