fix(stm32cubeprog): harden getopt support on MacOS

- brew prefix usage
- MacPorts support

Fixes #115

Signed-off-by: Frederic Pillon <frederic.pillon@st.com>

Author: fpistm

stm32CubeProg.sh

@@ -3,7 +3,7 @@ set -o nounset # Treat unset variables as an error
 # set -o xtrace  # Print command traces before executing command.
 
 UNAME_OS="$(uname -s)"
-GNU_GETOPT=
+GNU_GETOPT="n" # "y" if GNU getopt is available
 STM32CP_CLI=
 INTERFACE=
 PORT=
@@ -73,6 +73,7 @@ aborting() {
 case "${UNAME_OS}" in
   Linux*)
     STM32CP_CLI=STM32_Programmer.sh
+    GNU_GETOPT="y"
     if ! command -v $STM32CP_CLI >/dev/null 2>&1; then
       export PATH="$HOME/STMicroelectronics/STM32Cube/STM32CubeProgrammer/bin":"$PATH"
     fi
@@ -91,19 +92,47 @@ case "${UNAME_OS}" in
     if ! command -v $STM32CP_CLI >/dev/null 2>&1; then
       aborting
     fi
-    if ! command -v /usr/local/opt/gnu-getopt/bin/getopt >/dev/null 2>&1; then
-      if ! command -v /opt/homebrew/opt/gnu-getopt/bin/getopt >/dev/null 2>&1; then
-        echo "Warning: long options not supported due to getopt from FreeBSD usage."
-        GNU_GETOPT=n
-      else
-        export PATH="/opt/homebrew/opt/gnu-getopt/bin":"$PATH"
+    # Manage getopt
+    # Check if getopt is in the path and if it is gnu or BSD version in the path
+    if command -v getopt >/dev/null 2>&1; then
+      if ! getopt --test 2>/dev/null; then
+        GNU_GETOPT="y"
+      fi
+    fi
+    # If getopt is not gnu version, check if it is installed via brew or macports and add it to the path
+    if [ "${GNU_GETOPT}" != "y" ]; then
+      if command -v brew >/dev/null 2>&1; then
+        BREW_PREFIX=$(brew --prefix)
+        # Only check in bin as brew symlinks its files into the prefix,
+        if command -v "${BREW_PREFIX}/bin/getopt" >/dev/null 2>&1; then
+          export PATH="${BREW_PREFIX}/bin":"$PATH"
+          GNU_GETOPT="y"
+        fi
+      fi
+      # Check if GNU_GETOPT is set then check for MacPorts getopt
+      if [ "${GNU_GETOPT}" != "y" ]; then
+        if command -v /opt/local/bin/getopt >/dev/null 2>&1; then
+          export PATH="/opt/local/bin:$PATH"
+          GNU_GETOPT="y"
+        fi
+      fi
+      # Fallback to previous check
+      if [ "${GNU_GETOPT}" != "y" ]; then
+        if ! command -v /usr/local/opt/gnu-getopt/bin/getopt >/dev/null 2>&1; then
+          if command -v /opt/homebrew/opt/gnu-getopt/bin/getopt >/dev/null 2>&1; then
+            export PATH="/opt/homebrew/opt/gnu-getopt/bin":"$PATH"
+            GNU_GETOPT="y"
+          fi
+        else
+          export PATH="/usr/local/opt/gnu-getopt/bin":"$PATH"
+          GNU_GETOPT="y"
+        fi
       fi
-    else
-      export PATH="/usr/local/opt/gnu-getopt/bin":"$PATH"
     fi
     ;;
   Windows*)
     STM32CP_CLI=STM32_Programmer_CLI.exe
+    GNU_GETOPT="y"
     if ! command -v $STM32CP_CLI >/dev/null 2>&1; then
       if [ -n "${PROGRAMFILES+x}" ]; then
         STM32CP86=${PROGRAMFILES}/STMicroelectronics/STM32Cube/STM32CubeProgrammer/bin
@@ -126,13 +155,15 @@ esac
 
 # parse command line arguments
 # options may be followed by one colon to indicate they have a required arg
-if [ -n "${GNU_GETOPT}" ]; then
-  if ! options=$(getopt hi:b:a:es:o:f:m:n:c:d:p:r:v: "$@"); then
+if [ "${GNU_GETOPT}" = "y" ]; then
+  if ! options=$(getopt -a -o hi:b:a:es:o:f:m:n:c:d:p:r:v: --long help,interface:,bin:,address:,erase,start:,offset:,freq:,mode:,snum:,com:,dtr:,parity:,rts:,pid:,vid: -- "$@"); then
     echo "Terminating..." >&2
     exit 1
   fi
 else
-  if ! options=$(getopt -a -o hi:b:a:es:o:f:m:n:c:d:p:r:v: --long help,interface:,bin:,address:,erase,start:,offset:,freq:,mode:,snum:,com:,dtr:,parity:,rts:,pid:,vid: -- "$@"); then
+  echo "Warning: long options not supported due to getopt from FreeBSD usage."
+  echo "         Space in arguments is not supported."
+  if ! options=$(getopt hi:b:a:es:o:f:m:n:c:d:p:r:v: "$@"); then
     echo "Terminating..." >&2
     exit 1
   fi