Improve platform detection and handling in install.sh

Refactor `select_archive_format` to use a case statement for better
readability and extend functionality for more robust platform detection.
Ensure `sudo` is correctly handled only when required in the main
installation block.

Changes:
- Use case statement in `select_archive_format` for clarity
- Add a root check before using `sudo` in the installation block
- Maintain compatibility for both `tar` and `unzip` commands

Author: coadler

install.sh

@@ -56,18 +56,21 @@ detect_platform() {
 select_archive_format() {
   PLATFORM_ARCH=$1
 
-  if [ "$PLATFORM_ARCH" = windows-* ]; then
-    echo "zip"
-  else
-    if command -v tar >/dev/null 2>&1; then
-      echo "tar.gz"
-    elif command -v unzip >/dev/null 2>&1; then
+  case "$PLATFORM_ARCH" in
+    windows-*)
       echo "zip"
-    else
-      echo "Unsupported: neither tar nor unzip are available."
-      exit 1
-    fi
-  fi
+      ;;
+    *)
+      if command -v tar >/dev/null 2>&1; then
+        echo "tar.gz"
+      elif command -v unzip >/dev/null 2>&1; then
+        echo "zip"
+      else
+        echo "Unsupported: neither tar nor unzip are available."
+        exit 1
+      fi
+      ;;
+  esac
 }
 
 main() {
@@ -119,10 +122,16 @@ main() {
     mkdir -p "$INSTALL_DIR"
     mv "$BINARY_PATH" "$INSTALL_DIR/$BINARY_NAME.exe"
   else
-    sudo su <<EOF
-      [ $(uname -s) = "Linux" ] && setcap cap_net_admin=eip $BINARY_PATH
-      mv "$BINARY_PATH" "$INSTALL_DIR/$BINARY_NAME"
+    # Run using sudo if not root
+    if [ "$(id -u)" -ne 0 ]; then
+      sudo sh <<EOF
+        [ "$(uname -s)" = "Linux" ] && setcap cap_net_admin=eip "$BINARY_PATH"
+        mv "$BINARY_PATH" "$INSTALL_DIR/$BINARY_NAME"
 EOF
+    else
+        [ "$(uname -s)" = "Linux" ] && setcap cap_net_admin=eip "$BINARY_PATH"
+        mv "$BINARY_PATH" "$INSTALL_DIR/$BINARY_NAME"
+    fi
   fi
 
   # Clean up