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

Side by Side Diff: build/config/BUILDCONFIG.gn

Issue 1312363011: Add a flag to BUILDCONFIG for updating some global configs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 3 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 | 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) 2013 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2013 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 # PLATFORM SELECTION 6 # PLATFORM SELECTION
7 # ============================================================================= 7 # =============================================================================
8 # 8 #
9 # There are two main things to set: "os" and "cpu". The "toolchain" is the name 9 # There are two main things to set: "os" and "cpu". The "toolchain" is the name
10 # of the GN thing that encodes combinations of these things. 10 # of the GN thing that encodes combinations of these things.
(...skipping 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
148 if (current_os == "chromeos") { 148 if (current_os == "chromeos") {
149 # Allows the target toolchain to be injected as arguments. This is needed 149 # Allows the target toolchain to be injected as arguments. This is needed
150 # to support the CrOS build system which supports per-build-configuration 150 # to support the CrOS build system which supports per-build-configuration
151 # toolchains. 151 # toolchains.
152 cros_use_custom_toolchain = false 152 cros_use_custom_toolchain = false
153 } 153 }
154 154
155 # DON'T ADD MORE FLAGS HERE. Read the comment above. 155 # DON'T ADD MORE FLAGS HERE. Read the comment above.
156 } 156 }
157 157
158 # This flag indicates if the build is using the new optimization and symbol
159 # level configs, or the old ones.
160 #
161 # This is part of a multi-repo landing. When all required deps (WebKit and
162 # ffmpeg) have been updated to handle this flag, we can change the optimization
163 # and symbol level configs and set this flag to true. Then delete the users of
164 # the flag, wait for all deps rolls, then remove the flag.
165 #
166 # In the old scheme, this file sets the appropriate optimization or symbol
167 # level config on targets depending on what the current build setup is. This
168 # requires that targets know what the default is, and if they want to change it,
169 # they have to figure out which one to remove depending on the state of the
170 # current build. It also means that symbol_level and the sanitizer flags need
171 # to be global.
172 #
173 # In the new scheme, this file sets a "default_symbols" and
174 # "default_optimization" configs, which then expands to the right thing
175 # depending on the current build. This means less information has to be global,
176 # and there's only one config to remove that's the same in all cases for
177 # targets that want to override it. It also means that the associated flags can
178 # be non-global.
179 using_new_global_compiler_configs = false
180
158 # ============================================================================= 181 # =============================================================================
159 # OS DEFINITIONS 182 # OS DEFINITIONS
160 # ============================================================================= 183 # =============================================================================
161 # 184 #
162 # We set these various is_FOO booleans for convenience in writing OS-based 185 # We set these various is_FOO booleans for convenience in writing OS-based
163 # conditions. 186 # conditions.
164 # 187 #
165 # - is_android, is_chromeos, is_ios, and is_win should be obvious. 188 # - is_android, is_chromeos, is_ios, and is_win should be obvious.
166 # - is_mac is set only for desktop Mac. It is not set on iOS. 189 # - is_mac is set only for desktop Mac. It is not set on iOS.
167 # - is_posix is true for mac and any Unix-like system (basically everything 190 # - is_posix is true for mac and any Unix-like system (basically everything
(...skipping 450 matching lines...) Expand 10 before | Expand all | Expand 10 after
618 forward_variables_from(invoker, "*") 641 forward_variables_from(invoker, "*")
619 642
620 # All shared libraries must have the sanitizer deps to properly link in 643 # All shared libraries must have the sanitizer deps to properly link in
621 # asan mode (this target will be empty in other cases). 644 # asan mode (this target will be empty in other cases).
622 if (!defined(deps)) { 645 if (!defined(deps)) {
623 deps = [] 646 deps = []
624 } 647 }
625 deps += [ "//build/config/sanitizers:deps" ] 648 deps += [ "//build/config/sanitizers:deps" ]
626 } 649 }
627 } 650 }
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