[python] Remove dependency only used in testing

PR project-chip#21567 added mobly as a default dependency of the Python Core CHIP library. The mobly package is only used by the Python testing framework in `src/python_testing/`. Allow to install extra dependency via `script/build_python.sh` flag and install them only when needed.
agners · May 26, 2023 · 6e2e885 · 6e2e885
1 parent ad5253a
commit 6e2e885
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/.github/workflows/tests.yaml b/.github/workflows/tests.yaml
@@ -487,7 +487,7 @@ jobs:
             - name: Build Python REPL and example apps
               timeout-minutes: 50
               run: |
-                  scripts/run_in_build_env.sh './scripts/build_python.sh --install_wheel build-env'
+                  scripts/run_in_build_env.sh './scripts/build_python.sh --install_wheel build-env --extra_packages "mobly"'
                   ./scripts/run_in_build_env.sh \
                    "./scripts/build/build_examples.py \
                       --target linux-x64-all-clusters-ipv6only-no-ble-no-wifi-tsan-clang-test \

diff --git a/scripts/build_python.sh b/scripts/build_python.sh
@@ -63,6 +63,7 @@ Input Options:
                                                             no: Do not install
                                                             build-env: install to virtual env for build matter
                                                             separate: install to another virtual env (out/python_env)
+  --extra_packages PACKAGES                                 Install extra Python packages from PyPI
   --include_yamltests                                       Whether to install the matter_yamltests wheel.
   -z --pregen_dir DIRECTORY                                 Directory where generated zap files have been pre-generated.
 "
@@ -96,6 +97,10 @@ while (($#)); do
             install_wheel=$2
             shift
             ;;
+        --extra_packages)
+            extra_packages=$2
+            shift
+            ;;
         --include_yamltests)
             include_yamltests="yes"
             ;;
@@ -160,6 +165,10 @@ if [ -n "$include_yamltests" ]; then
     )
 fi
 
+if [ -n "$extra_packages" ]; then
+    WHEEL+=(${extra_packages})
+fi
+
 if [ "$install_wheel" = "no" ]; then
     exit 0
 elif [ "$install_wheel" = "separate" ]; then