| OLD | NEW |
| 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 Loading... |
| 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 Loading... |
| 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) $@ |
| OLD | NEW |