add clang-tidy support

mapbox · Dec 8, 2015 · b437804 · b437804
1 parent 8d721f6
commit b437804
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 0 deletions.
diff --git a/Makefile b/Makefile
@@ -1,5 +1,16 @@
 BUILDTYPE ?= Release
 
+ifeq ($(shell uname -s), Darwin)
+export BUILD = osx
+else ifeq ($(shell uname -s), Linux)
+export BUILD = linux
+else
+$(error Cannot determine build platform)
+endif
+
+
+CLANG_TIDY ?= clang-tidy
+
 all: lib
 
 config.gypi:
@@ -34,6 +45,12 @@ xproj: config.gypi
 	deps/run_gyp geojsonvt.gyp -Iconfig.gypi --depth=. -Goutput_dir=. --generator-output=./build -f xcode
 	open build/geojsonvt.xcodeproj
 
+.PHONY: tidy
+tidy: config.gypi
+	deps/run_gyp geojsonvt.gyp -Iconfig.gypi --depth=. -Goutput_dir=. -Gconfig=$(BUILDTYPE) --generator-output=./build -f ninja
+	deps/ninja/ninja-$(BUILD) -C build/$(BUILDTYPE) -t compdb cc cc_s cxx objc objcxx > build/$(BUILDTYPE)/compile_commands.json
+	(cd build/$(BUILDTYPE) && git ls-files '../../*.cpp' | xargs -I{} $(CLANG_TIDY) -checks=* {} )
+
 .PHONY: clean
 clean:
 	-rm -rf build

diff --git a/deps/ninja/ninja-linux b/deps/ninja/ninja-linux
diff --git a/deps/ninja/ninja-osx b/deps/ninja/ninja-osx