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

Side by Side Diff: Makefile

Issue 856103002: Revert "Revert "delete old things!"" (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 11 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 | expectations/gm/Test-Android-GalaxyS4-SGX544-Arm7-Debug/expected-results.json » ('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 26 matching lines...) Expand all
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 debugger \ 44 debugger \
45 dm \ 45 dm \
46 everything \ 46 everything \
47 gm \
48 most \ 47 most \
49 pathops_unittest \ 48 pathops_unittest \
50 pdfviewer \ 49 pdfviewer \
51 SampleApp \ 50 SampleApp \
52 SampleApp_APK \ 51 SampleApp_APK \
53 skhello \ 52 skhello \
54 skia_lib \ 53 skia_lib \
55 skpskgr_test \ 54 skpskgr_test \
56 tools \ 55 tools \
57 skpdiff 56 skpdiff
(...skipping 29 matching lines...) Expand all
87 # Run gyp no matter what. 86 # Run gyp no matter what.
88 .PHONY: gyp 87 .PHONY: gyp
89 gyp: 88 gyp:
90 $(CWD)/gyp_skia 89 $(CWD)/gyp_skia
91 90
92 # For all specific targets: run gyp if necessary, and then pass control to 91 # For all specific targets: run gyp if necessary, and then pass control to
93 # the gyp-generated buildfiles. 92 # the gyp-generated buildfiles.
94 .PHONY: $(VALID_TARGETS) 93 .PHONY: $(VALID_TARGETS)
95 $(VALID_TARGETS):: gyp 94 $(VALID_TARGETS):: gyp
96 ninja -C $(SKIA_OUT)/$(BUILDTYPE) $@ 95 ninja -C $(SKIA_OUT)/$(BUILDTYPE) $@
OLDNEW
« no previous file with comments | « no previous file | expectations/gm/Test-Android-GalaxyS4-SGX544-Arm7-Debug/expected-results.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698