Skip to content

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

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

czoido merged 5 commits into
conan-io:develop2from
uilianries:feature/conf-configure-args

Conversation

@uilianries

@uilianries uilianries commented May 21, 2025

Copy link
Copy Markdown
Member

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
Copy Markdown
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
Copy Markdown
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>
Comment thread reference/tools/gnu/autotoolstoolchain.rst
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
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