many: support the use of build-aux/snap
Metadata
Current evaluation
Merged PR snapcore/snapcraft#2496 (commit 4c92c1b89) by maintainer sergiusens, adding support for build-aux/snap as an alternative location for snapcraft.yaml and Snapcraft assets, following GNOME conventions (LP: #1805219).
Resolution: closed resolved
Reason: The PR itself was merged into master as commit 4c92c1b89 'many: support the use of build-aux/snap (#2496)', with review approved and no unresolved comments. It was later complemented by #2551 (7ca0398c5) which reads local plugins from build-aux.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Many projects required an alternate snap directory to store
snapcraft related assets, in particular Gnome relate projects
have consolidated with the use of build-aux which snapcraft
is now adopting as an alternative as well.
LP: #1805219
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR snapcore/snapcraft#2496 (commit 4c92c1b89) by maintainer sergiusens, adding support for build-aux/snap as an alternative location for snapcraft.yaml and Snapcraft assets, following GNOME conventions (LP: #1805219). | |
| stealth/space-bunny-alpha | — | closed resolved | Merged snapcore/snapcraft#2496 (commit 4c92c1b89), adding build-aux/snap as an alternative location for Snapcraft assets and snapcraft.yaml, following GNOME conventions. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Implements support for the build-aux/snap directory as an alternative snapcraft asset location, aligning with GNOME conventions. The change passed unit tests, static analysis, and achieved 86.66% diff coverage. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#2551
(confidence 85%)
Follow-up merged PR (7ca0398c5) extending the same feature to read local plugins from build-aux.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1805219 Support for looking in build-aux/snap | snapcraft (launchpad) | closed | Implemented and shipped: snapcore/snapcraft#2496 (commit 4c92c1b89) added build-aux/snap as an alternative location for snapcraft.yaml and assets; LP bug auto-closed by lpqabot after stable revision r18938. | |
| #274 fix: add build-aux location for snapcraft project | fetch-service | merged | Merged in PR #274 via commit 01a15fb, adding build-aux/snap/snapcraft.yaml as a recognized Snapcraft project path; included in 0.4.1 to address the OSV issue. | |
| #5282 docs: document the `build-aux` directory | snapcraft | closed | Documentation added: docs/explanation/snapcraft-yaml.rst now lists <project>/build-aux/snap/snapcraft.yaml as a valid project file location. Resolved by merged PR #5466 (commit 2670e8278); maintainer closed the issue citing it. | |
| #2499 many: support for "base: core" in snapcraft.yaml | snapcraft | merged | Merged PR #2499 added support for base: core in snapcraft.yaml and omitted it from generated snap.yaml; resolution: e1cc2b91c. | |
| #1772584 Having a "snap" directory with actual content causes build failures | snapcraft (launchpad) | closed | Resolved by adding build-aux as an alternative location for Snapcraft assets, preserving source-tree snap directories; merged in snapcraft#2496 and #2551. | |
| #1819290 Support for "base: core" | snapcraft (launchpad) | closed | Implemented by merged snapcore/snapcraft#2499 (commit e1cc2b91c), adding 'base: core' support in snapcraft.yaml and stripping it from generated snap.yaml; Launchpad base config updated by cjwatson. |