Merge pull request cpp-best-practices#131 from cpp-best-practices/work_on_variables_and_naming

Work on variables and naming

Author: lefticus

CPM.cmake

@@ -2,8 +2,8 @@
 #
 # SPDX-FileCopyrightText: Copyright (c) 2019-2023 Lars Melchior and contributors
 
-set(CPM_DOWNLOAD_VERSION 0.40.8)
-set(CPM_HASH_SUM "78ba32abdf798bc616bab7c73aac32a17bbd7b06ad9e26a6add69de8f3ae4791")
+set(CPM_DOWNLOAD_VERSION 0.42.1)
+set(CPM_HASH_SUM "f3a6dcc6a04ce9e7f51a127307fa4f699fb2bade357a8eb4c5b45df76e1dc6a5")
 
 if(CPM_SOURCE_CACHE)
   set(CPM_DOWNLOAD_LOCATION "${CPM_SOURCE_CACHE}/cpm/CPM_${CPM_DOWNLOAD_VERSION}.cmake")

Linker.cmake

@@ -1,31 +1,20 @@
 macro(myproject_configure_linker project_name)
-  include(CheckCXXCompilerFlag)
-
-  set(USER_LINKER_OPTION
-      "lld"
+  set(myproject_USER_LINKER_OPTION
+    "DEFAULT"
       CACHE STRING "Linker to be used")
-  set(USER_LINKER_OPTION_VALUES "lld" "gold" "bfd" "mold")
-  set_property(CACHE USER_LINKER_OPTION PROPERTY STRINGS ${USER_LINKER_OPTION_VALUES})
+    set(myproject_USER_LINKER_OPTION_VALUES "DEFAULT" "SYSTEM" "LLD" "GOLD" "BFD" "MOLD" "SOLD" "APPLE_CLASSIC" "MSVC")
+  set_property(CACHE myproject_USER_LINKER_OPTION PROPERTY STRINGS ${myproject_USER_LINKER_OPTION_VALUES})
   list(
     FIND
-    USER_LINKER_OPTION_VALUES
-    ${USER_LINKER_OPTION}
-    USER_LINKER_OPTION_INDEX)
+    myproject_USER_LINKER_OPTION_VALUES
+    ${myproject_USER_LINKER_OPTION}
+    myproject_USER_LINKER_OPTION_INDEX)
 
-  if(${USER_LINKER_OPTION_INDEX} EQUAL -1)
+  if(${myproject_USER_LINKER_OPTION_INDEX} EQUAL -1)
     message(
       STATUS
-        "Using custom linker: '${USER_LINKER_OPTION}', explicitly supported entries are ${USER_LINKER_OPTION_VALUES}")
-  endif()
-
-  if(NOT myproject_ENABLE_USER_LINKER)
-    return()
+        "Using custom linker: '${myproject_USER_LINKER_OPTION}', explicitly supported entries are ${myproject_USER_LINKER_OPTION_VALUES}")
   endif()
 
-  set(LINKER_FLAG "-fuse-ld=${USER_LINKER_OPTION}")
-
-  check_cxx_compiler_flag(${LINKER_FLAG} CXX_SUPPORTS_USER_LINKER)
-  if(CXX_SUPPORTS_USER_LINKER)
-    target_compile_options(${project_name} INTERFACE ${LINKER_FLAG})
-  endif()
+  set_target_properties(${project_name} PROPERTIES LINKER_TYPE "${myproject_USER_LINKER_OPTION}")
 endmacro()

StandardProjectSettings.cmake

@@ -19,20 +19,10 @@ set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
 
 # Enhance error reporting and compiler messages
 if(CMAKE_CXX_COMPILER_ID MATCHES ".*Clang")
-  if(WIN32)
-    # On Windows cuda nvcc uses cl and not clang
     add_compile_options($<$<COMPILE_LANGUAGE:C>:-fcolor-diagnostics> $<$<COMPILE_LANGUAGE:CXX>:-fcolor-diagnostics>)
-  else()
-    add_compile_options(-fcolor-diagnostics)
-  endif()
 elseif(CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
-  if(WIN32)
-    # On Windows cuda nvcc uses cl and not gcc
     add_compile_options($<$<COMPILE_LANGUAGE:C>:-fdiagnostics-color=always>
                         $<$<COMPILE_LANGUAGE:CXX>:-fdiagnostics-color=always>)
-  else()
-    add_compile_options(-fdiagnostics-color=always)
-  endif()
 elseif(CMAKE_CXX_COMPILER_ID STREQUAL "MSVC" AND MSVC_VERSION GREATER 1900)
   add_compile_options(/diagnostics:column)
 else()
@@ -42,4 +32,4 @@ endif()
 
 # run vcvarsall when msvc is used
 include("${CMAKE_CURRENT_LIST_DIR}/VCEnvironment.cmake")
-run_vcvarsall()
+myproject_run_vcvarsall()

StaticAnalyzers.cmake

@@ -47,7 +47,7 @@ macro(myproject_enable_cppcheck WARNINGS_AS_ERRORS CPPCHECK_OPTIONS)
       list(APPEND CMAKE_CXX_CPPCHECK --error-exitcode=2)
     endif()
   else()
-    message(${WARNING_MESSAGE} "cppcheck requested but executable not found")
+    message(WARNING "cppcheck requested but executable not found")
   endif()
 endmacro()
 
@@ -100,7 +100,7 @@ macro(myproject_enable_clang_tidy target WARNINGS_AS_ERRORS)
     message("Also setting clang-tidy globally")
     set(CMAKE_CXX_CLANG_TIDY ${CLANG_TIDY_OPTIONS})
   else()
-    message(${WARNING_MESSAGE} "clang-tidy requested but executable not found")
+    message(WARNING "clang-tidy requested but executable not found")
   endif()
 endmacro()
 
@@ -109,6 +109,6 @@ macro(myproject_enable_include_what_you_use)
   if(INCLUDE_WHAT_YOU_USE)
     set(CMAKE_CXX_INCLUDE_WHAT_YOU_USE ${INCLUDE_WHAT_YOU_USE})
   else()
-    message(${WARNING_MESSAGE} "include-what-you-use requested but executable not found")
+    message(WARNING "include-what-you-use requested but executable not found")
   endif()
 endmacro()

Tests.cmake

@@ -1,6 +1,6 @@
 function(myproject_enable_coverage project_name)
   if(CMAKE_CXX_COMPILER_ID STREQUAL "GNU" OR CMAKE_CXX_COMPILER_ID MATCHES ".*Clang")
-    target_compile_options(${project_name} INTERFACE --coverage -O0 -g)
+    target_compile_options(${project_name} INTERFACE --coverage -g)
     target_link_libraries(${project_name} INTERFACE --coverage)
   endif()
 endfunction()

VCEnvironment.cmake

@@ -1,6 +1,6 @@
 include("${CMAKE_CURRENT_LIST_DIR}/Utilities.cmake")
 
-macro(detect_architecture)
+macro(myproject_detect_architecture)
   # detect the architecture
   string(TOLOWER "${CMAKE_SYSTEM_PROCESSOR}" CMAKE_SYSTEM_PROCESSOR_LOWER)
   if(CMAKE_SYSTEM_PROCESSOR_LOWER STREQUAL x86 OR CMAKE_SYSTEM_PROCESSOR_LOWER MATCHES "^i[3456]86$")
@@ -25,7 +25,7 @@ macro(detect_architecture)
 endmacro()
 
 # Run vcvarsall.bat and set CMake environment variables
-function(run_vcvarsall)
+function(myproject_run_vcvarsall)
   # if MSVC but VSCMD_VER is not set, which means vcvarsall has not run
   if(MSVC AND "$ENV{VSCMD_VER}" STREQUAL "")
 
@@ -43,7 +43,7 @@ function(run_vcvarsall)
 
     if(EXISTS ${VCVARSALL_FILE})
       # detect the architecture
-      detect_architecture()
+      myproject_detect_architecture()
 
       # run vcvarsall and print the environment variables
       message(STATUS "Running `${VCVARSALL_FILE} ${VCVARSALL_ARCH}` to set up the MSVC environment")