From d790cf01d701a5f66a15163a7912206d137bbaf6 Mon Sep 17 00:00:00 2001 From: Andrey Prokopenko Date: Tue, 12 Nov 2024 10:46:01 -0500 Subject: [PATCH 1/7] CMake AnyNewerVersion -> SameMajorVersion --- CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 163291c58..15d3f2c12 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -137,7 +137,7 @@ configure_package_config_file(cmake/ArborXConfig.cmake.in ) write_basic_package_version_file(${CMAKE_CURRENT_BINARY_DIR}/ArborXConfigVersion.cmake VERSION ${ARBORX_VERSION_STRING} - COMPATIBILITY AnyNewerVersion + COMPATIBILITY SameMajorVersion ) install(FILES ${CMAKE_CURRENT_BINARY_DIR}/ArborXConfig.cmake From e103ab0ee1d28e297dda481a106c580046d96797 Mon Sep 17 00:00:00 2001 From: Andrey Prokopenko Date: Wed, 11 Dec 2024 10:24:34 -0500 Subject: [PATCH 2/7] Bump minimum required CMake version to 3.22 --- CMakeLists.txt | 2 +- examples/CMakeLists.txt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 15d3f2c12..684255482 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -1,4 +1,4 @@ -cmake_minimum_required(VERSION 3.16) +cmake_minimum_required(VERSION 3.22) project(ArborX CXX) if(CMAKE_VERSION VERSION_GREATER_EQUAL 3.30.0) diff --git a/examples/CMakeLists.txt b/examples/CMakeLists.txt index 62f77dc5c..725eb19cf 100644 --- a/examples/CMakeLists.txt +++ b/examples/CMakeLists.txt @@ -1,5 +1,5 @@ if(CMAKE_SOURCE_DIR STREQUAL CMAKE_CURRENT_SOURCE_DIR) - cmake_minimum_required(VERSION 3.16) + cmake_minimum_required(VERSION 3.22) project(ArborXExamples CXX) find_package(ArborX 2.0 REQUIRED) enable_testing() From 52956f67376d838dc9b0d97ac1a25218d2d9ee9a Mon Sep 17 00:00:00 2001 From: Andrey Prokopenko Date: Wed, 11 Dec 2024 10:44:07 -0500 Subject: [PATCH 3/7] Update CMake in docker images --- docker/Dockerfile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/docker/Dockerfile b/docker/Dockerfile index 63f17494f..7ea67c97a 100644 --- a/docker/Dockerfile +++ b/docker/Dockerfile @@ -35,7 +35,7 @@ RUN KEYDUMP_URL=https://cloud1.cees.ornl.gov/download && \ # Install CMake ENV CMAKE_DIR=/opt/cmake -RUN CMAKE_VERSION=3.16.9 && \ +RUN CMAKE_VERSION=3.22.6 && \ CMAKE_KEY=2D2CEF1034921684 && \ CMAKE_URL=https://github.com/Kitware/CMake/releases/download/v${CMAKE_VERSION} && \ CMAKE_SCRIPT=cmake-${CMAKE_VERSION}-Linux-x86_64.sh && \ From 316fff88577a89976f3cb2bb0557cec716fdbe43 Mon Sep 17 00:00:00 2001 From: Andrey Prokopenko Date: Wed, 11 Dec 2024 13:51:08 -0500 Subject: [PATCH 4/7] Specify Kokkos cmake build type for Windows workflow --- .github/workflows/windows.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml index f1be50dff..80ab4003c 100644 --- a/.github/workflows/windows.yml +++ b/.github/workflows/windows.yml @@ -32,7 +32,8 @@ jobs: run: | mkdir build cd build - cmake -D CMAKE_INSTALL_PREFIX=C:\kokkos-install \ + cmake -D CMAKE_BUILD_TYPE=Debug \ + -D CMAKE_INSTALL_PREFIX=C:\kokkos-install \ -D Kokkos_ENABLE_THREADS=ON \ .. cmake --build . --target install -- -m From ba2667dce81164ce46d144b74613581602be2570 Mon Sep 17 00:00:00 2001 From: Andrey Prokopenko Date: Wed, 11 Dec 2024 13:53:12 -0500 Subject: [PATCH 5/7] Switch CMP0167 policy to NEW --- CMakeLists.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 684255482..80402e876 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -3,7 +3,7 @@ project(ArborX CXX) if(CMAKE_VERSION VERSION_GREATER_EQUAL 3.30.0) message(STATUS "Setting policy CMP0167 to use FindBoost module") - cmake_policy(SET CMP0167 OLD) + cmake_policy(SET CMP0167 NEW) endif() # use gnu standard install directories From ba11f003f9e729e3a09f025f750b66d7ca27233d Mon Sep 17 00:00:00 2001 From: Andrey Prokopenko Date: Wed, 11 Dec 2024 14:39:21 -0500 Subject: [PATCH 6/7] Try explicitly building Kokkos with Debug --- .github/workflows/windows.yml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml index 80ab4003c..349b0c8ac 100644 --- a/.github/workflows/windows.yml +++ b/.github/workflows/windows.yml @@ -32,11 +32,10 @@ jobs: run: | mkdir build cd build - cmake -D CMAKE_BUILD_TYPE=Debug \ - -D CMAKE_INSTALL_PREFIX=C:\kokkos-install \ + cmake -D CMAKE_INSTALL_PREFIX=C:\kokkos-install \ -D Kokkos_ENABLE_THREADS=ON \ .. - cmake --build . --target install -- -m + cmake --build . --config Debug --target install -- -m - name: Configure ArborX run: | mkdir build From 0dbd7234171a320c3b845932a60f8c8924f35a19 Mon Sep 17 00:00:00 2001 From: Daniel Arndt Date: Wed, 11 Dec 2024 15:54:04 -0500 Subject: [PATCH 7/7] Try building ArborX explicityl in Debug mode in Windows CI --- .github/workflows/windows.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml index 349b0c8ac..76ae0d99a 100644 --- a/.github/workflows/windows.yml +++ b/.github/workflows/windows.yml @@ -40,9 +40,9 @@ jobs: run: | mkdir build cd build - cmake -DCMAKE_BUILD_TYPE=Debug -DCMAKE_CXX_FLAGS="/EHsc /bigobj" -DKokkos_ROOT="C:\kokkos-install" ${{ steps.vcpkg.outputs.vcpkg-cmake-config }} -DARBORX_ENABLE_MPI=OFF -DARBORX_ENABLE_TESTS=ON -DARBORX_ENABLE_EXAMPLES=ON -DARBORX_ENABLE_BENCHMARKS=ON -DARBORX_ENABLE_HEADER_SELF_CONTAINMENT_TESTS=OFF .. + cmake -DCMAKE_CXX_FLAGS="/EHsc /bigobj" -DKokkos_ROOT="C:\kokkos-install" ${{ steps.vcpkg.outputs.vcpkg-cmake-config }} -DARBORX_ENABLE_MPI=OFF -DARBORX_ENABLE_TESTS=ON -DARBORX_ENABLE_EXAMPLES=ON -DARBORX_ENABLE_BENCHMARKS=ON -DARBORX_ENABLE_HEADER_SELF_CONTAINMENT_TESTS=OFF .. - name: Build ArborX shell: bash run: | - cmake --build build --target install -- -m + cmake --build build --config Debug --target install -- -m cd build