Uh oh!
There was an error while loading. Please reload this page.
deps: backport char16ptr.h to 57 - #19658
Conversation
srl295
commented
Mar 28, 2018
note that #19624 is also needed |
- char16ptr.h is appended to the ICU 57 unistr.h - this way, no v8 code change is needed. Fixes: nodejs#19656
9d59bd5 to
c53a1e2Comparesrl295
commented
Mar 28, 2018
^ doesn't actually work (why did it seem to work locally?)… my 'floating patch' mechanism doesn't support header file patching. |
bnoordhuis
commented
Mar 28, 2018
For my curiosity: is there a reason we try to support old versions of ICU? It doesn't work when you try to link against the system ICU (logical) so it seems like the only use case is people downloading an old tarball manually and extracting that to deps/icu - but why would anyone do that? |
Can test with:
@bnoordhuis good point… my point was for system-icu. But, floating patches against icu don't affect an icu that's already built and part of the system.
The only benefit for doing "that" would be to verify compatibility (see #19657 ). But I think more importantly, the benefit of this PR is misguided: Fixing v8 would actually allow linking against the older system-icu. |
srl295
commented
Apr 3, 2018
#19710 actually solves the issue instead of hacking around it. The only benefit to this PR is that it fixes the floating patch mechanism to support patching header files (below). I'll close this for now. diff --git a/configure b/configure
index df59b59e207..13fca229bf6 100755
--- a/configure+++ b/configure@@ -34,6 +34,8 @@ import shlex
import subprocess
import shutil
import string
+import distutils+from distutils import dir_util
# If not run from node/, cd to node/.
os.chdir(os.path.dirname(__file__) or '.')
@@ -1354,6 +1356,10 @@ def configure_intl(o):
print('See the README.md.')
# .. and we're not about to build it from .gyp!
sys.exit(1)
+ # If we're not using the in-tree ICU, copy over any patched files+ if with_icu_source and os.path.isdir('tools/icu/patches/%s' % icu_ver_major):+ print('Copying ICU patches from tools/icu/patches/%s to %s' % (icu_ver_major, icu_full_path))+ distutils.dir_util.copy_tree('tools/icu/patches/%s' % icu_ver_major, icu_full_path)
# map from variable name to subdirs
icu_src = {
'stubdata': 'stubdata', |
Fixes: #19656
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passes