fixed #14922 - do not use absolute paths in CMake INSTALL / added TODOs

Author: firewave

.github/workflows/CI-unixish.yml

@@ -150,6 +150,7 @@ jobs:
       - name: Run CMake install
         run: |
           cmake --build cmake.output --target install
+          # TODO: validate the installed files
 
       - name: Run CMake on ubuntu (no CLI)
         if: matrix.os == 'ubuntu-22.04'

.github/workflows/CI-windows.yml

@@ -69,6 +69,7 @@ jobs:
       - name: Run CMake install
         run: |
           cmake --build build --target install
+          # TODO: validate the installed files
 
   build_cmake_cxxstd:
     strategy:

cli/CMakeLists.txt

@@ -40,13 +40,15 @@ if (BUILD_CLI)
     endif()
 
     install(TARGETS cppcheck
-        RUNTIME DESTINATION ${CMAKE_INSTALL_FULL_BINDIR}
+        RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
         COMPONENT applications)
 
     install(PROGRAMS ${CMAKE_SOURCE_DIR}/htmlreport/cppcheck-htmlreport
-        DESTINATION ${CMAKE_INSTALL_FULL_BINDIR}
+        DESTINATION ${CMAKE_INSTALL_BINDIR}
         COMPONENT applications)
 
+    # TODO: leverage CMAKE_INSTALL_DATAROOTDIR (share)?
+
     install(FILES ${addons_py}
        DESTINATION ${FILESDIR_DEF}/addons
        COMPONENT headers)
@@ -63,4 +65,7 @@ if (BUILD_CLI)
        DESTINATION ${FILESDIR_DEF}/platforms
        COMPONENT headers)
 
+    # TODO: install manpage into CMAKE_INSTALL_MANDIR
+    # TODO: install documentation into CMAKE_INSTALL_DOCDIR?
+
 endif()

cmake/options.cmake

@@ -134,9 +134,11 @@ set(CMAKE_DISABLE_PRECOMPILE_HEADERS Off CACHE BOOL "Disable precompiled headers
 # see https://gitlab.kitware.com/cmake/cmake/-/issues/21219
 set(CMAKE_PCH_PROLOGUE "")
 
+# TODO: do we need to set these?
 set(CMAKE_INCLUDE_DIRS_CONFIGCMAKE ${CMAKE_INSTALL_PREFIX}/include      CACHE PATH "Output directory for headers")
 set(CMAKE_LIB_DIRS_CONFIGCMAKE     ${CMAKE_INSTALL_PREFIX}/lib          CACHE PATH "Output directory for libraries")
 
+# TODO: do we need to set these?
 set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}/bin)
 set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}/lib)
 set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${PROJECT_BINARY_DIR}/lib)

cmake/printInfo.cmake

@@ -8,6 +8,7 @@ message(STATUS "Compiler Version =      ${CMAKE_CXX_COMPILER_VERSION}")
 message(STATUS "Build type =            ${CMAKE_BUILD_TYPE}")
 message(STATUS "CMake C++ Standard =    ${CMAKE_CXX_STANDARD}")
 message(STATUS "CMAKE_INSTALL_PREFIX =  ${CMAKE_INSTALL_PREFIX}")
+message(STATUS "CMAKE_INSTALL_BINDIR =  ${CMAKE_INSTALL_BINDIR}")
 message(STATUS "CMAKE_DISABLE_PRECOMPILE_HEADERS = ${CMAKE_DISABLE_PRECOMPILE_HEADERS}")
 message(STATUS "C++ flags (General) =   ${CMAKE_CXX_FLAGS}")
 message(STATUS "C++ flags (Release) =   ${CMAKE_CXX_FLAGS_RELEASE}")

gui/CMakeLists.txt

@@ -71,8 +71,8 @@ CheckOptions:
         add_dependencies(cppcheck-gui online-help.qhc)
     endif()
 
-    install(TARGETS cppcheck-gui RUNTIME DESTINATION ${CMAKE_INSTALL_FULL_BINDIR} COMPONENT applications)
-    install(FILES ${qms} DESTINATION ${CMAKE_INSTALL_FULL_BINDIR} COMPONENT applications)
+    install(TARGETS cppcheck-gui RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR} COMPONENT applications)
+    install(FILES ${qms} DESTINATION ${CMAKE_INSTALL_BINDIR} COMPONENT applications)
 
     install(FILES cppcheck-gui.desktop DESTINATION ${CMAKE_INSTALL_DATAROOTDIR}/applications)