Skip to content

Commit

Permalink
build: unbreak --with-intl=system-icu build
Browse files Browse the repository at this point in the history
Include directories are configured by the tools/icu/icu-*.gyp files.
The v8.gyp file doesn't need to add them and in fact the way it did
that breaks when building against an external copy of ICU.

Fixes: nodejs#28052
  • Loading branch information
bnoordhuis committed Jun 7, 2019
1 parent 1a62bc8 commit 167f4b0
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions tools/v8_gypfiles/v8.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -393,13 +393,6 @@
'<(icu_gyp_path):icui18n',
'<(icu_gyp_path):icuuc',
],
'direct_dependent_settings': {
'include_dirs': [
'<(icu_path)/source/common',
'<(icu_path)/source/i18n',
'<(icu_path)/source/tools/toolutil',
],
},
}],
],
}, # v8_maybe_icu
Expand Down

0 comments on commit 167f4b0

Please sign in to comment.