Skip to content

Commit

Permalink
tests: Run pip3 install with --user for Ubuntu
Browse files Browse the repository at this point in the history
Modern Ubuntu errors out if you pip3 install system-wide without passing
--break-system-packages. We could add that flag, but we can just install
them for the current user instead (which more closely matches how actual
users will use cheribuild anyway). Although our current baseline doesn't
require this, be consistent and do it there too.

Fixes:	db30c92 ("Bump Python minimum to 3.8 and rework CI a bit")
  • Loading branch information
jrtc27 committed May 21, 2024
1 parent 4bc0f72 commit fd9cbc4
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion tests/ubuntu-baseline.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
python3-minimal python3-pip python3-setuptools

COPY requirements.txt /tmp/requirements.txt
RUN pip3 install -r /tmp/requirements.txt && rm -f /tmp/requirements.txt
RUN pip3 install --user -r /tmp/requirements.txt && rm -f /tmp/requirements.txt
2 changes: 1 addition & 1 deletion tests/ubuntu-latest.Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
python3-minimal python3-pip python3-setuptools

COPY requirements.txt /tmp/requirements.txt
RUN pip3 install -r /tmp/requirements.txt && rm -f /tmp/requirements.txt
RUN pip3 install --user -r /tmp/requirements.txt && rm -f /tmp/requirements.txt

0 comments on commit fd9cbc4

Please sign in to comment.