Add a new 'analyze' target to the makefile. This will make use of a new

'PATH_TO_SCAN_BUILD' which, if set will wrap xcodebuild in the clang
static analyzer.
https://bugs.webkit.org/show_bug.cgi?id=135057
<rdar://problem/10193187>

Reviewed by David Kilzer.

* Makefile:
* Makefile.shared:
* Source/Makefile:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@171247 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/ChangeLog b/ChangeLog
index 4b6e06a..11b7984 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2014-07-18  Dana Burkart  <dburkart@apple.com>
+
+        Add a new 'analyze' target to the makefile. This will make use of a new
+        'PATH_TO_SCAN_BUILD' which, if set will wrap xcodebuild in the clang
+        static analyzer.
+        https://bugs.webkit.org/show_bug.cgi?id=135057
+        <rdar://problem/10193187>
+
+        Reviewed by David Kilzer.
+
+        * Makefile:
+        * Makefile.shared:
+        * Source/Makefile:
+
 2014-07-15  Ryuan Choi  <ryuan.choi@samsung.com>
 
         [CMAKE] ENABLE_ENCRYPTED_MEDIA_V2 should depend on ENABLE_VIDEO

diff --git a/Makefile b/Makefile
index 3263364..1c57470 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -12,6 +12,10 @@ release r:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
 
+analyze:
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+
 clean:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done

diff --git a/Makefile.shared b/Makefile.shared
index 2d9a2de..ef5c0a5 100644 (file)
--- a/Makefile.shared
+++ b/Makefile.shared
@@ -36,6 +36,14 @@ release r deployment dep deploy: force
        $(SCRIPTS_PATH)/set-webkit-configuration --release
        ( $(SET_COLOR_DIAGNOSTICS_ARG); xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
 
+analyze:
+       $(SCRIPTS_PATH)/set-webkit-configuration --release
+ifndef PATH_TO_SCAN_BUILD
+       ( $(SET_COLOR_DIAGNOSTICS_ARG); xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) RUN_CLANG_STATIC_ANALYZER=YES | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
+else
+       ( $(SET_COLOR_DIAGNOSTICS_ARG); $(PATH_TO_SCAN_BUILD) xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
+endif
+
 clean:
        ( $(SET_COLOR_DIAGNOSTICS_ARG); xcodebuild $(OTHER_OPTIONS) -alltargets clean $(XCODE_OPTIONS) | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
 

diff --git a/Source/Makefile b/Source/Makefile
index d1d1a4b..b8b0bbe 100644 (file)
--- a/Source/Makefile
+++ b/Source/Makefile
@@ -18,6 +18,10 @@ release r:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
 
+analyze:
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+
 clean:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done

diff --git a/Tools/ChangeLog b/Tools/ChangeLog
index 4666d2f..698bd3b 100644 (file)
--- a/Tools/ChangeLog
+++ b/Tools/ChangeLog
@@ -1,3 +1,15 @@
+2014-07-18  Dana Burkart  <dburkart@apple.com>
+
+        Add a new 'analyze' target to the makefile. This will make use of a new
+        'PATH_TO_SCAN_BUILD' which, if set will wrap xcodebuild in the clang
+        static analyzer.
+        https://bugs.webkit.org/show_bug.cgi?id=135057
+        <rdar://problem/10193187>
+
+        Reviewed by David Kilzer.
+
+        * Makefile:
+
 2014-07-18  Filip Pizlo  <fpizlo@apple.com>
 
         Fix EFL+GTK.

diff --git a/Tools/Makefile b/Tools/Makefile
index bcf52fb..8d8d833 100644 (file)
--- a/Tools/Makefile
+++ b/Tools/Makefile
@@ -16,6 +16,10 @@ release r deployment dep deploy:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
 
+analyze:
+       @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
+       if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
+
 clean:
        @for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
        if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done

diff --git a/WebKitLibraries/ChangeLog b/WebKitLibraries/ChangeLog
index c038035..2dc8a5c 100644 (file)
--- a/WebKitLibraries/ChangeLog
+++ b/WebKitLibraries/ChangeLog
@@ -1,3 +1,17 @@
+2014-07-18  Dana Burkart  <dburkart@apple.com>
+
+        Add a new 'analyze' target to the makefile. This will make use of a new
+        'PATH_TO_SCAN_BUILD' which, if set will wrap xcodebuild in the clang
+        static analyzer.
+        https://bugs.webkit.org/show_bug.cgi?id=135057
+        <rdar://problem/10193187>
+
+        Reviewed by David Kilzer.
+
+        * Makefile:
+        * Makefile.shared:
+        * Source/Makefile:
+
 2014-07-10  Alex Christensen  <achristensen@webkit.org>
 
         Enable same features on all Windows ports.

diff --git a/WebKitLibraries/Makefile b/WebKitLibraries/Makefile
index e4df374..ed0ffba 100644 (file)
--- a/WebKitLibraries/Makefile
+++ b/WebKitLibraries/Makefile
@@ -18,3 +18,6 @@ release r:
        $(SCRIPTS_PATH)/set-webkit-configuration --release
        @$(MAKE) libs
 
+analyze:
+       $(SCRIPTS_PATH)/set-webkit-configuration --release
+       @$(MAKE) libs
\ No newline at end of file