Only disable javascript tests if node-gyp is missing when testing node
This commit is contained in:
parent
e730b16fac
commit
e6d5abb766
2 changed files with 11 additions and 5 deletions
|
|
@ -661,12 +661,14 @@ javascript_version:
|
|||
ifeq (, $(ENGINE))
|
||||
@if [ "$(NODEJS)" != "" ]; then \
|
||||
echo "Node.js: `($(NODEJS) --version)`"; \
|
||||
echo "node-gyp: `($(NODEGYP) --version)`"; \
|
||||
else \
|
||||
echo "Version depends on the interpreter"; \
|
||||
fi
|
||||
endif
|
||||
ifeq (node, $(ENGINE))
|
||||
$(NODEJS) --version
|
||||
echo "Node.js: `($(NODEJS) --version)`"
|
||||
echo "node-gyp: `($(NODEGYP) --version)`"
|
||||
endif
|
||||
ifeq (jsc, $(ENGINE))
|
||||
@if [ "@JSCOREVERSION@" != "" ]; then \
|
||||
|
|
@ -678,7 +680,6 @@ endif
|
|||
ifeq (v8, $(ENGINE))
|
||||
echo "Unknown v8 version."
|
||||
endif
|
||||
echo "node-gyp: `($(NODEGYP) --version)`"
|
||||
|
||||
# -----------------------------------------------------------------
|
||||
# Cleaning the Javascript examples
|
||||
|
|
|
|||
11
configure.ac
11
configure.ac
|
|
@ -1136,8 +1136,13 @@ else
|
|||
|
||||
AC_CHECK_PROGS(NODEJS, node)
|
||||
|
||||
# node-gyp is needed to run the test-suite/examples for all Javascript engines
|
||||
AC_CHECK_PROGS(NODEGYP, node-gyp)
|
||||
if test -n "$NODEJS"; then
|
||||
# node-gyp is needed to run the test-suite/examples
|
||||
AC_CHECK_PROGS(NODEGYP, node-gyp)
|
||||
if test -z "$NODEGYP"; then
|
||||
NODEJS=
|
||||
fi
|
||||
fi
|
||||
|
||||
#----------------------------------------------------------------
|
||||
# Look for JavascriptCore (Webkit) settings (JSCOREINCDIR, JSCOREDYNAMICLINKING)
|
||||
|
|
@ -2459,7 +2464,7 @@ fi
|
|||
AC_SUBST(SKIP_JAVA)
|
||||
|
||||
SKIP_JAVASCRIPT=
|
||||
if test -z "$JAVASCRIPT" || test -z "$NODEJS" || test -z "$NODEGYP"; then
|
||||
if test -z "$JAVASCRIPT" || test -z "$NODEJS" ; then
|
||||
SKIP_JAVASCRIPT="1"
|
||||
fi
|
||||
AC_SUBST(SKIP_JAVASCRIPT)
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue