Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(164)

Side by Side Diff: Makefile

Issue 517133002: Update LSAN suppressions. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: space Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | tools/lsan.supp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Makefile that wraps the Gyp and build steps for Unix and Mac (but not Windows) 1 # Makefile that wraps the Gyp and build steps for Unix and Mac (but not Windows)
2 # Uses "ninja" to build the code. 2 # Uses "ninja" to build the code.
3 # 3 #
4 # Some usage examples (tested on both Linux and Mac): 4 # Some usage examples (tested on both Linux and Mac):
5 # 5 #
6 # # Clean everything 6 # # Clean everything
7 # make clean 7 # make clean
8 # 8 #
9 # # Build and run tests (in Debug mode) 9 # # Build and run tests (in Debug mode)
10 # make dm 10 # make dm
(...skipping 22 matching lines...) Expand all
33 33
34 # Soon we should be able to get rid of VALID_TARGETS, and just pass control 34 # Soon we should be able to get rid of VALID_TARGETS, and just pass control
35 # to the gyp-generated Makefile for *any* target name. 35 # to the gyp-generated Makefile for *any* target name.
36 # But that will be a bit complicated, so let's keep it for a future CL. 36 # But that will be a bit complicated, so let's keep it for a future CL.
37 # Tracked as https://code.google.com/p/skia/issues/detail?id=947 ('eliminate 37 # Tracked as https://code.google.com/p/skia/issues/detail?id=947 ('eliminate
38 # need for VALID_TARGETS in toplevel Makefile') 38 # need for VALID_TARGETS in toplevel Makefile')
39 # 39 #
40 # TODO(epoger): I'm not sure if the above comment is still valid in a ninja 40 # TODO(epoger): I'm not sure if the above comment is still valid in a ninja
41 # world. 41 # world.
42 VALID_TARGETS := \ 42 VALID_TARGETS := \
43 bench \ 43 nanobench \
44 debugger \ 44 debugger \
45 dm \ 45 dm \
46 everything \ 46 everything \
47 gm \ 47 gm \
48 most \ 48 most \
49 pathops_unittest \ 49 pathops_unittest \
50 pdfviewer \ 50 pdfviewer \
51 SampleApp \ 51 SampleApp \
52 SampleApp_APK \ 52 SampleApp_APK \
53 skhello \ 53 skhello \
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 # Run gyp no matter what. 87 # Run gyp no matter what.
88 .PHONY: gyp 88 .PHONY: gyp
89 gyp: 89 gyp:
90 $(CWD)/gyp_skia 90 $(CWD)/gyp_skia
91 91
92 # For all specific targets: run gyp if necessary, and then pass control to 92 # For all specific targets: run gyp if necessary, and then pass control to
93 # the gyp-generated buildfiles. 93 # the gyp-generated buildfiles.
94 .PHONY: $(VALID_TARGETS) 94 .PHONY: $(VALID_TARGETS)
95 $(VALID_TARGETS):: gyp 95 $(VALID_TARGETS):: gyp
96 ninja -C $(SKIA_OUT)/$(BUILDTYPE) $@ 96 ninja -C $(SKIA_OUT)/$(BUILDTYPE) $@
OLDNEW
« no previous file with comments | « no previous file | tools/lsan.supp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698