← Back to issue list

snap: set PATH for snapcraft command

View original Github issue

Metadata

Project
snapcraft
Number
#3208
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Merged to configure the PATH for the snapcraft command in the build environment, ensuring parity with launchpad buildd and destructive-mode instances. Approved, passed CI, and modified three files.

Suggested action:

No scores available.

Issue body

Use the fixed path setup in the build environment, this brings parity to the launchpad buildd environment and any other --destructive-mode instance. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged to configure the PATH for the snapcraft command in the build environment, ensuring parity with launchpad buildd and destructive-mode instances. Approved, passed CI, and modified three files.
qwen3.6-35b-a3b-mtp-q6 Merged to set a fixed PATH for the snapcraft command in the build environment, ensuring parity with launchpad buildd and destructive-mode configurations.
qwen3.6-35b-a3b-mtp-q6 Merged. Sets the PATH for the snapcraft command in the build environment to match Launchpad buildd and destructive-mode setups. Integrated by maintainer sergiusens without discussion.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1635 snap: remove leaking LD_LIBRARY_PATH snapcraft merged Removed leaking LD_LIBRARY_PATH from the snapcraft-classic wrapper, resolving Launchpad bug #1723208. The variable leaked into the environment, breaking snaps on distros like Trusty, and was redundant given existing rpaths. Approved and merged.
73%
#2425 snap: re-add pyc files for snapcraft snapcraft merged Merged to master after approval and passing CI. The change re-adds pyc files for snapcraft to resolve Launchpad bug #1808199, modifying two files with minor adjustments.
72%
#3741 parts: replace snapcraft command-chain with environment snapcraft merged Merged updates replacing snapcraft command-chain with global environment variables (LD_LIBRARY_PATH and PATH) for core22 snaps. The adapter field is deprecated. Approved and merged after passing CI checks.
72%
#2256 local source: don't include .snapcraft directory snapcraft merged Resolved LP: #1791773 by merging a change that excludes the .snapcraft directory from copying and part staleness checks. Approved, passed CI, and merged with a six-line update across two files.
72%
#2278 snap: pull early snapcraft merged Merged a one-line change to execute snapcraftctl pull before snapcraftctl set-version. Approved by one reviewer, passed CI checks, and modified a single file.
72%
#2411 cli: snapcraft init with a base snapcraft merged Merged. Adds a syntax documentation link after snapcraft init execution. Approved by one reviewer, passed CI checks, modified five files, and achieved 100% diff coverage. Resolves LP #1803776.
72%
#2496 many: support the use of build-aux/snap snapcraft merged Merged implementation adding build-aux/snap as an alternate directory for snapcraft assets. Approved by reviewer, passed CI, and modified 7 files (+207/-64 lines) to align with GNOME project conventions.
72%
#1601 snapcraft.yaml: don't re-use build dir snapcraft merged Merged a fix for a source-build directory clash in snapcraft.yaml. The build directory was renamed to apt-build to avoid scriptlet conflicts. Approved by reviewers and passed CI, resolving the failure.
71%
#1128 feat(remotebuild): add build-path to snap recipes craft-application merged Merged. Adds build-path to snap recipes and RemoteBuildService, resolving #1106 and unblocking snapcraft #6287. Exposed as --project-dir in the remote-build command. Approved by two reviewers and passed CI.
71%
#228 packages: run snap operations silently craft-parts merged Merged to fix #218 by running snap subprocesses silently, preventing stdout leakage that conflicts with craft-cli during lifecycle execution. Approved by one reviewer, passed CI, and modified two files (+32/-15 lines).
70%