Skip to content

configure: make sure to use C++17 standard - #7934

Merged
nilason merged 2 commits into
OSGeo:mainfrom
nilason:fix_c++17_config
Sep 23, 2026
Merged

nilason merged 2 commits into
OSGeo:mainfrom
nilason:fix_c++17_config

Conversation

@nilason

@nilason nilason commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

Make sure to use C++17 standard while configuring with both Autotools and CMake.

The Autotools macros are fetched from https://www.gnu.org/software/autoconf-archive/ax_cxx_compile_stdcxx.html and https://www.gnu.org/software/autoconf-archive/ax_cxx_compile_stdcxx_17.html on 21 September 2026.

Closes #7933

@github-actions github-actions Bot added the CMake label Sep 21, 2026
@metzm metzm added the backport to 8.5 PR needs to be backported to release branch 8.5 label Sep 21, 2026
metzm
metzm previously approved these changes Sep 21, 2026

@metzm metzm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Works for me with both configure and CMake, thanks for the super-fast improvement!

@nilason

nilason commented Sep 22, 2026

Copy link
Copy Markdown
Contributor Author

I removed the file ax_cxx_compile_stdcxx_17.m4 which was not used anyway. Please re-approve for merging.

@nilason
nilason enabled auto-merge (squash) September 22, 2026 06:52
@nilason

nilason commented Sep 23, 2026

Copy link
Copy Markdown
Contributor Author

ping @metzm

@nilason
nilason merged commit 243ec75 into OSGeo:main Sep 23, 2026
27 checks passed
@github-actions github-actions Bot added this to the 8.6.0 milestone Sep 23, 2026
@nilason
nilason deleted the fix_c++17_config branch September 24, 2026 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport to 8.5 PR needs to be backported to release branch 8.5 CMake

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] PDAL 2.10 needs C++ standard 2017

2 participants