Skip to content

Commit c006888

Browse files
committed
build: use ICU_UTIL_DATA_STATIC across V8
ICU_UTIL_DATA_SHARED had been removed since https://crrev.com/c/1513615, but Node.js still defined it and relied on the removed path on Windows, so the ICU initialization in mksnapshot had been silently failing since then. https://crrev.com/c/7679153 made the failure visible so the build started breaking on Windows. Fix it by always using ICU_UTIL_DATA_STATIC since we already compile the ICU data statically in. Signed-off-by: Joyee Cheung <joyeec9h3@gmail.com>
1 parent edf8bca commit c006888

1 file changed

Lines changed: 1 addition & 7 deletions

File tree

tools/v8_gypfiles/v8.gyp

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1309,13 +1309,7 @@
13091309
['icu_use_data_file_flag', {
13101310
'defines': ['ICU_UTIL_DATA_IMPL=ICU_UTIL_DATA_FILE'],
13111311
}, {
1312-
'conditions': [
1313-
['OS=="win"', {
1314-
'defines': ['ICU_UTIL_DATA_IMPL=ICU_UTIL_DATA_SHARED'],
1315-
}, {
1316-
'defines': ['ICU_UTIL_DATA_IMPL=ICU_UTIL_DATA_STATIC'],
1317-
}],
1318-
],
1312+
'defines': ['ICU_UTIL_DATA_IMPL=ICU_UTIL_DATA_STATIC'],
13191313
}],
13201314
['OS=="win"', {
13211315
'dependencies': [

0 commit comments

Comments
 (0)