Skip to content

Commit 92ecd34

Browse files
committed
Fix verification to use venv Python instead of system Python
- Use /opt/esp/python_env/.../python3 -m pytest for verification - System python3 doesn't have packages (they're in venv) - Combines python -m portability with venv path specificity
1 parent d85b29b commit 92ecd34

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

images/esp-idf/Dockerfile

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,14 +91,14 @@ RUN /opt/esp/python_env/idf5.4_py3.12_env/bin/pip install --no-cache-dir \
9191
WORKDIR /workspace
9292

9393
# Verify installation
94-
# Use python -m to run installed packages (more portable than hardcoded paths)
94+
# Use venv python -m to verify packages (packages are installed in venv)
9595
ARG ESP_IDF_VERSION
9696
RUN python3 --version && \
97-
python3 -m pytest --version && \
97+
/opt/esp/python_env/idf5.4_py3.12_env/bin/python3 -m pytest --version && \
9898
jq --version && \
9999
qemu-system-xtensa --version && \
100100
qemu-system-riscv32 --version && \
101-
python3 -m gcovr --version && \
101+
/opt/esp/python_env/idf5.4_py3.12_env/bin/python3 -m gcovr --version && \
102102
ccache --version && \
103103
echo "ESP-IDF ${ESP_IDF_VERSION} installation verified successfully"
104104

0 commit comments

Comments
 (0)