Skip to content

Add install test without toolchain - #700

Draft
timokoessler wants to merge 2 commits into
mainfrom
add-test-without-toolchain
Draft

Add install test without toolchain#700
timokoessler wants to merge 2 commits into
mainfrom
add-test-without-toolchain

Conversation

@timokoessler

Copy link
Copy Markdown
Member

No description provided.

@timokoessler
timokoessler marked this pull request as draft August 21, 2026 11:59
@timokoessler
timokoessler force-pushed the add-test-without-toolchain branch from 7019b45 to 66ef7cb Compare August 21, 2026 12:02
WORKDIR /app
COPY . .

RUN pip install --no-cache-dir --only-binary=:all: .

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Medium - Install check cannot pass because pytricia has no wheel

The image uses Python 3.14 on Linux, where the package's required pytricia dependency is selected, but the repository lock records version 1.3.0 with only a source tarball. Because --only-binary=:all: forbids that distribution and the slim image has no compiler to build it, every run reaches pip's resolution error before installing the project, leaving this new workflow permanently red rather than providing a usable toolchain-free install check.

Show fix

Provide and select a Linux/Python 3.14 wheel-capable replacement or release wheels for the required pytricia dependency before enforcing this check. If source builds are intentionally supported instead, remove the all-binary restriction and explicitly install the required build toolchain; otherwise use a dependency set that is actually satisfiable in the restricted image.

More info - Reply on this comment to give feedback or ignore the issue.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that is why we add a test for it. We will wait with merging this until we can enforce it.

Comment thread .dockerignore Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant