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=y
 STM32CP_CLI=
 INTERFACE=
 PORT=
@@ -91,15 +91,42 @@ 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."
+    # 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=n
-      else
-        export PATH="/opt/homebrew/opt/gnu-getopt/bin":"$PATH"
       fi
-    else
-      export PATH="/usr/local/opt/gnu-getopt/bin":"$PATH"
+    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}/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
     fi
     ;;
   Windows*)
@@ -126,13 +153,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