← Back to issue list

fix(part_handler): fetch correct target stage packages

View original Github issue

Metadata

Project
craft-parts
Number
#721
Type
pull request
State
open
Author
dilyn-corner
Labels
—
Created
Updated
Closed
—

Current evaluation

Targets architecture for stage-package fetches, but omits architecture:all handling; approved once with an unresolved regression-review thread and no CI after 889 days.

Suggested action: close stale

Reason: A maintainer explicitly required regression testing against LP #1983009 before merging, but no such check or CI run is listed, and the author states architecture:all remains unsupported. After that April 2024 request, the external contributor made no implementation follow-up for 889 days, while current main still uses the merged host-architecture behavior from #258.

Evaluated by: mr-cal-bot

Impact: 58 Quick Win: 9 Complexity: 42 Confidence: 82 Actionability: 28

Issue body

- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- This is a partial attempt at fixing https://github.com/canonical/snapcraft/issues/4542 specifically [Alex's comment](https://github.com/canonical/snapcraft/issues/4542#issuecomment-1934943478). Note that this change does not take into consideration cases where the stage package is only available for `architecture: all`.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha
Impact: 58
Quick Win: 9
Complexity: 42
Confidence: 82
Actionability: 28
close stale Targets architecture for stage-package fetches, but omits architecture:all handling; approved once with an unresolved regression-review thread and no CI after 889 days.
qwen/qwen3.6-35b-a3b
Impact: 65
Quick Win: 25
Complexity: 40
Confidence: 75
Actionability: 65
needs review Fixes cross-compilation fetching wrong architecture stage-packages. Approved by 1 reviewer with 1 unresolved comment, awaiting maintainer review.
qwen3.6-35b-a3b-mtp-q6
Impact: 65
Quick Win: 29
Staleness: 35
Complexity: 55
Confidence: 70
needs review Fixes cross-compilation stage-package fetching to use target architecture. Approved with 1 unresolved review comment; regression testing requested but not yet confirmed.
qwen/qwen3.8-27b
Impact: 55
Quick Win: 36
Staleness: 85
Complexity: 35
Confidence: 72
needs review Fixes cross-compilation stage-packages fetching by using target arch instead of host arch in part_handler. Approved by 1 reviewer but inactive for ~2 years; related snapcraft issue #4542 closed with root cause tracked in craft-parts.
qwen/qwen3.6-35b-a3b
Staleness: 92
Complexity: 15
Confidence: 85
close stale Small fix to fetch correct target stage packages. Approved by one reviewer but inactive for over 1.8 years. Maintainers requested regression testing and additional context before merging.
qwen3.6-35b-a3b-mtp-q6
Staleness: 88
Complexity: 50
Confidence: 85
close stale Addresses incorrect target stage package fetching in part_handler. Currently incomplete, lacks architecture: all support, and has been inactive since July 2024 with unresolved maintainer requests for regression testing.

Update history

Date Change
updated

Related work

  • Related To: canonical/snapcraft#4542 (confidence 100%)

    The PR explicitly targets Alex's architecture-selection comment in this snapcraft issue.

  • Related To: canonical/craft-parts#258 (confidence 98%)

    A reviewer linked this merged PR as regression context; it established the host-architecture fetch behavior still present on main.

  • Related To: canonical/craft-archives#104 (confidence 96%)

    A reviewer linked this issue, whose foreign-architecture support was subsequently implemented and shipped in craft-archives 2.1.0.

Related issues

Issue Project State Summary Similarity
#258 packages: fetch stage-packages from host arch's repo craft-parts merged Merged in canonical/craft-parts#258 via ef86c802, fetching stage-packages from host-architecture repositories to fix cross-compilation failures.
72%
#876 repo: allow for architecture-specific stage-packages snapcraft merged Merged as 9d6f953a1 with package:arch support for stage-packages; the change was later reverted by #898 in 5ac55528f.
70%