Skip to content

Commit

Permalink
Merge pull request #1201 from aprokop/arborx_version_followup
Browse files Browse the repository at this point in the history
aprokop authored Jan 13, 2025
2 parents aefa0f5 + 31857fd commit 09c9c1f
Showing 5 changed files with 39 additions and 3 deletions.
8 changes: 6 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -100,8 +100,12 @@ install(EXPORT ArborXTargets
DESTINATION ${CMAKE_INSTALL_DATADIR}/cmake/ArborX
)

set(ARBORX_VERSION "10799")
set(ARBORX_VERSION_STRING "2.0 (dev)")
set(ARBORX_VERSION_MAJOR 1)
set(ARBORX_VERSION_MINOR 7)
set(ARBORX_VERSION_PATCH 99)
math(EXPR ARBORX_VERSION "${ARBORX_VERSION_MAJOR} * 10000 + ${ARBORX_VERSION_MINOR} * 100 + ${ARBORX_VERSION_PATCH}")
set(ARBORX_VERSION_STRING "${ARBORX_VERSION_MAJOR}.${ARBORX_VERSION_MINOR}.${ARBORX_VERSION_PATCH}")
message(STATUS "ArborX version: ${ARBORX_VERSION_STRING}")

# Make sure that the git hash in ArborX_Version.hpp is considered to be always
# out of date, and thus is updated every recompile.
2 changes: 1 addition & 1 deletion examples/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
if(CMAKE_SOURCE_DIR STREQUAL CMAKE_CURRENT_SOURCE_DIR)
cmake_minimum_required(VERSION 3.22)
project(ArborXExamples CXX)
find_package(ArborX 2.0 REQUIRED)
find_package(ArborX 1.7.99 REQUIRED)
enable_testing()
endif()

7 changes: 7 additions & 0 deletions src/ArborX_Config.hpp.in
Original file line number Diff line number Diff line change
@@ -12,7 +12,14 @@
#ifndef ARBORX_CONFIG_HPP
#define ARBORX_CONFIG_HPP

// ARBORX_VERSION % 100 is the patch level
// ARBORX_VERSION / 100 % 100 is the minor version
// ARBORX_VERSION / 10000 is the major version
#cmakedefine ARBORX_VERSION @ARBORX_VERSION@
#cmakedefine ARBORX_VERSION_MAJOR @ARBORX_VERSION_MAJOR@
#cmakedefine ARBORX_VERSION_MINOR @ARBORX_VERSION_MINOR@
#cmakedefine ARBORX_VERSION_PATCH @ARBORX_VERSION_PATCH@

#cmakedefine ARBORX_ENABLE_ROCTHRUST
#cmakedefine ARBORX_ENABLE_ONEDPL
#cmakedefine ARBORX_ENABLE_MPI
1 change: 1 addition & 0 deletions test/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -41,6 +41,7 @@ add_executable(ArborX_Test_CompileOnly.exe
tstCompileOnlyCallbacks.cpp
tstCompileOnlyTypeRequirements.cpp
tstCompileOnlyWeightedEdges.cpp
tstCompileOnlyVersionMacros.cpp
tstCompileOnlyMain.cpp
)
target_link_libraries(ArborX_Test_CompileOnly.exe PRIVATE ArborX)
24 changes: 24 additions & 0 deletions test/tstCompileOnlyVersionMacros.cpp
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
#include <ArborX_Config.hpp>

#ifndef ARBORX_VERSION
static_assert(false, "ARBORX_VERSION macro is not defined!");
#endif

#ifndef ARBORX_VERSION_MAJOR
static_assert(false, "ARBORX_VERSION_MAJOR macro is not defined!");
#endif

#ifndef ARBORX_VERSION_MINOR
static_assert(false, "ARBORX_VERSION_MINOR macro is not defined!");
#endif

#ifndef ARBORX_VERSION_PATCH
static_assert(false, "ARBORX_VERSION_PATCH macro is not defined!");
#endif

static_assert(0 <= ARBORX_VERSION_MAJOR && ARBORX_VERSION_MAJOR <= 99);
static_assert(0 <= ARBORX_VERSION_MINOR && ARBORX_VERSION_MINOR <= 99);
static_assert(0 <= ARBORX_VERSION_PATCH && ARBORX_VERSION_PATCH <= 99);
static_assert(ARBORX_VERSION == ARBORX_VERSION_MAJOR * 10000 +
ARBORX_VERSION_MINOR * 100 +
ARBORX_VERSION_PATCH);

0 comments on commit 09c9c1f

Please sign in to comment.