diff options
Diffstat (limited to 'cmake/Lint.cmake')
-rw-r--r-- | cmake/Lint.cmake | 75 |
1 files changed, 48 insertions, 27 deletions
diff --git a/cmake/Lint.cmake b/cmake/Lint.cmake index 73b4795..4453fe3 100644 --- a/cmake/Lint.cmake +++ b/cmake/Lint.cmake @@ -4,46 +4,67 @@ # Copyright (C) 2021 Manuel Bentele <development@manuel-bentele.de> # -find_package(ClangFormat REQUIRED) +find_package(ClangFormat) +find_package(CheckPatch) -# add target to trigger all linter targets -add_custom_target(lint) +if(ClangFormat_FOUND OR CheckPatch_FOUND) + # add target to trigger all linter targets + add_custom_target(lint) +endif(ClangFormat_FOUND OR CheckPatch_FOUND) # macro to define lint targets macro(add_linter LINT_NAME LINT_SOURCE_FILES) - add_custom_target(${LINT_NAME} - COMMAND clang-format --Werror --dry-run ${LINT_SOURCE_FILES} ${ARGN} - WORKING_DIRECTORY ${CMAKE_SOURCE_DIR} - DEPENDS ${LINT_SOURCE_FILES} ${ARGN}) - add_dependencies(lint ${LINT_NAME}) + if(ClangFormat_FOUND) + add_custom_target(${LINT_NAME} + COMMAND ${ClangFormat_EXECUTABLE} --Werror --dry-run ${LINT_SOURCE_FILES} ${ARGN} + WORKING_DIRECTORY ${CMAKE_SOURCE_DIR} + DEPENDS ${LINT_SOURCE_FILES} ${ARGN}) + add_dependencies(lint ${LINT_NAME}) + endif(ClangFormat_FOUND) endmacro(add_linter) # macro to define lint targets for kernel source code files -macro(add_kernel_linter LINT_NAME KERNEL_BUILD_DIR LINT_SOURCE_FILES LINT_HEADER_FILES) - add_custom_target(${LINT_NAME} - COMMAND ${KERNEL_BUILD_DIR}/scripts/checkpatch.pl --no-tree -f ${LINT_SOURCE_FILES} ${LINT_HEADER_FILES} - WORKING_DIRECTORY ${CMAKE_SOURCE_DIR} - DEPENDS ${LINT_SOURCE_FILES} ${LINT_HEADER_FILES}) - add_dependencies(lint ${LINT_NAME}) +macro(add_kernel_linter LINT_NAME LINT_IGNORE_OPTIONS LINT_SOURCE_FILES LINT_HEADER_FILES) + if(CheckPatch_FOUND) + set(LINT_IGNORE_ARGS "") + foreach(IGNORE_OPTION ${LINT_IGNORE_OPTIONS}) + list(APPEND LINT_IGNORE_ARGS "--ignore" "${IGNORE_OPTION}") + endforeach(IGNORE_OPTION ${LINT_IGNORE_OPTIONS}) + add_custom_target(${LINT_NAME} + COMMAND ${CheckPatch_EXECUTABLE} --no-tree --max-line-length=120 ${LINT_IGNORE_ARGS} -f ${LINT_SOURCE_FILES} ${LINT_HEADER_FILES} + WORKING_DIRECTORY ${CMAKE_SOURCE_DIR} + DEPENDS ${LINT_SOURCE_FILES} ${LINT_HEADER_FILES}) + add_dependencies(lint ${LINT_NAME}) + endif(CheckPatch_FOUND) endmacro(add_kernel_linter) -# add target to trigger all formatter targets -add_custom_target(lint-fix) +if(ClangFormat_FOUND OR CheckPatch_FOUND) + # add target to trigger all formatter targets + add_custom_target(lint-fix) +endif(ClangFormat_FOUND OR CheckPatch_FOUND) # macro to define formatter targets macro(add_linter_fix LINT_FIX_NAME LINT_FIX_SOURCE_FILES) - add_custom_target(${LINT_FIX_NAME} - COMMAND clang-format --Werror -i ${LINT_FIX_SOURCE_FILES} ${ARGN} - WORKING_DIRECTORY ${CMAKE_SOURCE_DIR} - DEPENDS ${LINT_FIX_SOURCE_FILES} ${ARGN}) - add_dependencies(lint-fix ${LINT_FIX_NAME}) + if(ClangFormat_FOUND) + add_custom_target(${LINT_FIX_NAME} + COMMAND ${ClangFormat_EXECUTABLE} --Werror -i ${LINT_FIX_SOURCE_FILES} ${ARGN} + WORKING_DIRECTORY ${CMAKE_SOURCE_DIR} + DEPENDS ${LINT_FIX_SOURCE_FILES} ${ARGN}) + add_dependencies(lint-fix ${LINT_FIX_NAME}) + endif(ClangFormat_FOUND) endmacro(add_linter_fix) # macro to define formatter targets for kernel source code files -macro(add_kernel_linter_fix LINT_FIX_NAME KERNEL_BUILD_DIR LINT_FIX_SOURCE_FILES LINT_FIX_HEADER_FILES) - add_custom_target(${LINT_FIX_NAME} - COMMAND ${KERNEL_BUILD_DIR}/scripts/checkpatch.pl --no-tree --fix-inplace -f ${LINT_FIX_SOURCE_FILES} ${LINT_FIX_HEADER_FILES} - WORKING_DIRECTORY ${CMAKE_SOURCE_DIR} - DEPENDS ${LINT_FIX_SOURCE_FILES} ${LINT_FIX_HEADER_FILES}) - add_dependencies(lint-fix ${LINT_FIX_NAME}) +macro(add_kernel_linter_fix LINT_FIX_NAME LINT_FIX_IGNORE_OPTIONS LINT_FIX_SOURCE_FILES LINT_FIX_HEADER_FILES) + if(CheckPatch_FOUND) + set(LINT_FIX_IGNORE_ARGS "") + foreach(IGNORE_OPTION ${LINT_FIX_IGNORE_OPTIONS}) + list(APPEND LINT_FIX_IGNORE_ARGS "--ignore" "${IGNORE_OPTION}") + endforeach(IGNORE_OPTION ${LINT_FIX_IGNORE_OPTIONS}) + add_custom_target(${LINT_FIX_NAME} + COMMAND ${CheckPatch_EXECUTABLE} --no-tree --max-line-length=120 ${LINT_FIX_IGNORE_ARGS} --fix-inplace -f ${LINT_FIX_SOURCE_FILES} ${LINT_FIX_HEADER_FILES} + WORKING_DIRECTORY ${CMAKE_SOURCE_DIR} + DEPENDS ${LINT_FIX_SOURCE_FILES} ${LINT_FIX_HEADER_FILES}) + add_dependencies(lint-fix ${LINT_FIX_NAME}) + endif(CheckPatch_FOUND) endmacro(add_kernel_linter_fix) |