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

Unified Diff: build.py

Issue 5710085762318336: 1757 - Integrate ensure_dependencies.py with the adblockpluschrome repository (Closed)
Patch Set: '1757 - Integrate ensure_dependencies.py with the adblockpluschrome repository' Created Jan. 14, 2015, 4:08 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 | « .hgsubstate ('k') | dependencies » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build.py
===================================================================
--- a/build.py
+++ b/build.py
@@ -3,5 +3,13 @@
import os, sys, subprocess
+DEPENDENCY_SCRIPT = os.path.join(os.path.dirname(__file__), "ensure_dependencies.py")
+
+try:
+ subprocess.check_call([sys.executable, DEPENDENCY_SCRIPT])
+except subprocess.CalledProcessError as e:
+ print >>sys.stderr, e
+ print >>sys.stderr, "Failed to ensure dependencies being up-to-date!"
+
Sebastian Noack 2015/01/14 16:34:13 Maybe exit with a non-0 status code?
Sebastian Noack 2015/01/14 17:28:36 Thinking again about it, I think it's OK to just s
Wladimir Palant 2015/01/14 17:42:46 Making dependency issues non-fatal was one of the
import buildtools.build
buildtools.build.processArgs('.', sys.argv)
« no previous file with comments | « .hgsubstate ('k') | dependencies » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld