]> https://gitweb.dealii.org/ - dealii.git/commitdiff
fix clang tidy script
authorTimo Heister <timo.heister@gmail.com>
Tue, 15 Jan 2019 16:17:49 +0000 (09:17 -0700)
committerTimo Heister <timo.heister@gmail.com>
Tue, 15 Jan 2019 16:17:49 +0000 (09:17 -0700)
- fix additional args ($@)
- do not quote ARGS, otherwise it doesn't work

contrib/utilities/run_clang_tidy.sh

index ebf6f8e5f2babbe735550fc5c90855f3122a1d2c..8741032d7aaf090cd7065454825e4ac5b75ec2df 100755 (executable)
@@ -43,7 +43,7 @@ echo "SRC-DIR=$SRC"
 # disable muparser (to avoid warnings with bundled one)
 # enable MPI (to get MPI warnings)
 # export compile commands (so that run-clang-tidy.py works)
-ARGS="-D DEAL_II_WITH_MUPARSER=OFF -D DEAL_II_WITH_MPI=ON -D CMAKE_EXPORT_COMPILE_COMMANDS=ON @$"
+ARGS="-D DEAL_II_WITH_MUPARSER=OFF -D DEAL_II_WITH_MPI=ON -D CMAKE_EXPORT_COMPILE_COMMANDS=ON $@"
 
 # disable performance-inefficient-string-concatenation because we don't care about "a"+to_string(5)+...
 CHECKS="-*,
@@ -64,7 +64,7 @@ if ! [ -x "$(command -v run-clang-tidy.py)" ] || ! [ -x "$(command -v clang++)"
     exit 2
 fi
 
-CC=clang CXX=clang++ cmake "$ARGS" "$SRC" || (echo "cmake failed!"; false) || exit 2
+CC=clang CXX=clang++ cmake $ARGS "$SRC" || (echo "cmake failed!"; false) || exit 2
 
 cmake --build . --target expand_all_instantiations || (echo "make expand_all_instantiations failed!"; false) || exit 3
 

In the beginning the Universe was created. This has made a lot of people very angry and has been widely regarded as a bad move.

Douglas Adams


Typeset in Trocchi and Trocchi Bold Sans Serif.