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

Side by Side Diff: BUILD.gn

Issue 2301213002: Don't allow official component builds (Closed)
Patch Set: Created 4 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 # This is the root build file for GN. GN will start processing by loading this 5 # This is the root build file for GN. GN will start processing by loading this
6 # file, and recursively load all dependencies until all dependencies are either 6 # file, and recursively load all dependencies until all dependencies are either
7 # resolved or known not to exist (which will cause the build to fail). So if 7 # resolved or known not to exist (which will cause the build to fail). So if
8 # you add a new build file, there must be some path of dependencies from this 8 # you add a new build file, there must be some path of dependencies from this
9 # file to your new one or GN won't know about it. 9 # file to your new one or GN won't know about it.
10 10
(...skipping 12 matching lines...) Expand all
23 import("//build/config/android/config.gni") 23 import("//build/config/android/config.gni")
24 } 24 }
25 25
26 declare_args() { 26 declare_args() {
27 # A list of extra dependencies to add to the root target. This allows a 27 # A list of extra dependencies to add to the root target. This allows a
28 # checkout to add additional targets without explicitly changing any checked- 28 # checkout to add additional targets without explicitly changing any checked-
29 # in files. 29 # in files.
30 root_extra_deps = [] 30 root_extra_deps = []
31 } 31 }
32 32
33 if (is_official_build) {
34 # An official (maximally optimized!) component (optimized for build times)
35 # build doesn't make sense and usually doesn't work.
36 assert(!is_component_build)
37 }
38
33 # This file defines the following five main targets: 39 # This file defines the following five main targets:
34 # 40 #
35 # "both_gn_and_gyp" should list every root target (target that nothing else 41 # "both_gn_and_gyp" should list every root target (target that nothing else
36 # depends on) built by GN that is also built in the GYP build. 42 # depends on) built by GN that is also built in the GYP build.
37 # 43 #
38 # "gn_all" should (transitively) cause everything to be built; if you run 44 # "gn_all" should (transitively) cause everything to be built; if you run
39 # 'ninja gn_all' and then 'ninja all', the second build should do no work. 45 # 'ninja gn_all' and then 'ninja all', the second build should do no work.
40 # 46 #
41 # "gn_only" should list every root target that is *not* intended to be built in 47 # "gn_only" should list every root target that is *not* intended to be built in
42 # a GYP build. Because GN has different rules for deciding what an 'all' build 48 # a GYP build. Because GN has different rules for deciding what an 'all' build
(...skipping 1083 matching lines...) Expand 10 before | Expand all | Expand 10 after
1126 "//ui/display:display_unittests", 1132 "//ui/display:display_unittests",
1127 "//ui/events:events_unittests", 1133 "//ui/events:events_unittests",
1128 "//ui/gfx:gfx_unittests", 1134 "//ui/gfx:gfx_unittests",
1129 "//ui/gl:gl_unittests", 1135 "//ui/gl:gl_unittests",
1130 "//ui/keyboard:keyboard_unittests", 1136 "//ui/keyboard:keyboard_unittests",
1131 "//ui/touch_selection:ui_touch_selection_unittests", 1137 "//ui/touch_selection:ui_touch_selection_unittests",
1132 "//url:url_unittests", 1138 "//url:url_unittests",
1133 ] 1139 ]
1134 } 1140 }
1135 } 1141 }
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