The Wayback Machine - https://web.archive.org/web/20211023154328/https://github.com/opencv/opencv/pull/19592
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.

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

Use C++ 14 in Cmake and related cleanup #19592

Open
wants to merge 2 commits into
base: next
Choose a base branch
from
Open

Use C++ 14 in Cmake and related cleanup #19592

wants to merge 2 commits into from
+15 −34

Conversation

@cyyever
Copy link
Contributor

@cyyever cyyever commented Feb 21, 2021 •
edited by alalek

I just change CMAKE_CXX_STANDARD to 14.

build_image:Custom=centos:7
buildworker:Custom=linux-6
@cyyever cyyever changed the title Use C++ 14 Use C++ 14 in Cmake and related cleanup Feb 21, 2021
alalek
alalek reviewed Feb 21, 2021
View changes
cmake/OpenCVDetectCXXCompiler.cmake Show resolved Hide resolved
cmake/OpenCVDetectCXXCompiler.cmake Outdated Show resolved Hide resolved
@cyyever
Copy link
Contributor Author

@cyyever cyyever commented Feb 22, 2021

@alalek The previous change breaked BC. I try to keep the check now.

@cyyever cyyever force-pushed the cxx14 branch 2 times, most recently from 0225058 to a2af650 Feb 22, 2021
@asmorkalov asmorkalov added this to the 5.0 milestone Feb 24, 2021
@asenyaev
Copy link

@asenyaev asenyaev commented Apr 8, 2021

jenkins cn please retry a build

@cyyever
use c++14 and improve checking
0b1a227
@cyyever
remove unneeded HAVE_CXX11 conditions
82e6d8f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alalek alalek

Assignees
No one assigned
Labels
category: build/install
Milestone
5.0
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
@cyyever @asenyaev @alalek @asmorkalov