Skip to content

Commit

Permalink
Fix linting image build failure due to PEP 668
Browse files Browse the repository at this point in the history
Pip had a check enabled that fails the install due to packages being
managed by the OS.
The added flag ignores this check.

Otherwise, the build fails with:
```
11.85 × This environment is externally managed
11.85 ╰─>
11.85     The system-wide python installation should be maintained using the
11.85     system package manager (apk) only.
...
11.85 note: If you believe this is a mistake, please contact your Python
  installation or OS distribution provider. You can override this, at the risk
  of breaking your Python installation or OS, by passing
  --break-system-packages.
11.85 hint: See PEP 668 for the detailed specification.
```

Signed-off-by: Mike Kolesnik <[email protected]>
  • Loading branch information
mkolesnik authored and skitt committed Dec 18, 2023
1 parent e067789 commit e284eaa
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion package/Dockerfile.shipyard-linting
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ ENV MARKDOWNLINT_VERSION=0.33.0 \
RUN apk add --no-cache bash findutils git grep make nodejs py3-six shellcheck upx yamllint yq && \
apk add --no-cache --virtual installers npm py3-pip && \
npm install -g markdownlint-cli@${MARKDOWNLINT_VERSION} && \
pip install gitlint==${GITLINT_VERSION} && \
pip install --break-system-packages gitlint==${GITLINT_VERSION} && \
find /usr/bin/ -type f -executable -newercc /proc -size +1M \( -execdir upx {} \; -o -true \) && \
find /usr/lib/ -name __pycache__ -type d -exec rm -rf {} + && \
apk del installers
Expand Down

0 comments on commit e284eaa

Please sign in to comment.