Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: ensure_dependencies.py

Issue 29584596: Issue 5897 - Update buildtools dependency to revision ebdc3801dd98 (Closed)
Patch Set: Created Oct. 20, 2017, 1:31 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « dependencies ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ensure_dependencies.py
diff --git a/ensure_dependencies.py b/ensure_dependencies.py
index 9c9a9ca83dc7efe936204802a8b344b0159174cc..1cbbf09522eef68283123f634370064352ec49f1 100755
--- a/ensure_dependencies.py
+++ b/ensure_dependencies.py
@@ -278,8 +278,8 @@ def resolve_npm_dependencies(target, vcs):
lockfile_path = os.path.join(target, NPM_LOCKFILE)
open(lockfile_path, 'a').close()
- cmd = ['npm', 'install', '--only=production',
- '--loglevel=warn', '--no-package-lock']
+ cmd = ['npm', 'install', '--only=production', '--loglevel=warn',
+ '--no-package-lock', '--no-optional']
subprocess.check_output(cmd, cwd=target)
repo_types[vcs].ignore(os.path.join(target, NPM_LOCKFILE), target)
« no previous file with comments | « dependencies ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld