update for clang-tidy

Use `clang` compiler instead of `gcc`. Remove ansi color from `run-clang-tidy` result. Signed-off-by: Inho Oh <[email protected]>
nugulinux · Nov 2, 2023 · e0ea508 · e0ea508
1 parent 8ae9848
commit e0ea508
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -5,7 +5,8 @@ RUN apt-get update \
 	&& apt-get update && apt-get install -y --no-install-recommends \
 	    apt-utils \
 	    automake \
-	    clang-format clang-tidy clang-tools \
+	    clang clang-format clang-tidy clang-tools \
+	    colorized-logs \
 	    cppcheck \
 	    debianutils \
 	    debhelper \
@@ -24,7 +25,7 @@ RUN apt-get update \
 	    gstreamer1.0-pulseaudio \
 	    iputils-ping \
 	    jq \
-		rapidjson-dev \
+	    rapidjson-dev \
 	    less \
 	    libnugu-epd-dbg \
 	    libnugu-kwd-dbg \

diff --git a/run_codereview.sh b/run_codereview.sh
@@ -1,4 +1,4 @@
-#!/bin/sh
+#!/bin/bash
 
 mkdir -p temp_build
 cd temp_build
@@ -7,10 +7,10 @@ CLANG_RESULT_FILE="result.log"
 CHECKER_REPORT_FILE="report.json"
 
 echo "step-1. generate compile_commands.json"
-cmake -DCMAKE_EXPORT_COMPILE_COMMANDS=ON ../ > /dev/null
+cmake -DCMAKE_EXPORT_COMPILE_COMMANDS=ON -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ ../ > /dev/null
 
 echo "step-2. run clang-tidy tools"
-run-clang-tidy -checks='-*,clang*,perf*,cert*' > $CLANG_RESULT_FILE
+run-clang-tidy -checks='-*,clang*,perf*,cert*' | tee >(ansi2txt > $CLANG_RESULT_FILE)
 
 echo "step-3. run code checker"
 set -e