Skip to content

Commit d509a9f

Browse files
committed
Use python -m for package verification instead of hardcoded paths
- Replace hardcoded /opt/esp/python_env/.../pytest with python3 -m pytest - Replace hardcoded /opt/esp/python_env/.../gcovr with python3 -m gcovr - More portable and follows Python best practices - Install still uses direct pip path to avoid QEMU emulation issues
1 parent 8ec6e83 commit d509a9f

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

images/esp-idf/Dockerfile

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,13 +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)
9495
ARG ESP_IDF_VERSION
9596
RUN python3 --version && \
96-
/opt/esp/python_env/idf5.4_py3.12_env/bin/pytest --version && \
97+
python3 -m pytest --version && \
9798
jq --version && \
9899
qemu-system-xtensa --version && \
99100
qemu-system-riscv32 --version && \
100-
/opt/esp/python_env/idf5.4_py3.12_env/bin/gcovr --version && \
101+
python3 -m gcovr --version && \
101102
ccache --version && \
102103
echo "ESP-IDF ${ESP_IDF_VERSION} installation verified successfully"
103104

0 commit comments

Comments
 (0)