From 1de8ca932eb3d52ee08bbb9ce45def246e1dd8df Mon Sep 17 00:00:00 2001
From: Simon Praetorius <simon.praetorius@tu-dresden.de>
Date: Fri, 20 Sep 2024 17:37:33 +0200
Subject: [PATCH] Update set_package_properties for ARPack and SuperLU

---
 cmake/modules/AddARPACKPPFlags.cmake |  7 +++++++
 cmake/modules/AddSuperLUFlags.cmake  |  4 ++++
 cmake/modules/FindARPACK.cmake       | 10 +++++-----
 cmake/modules/FindARPACKPP.cmake     | 10 +++++-----
 cmake/modules/FindSuperLU.cmake      |  4 ++--
 5 files changed, 23 insertions(+), 12 deletions(-)

diff --git a/cmake/modules/AddARPACKPPFlags.cmake b/cmake/modules/AddARPACKPPFlags.cmake
index 12de582f..fa0d42cc 100644
--- a/cmake/modules/AddARPACKPPFlags.cmake
+++ b/cmake/modules/AddARPACKPPFlags.cmake
@@ -12,6 +12,13 @@
 #
 #       A list of targets to use ARPACKPP with.
 #
+include_guard(GLOBAL)
+
+set_package_properties("ARPACK" PROPERTIES
+  PURPOSE "Solve large scale eigenvalue problems")
+
+set_package_properties("ARPACKPP" PROPERTIES
+  PURPOSE "C++ interface for ARPACK")
 
 function(add_dune_arpackpp_flags _targets)
   if(ARPACKPP_FOUND)
diff --git a/cmake/modules/AddSuperLUFlags.cmake b/cmake/modules/AddSuperLUFlags.cmake
index fd56795e..b4cc7d2e 100644
--- a/cmake/modules/AddSuperLUFlags.cmake
+++ b/cmake/modules/AddSuperLUFlags.cmake
@@ -12,6 +12,10 @@
 #
 #       A list of targets to use SuperLU with.
 #
+include_guard(GLOBAL)
+
+set_package_properties("SuperLU" PROPERTIES
+  PURPOSE "Direct solver for linear system, based on LU decomposition")
 
 # set HAVE_SUPERLU for config.h
 set(HAVE_SUPERLU ${SuperLU_FOUND})
diff --git a/cmake/modules/FindARPACK.cmake b/cmake/modules/FindARPACK.cmake
index 5986749a..ca3d3645 100644
--- a/cmake/modules/FindARPACK.cmake
+++ b/cmake/modules/FindARPACK.cmake
@@ -25,6 +25,11 @@
 #    system paths.
 #
 
+# text for feature summary
+set_package_properties("ARPACK" PROPERTIES
+  URL "https://www.arpack.org"
+  DESCRIPTION "ARnoldi PACKage")
+
 # look for library, only at positions given by the user
 find_library(ARPACK_LIBRARY
   NAMES "arpack"
@@ -81,8 +86,3 @@ else()
     "Determining location of ARPACK failed:\n"
     "Libraries to link against: ${ARPACK_LIBRARIES}\n\n")
 endif()
-
-# text for feature summary
-set_package_properties("ARPACK" PROPERTIES
-  DESCRIPTION "ARnoldi PACKage"
-  PURPOSE "Solve large scale eigenvalue problems")
diff --git a/cmake/modules/FindARPACKPP.cmake b/cmake/modules/FindARPACKPP.cmake
index 3be07ff4..36516e84 100644
--- a/cmake/modules/FindARPACKPP.cmake
+++ b/cmake/modules/FindARPACKPP.cmake
@@ -28,6 +28,11 @@
 #    system paths.
 #
 
+# text for feature summary
+set_package_properties("ARPACKPP" PROPERTIES
+  URL "https://github.com/m-reuter/arpackpp"
+  DESCRIPTION "ARPACK++")
+
 # find ARPACK which is required by ARPACK++
 find_package(ARPACK)
 
@@ -132,8 +137,3 @@ if(ARPACKPP_FOUND)
                               LIBRARIES "${ARPACKPP_LIBRARIES}"
                               COMPILE_OPTIONS "${ARPACKPP_DUNE_COMPILE_FLAGS}")
 endif()
-
-# text for feature summary
-set_package_properties("ARPACKPP" PROPERTIES
-  DESCRIPTION "ARPACK++"
-  PURPOSE "C++ interface for ARPACK")
diff --git a/cmake/modules/FindSuperLU.cmake b/cmake/modules/FindSuperLU.cmake
index 50984293..8e5f5345 100644
--- a/cmake/modules/FindSuperLU.cmake
+++ b/cmake/modules/FindSuperLU.cmake
@@ -25,8 +25,8 @@
 # text for feature summary
 include(FeatureSummary)
 set_package_properties("SuperLU" PROPERTIES
-  DESCRIPTION "Supernodal LU"
-  PURPOSE "Direct solver for linear system, based on LU decomposition")
+  URL "https://portal.nersc.gov/project/sparse/superlu/"
+  DESCRIPTION "Supernodal LU")
 
 set(SUPERLU_INT_TYPE "int" CACHE STRING
   "The integer version that SuperLU was compiled for (Default is int.