Skip to content

build: run `npm install` for doc builds in tarball

Rodrigo Muino Tomonari requested to merge github/fork/addaleax/build-doctool-npm-install into master
Checklist
  • make -j4 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

build

Description of change

Run npm install before building the documentation from release tarballs. The doctool currently depends on js-yaml, which is imported from the tools/eslint subtree; however, release tarballs don’t contain that directory.

Running npm install is clearly not a beautiful solution, but it works, and this has been on my to-do list for long enough… If anyone has better ideas, please voice them!

Fixes: https://github.com/nodejs/node/issues/7872

Merge request reports