Skip to content

Commit

Permalink
docs: fix references to node-waf
Browse files Browse the repository at this point in the history
The behavior described here has been inaccurate for a long time. Not only was
the `wscript` bit removed from `read-package-json` over 2 years ago, the
`install` script is what is set to the default (now `node-gyp rebuild`), not
`preinstall`.

Fixes npm#9583

PR-URL: npm#9584
  • Loading branch information
kenany authored and iarna committed Sep 17, 2015
1 parent 8a942e0 commit 8b3859a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions doc/misc/npm-scripts.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,10 +69,10 @@ npm will default some script values based on package contents.
If there is a `server.js` file in the root of your package, then npm
will default the `start` command to `node server.js`.

* `"preinstall": "node-waf clean || true; node-waf configure build"`:
* `"install": "node-gyp rebuild"`:

If there is a `wscript` file in the root of your package, npm will
default the `preinstall` command to compile using node-waf.
If there is a `bindings.gyp` file in the root of your package, npm will
default the `install` command to compile using node-gyp.

## USER

Expand Down

0 comments on commit 8b3859a

Please sign in to comment.