package/nodejs: use system-icu for host-nodejs
authorJames Hilliard <james.hilliard1@gmail.com>
Mon, 27 Jul 2020 11:51:55 +0000 (05:51 -0600)
committerYann E. MORIN <yann.morin.1998@free.fr>
Mon, 27 Jul 2020 12:24:39 +0000 (14:24 +0200)
commit319f7b0dab761483c7bf60dd933aa8ec485a06e1
tree61e9b06edce0055b30079f261dd383ff476cd4de
parent93a28709664623bb394e7e5c9a0be6fc6b52ae5f
package/nodejs: use system-icu for host-nodejs

The nodejs configure.py file orders zlib headers before the bundled ICU
headers. The zlib headers happen to be located in the system include
directory, next to some system ICU headers (not bundled). If these are
built before nodejs is, nodejs will get confused and try to use the
system ICU headers instead of the bundled ones.

Fix this by always using host-icu.

Set CXXFLAGS to -DU_DISABLE_RENAMING=1 when building with
system-icu since host-icu is built with --disable-renaming.

Fixes:
 - http://autobuild.buildroot.net/results/1ef947553ec762dba6a6202b1cfc84ceed75dbb2/

Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
[yann.morin.1998@free.fr:
  - keep alphabetical order in _DEPENDENCIES
  - don't introduce HOST_NODEJS_CONF_OPTS
]
Signed-off-by: Yann E. MORIN <yann.morin.1998@free.fr>
package/nodejs/nodejs.mk