Differential D2840 Diff 7239 cmake/ClangFormatAll.cmake

Changeset View

Standalone View

View Options

cmake/ClangFormatAll.cmake

Context not available.
10 NAMES "clang-format" 10​ NAMES "clang-format"
11 ) 11​ )
12if(NOT CLANG_FMT_CMD) 12​if(NOT CLANG_FMT_CMD)
13 message(ERROR "clang-format not found!") 13​ message(WARNING "clang-format not found!")
kfunkUnsubmitted
Done

Does that need to be an error?

kfunk: Does that need to be an error?
coliveiraAuthorUnsubmitted
Not Done

Well, WARNING should be enough, since it's just for developers to auto format code.

coliveira: Well, WARNING should be enough, since it's just for developers to auto format code.
14else() 14​else()
15 add_custom_target( ClangFormatAll 15​ add_custom_target( ClangFormatAll
16 COMMAND ${CLANG_FMT_CMD} -style=file -i ${ALL_SOURCE_FILES} 16​ COMMAND ${CLANG_FMT_CMD} -style=file -i ${ALL_SOURCE_FILES}
Context not available.