test: Node 8.9.0 cannot 'make test' when building from source code

[BobCochran]

• Version: 8.9.0
• Platform: linux 4.13.9-200.fc26.x86_64 deps: update openssl to 1.0.1j #1 SMP Mon Oct 23 13:52:45 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
• Subsystem:

I downloaded the source code tarball node-v8.9.0.tar.gz to build node on a Fedora 26 workstation. 'make test' fails at this point:

`Building addon /home/bcochranc/Downloads/node-v8.9.0/test/addons-napi/test_warning/
make[2]: Entering directory '/home/bcochranc/Downloads/node-v8.9.0/test/addons-napi/test_warning/build'
CC(target) Release/obj.target/test_warning/test_warning.o
SOLINK_MODULE(target) Release/obj.target/test_warning.node
COPY Release/test_warning.node
CC(target) Release/obj.target/test_warning2/test_warning2.o
SOLINK_MODULE(target) Release/obj.target/test_warning2.node
COPY Release/test_warning2.node
make[2]: Leaving directory '/home/bcochranc/Downloads/node-v8.9.0/test/addons-napi/test_warning/build'
touch test/addons-napi/.buildstamp
make doc-only
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN node-doc-generator@0.0.0 No repository field.
npm WARN node-doc-generator@0.0.0 No license field.

added 3 packages and updated 1 package in 0.872s
/bin/sh: out/doc/api/debugger.html: No such file or directory
make[1]: *** [Makefile:579: out/doc/api/debugger.html] Error 1
make: *** [Makefile:215: test] Error 2
`

I was last able to make test successfully with version 8.7.0. Node releases since then have failed when attempting to make test.

[vsemozhetbyt]

@joyeecheung, is it test doc related?

[drewfish]

In my company we run tests using gmake test-ci-native test-ci-js.

[joyeecheung]

@BobCochran I've tried it on Ubuntu and looks like if you run make doc-only, it will fail on the first attempt but will work the second time. Can you try running it again?

I'll investigate why this fails on the first attempt..Also this works on Mac so I think there are probably some differences in gnu make.

[joyeecheung]

diff --git a/Makefile b/Makefile
index c2f872daed..f7fc5b676d 100644
--- a/Makefile
+++ b/Makefile
@@ -544,7 +544,8 @@ apidoc_dirs = out/doc out/doc/api/ out/doc/api/assets

 apiassets = $(subst api_assets,api/assets,$(addprefix out/,$(wildcard doc/api_assets/*)))

-doc-only: $(apidocs_html) $(apidocs_json)
+doc-targets: $(apidocs_html) $(apidocs_json)
+doc-only: | install-yaml doc-targets
 doc: $(NODE_EXE) doc-only

 $(apidoc_dirs):
@@ -561,15 +562,16 @@ gen-json = tools/doc/generate.js --format=json $< > $@
 gen-html = tools/doc/generate.js --node-version=$(FULLVERSION) --format=html \
 			--template=doc/template.html --analytics=$(DOCS_ANALYTICS) $< > $@

-gen-doc =	\
+install-yaml:
 	[ -e tools/doc/node_modules/js-yaml/package.json ] || \
 		[ -e tools/eslint/node_modules/js-yaml/package.json ] || \
 		if [ -x $(NODE) ]; then \
 			cd tools/doc && ../../$(NODE) ../../$(NPM) install; \
 		else \
 			cd tools/doc && node ../../$(NPM) install; \
-		fi;\
-	[ -x $(NODE) ] && $(NODE) $(1) || node $(1)
+		fi;
+
+gen-doc = [ -x $(NODE) ] && $(NODE) $(1) || node $(1)

 out/doc/api/%.json: doc/api/%.md
 	@$(call gen-doc, $(gen-json))

There is probably some problems in order of dependencies, this patch seems to work for me on Ubuntu, I will open a PR shortly

[joyeecheung]

By the way, make test is more for developers now because it runs linters and builds docs. If you only want to test if the binary is working, you can run make test-only.

[BobCochran]

@joyeecheung here is the build sequence I followed: ./configure —prefix=$MY_NODE_PREFIX make make test MY_NODE_PREFIX resolves to my home directory $HOME/local I vaguely recall seeing a ‘make doc-only’ that was emitted by the Makefile as part of the ‘make test’ output. I will try both ‘make doc-only’ and ‘make test’ and report back. All I really wanted to do was to execute ‘make test ‘ and see normal test completion. Thanks! Bob

…

On Oct 31, 2017, at 11:41 PM, Joyee Cheung ***@***.***> wrote: @BobCochran I've tried it on Ubuntu and looks like if you run make doc-only, it will fail on the first attempt but will work the second time. Can you try running it again? I'll investigate why this fails on the first attempt.. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[BobCochran]

@joyeecheung I tried your suggestion by running 'make test' a second time, and it worked fine. My first execution of 'make test' ended in an error, at which point I created this issue to report it in. Then following your advice I ran 'make test' a second time. That worked fine. This is the output I received at the console.

If you would like me to assist in resolving this issue in any way (such as to test a patch) please let me know.

Thank you for telling me that I can run make test-only if I just want to execute tests. I am really a javascript user of node.js, rather than a developer of node.js itself, so I appreciate the tips.

make_test_results.txt

[brettsheffield]

Running make test prior to running make doc results in this error:

added 3 packages and updated 1 package in 1.717s
/bin/sh: out/doc/api/console.html: No such file or directory
Makefile:579: recipe for target 'out/doc/api/console.html' failed
make[1]: *** [out/doc/api/console.html] Error 1
Makefile:213: recipe for target 'test' failed
make: *** [test] Error 2

After running make doc, make test succeeds.

I suggest an update to https://github.com/nodejs/node/blob/master/BUILDING.md#building-nodejs-on-supported-platforms to clarify that docs need to be built prior to running make test

[BobCochran]

For a long time — until now — ‘make test ‘ has just worked without requiring the user to run ‘make doc’ as a prerequisite step.

…

On Nov 17, 2017, at 6:09 AM, Brett Sheffield ***@***.***> wrote: Running make test prior to running make doc results in this error: added 3 packages and updated 1 package in 1.717s /bin/sh: out/doc/api/console.html: No such file or directory Makefile:579: recipe for target 'out/doc/api/console.html' failed make[1]: *** [out/doc/api/console.html] Error 1 Makefile:213: recipe for target 'test' failed make: *** [test] Error 2 After running make doc, make test succeeds. I suggest an update to https://github.com/nodejs/node/blob/master/BUILDING.md#building-nodejs-on-supported-platforms to clarify that docs need to be built prior to running make test — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[joyeecheung]

@BobCochran @brettsheffield Yes, and it's supposed to work without running make doc first, this should be fixed by #16661.

Also this seems specific to GNU make v4.x, I suspect it's the conditionals in gen-doc that are interpreted differently.