Make nlohmman_json version a single point of configuration

Co-authored-by: Andre Sailer <andre.philippe.sailer@cern.ch>
key4hep · Oct 23, 2024 · 2b9f971 · 2b9f971
1 parent 5bb65d2
commit 2b9f971
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -14,7 +14,8 @@ find_package(ROOT REQUIRED COMPONENTS RIO Tree Physics)
 
 option(EDM4HEP_WITH_JSON "Whether or not to build EDM4HEP with JSON support (and edm4hep2json)" ON)
 if (EDM4HEP_WITH_JSON)
- find_package(nlohmann_json 3.10.5 REQUIRED)
+ set(EDM4HEP_NLOHMANN_JSON_VERSION 3.10.5)
+ find_package(nlohmann_json ${EDM4HEP_NLOHMANN_JSON_VERSION} REQUIRED)
 endif()
 
 include(cmake/Key4hepConfig.cmake)

diff --git a/cmake/EDM4HEPConfig.cmake.in b/cmake/EDM4HEPConfig.cmake.in
@@ -16,7 +16,7 @@ find_dependency(podio REQUIRED)
 
 set(EDM4HEP_WITH_JSON @EDM4HEP_WITH_JSON@)
 if (EDM4HEP_WITH_JSON)
-  find_dependency(nlohmann_json 3.10.5 REQUIRED)
+  find_dependency(nlohmann_json @EDM4HEP_NLOHMANN_JSON_VERSION@ REQUIRED)
 endif()
 
 # - Include the targets file to create the imported targets that a client can