upgrade: Stop trying to copy node_modules out of tarballs.

Now that we no longer use node_modules at all in production (it's only
used to generate static assets), we don't include `node_modules` in
the production tarballs, and thus we shouldn't attempt to copy
`node_modules` out of the production tarballs when installing.

Fixes a regression introduced in
d71f2e7b9b.
This commit is contained in:
Tim Abbott
2017-02-07 09:37:31 -08:00
parent e79520c593
commit 3c3a8747c3

View File

@@ -79,8 +79,6 @@ else:
logging.info("Installing static assets...")
subprocess.check_call(["cp", "-rT", os.path.join(deploy_path, 'prod-static/serve'),
'/home/zulip/prod-static'], preexec_fn=su_to_zulip)
# Sets up npm cache
setup_node_modules(npm_args=['--production'], copy_modules=True)
# Our next optimization is to check whether any migrations are needed
# before we start the critical section of the restart. This saves