removing static analysis

Author: snhobbs

.gitignore

@@ -2,3 +2,17 @@
 cmake_install.cmake
 Makefile
 build
+CMakeLists.txt.user
+CMakeCache.txt
+CMakeFiles
+CMakeScripts
+Testing
+Makefile
+cmake_install.cmake
+install_manifest.txt
+compile_commands.json
+CTestTestfile.cmake
+_deps
+CMakeUserPresets.json
+lib
+tests/modbus_basic_tests

.gitmodules

@@ -1,15 +1,9 @@
-[submodule "examples/libmodbus_tests/-f"]
-	path = examples/libmodbus_tests/-f
-	url = git@github.com:stephane/libmodbus.git
 [submodule "examples/libmodbus_tests/libmodbus"]
 	path = examples/libmodbus_tests/libmodbus
 	url = git@github.com:stephane/libmodbus.git
 [submodule "examples/pymodbus_tests/pymodbus"]
 	path = examples/pymodbus_tests/pymodbus
 	url = git@github.com:riptideio/pymodbus.git
-[submodule "tests/cmake/CMakeStaticAnalysis"]
-	path = tests/CMakeStaticAnalysis
-	url = git@github.com:snhobbs/CMakeStaticAnalysis.git
 [submodule "lib/CppUtilities"]
 	path = external/CppUtilities
 	url = git@github.com:snhobbs/CppUtilities

CMakeLists.txt

@@ -1,6 +1,60 @@
 cmake_minimum_required(VERSION 3.15.0)
 
 project(ModbusBasic)
+
+# Enable testing
 enable_testing()
+
+# Set up GoogleTest as a subdirectory. It assumes you have googletest located in external/googletest.
+# This will build GoogleTest inside the build directory as well.
 add_subdirectory(external/googletest)
+
+# Add the tests subdirectory.
+# This assumes that you have a 'tests' directory in your project, where your test code resides.
 add_subdirectory(tests)
+
+# Optional: Set CMake build type (Release, Debug, etc.)
+set(CMAKE_BUILD_TYPE Release)  # You can also set it to Debug, RelWithDebInfo, etc.
+
+# Optional: Define output directories for binaries and libraries (useful for keeping everything in the build folder)
+set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
+set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/lib)
+set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/lib)
+
+# Function to detect changed source files using `git diff`
+function(GetChangedFiles changed_files)
+    # Detect changed files that are tracked in the git repository (only .cpp, .h files)
+    execute_process(
+        COMMAND git diff --name-only HEAD~1 HEAD -- '*.cpp' '*.h'
+        WORKING_DIRECTORY ${CMAKE_SOURCE_DIR}
+        OUTPUT_VARIABLE changed_files
+        OUTPUT_STRIP_TRAILING_WHITESPACE
+    )
+    # Split the output into a list of changed files
+    string(REPLACE "\n" ";" changed_files "${changed_files}")
+    list(REMOVE_ITEM changed_files "")  # Remove empty items
+endfunction()
+
+# Run static analysis only on the changed files
+function(RunStaticAnalysisOnChangedFiles)
+    GetChangedFiles(changed_files)
+
+    if(changed_files)
+        message(STATUS "Running static analysis on the following files: ${changed_files}")
+        # Run tools like cpplint, cppcheck, etc. only on the changed files
+        cpplint("${CMAKE_SOURCE_DIR}" "${changed_files}" cpplint)
+        Cppcheck("${CMAKE_SOURCE_DIR}" "${changed_files}" cppcheck)
+        flint("${CMAKE_SOURCE_DIR}" "${changed_files}" flint)
+        flawfinder("${CMAKE_SOURCE_DIR}" "${changed_files}" flawfinder)
+        cppclean("${CMAKE_SOURCE_DIR}" "${changed_files}" cppclean)
+        clangcheck("${CMAKE_SOURCE_DIR}" "${changed_files}" clang-check)
+    else()
+        message(STATUS "No changes detected, skipping static analysis.")
+    endif()
+endfunction()
+
+# Add a target for static analysis
+add_custom_target(StaticAnalysis
+    COMMAND ${CMAKE_COMMAND} -P ${CMAKE_BINARY_DIR}/run_static_analysis.cmake
+    COMMENT "Running Static Analysis on changed files..."
+)