rockdrilla

Google V8: use system ICU instead of supplied as 3rd pty

Jan 14th, 2014
517
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
Diff 4.09 KB | None | 0 0
  1. --- a/DEPS
  2. +++ b/DEPS
  3. @@ -5,10 +5,7 @@
  4.  deps = {
  5.    # Remember to keep the revision in sync with the Makefile.
  6.    "v8/build/gyp":
  7. -    "http://gyp.googlecode.com/svn/trunk@1831",
  8. -
  9. -  "v8/third_party/icu":
  10. -    "https://src.chromium.org/chrome/trunk/deps/third_party/icu46@239289",
  11. +    "http://gyp.googlecode.com/svn/trunk@1836",
  12.  }
  13.  
  14.  deps_os = {
  15. --- a/Makefile
  16. +++ b/Makefile
  17. @@ -441,7 +441,4 @@ grokdump: ia32.release
  18.  # Remember to keep these in sync with the DEPS file.
  19.  dependencies:
  20.     svn checkout --force http://gyp.googlecode.com/svn/trunk build/gyp \
  21. -       --revision 1831
  22. -   svn checkout --force \
  23. -       https://src.chromium.org/chrome/trunk/deps/third_party/icu46 \
  24. -       third_party/icu --revision 239289
  25. +       --revision 1836
  26. --- a/build/standalone.gypi
  27. +++ b/build/standalone.gypi
  28. @@ -93,7 +93,7 @@
  29.      'v8_optimized_debug%': 0,
  30.  
  31.      # Relative path to icu.gyp from this file.
  32. -    'icu_gyp_path': '../third_party/icu/icu.gyp',
  33. +#   'icu_gyp_path': '../third_party/icu/icu.gyp',
  34.  
  35.      'conditions': [
  36.        ['(v8_target_arch=="arm" and host_arch!="arm") or \
  37. --- a/samples/samples.gyp
  38. +++ b/samples/samples.gyp
  39. @@ -41,15 +41,15 @@
  40.      ],
  41.      'conditions': [
  42.        ['v8_enable_i18n_support==1', {
  43. -        'dependencies': [
  44. -          '<(icu_gyp_path):icui18n',
  45. -          '<(icu_gyp_path):icuuc',
  46. -        ],
  47. +        'link_settings': { 'libraries': [
  48. +          '-licui18n',
  49. +          '-licuuc'
  50. +        ], },
  51.        }],
  52.        ['OS=="win" and v8_enable_i18n_support==1', {
  53. -        'dependencies': [
  54. -          '<(icu_gyp_path):icudata',
  55. -        ],
  56. +        'link_settings': { 'libraries': [
  57. +          '-licudata'
  58. +        ], },
  59.        }],
  60.      ],
  61.    },
  62. --- a/src/d8.gyp
  63. +++ b/src/d8.gyp
  64. @@ -81,15 +81,15 @@
  65.            ],
  66.          }],
  67.          ['v8_enable_i18n_support==1', {
  68. -          'dependencies': [
  69. -            '<(icu_gyp_path):icui18n',
  70. -            '<(icu_gyp_path):icuuc',
  71. -          ],
  72. +          'link_settings': { 'libraries': [
  73. +            '-licui18n',
  74. +            '-licuuc'
  75. +          ], },
  76.          }],
  77.          ['OS=="win" and v8_enable_i18n_support==1', {
  78. -          'dependencies': [
  79. -            '<(icu_gyp_path):icudata',
  80. -          ],
  81. +          'link_settings': { 'libraries': [
  82. +            '-licudata'
  83. +          ], },
  84.          }],
  85.        ],
  86.      },
  87. --- a/tools/gcmole/gcmole.lua
  88. +++ b/tools/gcmole/gcmole.lua
  89. @@ -105,8 +105,8 @@ local function MakeClangCommandLine(plugin, plugin_args, triple, arch_define)
  90.        .. " -DENABLE_DEBUGGER_SUPPORT"
  91.        .. " -DV8_I18N_SUPPORT"
  92.        .. " -Isrc"
  93. -      .. " -Ithird_party/icu/source/common"
  94. -      .. " -Ithird_party/icu/source/i18n"
  95. +--    .. " -Ithird_party/icu/source/common"
  96. +--    .. " -Ithird_party/icu/source/i18n"
  97.  end
  98.  
  99.  function InvokeClangPluginForEachFile(filenames, cfg, func)
  100. --- a/tools/gyp/v8.gyp
  101. +++ b/tools/gyp/v8.gyp
  102. @@ -952,10 +952,10 @@
  103.            ]
  104.          }],
  105.          ['v8_enable_i18n_support==1', {
  106. -          'dependencies': [
  107. -            '<(icu_gyp_path):icui18n',
  108. -            '<(icu_gyp_path):icuuc',
  109. -          ]
  110. +          'link_settings': { 'libraries': [
  111. +            '-licui18n',
  112. +            '-licuuc'
  113. +          ], },
  114.          }, {  # v8_enable_i18n_support==0
  115.            'sources!': [
  116.              '../../src/i18n.cc',
  117. @@ -963,9 +963,9 @@
  118.            ],
  119.          }],
  120.          ['OS=="win" and v8_enable_i18n_support==1', {
  121. -          'dependencies': [
  122. -            '<(icu_gyp_path):icudata',
  123. -          ],
  124. +          'link_settings': { 'libraries': [
  125. +            '-licudata'
  126. +          ], },
  127.          }],
  128.          ['v8_use_default_platform==0', {
  129.            'sources!': [
  130. --- a/tools/lexer-shell.gyp
  131. +++ b/tools/lexer-shell.gyp
  132. @@ -40,10 +40,10 @@
  133.        ],
  134.        'conditions': [
  135.          ['v8_enable_i18n_support==1', {
  136. -          'dependencies': [
  137. -            '<(icu_gyp_path):icui18n',
  138. -            '<(icu_gyp_path):icuuc',
  139. -          ],
  140. +          'link_settings': { 'libraries': [
  141. +            '-licui18n',
  142. +            '-licuuc'
  143. +          ], },
  144.          }],
  145.        ],
  146.        'include_dirs+': [
Add Comment
Please, Sign In to add comment