Detect dirty build step.
Metadata
Current evaluation
Merged. Adds dirty build step detection to Snapcraft, prompting manual cleanup when YAML changes affect the build step. Addresses LP #1477904. Auto-rebuilds were omitted to prevent wasting time on long builds. Approved by sergiusens.
Suggested action: —
No scores available.
Issue body
Currently Snapcraft notices YAML changes only for the strip and stage steps. For the other steps, if they've already run, it simply says so and bails. This PR makes progress on LP: [#1477904](https://bugs.launchpad.net/snapcraft/+bug/1477904) by changing that for the build step, where now if one changes any part of the YAML that the part's plugin says is associated with building, build will notice that it's out of date and error out asking the user to clean the build step to continue.
The build step is not automatically cleaned like the strip and stage steps because builds can take a very long time and we don't want to blow one away without the user explicitly asking to do so.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Adds dirty build step detection to Snapcraft, prompting manual cleanup when YAML changes affect the build step. Addresses LP #1477904. Auto-rebuilds were omitted to prevent wasting time on long builds. Approved by sergiusens. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged changes implement dirty build step detection in Snapcraft. YAML modifications affecting the build step now trigger an error requiring manual cleanup instead of auto-rebuilding, partially resolving LP: #1477904. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged implementation of dirty build step detection in Snapcraft. The build step now detects YAML changes, errors on modification, and requires manual cleanup to preserve long-running builds. Approved by sergiusens. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #448 Detect dirty pull step. | snapcraft | merged | Merged changes add dirty detection for the pull step in Snapcraft, resolving Launchpad bug #1477904. The update errors on YAML changes, requiring manual cleanup instead of automatic removal. Approved by sergiusens. | |
| #442 Automatically re-strip if dirty. | snapcraft | merged | Merged. Automatically re-runs the strip step when the snapcraft.yaml snap keyword changes. Detects dirty state via YAML property comparison, cleans, and re-executes. Resolves Launchpad bug #1477904. Documentation added during review. | |
| #443 Automatically re-stage if dirty. | snapcraft | merged | Merged PR automatically re-stages and strips Snapcraft steps upon YAML changes, resolving Launchpad bug #1477904. The update modifies six files to detect out-of-date configurations and trigger a clean re-run. |