build: increase scripts portability by migrating from bash to sh (#244)

Author: Djaytan

docs/user-guide/mkdocs.sh

@@ -1,20 +1,24 @@
-#!/usr/bin/env bash
+#!/usr/bin/env sh
 # This script aims to wrap MkDocs Material CLI through Docker in order to ensure portability across systems (Windows, Linux, Mac)
 
 set -eu
 
-SCRIPT_DIR=$(cd "$(dirname "${BASH_SOURCE[0]}")" &>/dev/null && pwd -P)
+SCRIPT_DIR=$(cd "$(dirname "$0")" > /dev/null 2>&1 && pwd -P)
 
-# Path conversion is required on Windows so that Docker volume mount will work
-# However, it shall be done in a controlled manner to avoid issues
-# Using Cygwin effectively with Windows: https://cygwin.com/cygwin-ug-net/using-effectively.html
-# Kernel detection is done based on the recommendation found here: https://stackoverflow.com/questions/3466166/how-to-check-if-running-in-cygwin-mac-or-linux
-if [[ "$(uname)" =~ (CYGWIN.*)|(MINGW.*)|(MSYS.*) ]]; then
-  # We disable automatic path conversion to avoid unexpected side effects
-  export MSYS_NO_PATHCONV=1
-  # We manually enforce a Windows path to prevent issues with Docker volume mounts
-  SCRIPT_DIR="$(cygpath --windows "${SCRIPT_DIR}")"
-fi
+# On Windows environments (Cygwin, MinGW, MSYS), Docker volume mounts require Windows-style paths.
+# However, automatic path conversion by MSYS can cause unexpected issues,
+# so we disable it and convert paths explicitly and deliberately.
+# See: https://cygwin.com/cygwin-ug-net/using-effectively.html
+# Kernel detection approach based on recommendations from:
+# https://stackoverflow.com/questions/3466166/how-to-check-if-running-in-cygwin-mac-or-linux
+case "$(uname)" in
+  CYGWIN*|MINGW*|MSYS*)
+    # We disable automatic path conversion to avoid unexpected side effects
+    export MSYS_NO_PATHCONV=1
+    # We manually enforce a Windows path to prevent issues with Docker volume mounts
+    SCRIPT_DIR="$(cygpath --windows "${SCRIPT_DIR}")"
+    ;;
+esac
 
 ROOT_PROJECT_DIR="${SCRIPT_DIR}/../.."
 

docs/user-guide/requirements.sh

@@ -1,20 +1,24 @@
-#!/usr/bin/env bash
+#!/usr/bin/env sh
 # This script aims to wrap pip-tools through Docker in order to ensure portability across systems (Windows, Linux, Mac)
 
 set -eu
 
-SCRIPT_DIR=$(cd "$(dirname "${BASH_SOURCE[0]}")" &>/dev/null && pwd -P)
+SCRIPT_DIR=$(cd "$(dirname "$0")" > /dev/null 2>&1 && pwd -P)
 
-# Path conversion is required on Windows so that Docker volume mount will work
-# However, it shall be done in a controlled manner to avoid issues
-# Using Cygwin effectively with Windows: https://cygwin.com/cygwin-ug-net/using-effectively.html
-# Kernel detection is done based on the recommendation found here: https://stackoverflow.com/questions/3466166/how-to-check-if-running-in-cygwin-mac-or-linux
-if [[ "$(uname)" =~ (CYGWIN.*)|(MINGW.*)|(MSYS.*) ]]; then
-  # We disable automatic path conversion to avoid unexpected side effects
-  export MSYS_NO_PATHCONV=1
-  # We manually enforce a Windows path to prevent issues with Docker volume mounts
-  SCRIPT_DIR="$(cygpath --windows "${SCRIPT_DIR}")"
-fi
+# On Windows environments (Cygwin, MinGW, MSYS), Docker volume mounts require Windows-style paths.
+# However, automatic path conversion by MSYS can cause unexpected issues,
+# so we disable it and convert paths explicitly and deliberately.
+# See: https://cygwin.com/cygwin-ug-net/using-effectively.html
+# Kernel detection approach based on recommendations from:
+# https://stackoverflow.com/questions/3466166/how-to-check-if-running-in-cygwin-mac-or-linux
+case "$(uname)" in
+  CYGWIN*|MINGW*|MSYS*)
+    # We disable automatic path conversion to avoid unexpected side effects
+    export MSYS_NO_PATHCONV=1
+    # We manually enforce a Windows path to prevent issues with Docker volume mounts
+    SCRIPT_DIR="$(cygpath --windows "${SCRIPT_DIR}")"
+    ;;
+esac
 
 docker build -t pip-tools -f Dockerfile.pip-tools "${SCRIPT_DIR}"