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

Side by Side Diff: build/gn_migration.gypi

Issue 1953523002: Address some TODO(GYP)s in the GN build. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: merge Created 4 years, 7 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
OLDNEW
1 # Copyright (c) 2015 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2015 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 file defines five targets that we are using to track the progress of the 5 # This file defines five targets that we are using to track the progress of the
6 # GYP->GN migration: 6 # GYP->GN migration:
7 # 7 #
8 # 'both_gn_and_gyp' lists what GN is currently capable of building and should 8 # 'both_gn_and_gyp' lists what GN is currently capable of building and should
9 # match the 'both_gn_and_gyp' target in //BUILD.gn. 9 # match the 'both_gn_and_gyp' target in //BUILD.gn.
10 # 10 #
11 # 'gyp_all' Should include everything built when building "all"; i.e., if you 11 # 'gyp_all' Should include everything built when building "all"; i.e., if you
12 # type 'ninja gyp_all' and then 'ninja all', the second build should do 12 # type 'ninja gyp_all' and then 'ninja all', the second build should do
13 # nothing. 'gyp_all' should just depend on the other four targets. 13 # nothing. 'gyp_all' should just depend on the other four targets.
14 # 14 #
15 # 'gyp_remaining' lists all of the targets that still need to be converted, 15 # 'gyp_remaining' lists all of the targets that still need to be converted,
16 # i.e., all of the other (non-empty) targets that a GYP build will build. 16 # i.e., all of the other (non-empty) targets that a GYP build will build.
17 #
18 # TODO(GYP): crbug.com/481694. Add a build step to the bot that enforces the
19 # above contracts.
20 17
21 { 18 {
22 'includes': [ 19 'includes': [
23 '../media/media_variables.gypi' 20 '../media/media_variables.gypi'
24 ], 21 ],
25 'targets': [ 22 'targets': [
26 { 23 {
27 'target_name': 'gyp_all', 24 'target_name': 'gyp_all',
28 'type': 'none', 25 'type': 'none',
29 'dependencies': [ 26 'dependencies': [
(...skipping 744 matching lines...) Expand 10 before | Expand all | Expand 10 after
774 }], 771 }],
775 ['OS=="android" and target_arch != "x64"', { 772 ['OS=="android" and target_arch != "x64"', {
776 'dependencies': [ 773 'dependencies': [
777 '../third_party/android_platform/relocation_packer.gyp:android_reloc ation_packer_unittests#host' 774 '../third_party/android_platform/relocation_packer.gyp:android_reloc ation_packer_unittests#host'
778 ], 775 ],
779 }], 776 }],
780 ], 777 ],
781 }, 778 },
782 ] 779 ]
783 } 780 }
OLDNEW
« no previous file with comments | « build/config/compiler/BUILD.gn ('k') | build/secondary/third_party/crashpad/crashpad/util/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698