← Back to issue list

autotools v2 plugin: rename configflags to autotools-configure-parame…

View original Github issue

Metadata

Project
snapcraft
Number
#3066
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Merged a maintainer change renaming configflags to autotools-configure-parameters in the autotools v2 plugin. This correctly namespaces the property to eliminate ambiguity. All checks passed and the code was integrated.

Suggested action:

No scores available.

Issue body

…ters This correctly namespaces the property to the autotools plugin removing any ambiguity on what processes it. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `./runtests.sh static`? - [ ] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a maintainer change renaming configflags to autotools-configure-parameters in the autotools v2 plugin. This correctly namespaces the property to eliminate ambiguity. All checks passed and the code was integrated.
qwen3.6-35b-a3b-mtp-q6 Merged by maintainer sergiusens: renames the configflags property to autotools-configure-parameters in the autotools v2 plugin. This properly namespaces the property to eliminate ambiguity about the processes it affects.
qwen3.6-35b-a3b-mtp-q6 Merged. Renamed configflags to autotools-configure-parameters in the autotools v2 plugin. This change correctly namespaces the property to the plugin, eliminating ambiguity about the processes it targets.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#521 Correct autotools documentation and tests to use configflags snapcraft merged Merged a fix correcting autotools plugin documentation and test cases to use configflags. Approved by maintainers sergiusens and kyrofa following IRC discussion.
75%
#1337 autotools: don't force uniqueness on configflags snapcraft merged Merged to stop forcing uniqueness on autotools configflags, resolving Launchpad bug #1694707. Approved by two reviewers, the change modifies two files with a net reduction of three lines.
74%
#3064 cmake v2 plugin: rename configflags to cmake-parameters snapcraft merged Merged. Renamed configflags to cmake-parameters in the cmake plugin to correctly namespace the property and remove ambiguity. Updated spread test accordingly.
73%
#35 plugins: add autotools plugin craft-parts merged Merged the addition of an autotools plugin for handling ./configure scripts. Approved by two reviewers, the change adds 195 lines across four files, successfully integrating new build system support.
72%
#1694707 autotools.configflags enforces uniqueness snapcraft (launchpad) closed Closed without resolution. The autotools plugin configflags property enforces uniqueness, blocking duplicate -skip arguments. No comments or fixes were provided, indicating abandonment.
71%