Modernize the Config.cmake export

Signed-off-by: Cristian Le <[email protected]>

Author: LecrisUT

CMAKE/scalapack-config-build.cmake.in

@@ -0,0 +1,21 @@
+cmake_minimum_required(VERSION 3.17...4.0)
+
+@PACKAGE_INIT@
+
+include(${CMAKE_CURRENT_LIST_DIR}/scalapackTargets.cmake)
+
+# Deprecated compatibility shims
+if(TARGET scalapack::scalapack)
+    add_library(scalapack INTERFACE IMPORTED)
+    target_link_libraries(scalapack INTERFACE scalapack::scalapack)
+    set_target_properties(scalapack PROPERTIES
+        DEPRECATION "Target scalapack is deprecated, use scalapack::scalapack instead."
+    )
+endif()
+if(TARGET scalapack::scalapack-F)
+    add_library(scalapack-F INTERFACE IMPORTED)
+    target_link_libraries(scalapack-F INTERFACE scalapack::scalapack-F)
+    set_target_properties(scalapack-F PROPERTIES
+        DEPRECATION "Target scalapack-F is deprecated, use scalapack::scalapack-F instead."
+    )
+endif()

CMAKE/scalapack-config-install.cmake.in

@@ -72,10 +72,11 @@ if (UNIX)
    endif ()
 endif ()
 
+# TODO: Remove this macro
 macro(SCALAPACK_install_library lib)
-  install(TARGETS ${lib} EXPORT scalapack-targets
-    RUNTIME DESTINATION Testing
-  )
+install(TARGETS ${lib}
+    EXPORT scalapackTargets
+)
 endmacro()
 
 # --------------------------------------------------
@@ -227,6 +228,7 @@ append_subdir_files(src-C "SRC")
 if (NOT MSVC)
    set_source_files_properties(TOOLS/SL_gridreshape.c PROPERTIES COMPILE_OPTIONS "-Wno-error=implicit-function-declaration")
    add_library(scalapack ${blacs} ${tools} ${tools-C} ${extra_lapack} ${pblas} ${pblas-F} ${ptzblas} ${ptools} ${pbblas} ${redist} ${src} ${src-C})
+   add_library(scalapack::scalapack ALIAS scalapack)
    set_target_properties(scalapack PROPERTIES
     VERSION ${SCALAPACK_VERSION}
     SOVERSION ${SCALAPACK_VERSION_MAJOR}.${SCALAPACK_VERSION_MINOR})
@@ -235,11 +237,13 @@ if (NOT MSVC)
 else () # Need to separate Fortran and C Code
    OPTION(BUILD_SHARED_LIBS "Build shared libraries" ON )
    add_library(scalapack ${blacs} ${tools-C} ${pblas} ${ptools} ${redist} ${src-C})
+   add_library(scalapack::scalapack ALIAS scalapack)
    set_target_properties(scalapack PROPERTIES
     VERSION ${SCALAPACK_VERSION}
     SOVERSION ${SCALAPACK_VERSION_MAJOR}.${SCALAPACK_VERSION_MINOR})
    target_link_libraries( scalapack ${LAPACK_LIBRARIES} ${BLAS_LIBRARIES} MPI::MPI_C)
    add_library(scalapack-F ${pblas-F} ${pbblas} ${ptzblas} ${tools} ${src} ${extra_lapack} )
+   add_library(scalapack::scalapack-F ALIAS scalapack-F)
    set_target_properties(scalapack-F PROPERTIES
     VERSION ${SCALAPACK_VERSION}
     SOVERSION ${SCALAPACK_VERSION_MAJOR}.${SCALAPACK_VERSION_MINOR})
@@ -289,33 +293,36 @@ INCLUDE(CPack)
 # --------------------------------------------------
 
 
-export(TARGETS scalapack FILE scalapack-targets.cmake)
-
 if( NOT LAPACK_FOUND )
  install(FILES
   ${BLAS_LIBRARIES} ${LAPACK_LIBRARIES}
   DESTINATION ${CMAKE_INSTALL_LIBDIR}
    )
 endif( NOT LAPACK_FOUND )
 
-configure_file(${SCALAPACK_SOURCE_DIR}/CMAKE/scalapack-config-version.cmake.in
-  ${SCALAPACK_BINARY_DIR}/scalapack-config-version.cmake @ONLY)
-configure_file(${SCALAPACK_SOURCE_DIR}/CMAKE/scalapack-config-build.cmake.in
-  ${SCALAPACK_BINARY_DIR}/scalapack-config.cmake @ONLY)
-
-configure_file(${CMAKE_CURRENT_SOURCE_DIR}/scalapack.pc.in ${CMAKE_CURRENT_BINARY_DIR}/scalapack.pc)
- install(FILES
-  ${CMAKE_CURRENT_BINARY_DIR}/scalapack.pc
-  DESTINATION ${PKG_CONFIG_DIR}
-   )
-
-configure_file(${SCALAPACK_SOURCE_DIR}/CMAKE/scalapack-config-install.cmake.in
-  ${SCALAPACK_BINARY_DIR}/CMakeFiles/scalapack-config.cmake @ONLY)
+include(CMakePackageConfigHelpers)
+configure_package_config_file(
+    CMAKE/scalapackConfig.cmake.in
+    scalapackConfig.cmake
+    INSTALL_DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/scalapack
+)
+write_basic_package_version_file(
+    scalapackConfigVersion.cmake
+    COMPATIBILITY SameMajorVersion
+)
+
+install(EXPORT scalapackTargets
+    DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/scalapack
+    NAMESPACE scalapack
+)
+
+configure_file(scalapack.pc.in scalapack.pc @ONLY)
 install(FILES
-  ${SCALAPACK_BINARY_DIR}/CMakeFiles/scalapack-config.cmake
-  ${SCALAPACK_BINARY_DIR}/scalapack-config-version.cmake
-  DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/scalapack-${SCALAPACK_VERSION}
-  )
-
-install(EXPORT scalapack-targets
-  DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/scalapack-${SCALAPACK_VERSION})
+    ${CMAKE_CURRENT_BINARY_DIR}/scalapack.pc
+    DESTINATION ${PKG_CONFIG_DIR}
+)
+install(FILES
+    ${CMAKE_CURRENT_BINARY_DIR}/scalapackConfig.cmake
+    ${CMAKE_CURRENT_BINARY_DIR}/scalapackConfigVersion.cmake
+    DESTINATION ${CMAKE_INSTALL_LIBDIR}/cmake/scalapack
+)

CMAKE/scalapack-config-version.cmake.in

@@ -1,16 +1,14 @@
 cmake_minimum_required(VERSION 3.26...4.0)
 
-## Old test-suite to be modernized
-# include only if this is included via `add_subdirectory`
-
-if(DEFINED SCALAPACK_SOURCE_DIR)
-    add_subdirectory(traditional)
-endif()
-
-## Modern test-suite is an independent project
-
-# TODO: Add the implementations
 project(scalapack-tests
     LANGUAGES C Fortran
 )
+
+if(NOT TARGET scalapack::scalapack)
+    find_package(scalapack REQUIRED CONFIG)
+endif()
+
 enable_testing()
+
+# Old test-suite
+add_subdirectory(traditional)

CMAKE/scalapackConfig.cmake.in

@@ -40,7 +40,7 @@ set(genex_gfortran_10
 function(setup_test_target)
     foreach(target IN LISTS ARGN)
         target_link_libraries(${target} PRIVATE
-            scalapack
+            scalapack::scalapack
             ${LAPACK_LIBRARIES}
             ${BLAS_LIBRARIES}
             MPI::MPI_Fortran