-
Notifications
You must be signed in to change notification settings - Fork 0
Dphan/update to cpp17 #4
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
base: main
Are you sure you want to change the base?
Conversation
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.
Minor docs change.
docs/docs/features/conan.md
Outdated
@@ -8,6 +8,9 @@ import helloWorldModuleComponent from '!!raw-loader!./data/helloworld.module.yam | |||
|
|||
With this feature you we generate a conan support for your project. It helps with packaging and to managing dependencies for different platforms, OSs, compilers, architecture, cross-builds. Read more about conan and its all benefits [here](https://conan.io/). | |||
|
|||
::tip:: | |||
Make sure your conan profile has all the required settings set to desired values e.g. the cpp standard set to 17. |
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.
Keep it simple and specific.
Please make sure your C++ standard is set to 17 in your conan configuration, e.g.:
compiler.cppstd=17
For more details, please check https://docs.conan.io/2/reference/config_files/profiles.html#profiles
The version of cmake used across the project was varying. With this commit it is changed to same, highest used version.
the conan profile by default may provide inaccurate settings. For windows it often sets cpp version to 14, where for this project the 17 is the minimal required version. The profile is dependent on the user environment and cannot be provided by apigear, hence we chose to add this information in documentation.
4c57139
to
9b6e73f
Compare
https://github.com/apigear-io/internal-issue-tracker/issues/12
Closes #
π Description
β Checks
βΉ Additional Information