Skip to content

AutotoolsToolchain: Add note about the new conf tools.gnu:configure_args #4100

Merged
czoido merged 5 commits intoconan-io:develop2from
uilianries:feature/conf-configure-args
May 26, 2025
Merged

AutotoolsToolchain: Add note about the new conf tools.gnu:configure_args #4100
czoido merged 5 commits intoconan-io:develop2from
uilianries:feature/conf-configure-args

Conversation

@uilianries
Copy link
Member

@uilianries uilianries commented May 21, 2025

Hello! This PR is related to conan-io/conan#18333

I just added a note to illustrate that configure arguments can be extended via Conan conf.

Page preview - Commit ae275a4

Screenshot 2025-05-23 at 09-45-01 AutotoolsToolchain — conan 2 16 1 documentation

Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Comment on lines +239 to +240
The ``AutotoolsToolchain`` will listen to ``tools.gnu:extra_configure_args`` from the :ref:`reference_config_files_global_conf` to extend the
``configure_args`` attribute.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's fine to add a comment here, saying that you can add your own configure args with this conf but this should also go down below in the conf section

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, please, see the commit 2fe9006

Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
@czoido czoido added this to the 2.17 milestone May 23, 2025
@czoido czoido removed the request for review from memsharded May 26, 2025 05:23
@czoido czoido merged commit 1f647d1 into conan-io:develop2 May 26, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants