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

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

Issue 324403006: Add a new FORCE_DETERMINISTIC_BUILD define. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rename to force_deterministic_build Created 6 years, 6 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 | « build/common.gypi ('k') | 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 import("//build/config/allocator.gni") 5 import("//build/config/allocator.gni")
6 import("//build/config/crypto.gni") 6 import("//build/config/crypto.gni")
7 import("//build/config/features.gni") 7 import("//build/config/features.gni")
8 import("//build/config/ui.gni") 8 import("//build/config/ui.gni")
9 9
10 declare_args() { 10 declare_args() {
11 # When set, turns off the (normally-on) iterator debugging and related stuff 11 # When set, turns off the (normally-on) iterator debugging and related stuff
12 # that is normally turned on for Debug builds. These are generally useful for 12 # that is normally turned on for Debug builds. These are generally useful for
13 # catching bugs but in some cases may cause conflicts or excessive slowness. 13 # catching bugs but in some cases may cause conflicts or excessive slowness.
14 disable_iterator_debugging = false 14 disable_iterator_debugging = false
15
16 # Set to true to force deterministic builds (this isn't working yet but this
17 # flag will help us to get there). See http://crbug.com/314403.
18 # TODO(sebmarchand): Update this comment once this flag guarantee a
19 # deterministic build.
20 force_deterministic_build = false
15 } 21 }
16 22
17 # TODO(brettw) Most of these should be removed. Instead of global feature 23 # TODO(brettw) Most of these should be removed. Instead of global feature
18 # flags, we should have more modular flags that apply only to a target and its 24 # flags, we should have more modular flags that apply only to a target and its
19 # dependents. For example, depending on the "x11" meta-target should define 25 # dependents. For example, depending on the "x11" meta-target should define
20 # USE_X11 for all dependents so that everything that could use X11 gets the 26 # USE_X11 for all dependents so that everything that could use X11 gets the
21 # define, but anything that doesn't depend on X11 doesn't see it. 27 # define, but anything that doesn't depend on X11 doesn't see it.
22 # 28 #
23 # For now we define these globally to match the current GYP build. 29 # For now we define these globally to match the current GYP build.
24 config("feature_flags") { 30 config("feature_flags") {
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 } 70 }
65 if (enable_printing > 0) { 71 if (enable_printing > 0) {
66 defines += [ "ENABLE_PRINTING=1" ] 72 defines += [ "ENABLE_PRINTING=1" ]
67 if (enable_printing < 2) { 73 if (enable_printing < 2) {
68 defines += [ "ENABLE_FULL_PRINTING=1" ] 74 defines += [ "ENABLE_FULL_PRINTING=1" ]
69 } 75 }
70 } 76 }
71 if (enable_spellcheck) { 77 if (enable_spellcheck) {
72 defines += [ "ENABLE_SPELLCHECK=1" ] 78 defines += [ "ENABLE_SPELLCHECK=1" ]
73 } 79 }
80 if (force_deterministic_build) {
81 defines += [ "FORCE_DETERMINISTIC_BUILD" ]
82 }
74 if (use_udev) { 83 if (use_udev) {
75 # TODO(brettw) should probably be "=1". 84 # TODO(brettw) should probably be "=1".
76 defines += [ "USE_UDEV" ] 85 defines += [ "USE_UDEV" ]
77 } 86 }
78 if (toolkit_views) { 87 if (toolkit_views) {
79 defines += [ "TOOLKIT_VIEWS=1" ] 88 defines += [ "TOOLKIT_VIEWS=1" ]
80 } 89 }
81 if (ui_compositor_image_transport) { 90 if (ui_compositor_image_transport) {
82 # TODO(brettw) should probably be "=1". 91 # TODO(brettw) should probably be "=1".
83 defines += [ "UI_COMPOSITOR_IMAGE_TRANSPORT" ] 92 defines += [ "UI_COMPOSITOR_IMAGE_TRANSPORT" ]
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
225 "CoreText.framework", 234 "CoreText.framework",
226 "Foundation.framework", 235 "Foundation.framework",
227 "UIKit.framework", 236 "UIKit.framework",
228 ] 237 ]
229 } else if (is_linux) { 238 } else if (is_linux) {
230 libs = [ 239 libs = [
231 "dl", 240 "dl",
232 ] 241 ]
233 } 242 }
234 } 243 }
OLDNEW
« no previous file with comments | « build/common.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698