Closed Bidski closed 3 years ago
@Bidski that's great, thanks a lot for digging this out.
Could I ask you to make the following further additions to the PR:
CMakeLists.txt
of tests
, benchmarks
and tutorials
,CMakeLists.txt
,doc/sphinx/install.rst
to reflect the fact that the minimum required CMake version is now 3.8.If you don't have time/bandwidth for this, I'll do this myself, just let me know.
@bluescarni I have applied the changes you requested
@Bidski great, thanks a lot! I'll merge as soon as the CI runs.
See #456
Setting the C++ standard in this way causes it to show up in
pagmo_export.cmake
as anINTERFACE_COMPILE_FEATURE
which, in turn, will instruct anyone linking against thepagmo
cmake target to compile using the C++17 standard (at minimum). See cmake docs