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

Side by Side Diff: native_client_sdk/src/tools/common.mk

Issue 60343003: [NaCl SDK] Check for valid CONFIG parameter to Makefile. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month 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 | Annotate | Revision Log
« 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 # Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 # 5 #
6 # GNU Make based build file. For details on GNU Make see: 6 # GNU Make based build file. For details on GNU Make see:
7 # http://www.gnu.org/software/make/manual/make.html 7 # http://www.gnu.org/software/make/manual/make.html
8 # 8 #
9 9
10 # 10 #
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 # build the debug configuration by default instead. 105 # build the debug configuration by default instead.
106 # 106 #
107 ifneq (,$(findstring debug,$(MAKECMDGOALS))) 107 ifneq (,$(findstring debug,$(MAKECMDGOALS)))
108 CONFIG ?= Debug 108 CONFIG ?= Debug
109 else 109 else
110 CONFIG ?= Release 110 CONFIG ?= Release
111 endif 111 endif
112 112
113 113
114 # 114 #
115 # Verify we selected a valid configuration for this example.
116 #
117 VALID_CONFIGS ?= Debug Release
118 ifeq (,$(findstring $(CONFIG),$(VALID_CONFIGS)))
119 $(warning Availbile choices are: $(VALID_CONFIGS))
120 $(error Can not use CONFIG=$(CONFIG) on this example.)
121 endif
122
123
124 #
115 # Note for Windows: 125 # Note for Windows:
116 # The GCC and LLVM toolchains (include the version of Make.exe that comes 126 # The GCC and LLVM toolchains (include the version of Make.exe that comes
117 # with the SDK) expect and are capable of dealing with the '/' seperator. 127 # with the SDK) expect and are capable of dealing with the '/' seperator.
118 # For this reason, the tools in the SDK, including Makefiles and build scripts 128 # For this reason, the tools in the SDK, including Makefiles and build scripts
119 # have a preference for POSIX style command-line arguments. 129 # have a preference for POSIX style command-line arguments.
120 # 130 #
121 # Keep in mind however that the shell is responsible for command-line escaping, 131 # Keep in mind however that the shell is responsible for command-line escaping,
122 # globbing, and variable expansion, so those may change based on which shell 132 # globbing, and variable expansion, so those may change based on which shell
123 # is used. For Cygwin shells this can include automatic and incorrect expansion 133 # is used. For Cygwin shells this can include automatic and incorrect expansion
124 # of response files (files starting with '@'). 134 # of response files (files starting with '@').
(...skipping 384 matching lines...) Expand 10 before | Expand all | Expand 10 after
509 # uppercase aliases (for backward compatibility) 519 # uppercase aliases (for backward compatibility)
510 .PHONY: CHECK_FOR_CHROME DEBUG LAUNCH RUN 520 .PHONY: CHECK_FOR_CHROME DEBUG LAUNCH RUN
511 CHECK_FOR_CHROME: check_for_chrome 521 CHECK_FOR_CHROME: check_for_chrome
512 DEBUG: debug 522 DEBUG: debug
513 LAUNCH: run 523 LAUNCH: run
514 RUN: run 524 RUN: run
515 525
516 endif # TOOLCHAIN is valid... 526 endif # TOOLCHAIN is valid...
517 527
518 endif # TOOLCHAIN=all 528 endif # TOOLCHAIN=all
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