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

Side by Side Diff: icu.gyp

Issue 317373005: Fix multiple rules generating icudtl.dat (Closed) Base URL: https://chromium.googlesource.com/chromium/deps/icu46.git@master
Patch Set: 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 | « 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) 2012 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2012 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 'includes': [ 6 'includes': [
7 'icu.gypi', 7 'icu.gypi',
8 ], 8 ],
9 'variables': { 9 'variables': {
10 'use_system_icu%': 0, 10 'use_system_icu%': 0,
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
62 'include_dirs': [ 62 'include_dirs': [
63 'source/common', 63 'source/common',
64 'source/i18n', 64 'source/i18n',
65 ], 65 ],
66 'msvs_disabled_warnings': [4005, 4068, 4355, 4996, 4267], 66 'msvs_disabled_warnings': [4005, 4068, 4355, 4996, 4267],
67 }, 67 },
68 'conditions': [ 68 'conditions': [
69 ['use_system_icu==0 or want_separate_host_toolset==1', { 69 ['use_system_icu==0 or want_separate_host_toolset==1', {
70 'targets': [ 70 'targets': [
71 { 71 {
72 'target_name': 'copy_icudtl_dat',
73 'type': 'none',
74 # icudtl.dat is the same for both host/target, so this only supports a
75 # single toolset. If a target requires that the .dat file be copied
76 # to the output directory, it should explicitly depend on this target
77 # with the host toolset (like copy_icudtl_dat#host).
78 'toolsets': [ 'host' ],
79 'copies': [{
80 'destination': '<(PRODUCT_DIR)',
81 'conditions': [
82 ['OS == "android"', {
83 'files': [
84 'android/icudtl.dat',
85 ],
86 } , { # else: OS != android
87 'files': [
88 'source/data/in/icudtl.dat',
89 ],
90 }],
91 ],
92 }],
93 },
94 {
72 'target_name': 'icudata', 95 'target_name': 'icudata',
73 'type': 'static_library', 96 'type': 'static_library',
74 'defines': [ 97 'defines': [
75 'U_HIDE_DATA_SYMBOL', 98 'U_HIDE_DATA_SYMBOL',
76 ], 99 ],
77 'sources': [ 100 'sources': [
78 # These are hand-generated, but will do for now. The linux 101 # These are hand-generated, but will do for now. The linux
79 # version is an identical copy of the (mac) icudt46l_dat.S file, 102 # version is an identical copy of the (mac) icudt46l_dat.S file,
80 # modulo removal of the .private_extern and .const directives and 103 # modulo removal of the .private_extern and .const directives and
81 # with no leading underscore on the icudt46_dat symbol. 104 # with no leading underscore on the icudt46_dat symbol.
(...skipping 24 matching lines...) Expand all
106 }], 129 }],
107 [ 'icu_use_data_file_flag==1', { 130 [ 'icu_use_data_file_flag==1', {
108 # Remove any assembly data file. 131 # Remove any assembly data file.
109 'sources/': [['exclude', 'icudt46l_dat']], 132 'sources/': [['exclude', 'icudt46l_dat']],
110 # Compile in the stub data symbol. 133 # Compile in the stub data symbol.
111 'sources': ['source/stubdata/stubdata.c'], 134 'sources': ['source/stubdata/stubdata.c'],
112 135
113 # Make sure any binary depending on this gets the data file. 136 # Make sure any binary depending on this gets the data file.
114 'conditions': [ 137 'conditions': [
115 ['OS != "ios"', { 138 ['OS != "ios"', {
116 'copies': [{ 139 'dependencies': [
117 'destination': '<(PRODUCT_DIR)', 140 'copy_icudtl_dat#host',
118 'conditions': [ 141 ],
119 ['OS == "android"', {
120 'files': [
121 'android/icudtl.dat',
122 ],
123 } , { # else: OS != android
124 'files': [
125 'source/data/in/icudtl.dat',
126 ],
127 }],
128 ],
129 }],
130 } , { # else: OS=="ios" 142 } , { # else: OS=="ios"
131 'link_settings': { 143 'link_settings': {
132 'mac_bundle_resources': [ 144 'mac_bundle_resources': [
133 'source/data/in/icudtl.dat', 145 'source/data/in/icudtl.dat',
134 ], 146 ],
135 }, 147 },
136 }], # OS!=ios 148 }], # OS!=ios
137 ], # conditions 149 ], # conditions
138 }], # icu_use_data_file_flag 150 }], # icu_use_data_file_flag
139 ], # conditions 151 ], # conditions
(...skipping 455 matching lines...) Expand 10 before | Expand all | Expand 10 after
595 }, 607 },
596 'includes': [ 608 'includes': [
597 '../../build/shim_headers.gypi', 609 '../../build/shim_headers.gypi',
598 ], 610 ],
599 'toolsets': ['target'], 611 'toolsets': ['target'],
600 }, 612 },
601 ], # targets 613 ], # targets
602 }], 614 }],
603 ], # conditions 615 ], # conditions
604 } 616 }
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