Skip to content

Resolve "Update C++ standard requirement"

Lukas Riedel requested to merge 198-update-c-standard-requirement into master

What does this MR do?

Explicitly sets the C++ standard version for the compiled DORiE libraries. Removes the call to dune_require_cxx_standard(), which seems to have no helpful effect.

Is there something that needs to be double checked?

  • Compiles on macOS (@lriedel)
  • Compiles on macOS (@hbauser)

Can this MR be accepted?

  • Implemented CMake configuration for setting C++17 standard
  • [ ] Added/Updated tests
  • [ ] Added/Updated documentation
  • Pipeline passing
  • Squash option set
  • Delete branch option set
  • Added entry to CHANGELOG.md

Assignee: If the Squash option is set, check/update the commit message right before merging!

Related issues

Closes #198 (closed)

Edited by Lukas Riedel

Merge request reports