Skip to content

Specify the target required c++ version #3

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

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

jmigual
Copy link
Contributor

@jmigual jmigual commented Jun 12, 2024

Without this, the user importing the library is forced to set:

set(CMAKE_CXX_EXTENSIONS OFF)
set(CMAKE_CXX_STANDARD 17)

somewhere in their project. With this, the library can be just linked with:

target_link_libraries(<my_target> maxplus)

and the right c++ version is already set.

@jmigual
Copy link
Contributor Author

jmigual commented Jun 20, 2024

@MarcG-TUE

@MarcG-TUE
Copy link
Contributor

I did see it, but I haven't had time to fix the mess caused by the TU/e enterprise thingy...
when I tried I was not allowed to merge.

@MarcG-TUE MarcG-TUE merged commit f61d86c into Model-Based-Design-Lab:main Jun 20, 2024
@jmigual
Copy link
Contributor Author

jmigual commented Jun 20, 2024

Ah, I see. I still cannot see the TU/e enterprise organisation on GitHub. :/

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