Skip to content

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

dorotaphanSiili
Copy link
Collaborator

@dorotaphanSiili dorotaphanSiili commented Oct 22, 2024

https://github.com/apigear-io/internal-issue-tracker/issues/12

Closes #

πŸ“‘ Description

βœ… Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

β„Ή Additional Information

Copy link

@w4bremer w4bremer left a comment

Choose a reason for hiding this comment

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

Minor docs change.

@@ -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.

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.
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.

2 participants