Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1652)

Delta Between Two Patch Sets: Makefile

Issue 6651064: gyp: generate "everything" and "most" targets instead of "all" (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Left Patch Set: Created 11 years, 5 months ago
Right Patch Set: synced_to_r6115 Created 11 years, 5 months ago
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | gyp/all.gyp » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
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 "make" to build on Unix, and "xcodebuild" to build on Mac. 2 # Uses "make" to build on Unix, and "xcodebuild" to build on Mac.
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 tests 10 # make tests
(...skipping 15 matching lines...) Expand all
26 # gyp-generated projects yourself. 26 # gyp-generated projects yourself.
27 # 27 #
28 # See https://sites.google.com/site/skiadocs/ for complete documentation. 28 # See https://sites.google.com/site/skiadocs/ for complete documentation.
29 29
30 BUILDTYPE ?= Debug 30 BUILDTYPE ?= Debug
31 CWD := $(shell pwd) 31 CWD := $(shell pwd)
32 32
33 # Soon we should be able to get rid of VALID_TARGETS, and just pass control 33 # Soon we should be able to get rid of VALID_TARGETS, and just pass control
34 # to the gyp-generated Makefile for *any* target name. 34 # to the gyp-generated Makefile for *any* target name.
35 # But that will be a bit complicated, so let's keep it for a future CL. 35 # But that will be a bit complicated, so let's keep it for a future CL.
36 # Tracked as https://code.google.com/p/skia/issues/detail?id=947 ('eliminate
37 # need for VALID_TARGETS in toplevel Makefile')
36 VALID_TARGETS := \ 38 VALID_TARGETS := \
37 bench \ 39 bench \
38 debugger \ 40 debugger \
39 everything \ 41 everything \
40 gm \ 42 gm \
41 most \ 43 most \
42 SampleApp \ 44 SampleApp \
45 SkiaAndroidApp \
43 skia_base_libs \ 46 skia_base_libs \
44 tests \ 47 tests \
45 tools 48 tools
46 49
47 # Default target. This must be listed before all other targets. 50 # Default target. This must be listed before all other targets.
48 .PHONY: default 51 .PHONY: default
49 default: most 52 default: most
50 53
51 # As noted in http://code.google.com/p/skia/issues/detail?id=330 , building 54 # As noted in http://code.google.com/p/skia/issues/detail?id=330 , building
52 # multiple targets in parallel was failing. The special .NOTPARALLEL target 55 # multiple targets in parallel was failing. The special .NOTPARALLEL target
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
107 else ifneq (,$(findstring Linux, $(uname))) 110 else ifneq (,$(findstring Linux, $(uname)))
108 $(MAKE) -C out $@ BUILDTYPE=$(BUILDTYPE) 111 $(MAKE) -C out $@ BUILDTYPE=$(BUILDTYPE)
109 else ifneq (,$(findstring Darwin, $(uname))) 112 else ifneq (,$(findstring Darwin, $(uname)))
110 rm -f out/$(BUILDTYPE) || if test -d out/$(BUILDTYPE); then echo "run 'm ake clean' or otherwise delete out/$(BUILDTYPE)"; exit 1; fi 113 rm -f out/$(BUILDTYPE) || if test -d out/$(BUILDTYPE); then echo "run 'm ake clean' or otherwise delete out/$(BUILDTYPE)"; exit 1; fi
111 xcodebuild -project out/gyp/$@.xcodeproj -configuration $(BUILDTYPE) 114 xcodebuild -project out/gyp/$@.xcodeproj -configuration $(BUILDTYPE)
112 ln -s $(CWD)/xcodebuild/$(BUILDTYPE) out/$(BUILDTYPE) 115 ln -s $(CWD)/xcodebuild/$(BUILDTYPE) out/$(BUILDTYPE)
113 else 116 else
114 echo "unknown platform $(uname)" 117 echo "unknown platform $(uname)"
115 exit 1 118 exit 1
116 endif 119 endif
LEFTRIGHT
« no previous file | gyp/all.gyp » ('j') | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b