fix(pio-build): use python venv

See https://peps.python.org/pep-0668/ Signed-off-by: Frederic Pillon <frederic.pillon@st.com>
stm32duino · Apr 29, 2024 · 4782f07 · 4782f07
1 parent 2aa5cde
commit 4782f07
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/.github/actions/pio-build/Dockerfile b/.github/actions/pio-build/Dockerfile
@@ -5,14 +5,12 @@ ENV LANG C.UTF-8
 ENV LC_ALL C.UTF-8
 
 # Install prerequisites
-RUN apt-get --quiet=2 update && apt-get install --quiet=2 --assume-yes git python3 python3-pip wget
+RUN apt-get --quiet=2 update && apt-get install --quiet=2 --assume-yes git python3 python3-pip  python3-venv wget
 
-# Install PlatformIO
-RUN pip3 install --quiet --upgrade platformio
 CMD /bin/bash
 
 # Copies your code file from your action repository to the filesystem path `/` of the container
 COPY entrypoint.sh /entrypoint.sh
 
 # Code file to execute when the docker container starts up (`entrypoint.sh`)
-ENTRYPOINT ["/entrypoint.sh"]
+ENTRYPOINT ["/entrypoint.sh"]
diff --git a/.github/actions/pio-build/entrypoint.sh b/.github/actions/pio-build/entrypoint.sh
@@ -1,8 +1,14 @@
-#!/bin/bash
+#!/bin/sh
 
 readonly CMSIS_VERSION="$1"
 readonly CMSIS_ARCHIVE="CMSIS-${CMSIS_VERSION}.tar.bz2"
 
+# Use python venv
+python3 -m venv "$HOME/venv"
+. "$HOME/venv/bin/activate"
+# Install PlatformIO
+python3 -m pip install --quiet --upgrade platformio
+
 # Install the development version of ststm32 platform
 platformio platform install "https://github.com/platformio/platform-ststm32.git" || {
   exit 1