← Back to issue list

docs: document autotools disable-parallel key

View original Github issue

Metadata

Project
craft-parts
Number
#1631
Type
pull request
State
open
Author
GreyViperTooth
Labels
Created
Updated
Closed

Current evaluation

Documentation PR adding disable-parallel key docs for autotools plugin. Approved by 1 reviewer but has 2 unresolved comments and failing CI checks.

Suggested action: needs review

Reason: The PR has 1 approval but 2 unresolved review comments and failing CI checks (test-common / Minimum dependencies, test-java-plugins / Slow tests). The author responded to @lengau's nudge and @bepri's feedback but hasn't yet addressed the unresolved comments. Maintainer review is needed to resolve remaining feedback and CI issues before merging.

Impact: 20 Quick Win: 17.0 Staleness: 25 Complexity: 15 Confidence: 75

Issue body

The [Autotools plugin reference](https://documentation.ubuntu.com/craft-parts/en/latest/common/craft-parts/reference/plugins/autotools_plugin/) didn't mention that the plugin builds in parallel, nor did it document the `disable-parallel` key, even though the option is supported by the plugin. This PR documents both: - Adds a `disable-parallel` key entry under **Keys** (type boolean, default `false`), explaining that the plugin runs `make` with `-j` set to `CRAFT_PARALLEL_BUILD_COUNT` by default, and that setting the key to `true` forces a serial build for projects whose build system isn't parallel-safe. - Updates the **How it works** section so the `make` step notes the parallel behavior and the opt-out. - Adds a changelog entry under the Unreleased section. Docs only — no functional changes. Fixes #1603

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b
Impact: 20
Quick Win: 17.0
Staleness: 25
Complexity: 15
Confidence: 75
needs review Documentation PR adding disable-parallel key docs for autotools plugin. Approved by 1 reviewer but has 2 unresolved comments and failing CI checks.
qwen/qwen3.6-35b-a3b
Impact: 25
Quick Win: 22.5
Staleness: 10
Complexity: 10
Confidence: 85
needs review Documents the autotools plugin's disable-parallel key and parallel build behavior. Approved by 1 reviewer with 2 unresolved comments; CI passing.
qwen3.6-35b-a3b-mtp-q6
Impact: 15
Quick Win: 14.25
Staleness: 20
Complexity: 5
Confidence: 85
needs review Documents the disable-parallel key for the Autotools plugin and updates build behavior notes. Approved by one reviewer with 2 unresolved comments, CI passing.
qwen/qwen3.6-35b-a3b
Staleness: 45
Complexity: 10
Confidence: 85
needs review Documents the disable-parallel key for the Autotools plugin, updates build behavior notes, and adds a changelog entry. Currently approved by one reviewer with two unresolved comments, and CI is passing.
qwen/qwen3.6-35b-a3b
Staleness: 10
Complexity: 5
Confidence: 85
needs review Documents the disable-parallel key for the Autotools plugin and updates related build steps. Approved by one reviewer but has 5 unresolved comments and a failing OSV-scanner check.
qwen3.6-35b-a3b-mtp-q6
Staleness: 5
Complexity: 5
Confidence: 90
needs review Documents the disable-parallel key for the Autotools plugin, explaining its boolean type, default value, and usage for serial builds. Updates plugin reference and changelog. Docs-only change. Awaiting review.

Update history

Date Change
updated
updated
updated
updated

Related work

  • Related To: craft-parts#1603 (confidence 90%)

    This PR explicitly fixes #1603, which requested documentation for the disable-parallel key in the autotools plugin.

Related issues

Issue Project State Summary Similarity
#1603 docs: for the autotools plugin relevant flags are not described, e.g. disable_parallel craft-parts open Documentation missing for autotools plugin's disable_parallel option. PR #1631 opened to address this, currently under review with unresolved comments.
81%
#1167 feat: add parallel parameter to autotools plugin craft-parts merged Merged a feature adding a disable_parallel boolean option to the autotools plugin to suppress the -j make flag. Approved by three reviewers, passed all CI checks, and updated the changelog to resolve issue #1069.
77%