← Back to issue list

meta: fix management of snap/local

View original Github issue

Metadata

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

Current evaluation

Merged fix for snap/local management that prevents copying it into the snap and improves embedded directory copy logic. Approved, passed CI checks, and resolves SNAPCRAFT-J2 and SNAPCRAFT-JB.

Suggested action:

No scores available.

Issue body

Do not copy snap/local into the snap, furthermore, improve the copy logic to work better with embedded directories. Fixes SNAPCRAFT-J2 Fixes SNAPCRAFT-JB Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged fix for snap/local management that prevents copying it into the snap and improves embedded directory copy logic. Approved, passed CI checks, and resolves SNAPCRAFT-J2 and SNAPCRAFT-JB.
qwen3.6-35b-a3b-mtp-q6 Merged a fix preventing snap/local from being copied into the snap and improving copy logic for embedded directories. Resolves SNAPCRAFT-J2 and SNAPCRAFT-JB. Approved and merged by maintainer sergiusens.
qwen3.6-35b-a3b-mtp-q6 Merged fix preventing snap/local from being copied into the snap and improving embedded directory copy logic. Resolves SNAPCRAFT-J2 and SNAPCRAFT-JB. Approved by maintainer sergiusens after passing static and unit tests.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#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.
74%
#4662 fix(test): do not create 2 snapcraft.yaml and confuse location snapcraft merged Merged a test fix preventing duplicate snapcraft.yaml creation in conflicting root and snap directories. Resolves silent overrides and ensures hooks are detected by clarifying fixture paths. Approved and merged.
72%
#4638 fix(application): use SNAPCRAFT_MANAGED_MODE everywhere snapcraft merged Merged. Replaces CRAFT_MANAGED_MODE with SNAPCRAFT_MANAGED_MODE to fix log loss in nested LXD instances. Approved by two reviewers and passed CI. Does not address unrelated spread test failures.
71%
#5085 fix: allow parallel installations of signed snapcraft snaps snapcraft merged Merged a fix enabling parallel installations of signed snapcraft snaps, resolving issues #4683 and #4927. Approved by two reviewers, passed CI checks, and integrated.
70%