Skip to content

Commit

Permalink
Use the absolute path to the venv Python instead of modifying PATH
Browse files Browse the repository at this point in the history
`slc-cli` does not play well in a venv, it seems
  • Loading branch information
puddly committed Oct 29, 2024
1 parent 3db28b8 commit 6e5be68
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ jobs:
done
# Build it
python3 tools/build_project.py \
/opt/venv/bin/python3 tools/build_project.py \
$sdk_args \
$toolchain_args \
--manifest "${{ matrix.manifest }}" \
Expand Down
2 changes: 0 additions & 2 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,6 @@ RUN \
virtualenv /opt/venv \
&& /opt/venv/bin/pip install jq ruamel.yaml

ENV PATH="/opt/venv/bin:$PATH"

# Install Simplicity Commander (unfortunately no stable URL available, this
# is known to be working with Commander_linux_x86_64_1v15p0b1306.tar.bz).
RUN \
Expand Down

0 comments on commit 6e5be68

Please sign in to comment.