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

Side by Side Diff: icu.gyp

Issue 1639543006: ICU 56 step 6: Check in the pre-built ICU data (Closed) Base URL: https://chromium.googlesource.com/chromium/deps/icu.git@56local_patches
Patch Set: address review comments Created 4 years, 10 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 | « android/patch_locale.sh ('k') | icu.gypi » ('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) 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,
11 'icu_use_data_file_flag%': 0, 11 'icu_use_data_file_flag%': 0,
12 'want_separate_host_toolset%': 1, 12 'want_separate_host_toolset%': 1,
13 }, 13 },
14 'target_defaults': { 14 'target_defaults': {
15 'direct_dependent_settings': { 15 'direct_dependent_settings': {
16 'defines': [ 16 'defines': [
17 # Tell ICU to not insert |using namespace icu;| into its headers, 17 # Tell ICU to not insert |using namespace icu;| into its headers,
18 # so that chrome's source explicitly has to use |icu::|. 18 # so that chrome's source explicitly has to use |icu::|.
19 'U_USING_ICU_NAMESPACE=0', 19 'U_USING_ICU_NAMESPACE=0',
20 # We don't use ICU plugins and dyload is only necessary for them. 20 # We don't use ICU plugins and dyload is only necessary for them.
21 # NaCl-related builds also fail looking for dlfcn.h when it's enabled. 21 # NaCl-related builds also fail looking for dlfcn.h when it's enabled.
22 'U_ENABLE_DYLOAD=0', 22 'U_ENABLE_DYLOAD=0',
23 ], 23 ],
24 }, 24 },
25 'defines': [ 25 'defines': [
26 'U_USING_ICU_NAMESPACE=0', 26 'U_USING_ICU_NAMESPACE=0',
27 'HAVE_DLOPEN=0', 27 'HAVE_DLOPEN=0',
28 # Only build encoding coverters and detectors necessary for HTML5. 28 # Only build encoding coverters and detectors necessary for HTML5.
29 'UCONFIG_NO_NON_HTML5_CONVERSION=1', 29 'UCONFIG_ONLY_HTML_CONVERSION=1',
30 # No dependency on the default platform encoding. 30 # No dependency on the default platform encoding.
31 # Will cut down the code size. 31 # Will cut down the code size.
32 'U_CHARSET_IS_UTF8=1', 32 'U_CHARSET_IS_UTF8=1',
33 ], 33 ],
34 'conditions': [ 34 'conditions': [
35 ['component=="static_library"', { 35 ['component=="static_library"', {
36 'defines': [ 36 'defines': [
37 'U_STATIC_IMPLEMENTATION', 37 'U_STATIC_IMPLEMENTATION',
38 ], 38 ],
39 }], 39 }],
(...skipping 559 matching lines...) Expand 10 before | Expand all | Expand 10 after
599 }, 599 },
600 'includes': [ 600 'includes': [
601 '../../build/shim_headers.gypi', 601 '../../build/shim_headers.gypi',
602 ], 602 ],
603 'toolsets': ['target'], 603 'toolsets': ['target'],
604 }, 604 },
605 ], # targets 605 ], # targets
606 }], 606 }],
607 ], # conditions 607 ], # conditions
608 } 608 }
OLDNEW
« no previous file with comments | « android/patch_locale.sh ('k') | icu.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698