Skip to content

Commit

Permalink
nodejs: Restore c-ares and gtest include files
Browse files Browse the repository at this point in the history
Due to a bug in nodejs [1] we have to remove the c-ares and gtest
include files before building. Improve this to restore them after
building to prevent build breakage if building a package requiring
these afterwards.

[1] nodejs/node#2637
  • Loading branch information
fornwall committed Jan 2, 2016
1 parent 5bdfa51 commit d53dc64
Showing 1 changed file with 16 additions and 3 deletions.
19 changes: 16 additions & 3 deletions packages/nodejs/build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,15 @@ TERMUX_PKG_RM_AFTER_INSTALL="lib/node_modules/npm/html lib/node_modules/npm/make
TERMUX_PKG_BUILD_IN_SRC=yes

termux_step_configure () {
#FIXME: node.js build does not handle already installed headers
# https://github.com/nodejs/node/issues/2637
rm -Rf $TERMUX_PREFIX/{include/gtest/,/include/ares*}
#XXX: node.js build does not handle already installed headers
# https://github.com/nodejs/node/issues/2637
# So we remove them here and restore afterwards.
rm -Rf $TERMUX_PKG_CACHEDIR/gtest-include-dir $TERMUX_PKG_CACHEDIR/ares-includes
test -d $TERMUX_PREFIX/include/gtest &&
mv $TERMUX_PREFIX/include/gtest $TERMUX_PKG_CACHEDIR/gtest-include-dir
test -f $TERMUX_PREFIX/include/ares.h &&
mkdir $TERMUX_PKG_CACHEDIR/ares-includes/ &&
mv $TERMUX_PREFIX/include/ares* $TERMUX_PKG_CACHEDIR/ares-includes/

if [ $TERMUX_ARCH = "arm" ]; then
TERMUX_PKG_EXTRA_CONFIGURE_ARGS+=" --dest-cpu=arm"
Expand All @@ -27,3 +33,10 @@ termux_step_configure () {

./configure --prefix=$TERMUX_PREFIX ${TERMUX_PKG_EXTRA_CONFIGURE_ARGS}
}

termux_step_post_massage () {
test -d $TERMUX_PKG_CACHEDIR/gtest-include-dir &&
mv $TERMUX_PKG_CACHEDIR/gtest-include-dir $TERMUX_PREFIX/include/gtest
test -d $TERMUX_PKG_CACHEDIR/ares-includes &&
mv $TERMUX_PKG_CACHEDIR/ares-includes/* $TERMUX_PREFIX/include/
}

0 comments on commit d53dc64

Please sign in to comment.