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

Side by Side Diff: build/gn_migration.gypi

Issue 1279133006: Removed unused mojo_nacl code and GYP targets. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: - -l Created 5 years, 4 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/all.gyp ('k') | mojo/mojo_nacl.gyp » ('j') | 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) 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_only' lists any targets that are not meant to be ported over to the GN
16 # build.
17 #
18 # '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,
19 # 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.
20 # 17 #
21 # TODO(GYP): crbug.com/481694. Add a build step to the bot that enforces the 18 # TODO(GYP): crbug.com/481694. Add a build step to the bot that enforces the
22 # above contracts. 19 # above contracts.
23 20
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': [
30 'both_gn_and_gyp', 27 'both_gn_and_gyp',
31 'gyp_only',
32 'gyp_remaining', 28 'gyp_remaining',
33 ] 29 ]
34 }, 30 },
35 { 31 {
36 # This target should mirror the structure of //:both_gn_and_gyp 32 # This target should mirror the structure of //:both_gn_and_gyp
37 # in src/BUILD.gn as closely as possible, for ease of comparison. 33 # in src/BUILD.gn as closely as possible, for ease of comparison.
38 'target_name': 'both_gn_and_gyp', 34 'target_name': 'both_gn_and_gyp',
39 'type': 'none', 35 'type': 'none',
40 'dependencies': [ 36 'dependencies': [
41 '../base/base.gyp:base_i18n_perftests', 37 '../base/base.gyp:base_i18n_perftests',
(...skipping 441 matching lines...) Expand 10 before | Expand all | Expand 10 after
483 '../testing/gtest.gyp:gtest_main', 479 '../testing/gtest.gyp:gtest_main',
484 '../third_party/codesighs/codesighs.gyp:msdump2symdb', 480 '../third_party/codesighs/codesighs.gyp:msdump2symdb',
485 '../third_party/codesighs/codesighs.gyp:msmap2tsv', 481 '../third_party/codesighs/codesighs.gyp:msmap2tsv',
486 '../third_party/pdfium/samples/samples.gyp:pdfium_diff', 482 '../third_party/pdfium/samples/samples.gyp:pdfium_diff',
487 '../win8/win8.gyp:metro_viewer', 483 '../win8/win8.gyp:metro_viewer',
488 ], 484 ],
489 }], 485 }],
490 ], 486 ],
491 }, 487 },
492 { 488 {
493 'target_name': 'gyp_only',
494 'type': 'none',
495 'conditions': [
496 ['OS=="linux" or OS=="win"', {
497 'conditions': [
498 ['disable_nacl==0 and disable_nacl_untrusted==0', {
499 'dependencies': [
500 '../mojo/mojo_nacl.gyp:monacl_shell', # This should not be buil t in chromium.
501 ]
502 }],
503 ]
504 }],
505 ],
506 },
507 {
508 'target_name': 'gyp_remaining', 489 'target_name': 'gyp_remaining',
509 'type': 'none', 490 'type': 'none',
510 'conditions': [ 491 'conditions': [
511 ['remoting==1', { 492 ['remoting==1', {
512 'dependencies': [ 493 'dependencies': [
513 '../remoting/app_remoting_webapp.gyp:ar_sample_app', # crbug.com/47 1916 494 '../remoting/app_remoting_webapp.gyp:ar_sample_app', # crbug.com/47 1916
514 ], 495 ],
515 }], 496 }],
516 ['test_isolation_mode!="noop"', { 497 ['test_isolation_mode!="noop"', {
517 'dependencies': [ 498 'dependencies': [
(...skipping 198 matching lines...) Expand 10 before | Expand all | Expand 10 after
716 'dependencies': [ 697 'dependencies': [
717 # TODO(GYP): All of these targets need to be ported over. 698 # TODO(GYP): All of these targets need to be ported over.
718 '../components/components.gyp:policy_win64', 699 '../components/components.gyp:policy_win64',
719 ] 700 ]
720 }], 701 }],
721 ], 702 ],
722 }, 703 },
723 ] 704 ]
724 } 705 }
725 706
OLDNEW
« no previous file with comments | « build/all.gyp ('k') | mojo/mojo_nacl.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698