parser: support remote dependencies
Metadata
Current evaluation
Merged in snapcraft#930 (commit 6c012194b), adding remote-dependency support by retrying parts after valid remote parts load. Fixes LP: #1645350.
Resolution: closed resolved
Reason: PR #930 was approved and merged as commit 6c012194b. The parser now retries parts with unresolved dependencies after loading valid remote parts, fixing LP: #1645350.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Ensure that we re-parse the parts that have missing dependencies after we've added all the valid ones to the `master_parts_list`
I would have fixed this in a cleaner way (allowing to include parts of a snapcraft.yaml that are good, and ignoring the bad ones only) but it would have caused a major refactor of the parser which probably is better to avoid.
Fixes [LP: #1645350](https://bugs.launchpad.net/snapcraft/+bug/1645350)
@josepht, please have a look to this.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged in snapcraft#930 (commit 6c012194b), adding remote-dependency support by retrying parts after valid remote parts load. Fixes LP: #1645350. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in snapcraft#930 (commit 6c012194b): parser re-parses parts with missing dependencies after valid remote parts load, fixing LP: #1645350. Remote-part support was later removed in #2304. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in snapcraft#930 (commit 6c012194b), adding remote-dependency support by re-parsing parts with missing dependencies after valid wiki entries load. Fixes LP: #1645350. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in snapcraft#930 (commit 6c012194b), adding remote-dependency support by retrying parts after valid remote parts load. Fixes LP: #1645350. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged. Enables parser support for remote dependencies by re-parsing parts with missing dependencies after valid ones are processed. Fixes LP: #1645350. Approved after refactoring post-processing logic into a separate method. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Enables parser support for remote dependencies by re-parsing parts with missing dependencies after valid ones are added, fixing LP: #1645350. Approved after extracting post-processing into a separate method and retaining error handling. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft (launchpad)#1645350
(confidence 98%)
The PR body states that it fixes Launchpad bug #1645350.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1645350 Parts depending on other remote parts aren't properly added | snapcraft (launchpad) | closed | Remote-part dependencies were fixed by merged snapcore/snapcraft PR #930 (commit 6c012194b), adding parser support for parts defined in remote origins. | |
| #1750866 Remote parts can't depend on other remote parts | snapcraft (launchpad) | closed | Remote-part dependency support remained unresolved; proposed fix snapcore/snapcraft#2105 stalled, and Launchpad #1750866 was later closed without a merged resolution. | |
| #590 Integrate with new remote parts | snapcraft | merged | Merged PR #590 as commit 9c9fb6949, integrating Snapcraft with the new remote-parts service and updating part loading, caching, and related tests. |