← Back to issue list

executor: use source type specified in part (CRAFT-831)

View original Github issue

Metadata

Project
craft-parts
Number
#172
Type
pull request
State
merged
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Merged #172 (CRAFT-831) in cf99b276, honoring a part's declared source type before URI inference when collecting build packages; fixes #171.

Resolution: closed resolved

Reason: PR #172 was merged as commit cf99b276. Its implementation now uses part.spec.source_type when present and infers from the URI only as a fallback, resolving #171.

Evaluated by: mr-cal-bot

No scores available.

Issue body

When obtaining the consolidated list of build packages, honor the source type specified by the user before attempting to infer it from the source URI. Fixes #171. Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com> - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #172 (CRAFT-831) in cf99b276, honoring a part's declared source type before URI inference when collecting build packages; fixes #171.
stealth/space-bunny-alpha — closed resolved Merged PR #172, fixing source-type handling to honor the part's declared type before inferring it from the URI (CRAFT-831).
qwen/qwen3.6-35b-a3b — — Merged fix for #171 that prioritizes user-specified source types over URI inference during package builds. Approved by one reviewer, passed CI, and modified two files.
qwen3.6-35b-a3b-mtp-q6 — — Merged to fix #171 by prioritizing user-specified source types over automatic URI inference when compiling build package lists. Explicit configuration now takes precedence during executor initialization.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/craft-parts#171 (confidence 100%)

    Source-type recognition bug explicitly identified as fixed by PR #172.

Related issues

No related issues found above the similarity threshold.