Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ACE 7 requires C++17 #4605

Closed
wants to merge 1 commit into from
Closed

ACE 7 requires C++17 #4605

wants to merge 1 commit into from

Conversation

jwillemsen
Copy link
Member 

* cmake/init.cmake:

@jwillemsen
ACE 7 requires C++17
de1940c 
    * cmake/init.cmake:
@jwillemsen jwillemsen mentioned this pull request Apr 26, 2024
Merged
@iguessthislldo
Copy link
Member

I don't want to unconditionally require 2017 by default and we're still trying to evaluate what we want to do in both OpenDDS and ACE/TAO.

@jwillemsen
Copy link
Member Author

When you want to keep ACE7/TAO3 at C++14 I am ok with releasing ACE8/TAO4 as next release with C++17 as minimum required level.

@jrw972
Copy link
Contributor

jrw972 commented Apr 29, 2024

@jwillemsen We believe it would simplify things for users if we maintain the same requirements of the C++ language standard for all of the ACE7/TAO3 releases. Changing to C++17 in ACE8/TAO4 would be perfectly acceptable.

@jwillemsen
Copy link
Member Author

Ok, will update ACE/TAO master that the next release will be a major one

@jwillemsen
Copy link
Member Author

DOCGroup/ACE_TAO#2230

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jwillemsen @iguessthislldo @jrw972