Merge pull request #1053 from luxonis/fix/depthai_installer_for_mac

Fix/depthai installer for mac

Author: PetrNovota

docs/source/_static/install_dependencies.sh

@@ -117,6 +117,12 @@ readonly fedora_pkgs=(
     #           libsm6 libxext6 libgl1-mesa-glx
 )
 
+# Check Bash version
+if [[ "${BASH_VERSINFO:-0}" -lt 4 ]]; then
+    echo "This script requires Bash 4.0 or higher. You are using Bash ${BASH_VERSION}. Please upgrade your Bash version."
+    exit 1
+fi
+
 print_action () {
     green="\e[0;32m"
     reset="\e[0;0m"
@@ -155,16 +161,26 @@ lookup_debian_version_number() {
 }
 
 if [[ $(uname) == "Darwin" ]]; then
-    echo "During Homebrew install, certain commands need 'sudo'. Requesting access..."
-    sudo true
-    homebrew_install_url="https://raw.githubusercontent.com/Homebrew/install/master/install.sh"
-    print_action "Installing Homebrew from $homebrew_install_url"
-    # CI=1 will skip some interactive prompts
-    CI=1 /bin/bash -c "$(curl -fsSL $homebrew_install_url)"
-    print_and_exec brew install git
-    echo
-    echo "=== Installed successfully!  IMPORTANT: For changes to take effect,"
-    echo "please close and reopen the terminal window, or run:  exec \$SHELL"
+    if ! command -v brew &> /dev/null; then
+        echo "During Homebrew install, certain commands need 'sudo'. Requesting access..."
+        sudo true
+        homebrew_install_url="https://raw.githubusercontent.com/Homebrew/install/master/install.sh"
+        print_action "Installing Homebrew from $homebrew_install_url"
+        # CI=1 will skip some interactive prompts
+        CI=1 /bin/bash -c "$(curl -fsSL $homebrew_install_url)"
+        echo
+        echo "=== Installed successfully!  IMPORTANT: For changes to take effect,"
+        echo "please close and reopen the terminal window, or run:  exec \$SHELL"
+    else
+        echo "Homebrew is already installed."
+    fi
+
+    if ! command -v git &> /dev/null; then
+        echo "Git not found, installing using Homebrew..."
+        print_and_exec brew install git
+    else
+        echo "Git Already installed.."
+    fi
 
 elif [ -f /etc/os-release ]; then
     source /etc/os-release

docs/source/_static/install_depthai.sh

@@ -7,7 +7,7 @@ mkdir "$WORKING_DIR"
 install_path=""
 path_correct="false"
 
-trap 'RET=$? ; echo -e >&2 "\n\x1b[31mFailed installing dependencies. Could be a bug in the installer or unsupported platform. Open a bug report over at https://github.com/luxonis/depthai - exited with status $RET at line $LINENO \x1b[0m\n" ; exit $RET' ERR
+trap 'RET=$? ; echo -e >&2 "\n\x1b[31mFailed installing depthai. Could be a bug in the installer or unsupported platform. Open a bug report over at https://github.com/luxonis/depthai - exited with status $RET at line $LINENO \x1b[0m\n" ; exit $RET' ERR
 
 while [ "$path_correct" = "false" ]
 do
@@ -127,9 +127,6 @@ if [[ $(uname -s) == "Darwin" ]]; then
   echo "Installing global dependencies."
   bash -c "$(curl -fL https://docs.luxonis.com/install_dependencies.sh)"
 
-  echo "Upgrading brew."
-  brew update
-
   # clone depthai form git
   if [ -d "$DEPTHAI_DIR" ]; then
      echo "Demo app already downloaded. Checking out main and updating."
@@ -142,20 +139,13 @@ if [[ $(uname -s) == "Darwin" ]]; then
   git checkout main
   git pull
 
-  # install python 3.10 and python dependencies
-  brew update
 
   if [ "$install_python" == "true" ]; then
     echo "installing python 3.10"
     brew install [email protected]
     python_executable=$(which python3.10)
   fi
 
-  # pip does not have pyqt5 for arm
-  if [[ $(uname -m) == 'arm64' ]]; then
-    echo "Installing pyqt5 with homebrew."
-    brew install pyqt@5
-  fi
 
   # create python virtual environment
   echo "Creating python virtual environment in $VENV_DIR"
@@ -166,20 +156,14 @@ if [[ $(uname -s) == "Darwin" ]]; then
   python -m pip install --upgrade pip
 
   # install launcher dependencies
-  # only on mac silicon point PYTHONPATH to pyqt5 installation via homebrew, otherwise install pyqt5 with pip
-  if [[ $(uname -m) == 'arm64' ]]; then
-    if [[ ":$PYTHONPATH:" == *":/opt/homebrew/lib/python3.10/site-packages:"* ]]; then
-      echo "/opt/homebrew/lib/python$nr_1.$nr_2/site-packages already in PYTHONPATH"
-    else
-      export "PYTHONPATH=/opt/homebrew/lib/python$nr_1.$nr_2/site-packages:"$PYTHONPATH
-      echo "/opt/homebrew/lib/pythonv$nr_1.$nr_2/site-packages added to PYTHONPATH"
-    fi
-  else
-    pip install pyqt5
-  fi
-
+  pip install pyqt5
   pip install packaging
 
+  # Inform macOS users about PATH changes
+  echo "DepthAI has been added to your PATH in .bashrc and .zshrc (if present)."
+  echo "If you prefer, you can manually add the following line to your .bash_profile for it to be recognized in login shells:"
+  echo "export PATH=\$PATH:$ENTRYPOINT_DIR"
+
 elif [[ $(uname -s) == "Linux" ]]; then
   echo _____________________________
   echo "Calling linux_installer.sh"