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

Use target_compile_features to specify C++ standard requirement #3205

Merged
merged 1 commit into from
Nov 26, 2022
Merged

Use target_compile_features to specify C++ standard requirement #3205

merged 1 commit into from
Nov 26, 2022

Conversation

ChrisThrasher
Copy link
Contributor

@ChrisThrasher ChrisThrasher commented Nov 25, 2022

Closes #3189

I increase the minimum CMake version to 3.8 because that's when the cxx_std_11 compile feature was added. Realistically this requirement could be increased to at least 3.10 since 3.10 is what Ubuntu 18 ships with and it's unlikely that any users are locked to an older CMake version than that.

@vitaut
Copy link
Contributor

vitaut commented Nov 26, 2022

Thank you!

@ChrisThrasher ChrisThrasher deleted the cmake-3.8 branch November 26, 2022 19:11
OlivierLDff added a commit to OlivierLDff/date that referenced this pull request Dec 3, 2022
…ists.

I increase the minimum CMake version to 3.8 because that's when the cxx_std_11 compile feature was added. Realistically this requirement could be increased to at least 3.10 since 3.10 is what Ubuntu 18 ships with and it's unlikely that any users are locked to an older CMake version than that.

Text copied from fmtlib/fmt#3205
OlivierLDff added a commit to OlivierLDff/date that referenced this pull request Dec 3, 2022
…ists.

I increase the minimum CMake version to 3.8 because that's when the cxx_std_11 compile feature was added. Realistically this requirement could be increased to at least 3.10 since 3.10 is what Ubuntu 18 ships with and it's unlikely that any users are locked to an older CMake version than that.

Text copied from fmtlib/fmt#3205
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.

Hardcoding CMAKE_CXX_STANDARD breaks source-consuming downstream projects using target_compile_features
2 participants