← 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 #3208 in commit cd4f5d5cb, applying the build environment’s fixed PATH to the Snapcraft command for parity with Launchpad buildd and destructive-mode runs.

Resolution: closed resolved

Reason: PR #3208 was approved and merged as commit cd4f5d5cb, setting the Snapcraft command’s PATH consistently with the build environment.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged #3208 in commit cd4f5d5cb, applying the build environment’s fixed PATH to the Snapcraft command for parity with Launchpad buildd and destructive-mode runs.
stealth/space-bunny-alpha — closed resolved Merged #3208 in commit cd4f5d5cb, applying the build environment’s fixed PATH to the snapcraft command for parity with Launchpad buildd and destructive-mode instances.
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.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#564 fix(snap): use snap bin paths if in a snap craft-parts closed PR #564 was closed unmerged after unresolved review comments and prolonged inactivity; its Snap-aware binary PATH change never landed in craft-parts.
73%
#2889 meta: always generate snapcraft-runner to workaround classic PATH bug snapcraft merged Merged #2889 as commit 343ac2862, always adding an empty snapcraft-runner to app command chains to work around incorrect PATH in non-shell classic apps.
71%
#1128 feat(remotebuild): add build-path to snap recipes craft-application merged Merged PR #1128 as commit 90cc2c75, adding Launchpad build-path support to SnapRecipe and RemoteBuildService. It fixes #1106 and unblocks canonical/snapcraft#6287.
71%