snap: migrate builds to core26 - #7242
Closed
kvnloo wants to merge 2 commits into
Closed
Conversation
Assisted-by: Hermes Agent:gpt-5.6-sol
Assisted-by: Hermes Agent:gpt-5.6-sol
Contributor
Author
|
Closing this replacement draft. The cleaner repository practice is to keep the existing review thread and update the contributor-owned PR branch with a lease-guarded history rewrite. I will restore #7238 with the corrected, policy-compliant commits. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #7199
Description of the change
Migrates the Snap base from
core24tocore26. The existing Python build dependencies now run in a project-local virtual environment so the build does not invoke externally managed system pip under PEP 668.This replacement removes the unnecessary test file from #7238. Each AI-assisted commit includes the BOINC-required
Assisted-by: Hermes Agent:gpt-5.6-soltrailer.Verification completed
pip install --userfailing withexternally-managed-environmentin Ubuntu 26.04git diff --check upstream/master...HEADpassedDraft status
A complete Snap package build has not yet been verified at this exact head. This PR remains draft until that build path is green.
AI assistance
Hermes Agent using gpt-5.6-sol assisted with investigation, implementation, validation, and review. I reviewed the final one-file diff and the commands listed above.
Summary by cubic
Migrates the Snap base from
core24tocore26and isolates Python build dependencies in a project-local virtual environment. Previously the build usedpip --userand$HOME/.local/bin; now it creates.venvunder$CRAFT_PART_BUILDand installspipandjinja2there to avoid PEP 668 externally-managed-environment failures on Ubuntu 26.04.python3andpython3-piptobuild-packages; keepspython3-venv; updates PATH to use.venv/bin.pip --userinstalls withpython3 -m venvandpython -m pip install -U pip jinja2.base: core26in Snapcraft; verify a full snapcraft build before promotion.Written for commit 1964c1a. Summary will update on new commits.