tree ab6f6667996d85a48ff8243b0cd24776ef3319df
parent 6693617d2022af9b7212ff8ba56a3f201b972900
author Han Seoul-Oh <laughinghan@gmail.com> 1485294662 -0800
committer Han Seoul-Oh <laughinghan@gmail.com> 1485294662 -0800

Makefile: Wipe node_modules if not properly installed

If `make server` is already running (or port 9292 is someotherhow
already in use), and I run `make server` in a fresh clone, the test
server gets an EADDRINUSE and quits. However, it first starts up `make
test` in parallel, which does an `npm install`, which gets aborted
partway through. On my machine, this race condition results in the
packages seemingly all installed under `node_modules/*`, but not the
executables like `node_modules/.bin/lessc`.

The problem arises when subsequently building again (e.g. `PORT=9393
make server`), which runs `npm install` again, this time to completion.
`npm` sees all the packages and installed under `node_modules/*` and
doesn't try to complete the installation and install the
`node_modules/.bin/*` executables (it probably doesn't want to do a full
integrity check: https://github.com/npm/npm/issues/15562 ), instead
quitting and reports success, but the Makefile then fails when it tries
to do `node_modules/.bin/lessc`.

https://mathquill.slack.com/archives/mathquill/p1482616486000235