Move abacus_rename_option before cmake_dependent_option

Author: Growl1234

CMakeLists.txt

@@ -65,6 +65,29 @@ option(ENABLE_GOOGLEBENCH "Enable GOOGLE-benchmark usage" OFF)
 option(ENABLE_RAPIDJSON "Enable rapid-json usage" OFF)
 option(ENABLE_CNPY "Enable cnpy usage" OFF)
 
+# ==============================================================================
+# Rename deprecated options (TODO: Remove this section in the future release)
+#  This must appear before cmake_dependent_options so their cached value is
+#  preserved even when the dependencies are unavailable
+# ==============================================================================
+function(abacus_rename_option old_name new_name)
+  get_property(_old_defined CACHE "${old_name}" PROPERTY TYPE SET)
+  if(NOT _old_defined)
+    return()
+  endif()
+  message(DEPRECATION "${old_name} has been renamed to ${new_name}.")
+  get_property(_type CACHE "${new_name}" PROPERTY TYPE)
+  get_property(_help CACHE "${new_name}" PROPERTY HELPSTRING)
+  set("${new_name}" "${${old_name}}" CACHE "${_type}" "${_help}" FORCE)
+  unset("${old_name}" CACHE)
+endfunction()
+abacus_rename_option(USE_OPENMP ENABLE_OPENMP)
+abacus_rename_option(USE_ABACUS_LIBM ENABLE_ABACUS_LIBM)
+abacus_rename_option(USE_ELPA ENABLE_ELPA)
+abacus_rename_option(INFO MATH_INFO)
+abacus_rename_option(EXX_DEV ENABLE_EXX_DEV)
+# ==============================================================================
+
 # Options requiring MPI and LCAO
 cmake_dependent_option(ENABLE_ELPA "Enable ELPA for LCAO" ON "ENABLE_LCAO;ENABLE_MPI" OFF)
 cmake_dependent_option(ENABLE_LIBRI "Enable LibRI for hybrid functional"
@@ -90,26 +113,8 @@ cmake_dependent_option(ENABLE_NCCL_PARALLEL_DEVICE
                        "Enable NCCL-backed collectives in parallel_device; requires MPI"
                        OFF "USE_CUDA;ENABLE_MPI" OFF)
 
-# ==============================================================================
-# Deprecated options (TODO: Remove this section in the future release)
-# ==============================================================================
-function(abacus_rename_option old_name new_name)
-  get_property(_old_defined CACHE "${old_name}" PROPERTY TYPE SET)
-  if(NOT _old_defined)
-    return()
-  endif()
-  message(DEPRECATION "${old_name} has been renamed to ${new_name}.")
-  get_property(_type CACHE "${new_name}" PROPERTY TYPE)
-  get_property(_help CACHE "${new_name}" PROPERTY HELPSTRING)
-  set("${new_name}" "${${old_name}}" CACHE "${_type}" "${_help}" FORCE)
-  unset("${old_name}" CACHE)
-endfunction()
-abacus_rename_option(USE_OPENMP ENABLE_OPENMP)
-abacus_rename_option(USE_ABACUS_LIBM ENABLE_ABACUS_LIBM)
-abacus_rename_option(USE_ELPA ENABLE_ELPA)
-abacus_rename_option(INFO MATH_INFO)
-abacus_rename_option(EXX_DEV ENABLE_EXX_DEV)
-
+# Deprecated ENABLE_LIBCOMM
+# TODO: Remove this section in the future release
 if(DEFINED CACHE{ENABLE_LIBCOMM})
   message(
     DEPRECATION
@@ -126,7 +131,6 @@ if(DEFINED CACHE{ENABLE_LIBCOMM})
   endif()
   unset(ENABLE_LIBCOMM CACHE)
 endif()
-# ==============================================================================
 
 # CTest defines BUILD_TESTING when it is first included. Include it only after
 # ABACUS has declared its OFF-by-default option above.