From 35d5df14aa6b51c67de0fcd650a3d36b6e47dbe8 Mon Sep 17 00:00:00 2001 From: 3wc <3wc.git@doesthisthing.work> Date: Sat, 20 Mar 2021 18:58:16 +0200 Subject: [PATCH] Fix subcommand selection.. ..by sorting the list of subcommand function names in descending order of how many '_' are in them. This means that `abra app version` will always be matched before `abra version`. Ref #108 --- CHANGELOG.md | 2 + abra | 206 +++++++++++++++++++++++++++------------------------ 2 files changed, 111 insertions(+), 97 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 10265fa..5f21dbb 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,7 @@ # abra x.x.x (UNRELEASED) +- Search for subcommands in descending order of how many components there are ([#108](https://git.autonomic.zone/coop-cloud/abra/issues/108)) + # abra 0.6.0 (2021-03-17) - Show version and digest of app if labelled ([98e674b8e8](https://git.autonomic.zone/coop-cloud/abra/commit/98e674b8e83458a83dcbf331e8e34c7188559c4a)) diff --git a/abra b/abra index 8c9cc38..1716b14 100755 --- a/abra +++ b/abra @@ -20,6 +20,7 @@ Usage: abra [options] app backup (|--all) abra [options] app deploy [--update] abra [options] app check + abra [options] app version abra [options] app config abra [options] app cp abra [options] app logs [] @@ -157,16 +158,16 @@ eval "var_$1+=($value)"; else eval "var_$1=$value"; fi; return 0; fi; done return 1; }; stdout() { printf -- "cat <<'EOM'\n%s\nEOM\n" "$1"; }; stderr() { printf -- "cat <<'EOM' >&2\n%s\nEOM\n" "$1"; }; error() { [[ -n $1 ]] && stderr "$1"; stderr "$usage"; _return 1; }; _return() { -printf -- "exit %d\n" "$1"; exit "$1"; }; set -e; trimmed_doc=${DOC:1:2017} -usage=${DOC:40:1481}; digest=fd466 -shorts=(-e -U -d -v -C -s -h -b '' '' '' '' '' '' '' '' '' '' '' '' '' '') -longs=(--env --skip-update --debug --verbose --skip-check --stack --help --branch --status --server --type --domain --app-name --pass --secrets --all --update --force --volumes --no-tty --user --dev) -argcounts=(1 0 0 0 0 1 0 1 0 1 1 1 1 0 0 0 0 0 0 0 1 0); node_0(){ value __env 0 -}; node_1(){ switch __skip_update 1; }; node_2(){ switch __debug 2; }; node_3(){ -switch __verbose 3; }; node_4(){ switch __skip_check 4; }; node_5(){ -value __stack 5; }; node_6(){ switch __help 6; }; node_7(){ value __branch 7; } -node_8(){ switch __status 8; }; node_9(){ value __server 9; }; node_10(){ -value __type 10; }; node_11(){ value __domain 11; }; node_12(){ +printf -- "exit %d\n" "$1"; exit "$1"; }; set -e; trimmed_doc=${DOC:1:2052} +usage=${DOC:40:1516}; digest=76d0e +shorts=(-U -d -s -C -e -h -v -b '' '' '' '' '' '' '' '' '' '' '' '' '' '') +longs=(--skip-update --debug --stack --skip-check --env --help --verbose --branch --status --server --type --domain --app-name --pass --secrets --all --update --force --volumes --no-tty --user --dev) +argcounts=(0 0 1 0 1 0 0 1 0 1 1 1 1 0 0 0 0 0 0 0 1 0); node_0(){ +switch __skip_update 0; }; node_1(){ switch __debug 1; }; node_2(){ +value __stack 2; }; node_3(){ switch __skip_check 3; }; node_4(){ value __env 4 +}; node_5(){ switch __help 5; }; node_6(){ switch __verbose 6; }; node_7(){ +value __branch 7; }; node_8(){ switch __status 8; }; node_9(){ value __server 9 +}; node_10(){ value __type 10; }; node_11(){ value __domain 11; }; node_12(){ value __app_name 12; }; node_13(){ switch __pass 13; }; node_14(){ switch __secrets 14; }; node_15(){ switch __all 15; }; node_16(){ switch __update 16; }; node_17(){ switch __force 17; }; node_18(){ @@ -181,82 +182,83 @@ value _user_ a; }; node_36(){ value _port_ a; }; node_37(){ value _provider_ a }; node_38(){ value _subcommands_ a true; }; node_39(){ _command app; } node_40(){ _command list; }; node_41(){ _command ls; }; node_42(){ _command new }; node_43(){ _command backup; }; node_44(){ _command deploy; }; node_45(){ -_command check; }; node_46(){ _command config; }; node_47(){ _command cp; } -node_48(){ _command logs; }; node_49(){ _command ps; }; node_50(){ -_command restore; }; node_51(){ _command rm; }; node_52(){ _command delete; } -node_53(){ _command run; }; node_54(){ _command rollback; }; node_55(){ -_command secret; }; node_56(){ _command generate; }; node_57(){ _command insert -}; node_58(){ _command undeploy; }; node_59(){ _command server; }; node_60(){ -_command add; }; node_61(){ _command init; }; node_62(){ _command apps; } -node_63(){ _command upgrade; }; node_64(){ _command version; }; node_65(){ -_command help; }; node_66(){ optional 0 1 2 3 4 5 6 7; }; node_67(){ optional 66 -}; node_68(){ either 40 41; }; node_69(){ required 68; }; node_70(){ optional 8 -}; node_71(){ optional 9; }; node_72(){ optional 10; }; node_73(){ -required 67 39 69 70 71 72; }; node_74(){ optional 11; }; node_75(){ optional 12 -}; node_76(){ optional 13; }; node_77(){ optional 14; }; node_78(){ -required 67 39 42 71 74 75 76 77 22; }; node_79(){ either 24 15; }; node_80(){ -required 79; }; node_81(){ required 67 39 23 43 80; }; node_82(){ optional 16; } -node_83(){ required 67 39 23 44 82; }; node_84(){ required 67 39 23 45; } -node_85(){ required 67 39 23 46; }; node_86(){ required 67 39 23 47 25 26; } -node_87(){ optional 24; }; node_88(){ required 67 39 23 48 87; }; node_89(){ -required 67 39 23 49; }; node_90(){ required 67 39 23 50 80; }; node_91(){ -either 51 52; }; node_92(){ required 91; }; node_93(){ optional 17; } -node_94(){ optional 18; }; node_95(){ required 67 39 23 92 93 94 77; } -node_96(){ optional 27; }; node_97(){ required 67 39 23 50 24 96; }; node_98(){ -optional 19; }; node_99(){ optional 20; }; node_100(){ oneormore 28; } -node_101(){ required 67 39 23 53 98 99 24 100; }; node_102(){ -required 67 39 23 54 24; }; node_103(){ required 29 30; }; node_104(){ -either 103 15; }; node_105(){ required 104; }; node_106(){ optional 31; } -node_107(){ required 67 39 23 55 56 105 106 76; }; node_108(){ -required 67 39 23 55 57 29 30 32 76; }; node_109(){ either 29 15; }; node_110(){ -required 109; }; node_111(){ required 67 39 23 55 92 110 76 93; }; node_112(){ -required 67 39 23 58; }; node_113(){ optional 100; }; node_114(){ -required 67 39 23 33 113; }; node_115(){ optional 35; }; node_116(){ optional 36 -}; node_117(){ required 67 59 60 34 115 116; }; node_118(){ required 67 59 42 37 -}; node_119(){ required 67 59 69; }; node_120(){ required 67 59 34 51; } -node_121(){ required 67 59 34 61; }; node_122(){ required 67 59 34 62 70; } -node_123(){ optional 21; }; node_124(){ required 67 63 123; }; node_125(){ -required 67 64; }; node_126(){ oneormore 38; }; node_127(){ optional 126; } -node_128(){ required 67 65 127; }; node_129(){ required 67; }; node_130(){ -either 73 78 81 83 84 85 86 88 89 90 95 97 101 102 107 108 111 112 114 117 118 119 120 121 122 124 125 128 129 -}; node_131(){ required 130; }; cat <<<' docopt_exit() { -[[ -n $1 ]] && printf "%s\n" "$1" >&2; printf "%s\n" "${DOC:40:1481}" >&2 -exit 1; }'; unset var___env var___skip_update var___debug var___verbose \ -var___skip_check var___stack var___help var___branch var___status var___server \ +_command check; }; node_46(){ _command version; }; node_47(){ _command config; } +node_48(){ _command cp; }; node_49(){ _command logs; }; node_50(){ _command ps +}; node_51(){ _command restore; }; node_52(){ _command rm; }; node_53(){ +_command delete; }; node_54(){ _command run; }; node_55(){ _command rollback; } +node_56(){ _command secret; }; node_57(){ _command generate; }; node_58(){ +_command insert; }; node_59(){ _command undeploy; }; node_60(){ _command server +}; node_61(){ _command add; }; node_62(){ _command init; }; node_63(){ +_command apps; }; node_64(){ _command upgrade; }; node_65(){ _command help; } +node_66(){ optional 0 1 2 3 4 5 6 7; }; node_67(){ optional 66; }; node_68(){ +either 40 41; }; node_69(){ required 68; }; node_70(){ optional 8; }; node_71(){ +optional 9; }; node_72(){ optional 10; }; node_73(){ required 67 39 69 70 71 72 +}; node_74(){ optional 11; }; node_75(){ optional 12; }; node_76(){ optional 13 +}; node_77(){ optional 14; }; node_78(){ required 67 39 42 71 74 75 76 77 22; } +node_79(){ either 24 15; }; node_80(){ required 79; }; node_81(){ +required 67 39 23 43 80; }; node_82(){ optional 16; }; node_83(){ +required 67 39 23 44 82; }; node_84(){ required 67 39 23 45; }; node_85(){ +required 67 39 23 46; }; node_86(){ required 67 39 23 47; }; node_87(){ +required 67 39 23 48 25 26; }; node_88(){ optional 24; }; node_89(){ +required 67 39 23 49 88; }; node_90(){ required 67 39 23 50; }; node_91(){ +required 67 39 23 51 80; }; node_92(){ either 52 53; }; node_93(){ required 92 +}; node_94(){ optional 17; }; node_95(){ optional 18; }; node_96(){ +required 67 39 23 93 94 95 77; }; node_97(){ optional 27; }; node_98(){ +required 67 39 23 51 24 97; }; node_99(){ optional 19; }; node_100(){ +optional 20; }; node_101(){ oneormore 28; }; node_102(){ +required 67 39 23 54 99 100 24 101; }; node_103(){ required 67 39 23 55 24; } +node_104(){ required 29 30; }; node_105(){ either 104 15; }; node_106(){ +required 105; }; node_107(){ optional 31; }; node_108(){ +required 67 39 23 56 57 106 107 76; }; node_109(){ +required 67 39 23 56 58 29 30 32 76; }; node_110(){ either 29 15; }; node_111(){ +required 110; }; node_112(){ required 67 39 23 56 93 111 76 94; }; node_113(){ +required 67 39 23 59; }; node_114(){ optional 101; }; node_115(){ +required 67 39 23 33 114; }; node_116(){ optional 35; }; node_117(){ optional 36 +}; node_118(){ required 67 60 61 34 116 117; }; node_119(){ required 67 60 42 37 +}; node_120(){ required 67 60 69; }; node_121(){ required 67 60 34 52; } +node_122(){ required 67 60 34 62; }; node_123(){ required 67 60 34 63 70; } +node_124(){ optional 21; }; node_125(){ required 67 64 124; }; node_126(){ +required 67 46; }; node_127(){ oneormore 38; }; node_128(){ optional 127; } +node_129(){ required 67 65 128; }; node_130(){ required 67; }; node_131(){ +either 73 78 81 83 84 85 86 87 89 90 91 96 98 102 103 108 109 112 113 115 118 119 120 121 122 123 125 126 129 130 +}; node_132(){ required 131; }; cat <<<' docopt_exit() { +[[ -n $1 ]] && printf "%s\n" "$1" >&2; printf "%s\n" "${DOC:40:1516}" >&2 +exit 1; }'; unset var___skip_update var___debug var___stack var___skip_check \ +var___env var___help var___verbose var___branch var___status var___server \ var___type var___domain var___app_name var___pass var___secrets var___all \ var___update var___force var___volumes var___no_tty var___user var___dev \ var__type_ var__app_ var__service_ var__src_ var__dst_ var__backup_file_ \ var__args_ var__secret_ var__version_ var__cmd_ var__data_ var__command_ \ var__host_ var__user_ var__port_ var__provider_ var__subcommands_ var_app \ -var_list var_ls var_new var_backup var_deploy var_check var_config var_cp \ -var_logs var_ps var_restore var_rm var_delete var_run var_rollback var_secret \ -var_generate var_insert var_undeploy var_server var_add var_init var_apps \ -var_upgrade var_version var_help; parse 131 "$@" -local prefix=${DOCOPT_PREFIX:-''}; unset "${prefix}__env" \ -"${prefix}__skip_update" "${prefix}__debug" "${prefix}__verbose" \ -"${prefix}__skip_check" "${prefix}__stack" "${prefix}__help" \ -"${prefix}__branch" "${prefix}__status" "${prefix}__server" "${prefix}__type" \ -"${prefix}__domain" "${prefix}__app_name" "${prefix}__pass" \ -"${prefix}__secrets" "${prefix}__all" "${prefix}__update" "${prefix}__force" \ -"${prefix}__volumes" "${prefix}__no_tty" "${prefix}__user" "${prefix}__dev" \ -"${prefix}_type_" "${prefix}_app_" "${prefix}_service_" "${prefix}_src_" \ -"${prefix}_dst_" "${prefix}_backup_file_" "${prefix}_args_" \ -"${prefix}_secret_" "${prefix}_version_" "${prefix}_cmd_" "${prefix}_data_" \ -"${prefix}_command_" "${prefix}_host_" "${prefix}_user_" "${prefix}_port_" \ -"${prefix}_provider_" "${prefix}_subcommands_" "${prefix}app" "${prefix}list" \ -"${prefix}ls" "${prefix}new" "${prefix}backup" "${prefix}deploy" \ -"${prefix}check" "${prefix}config" "${prefix}cp" "${prefix}logs" "${prefix}ps" \ -"${prefix}restore" "${prefix}rm" "${prefix}delete" "${prefix}run" \ -"${prefix}rollback" "${prefix}secret" "${prefix}generate" "${prefix}insert" \ -"${prefix}undeploy" "${prefix}server" "${prefix}add" "${prefix}init" \ -"${prefix}apps" "${prefix}upgrade" "${prefix}version" "${prefix}help" -eval "${prefix}"'__env=${var___env:-}' +var_list var_ls var_new var_backup var_deploy var_check var_version var_config \ +var_cp var_logs var_ps var_restore var_rm var_delete var_run var_rollback \ +var_secret var_generate var_insert var_undeploy var_server var_add var_init \ +var_apps var_upgrade var_help; parse 132 "$@"; local prefix=${DOCOPT_PREFIX:-''} +unset "${prefix}__skip_update" "${prefix}__debug" "${prefix}__stack" \ +"${prefix}__skip_check" "${prefix}__env" "${prefix}__help" \ +"${prefix}__verbose" "${prefix}__branch" "${prefix}__status" \ +"${prefix}__server" "${prefix}__type" "${prefix}__domain" \ +"${prefix}__app_name" "${prefix}__pass" "${prefix}__secrets" "${prefix}__all" \ +"${prefix}__update" "${prefix}__force" "${prefix}__volumes" \ +"${prefix}__no_tty" "${prefix}__user" "${prefix}__dev" "${prefix}_type_" \ +"${prefix}_app_" "${prefix}_service_" "${prefix}_src_" "${prefix}_dst_" \ +"${prefix}_backup_file_" "${prefix}_args_" "${prefix}_secret_" \ +"${prefix}_version_" "${prefix}_cmd_" "${prefix}_data_" "${prefix}_command_" \ +"${prefix}_host_" "${prefix}_user_" "${prefix}_port_" "${prefix}_provider_" \ +"${prefix}_subcommands_" "${prefix}app" "${prefix}list" "${prefix}ls" \ +"${prefix}new" "${prefix}backup" "${prefix}deploy" "${prefix}check" \ +"${prefix}version" "${prefix}config" "${prefix}cp" "${prefix}logs" \ +"${prefix}ps" "${prefix}restore" "${prefix}rm" "${prefix}delete" \ +"${prefix}run" "${prefix}rollback" "${prefix}secret" "${prefix}generate" \ +"${prefix}insert" "${prefix}undeploy" "${prefix}server" "${prefix}add" \ +"${prefix}init" "${prefix}apps" "${prefix}upgrade" "${prefix}help" eval "${prefix}"'__skip_update=${var___skip_update:-false}' eval "${prefix}"'__debug=${var___debug:-false}' -eval "${prefix}"'__verbose=${var___verbose:-false}' -eval "${prefix}"'__skip_check=${var___skip_check:-false}' eval "${prefix}"'__stack=${var___stack:-}' +eval "${prefix}"'__skip_check=${var___skip_check:-false}' +eval "${prefix}"'__env=${var___env:-}' eval "${prefix}"'__help=${var___help:-false}' +eval "${prefix}"'__verbose=${var___verbose:-false}' eval "${prefix}"'__branch=${var___branch:-}' eval "${prefix}"'__status=${var___status:-false}' eval "${prefix}"'__server=${var___server:-}' @@ -294,6 +296,7 @@ eval "${prefix}"'new=${var_new:-false}' eval "${prefix}"'backup=${var_backup:-false}' eval "${prefix}"'deploy=${var_deploy:-false}' eval "${prefix}"'check=${var_check:-false}' +eval "${prefix}"'version=${var_version:-false}' eval "${prefix}"'config=${var_config:-false}' eval "${prefix}"'cp=${var_cp:-false}'; eval "${prefix}"'logs=${var_logs:-false}' eval "${prefix}"'ps=${var_ps:-false}' @@ -311,26 +314,26 @@ eval "${prefix}"'add=${var_add:-false}' eval "${prefix}"'init=${var_init:-false}' eval "${prefix}"'apps=${var_apps:-false}' eval "${prefix}"'upgrade=${var_upgrade:-false}' -eval "${prefix}"'version=${var_version:-false}' eval "${prefix}"'help=${var_help:-false}'; local docopt_i=1 [[ $BASH_VERSION =~ ^4.3 ]] && docopt_i=2; for ((;docopt_i>0;docopt_i--)); do -declare -p "${prefix}__env" "${prefix}__skip_update" "${prefix}__debug" \ -"${prefix}__verbose" "${prefix}__skip_check" "${prefix}__stack" \ -"${prefix}__help" "${prefix}__branch" "${prefix}__status" "${prefix}__server" \ -"${prefix}__type" "${prefix}__domain" "${prefix}__app_name" "${prefix}__pass" \ -"${prefix}__secrets" "${prefix}__all" "${prefix}__update" "${prefix}__force" \ -"${prefix}__volumes" "${prefix}__no_tty" "${prefix}__user" "${prefix}__dev" \ -"${prefix}_type_" "${prefix}_app_" "${prefix}_service_" "${prefix}_src_" \ -"${prefix}_dst_" "${prefix}_backup_file_" "${prefix}_args_" \ -"${prefix}_secret_" "${prefix}_version_" "${prefix}_cmd_" "${prefix}_data_" \ -"${prefix}_command_" "${prefix}_host_" "${prefix}_user_" "${prefix}_port_" \ -"${prefix}_provider_" "${prefix}_subcommands_" "${prefix}app" "${prefix}list" \ -"${prefix}ls" "${prefix}new" "${prefix}backup" "${prefix}deploy" \ -"${prefix}check" "${prefix}config" "${prefix}cp" "${prefix}logs" "${prefix}ps" \ -"${prefix}restore" "${prefix}rm" "${prefix}delete" "${prefix}run" \ -"${prefix}rollback" "${prefix}secret" "${prefix}generate" "${prefix}insert" \ -"${prefix}undeploy" "${prefix}server" "${prefix}add" "${prefix}init" \ -"${prefix}apps" "${prefix}upgrade" "${prefix}version" "${prefix}help"; done; } +declare -p "${prefix}__skip_update" "${prefix}__debug" "${prefix}__stack" \ +"${prefix}__skip_check" "${prefix}__env" "${prefix}__help" \ +"${prefix}__verbose" "${prefix}__branch" "${prefix}__status" \ +"${prefix}__server" "${prefix}__type" "${prefix}__domain" \ +"${prefix}__app_name" "${prefix}__pass" "${prefix}__secrets" "${prefix}__all" \ +"${prefix}__update" "${prefix}__force" "${prefix}__volumes" \ +"${prefix}__no_tty" "${prefix}__user" "${prefix}__dev" "${prefix}_type_" \ +"${prefix}_app_" "${prefix}_service_" "${prefix}_src_" "${prefix}_dst_" \ +"${prefix}_backup_file_" "${prefix}_args_" "${prefix}_secret_" \ +"${prefix}_version_" "${prefix}_cmd_" "${prefix}_data_" "${prefix}_command_" \ +"${prefix}_host_" "${prefix}_user_" "${prefix}_port_" "${prefix}_provider_" \ +"${prefix}_subcommands_" "${prefix}app" "${prefix}list" "${prefix}ls" \ +"${prefix}new" "${prefix}backup" "${prefix}deploy" "${prefix}check" \ +"${prefix}version" "${prefix}config" "${prefix}cp" "${prefix}logs" \ +"${prefix}ps" "${prefix}restore" "${prefix}rm" "${prefix}delete" \ +"${prefix}run" "${prefix}rollback" "${prefix}secret" "${prefix}generate" \ +"${prefix}insert" "${prefix}undeploy" "${prefix}server" "${prefix}add" \ +"${prefix}init" "${prefix}apps" "${prefix}upgrade" "${prefix}help"; done; } # docopt parser above, complete command for generating this parser is `docopt.sh abra` PROGRAM_NAME=$(basename "$0") @@ -1109,6 +1112,12 @@ sub_app_config (){ $EDITOR "$ENV_FILE" } +###### .. app version +sub_app_version (){ + # shellcheck disable=SC2016 + echo '`abra app version` command working correctly!' +} + ###### .. app check help_app_check (){ echo "abra [options] app check @@ -1818,8 +1827,11 @@ abra() { load_abra_sh # Search for sub_* functions, and check if any of them matches enabled - # arguments (i.e. is a command and is specified) - for SUBCOMMAND in $(declare -Ff | grep 'sub_' | cut -d' ' -f3 | sort -r); do + # arguments (i.e. is a command and is specified). The `awk / sort` sorts by + # the number of occurrences of '_' in the function name, to ensure that + # `abra app version` will be matched before `abra version`. + SUBCOMMANDS=$(declare -Ff | grep 'sub_' | cut -d' ' -f3 | awk '{ print gsub("_","&"), $0 }' | sort -n -r | cut -d" " -f2-) + for SUBCOMMAND in $SUBCOMMANDS; do IFS='_' read -r -a PARTS <<< "$SUBCOMMAND" for PART in "${PARTS[@]:1}"; do # TODO 3wc: probably a better way to check if a variable is defined..