-
Notifications
You must be signed in to change notification settings - Fork 990
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] Add cstd to compiler settings #13777
Comments
Thanks for your proposal @PhilippFinke1988 I think this proposal makes sense, and we could consider it.
That means that we would need some recipe opt-in for this feature:
Taking this complexity in mind, it will be challenging to prioritize this for the short-term, there would be quite a few other priorities. I am assigning this as 2.X roadmap, in the meantime, it might be approached by injection of |
@memsharded Thanks for the fast reply and I'm happy to hear that you consider my proposal. You're totally correct that this is a rather complex feature request and I understand it cannot be done in the short-term. As we're currently still using Conan 1.X in our company and planning to migrate to 2.X in the long-term. So we'd be happy to see this feature in a future 2.X release. Until then we'll surely find a workaround for this issue based on the ideas you provided. |
@memsharded Is there already any decision to introduce the |
Thanks for the ping @PhilippFinke1988 We haven't been able to prioritize this yet, we have been very busy with the stabilization of 2.0, migration of ConanCenter to 2.0, and also releasing already 17 releases with some very demanded features (https://blog.conan.io) like package-lists, cache save/remote, sources backup, metadata files, etc. I think we want to tackle this when possible, let me assign it a fixed milestone and a look-into label. |
Resuming this to discuss with the team. Alternatives:
|
Quick questions: Would adding a new Would it be preferable that when a profile/cli defines |
I am starting the proposal in #16028, testing and feedback welcome. |
#16028 was merged, next Conan 2.4 will have |
Background
We encountered an issue with the GCC toolchain (at least GCC 9) pulling in BSD and SVr4 extensions if no C/C++ dialect via compiler flag -std= has been explicitly defined. This is unwanted on many embedded targets.
We found out that in contrast to the legacy Conan CMake build helper the new generate() automatically translates the infomation in setting compiler.cppstd to a toolchain specific flag and appends it to the compiler call. This works pretty well for C++ files, but as many of our packages contain C files or are written exclusively in C, there is no similar feature for C in Conan. Using the same flag as for C++ is no option as this will create warnings for C files.
Proposal
Have you read the CONTRIBUTING guide?
The text was updated successfully, but these errors were encountered: