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

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

Issue 1433033002: simplify the is_desktop_linux default value expression (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
« 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 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
111 # even if the value is overridden, which is wasteful. See first bullet. 111 # even if the value is overridden, which is wasteful. See first bullet.
112 112
113 declare_args() { 113 declare_args() {
114 # Component build. 114 # Component build.
115 is_component_build = false 115 is_component_build = false
116 116
117 # Debug build. 117 # Debug build.
118 is_debug = true 118 is_debug = true
119 119
120 # Whether we're a traditional desktop unix. 120 # Whether we're a traditional desktop unix.
121 is_desktop_linux = current_os == "linux" && current_os != "chromeos" 121 is_desktop_linux = current_os == "linux"
122 122
123 # Set to true when compiling with the Clang compiler. Typically this is used 123 # Set to true when compiling with the Clang compiler. Typically this is used
124 # to configure warnings. 124 # to configure warnings.
125 is_clang = current_os == "mac" || current_os == "ios" || 125 is_clang = current_os == "mac" || current_os == "ios" ||
126 current_os == "linux" || current_os == "chromeos" 126 current_os == "linux" || current_os == "chromeos"
127 127
128 # Allows the path to a custom target toolchain to be injected as a single 128 # Allows the path to a custom target toolchain to be injected as a single
129 # argument, and set as the default toolchain. 129 # argument, and set as the default toolchain.
130 custom_toolchain = "" 130 custom_toolchain = ""
131 131
(...skipping 449 matching lines...) Expand 10 before | Expand all | Expand 10 after
581 forward_variables_from(invoker, "*") 581 forward_variables_from(invoker, "*")
582 582
583 # All shared libraries must have the sanitizer deps to properly link in 583 # All shared libraries must have the sanitizer deps to properly link in
584 # asan mode (this target will be empty in other cases). 584 # asan mode (this target will be empty in other cases).
585 if (!defined(deps)) { 585 if (!defined(deps)) {
586 deps = [] 586 deps = []
587 } 587 }
588 deps += [ "//build/config/sanitizers:deps" ] 588 deps += [ "//build/config/sanitizers:deps" ]
589 } 589 }
590 } 590 }
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