Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ICU Toolset being compiled by cross compiler, not host compiler #22173

Closed
drumwhizz opened this issue Aug 7, 2018 · 2 comments
Closed

ICU Toolset being compiled by cross compiler, not host compiler #22173

drumwhizz opened this issue Aug 7, 2018 · 2 comments
Labels
build Issues and PRs related to build files or the CI. i18n-api Issues and PRs related to the i18n implementation.

Comments

@drumwhizz
Copy link

v8.11.3:
Linux KubuntuVM 3.19.0-59-generic #66~14.04.1-Ubuntu SMP Fri May 13 17:27:10 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux:
ICU_TOOLS:
Cross compiling onto arm

Same problem as this issue #7173
however, compiling without international support is NOT an option as we require that functionality.

The ICU Tools are getting built with the arm cross compiler and not the host, hence when they are run, we see the following error.

/home/phil/develop/ciws/elektron/checkit/hub/platform/obj/node-js/libs/obj/src/out/Release/icupkg: 1: /home/phil/develop/ciws/elektron/checkit/hub/platform/obj/node-js/libs/obj/src/out/Release/icupkg: Syntax error: word unexpected (expecting ")")
Options: {'verbose': 1, 'filterfile': 'icu_small.json', 'toolpath': '/home/phil/develop/ciws/elektron/checkit/hub/platform/obj/node-js/libs/obj/src/out/Release', 'deltmpdir': 1, 'outfile': 'icudt60l.dat', 'datfile': '../../deps/icu-small/source/data/in/icudt60l.dat', 'locales': 'en,root', 'endian': 'little', 'tmpdir': '/home/phil/develop/ciws/elektron/checkit/hub/platform/obj/node-js/libs/obj/src/out/Release/obj/gen/icutmp'}
icu_small.json: icutrim.py config: Trim down ICU to just a certain locale set, needed for node.js use.
FAILED: /home/phil/develop/ciws/elektron/checkit/hub/platform/obj/node-js/libs/obj/src/out/Release/icupkg -tl ../../deps/icu-small/source/data/in/icudt60l.dat /home/phil/develop/ciws/elektron/checkit/hub/platform/obj/node-js/libs/obj/src/out/Release/obj/gen/icutmp/icudt60l.dat

file /home/phil/develop/ciws/elektron/checkit/hub/platform/obj/node-js/libs/obj/src/out/Release/icupkg
/home/phil/develop/ciws/elektron/checkit/hub/platform/obj/node-js/libs/obj/src/out/Release/icupkg: ELF 32-bit LSB executable, ARM, EABI5 version 1 (SYSV), dynamically linked (uses shared libs), for GNU/Linux 4.4.3, BuildID[sha1]=aaf6d854098bed144ba06dbd0a7a2607a81fe022, not stripped

I can confirm the file icu-generic.gyp has the following settings:

'target_name': 'icupkg',
'toolsets': [ 'host' ],

It would appear that the build system is not picking up these settings.

Build config as follows:

CC=arm-unknown-linux-gnueabihf-gcc CXX=arm-unknown-linux-gnueabihf-g++
AR=arm-unknown-linux-gnueabihf-ar
./configure --prefix=/home/phil/develop/ciws/elektron/checkit/hub/platform/install/libs --dest-cpu=arm --dest-os=linux --with-arm-float-abi=hard --without-snapshot

@refack refack added build Issues and PRs related to build files or the CI. i18n-api Issues and PRs related to the i18n implementation. labels Aug 7, 2018
@jasnell
Copy link
Member

jasnell commented Jun 19, 2020

Hmm, looks like this was never followed up on in nearly two years :-/ .... is this still a problem?

@jasnell
Copy link
Member

jasnell commented Jun 25, 2020

Closing, can reopen if it is necessary

@jasnell jasnell closed this as completed Jun 25, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
build Issues and PRs related to build files or the CI. i18n-api Issues and PRs related to the i18n implementation.
Projects
None yet
Development

No branches or pull requests

3 participants