More build system tweaks, doc improvements.

Author: bluescarni

CMakeLists.txt

@@ -314,8 +314,11 @@ target_compile_options(pagmo PRIVATE
     "$<$<CONFIG:MinSizeRel>:${PAGMO_CXX_FLAGS_RELEASE}>"
 )
 
-# Set the minimum C++ standard to C++17
+# Set the minimum C++ standard to C++17, both when building
+# and consuming pagmo.
 target_compile_features(pagmo PUBLIC cxx_std_17)
+# Enforce vanilla C++17 when compiling pagmo.
+set_property(TARGET pagmo PROPERTY CXX_EXTENSIONS NO)
 # NOTE: make sure the include directories from the current build
 # are included first, so that if there is already a pagmo installation
 # in the prefix path we don't risk including the headers from that

benchmarks/CMakeLists.txt

@@ -8,7 +8,8 @@ function(ADD_PAGMO_BENCHMARK arg1)
         "$<$<CONFIG:MinSizeRel>:${PAGMO_CXX_FLAGS_RELEASE}>"
     )
     # Set the minimum C++ standard to C++17
-    target_compile_features(${arg1} PUBLIC cxx_std_17)
+    target_compile_features(${arg1} PRIVATE cxx_std_17)
+    set_property(TARGET ${arg1} PROPERTY CXX_EXTENSIONS NO)
 endfunction()
 
 ADD_PAGMO_BENCHMARK(thread_island_pool)

doc/sphinx/changelog.rst

@@ -3,6 +3,23 @@
 Changelog
 =========
 
+2.16.1 (unreleased)
+-------------------
+
+Changes
+~~~~~~~
+
+- pagmo now requires CMake >= 3.8
+  (`#458 <https://github.com/esa/pagmo2/pull/458>`__).
+
+Fix
+~~~
+
+- Various build system fixes/improvements
+  (`#459 <https://github.com/esa/pagmo2/pull/459>`__,
+  `#458 <https://github.com/esa/pagmo2/pull/458>`__,
+  `#457 <https://github.com/esa/pagmo2/pull/457>`__).
+
 2.16.0 (2020-09-25)
 -------------------
 

doc/sphinx/quickstart.rst

@@ -38,7 +38,7 @@ Using pagmo with CMake
 As a part of the pagmo installation, a group of CMake files is installed into
 ``CMAKE_INSTALL_PREFIX/lib/cmake/pagmo``.
 This bundle, which is known in the CMake lingo as a
-`config-file package <https://cmake.org/cmake/help/v3.3/manual/cmake-packages.7.html>`__,
+`config-file package <https://cmake.org/cmake/help/latest/manual/cmake-packages.7.html>`__,
 facilitates the detection and use of pagmo from other CMake-based projects.
 pagmo's config-file package, once loaded, provides
 an imported target called ``Pagmo::pagmo`` which encapsulates all the information
@@ -53,9 +53,6 @@ program presented earlier may look like this:
 
 .. code-block:: cmake
 
-   # pagmo needs at least CMake 3.3.
-   cmake_minimum_required(VERSION 3.3)
-
    # The name of our project.
    project(sample_project)
 

tests/CMakeLists.txt

@@ -18,7 +18,8 @@ function(ADD_PAGMO_TESTCASE arg1)
         "$<$<CONFIG:MinSizeRel>:${PAGMO_CXX_FLAGS_RELEASE}>"
     )
     # Set the minimum C++ standard to C++17
-    target_compile_features(${arg1} PUBLIC cxx_std_17)
+    target_compile_features(${arg1} PRIVATE cxx_std_17)
+    set_property(TARGET ${arg1} PROPERTY CXX_EXTENSIONS NO)
     add_test(${arg1} ${arg1})
 endfunction()
 

tutorials/CMakeLists.txt

@@ -8,7 +8,8 @@ function(ADD_PAGMO_TUTORIAL arg1)
         "$<$<CONFIG:MinSizeRel>:${PAGMO_CXX_FLAGS_RELEASE}>"
     )
     # Set the minimum C++ standard to C++17
-    target_compile_features(${arg1} PUBLIC cxx_std_17)
+    target_compile_features(${arg1} PRIVATE cxx_std_17)
+    set_property(TARGET ${arg1} PROPERTY CXX_EXTENSIONS NO)
     add_test(${arg1} ${arg1})
 endfunction()