From 3026b757e078fd1f768045917f73279e61c676a2 Mon Sep 17 00:00:00 2001 From: Mayur Raut <95993675+RazoBeckett@users.noreply.github.com> Date: Sun, 15 Sep 2024 19:16:43 +0530 Subject: [PATCH] feat(dwmtitus-setup): simplified --- tabs/applications-setup/dwmtitus-setup.sh | 38 +++++++++-------------- 1 file changed, 14 insertions(+), 24 deletions(-) diff --git a/tabs/applications-setup/dwmtitus-setup.sh b/tabs/applications-setup/dwmtitus-setup.sh index 961e8892..e5e954ac 100755 --- a/tabs/applications-setup/dwmtitus-setup.sh +++ b/tabs/applications-setup/dwmtitus-setup.sh @@ -296,30 +296,20 @@ setupDisplayManager() { } install_slstatus() { - while true; do - printf "Do you want to install slstatus? (y/n): " # using printf instead of 'echo' to avoid newline, -n flag for 'echo' is not supported in POSIX - read -r response # -r flag to prevent backslashes from being interpreted - case "$response" in - [Yy] | [Yy][Ee][Ss]) # Matches: Y, y, Yes, yes, YES, yEs, yeS, etc. - echo "Installing slstatus" - cd slstatus/ || { echo "Failed to change directory to slstatus"; return 1; } - if $ESCALATION_TOOL make clean install; then - echo "slstatus installed successfully" - else - echo "Failed to install slstatus" - return 1 - fi - break - ;; - [Nn] | [Nn][Oo]) # same logic as above, but for No - echo "Skipping slstatus installation" - break - ;; - *) - echo "Invalid input. Please enter y/yes or n/no." - ;; - esac - done + printf "Do you want to install slstatus? (y/N): " # using printf instead of 'echo' to avoid newline, -n flag for 'echo' is not supported in POSIX + read -r response # -r flag to prevent backslashes from being interpreted + if [ "$response" = "y" ] || [ "$response" = "Y" ]; then + echo "Installing slstatus" + cd slstatus/ || { echo "Failed to change directory to slstatus"; return 1; } + if $ESCALATION_TOOL make clean install; then + echo "slstatus installed successfully" + else + echo "Failed to install slstatus" + return 1 + fi + else + echo "Skipping slstatus installation" + fi cd .. }