Skip to content

Commit 52a6c6b

Browse files
author
Dana Burkart
committed
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: Canonical link: https://commits.webkit.org/152978@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@171247 268f45cc-cd09-0410-ab3c-d52691b4dbfc
1 parent b56595c commit 52a6c6b

8 files changed

Lines changed: 63 additions & 0 deletions

File tree

‎ChangeLog‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,17 @@
1+
2014-07-18 Dana Burkart <dburkart@apple.com>
2+
3+
Add a new 'analyze' target to the makefile. This will make use of a new
4+
'PATH_TO_SCAN_BUILD' which, if set will wrap xcodebuild in the clang
5+
static analyzer.
6+
https://bugs.webkit.org/show_bug.cgi?id=135057
7+
<rdar://problem/10193187>
8+
9+
Reviewed by David Kilzer.
10+
11+
* Makefile:
12+
* Makefile.shared:
13+
* Source/Makefile:
14+
115
2014-07-15 Ryuan Choi <ryuan.choi@samsung.com>
216

317
[CMAKE] ENABLE_ENCRYPTED_MEDIA_V2 should depend on ENABLE_VIDEO

‎Makefile‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,10 @@ release r:
1212
@for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
1313
if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
1414

15+
analyze:
16+
@for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
17+
if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
18+
1519
clean:
1620
@for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
1721
if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done

‎Makefile.shared‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,14 @@ release r deployment dep deploy: force
3636
$(SCRIPTS_PATH)/set-webkit-configuration --release
3737
( $(SET_COLOR_DIAGNOSTICS_ARG); xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
3838

39+
analyze:
40+
$(SCRIPTS_PATH)/set-webkit-configuration --release
41+
ifndef PATH_TO_SCAN_BUILD
42+
( $(SET_COLOR_DIAGNOSTICS_ARG); xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) RUN_CLANG_STATIC_ANALYZER=YES | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
43+
else
44+
( $(SET_COLOR_DIAGNOSTICS_ARG); $(PATH_TO_SCAN_BUILD) xcodebuild $(OTHER_OPTIONS) $(XCODE_OPTIONS) | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
45+
endif
46+
3947
clean:
4048
( $(SET_COLOR_DIAGNOSTICS_ARG); xcodebuild $(OTHER_OPTIONS) -alltargets clean $(XCODE_OPTIONS) | $(OUTPUT_FILTER) && exit $${PIPESTATUS[0]} )
4149

‎Source/Makefile‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,10 @@ release r:
1818
@for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
1919
if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
2020

21+
analyze:
22+
@for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
23+
if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
24+
2125
clean:
2226
@for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
2327
if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done

‎Tools/ChangeLog‎

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,15 @@
1+
2014-07-18 Dana Burkart <dburkart@apple.com>
2+
3+
Add a new 'analyze' target to the makefile. This will make use of a new
4+
'PATH_TO_SCAN_BUILD' which, if set will wrap xcodebuild in the clang
5+
static analyzer.
6+
https://bugs.webkit.org/show_bug.cgi?id=135057
7+
<rdar://problem/10193187>
8+
9+
Reviewed by David Kilzer.
10+
11+
* Makefile:
12+
113
2014-07-18 Filip Pizlo <fpizlo@apple.com>
214

315
Fix EFL+GTK.

‎Tools/Makefile‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@ release r deployment dep deploy:
1616
@for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
1717
if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
1818

19+
analyze:
20+
@for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
21+
if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done
22+
1923
clean:
2024
@for dir in $(MODULES); do ${MAKE} $@ -C $$dir; exit_status=$$?; \
2125
if [ $$exit_status -ne 0 ]; then exit $$exit_status; fi; done

‎WebKitLibraries/ChangeLog‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,17 @@
1+
2014-07-18 Dana Burkart <dburkart@apple.com>
2+
3+
Add a new 'analyze' target to the makefile. This will make use of a new
4+
'PATH_TO_SCAN_BUILD' which, if set will wrap xcodebuild in the clang
5+
static analyzer.
6+
https://bugs.webkit.org/show_bug.cgi?id=135057
7+
<rdar://problem/10193187>
8+
9+
Reviewed by David Kilzer.
10+
11+
* Makefile:
12+
* Makefile.shared:
13+
* Source/Makefile:
14+
115
2014-07-10 Alex Christensen <achristensen@webkit.org>
216

317
Enable same features on all Windows ports.

‎WebKitLibraries/Makefile‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,3 +18,6 @@ release r:
1818
$(SCRIPTS_PATH)/set-webkit-configuration --release
1919
@$(MAKE) libs
2020

21+
analyze:
22+
$(SCRIPTS_PATH)/set-webkit-configuration --release
23+
@$(MAKE) libs

0 commit comments

Comments
 (0)