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

Side by Side Diff: Makefile

Issue 1530233005: Add visualbench as valid target to Makefile (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: tabs = spaces Created 5 years 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 | no next file » | 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 23 matching lines...) Expand all
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 nanobench \ 43 nanobench \
44 visualbench \
44 debugger \ 45 debugger \
45 dm \ 46 dm \
46 everything \ 47 everything \
47 lua_app \ 48 lua_app \
48 lua_pictures \ 49 lua_pictures \
49 most \ 50 most \
50 pathops_unittest \ 51 pathops_unittest \
51 SampleApp \ 52 SampleApp \
52 SampleApp_APK \ 53 SampleApp_APK \
53 skhello \ 54 skhello \
(...skipping 33 matching lines...) Expand 10 before | Expand all | Expand 10 after
87 # Run gyp no matter what. 88 # Run gyp no matter what.
88 .PHONY: gyp 89 .PHONY: gyp
89 gyp: 90 gyp:
90 $(CWD)/gyp_skia --no-parallel -G config=$(BUILDTYPE) 91 $(CWD)/gyp_skia --no-parallel -G config=$(BUILDTYPE)
91 92
92 # For all specific targets: run gyp if necessary, and then pass control to 93 # For all specific targets: run gyp if necessary, and then pass control to
93 # the gyp-generated buildfiles. 94 # the gyp-generated buildfiles.
94 .PHONY: $(VALID_TARGETS) 95 .PHONY: $(VALID_TARGETS)
95 $(VALID_TARGETS):: gyp 96 $(VALID_TARGETS):: gyp
96 ninja -C $(SKIA_OUT)/$(BUILDTYPE) $@ 97 ninja -C $(SKIA_OUT)/$(BUILDTYPE) $@
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698