snap: pull early
Metadata
Current evaluation
Merged #2278 (commit 7386bed80) reordered snapcraftctl execution so pull runs before set-version.
Resolution: closed resolved
Reason: PR #2278 was approved and merged. Commit 7386bed80 applies the one-line change so snapcraftctl pull executes before snapcraftctl set-version.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Do "snapcraftctl pull" before "snapcraftctl set-version"
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/)?
- [ ] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [ ] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [ ] 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 #2278 (commit 7386bed80) reordered snapcraftctl execution so pull runs before set-version. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2278 (commit 7386bed80) reordered snapcraftctl commands so pull runs before set-version. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2278 in commit 7386bed80 to run snapcraftctl pull before snapcraftctl set-version. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a change to execute snapcraftctl pull before snapcraftctl set-version, ensuring correct build initialization order. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2063 many: add snapcraftctl set-version | snapcraft | merged | Merged as commit 278f9b79f, adding snapcraftctl set-version for scriptlets and reusable setter infrastructure; resolves #1676. | |
| #2653 scriplets: run override-pull on update_pull | snapcraft | merged | Merged in snapcraft#2653 (commit ae7ab338e), re-running override-pull scriptlets when pull repeats so snapcraftctl-generated metadata is preserved. | |
| #2103 package: make use of snapcraftctl snapcraft features | snapcraft | merged | Merged in #2103 as commit 60d505347 and shipped in 2.42, replacing prepare/build/install hooks with override-pull/override-build. | |
| #2045 many: add override-pull scriptlet | snapcraft | merged | Merged as commit 56c16cbc5: added override-pull and snapcraftctl pull; Local sources now merge into the existing directory to preserve pre-pull changes. |