-
Notifications
You must be signed in to change notification settings - Fork 993
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[feature] Export cppstd_flag as public tool #15710
[feature] Export cppstd_flag as public tool #15710
Conversation
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
@@ -7,6 +6,7 @@ def architecture_flag(settings): | |||
returns flags specific to the target architecture and compiler | |||
Used by CMakeToolchain and AutotoolsToolchain | |||
""" | |||
from conan.tools.apple.apple import _to_apple_arch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This lazy import is to avoid a cyclic import error.
This reverts commit a98b289.
Thanks! Will you backport it to Conan 1.X, to achieve recipe compatibility in 1.64? |
That's an excellent idea actually, otherwise, we will not be able to use it in CCI. I'll open a second PR. |
Done, the PR #15731 has the backport implementation. |
This PR exposes the cppstd_flag method from flags.py to builds:
Changelog: Feature: Make
cppstd_flag
public to return the corresponding C++ standard flag based on the settings.Docs: conan-io/docs#3599
closes #12603
develop
branch, documenting this one.