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

Unified Diff: sitescripts/extensions/bin/createNightlies.py

Issue 6627397512200192: Issue 1362 - createNightlies script should call ensure_dependencies (Closed)
Patch Set: Use set literals Created Sept. 18, 2014, 6:17 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/extensions/bin/createNightlies.py
===================================================================
--- a/sitescripts/extensions/bin/createNightlies.py
+++ b/sitescripts/extensions/bin/createNightlies.py
@@ -89,16 +89,27 @@ class NightlyBuild(object):
Create a repository copy in a temporary directory
'''
# We cannot use hg archive here due to
# http://bz.selenic.com/show_bug.cgi?id=3747, have to clone properly :-(
self.tempdir = tempfile.mkdtemp(prefix=self.config.repositoryName)
command = ['hg', 'clone', '-q', self.config.repository, '-u', 'default', self.tempdir]
subprocess.check_call(command)
+ # Make sure to process the dependencies file if it is present
+ import logging
+ logging.disable(logging.WARNING)
+ try:
+ from buildtools.ensure_dependencies import resolve_deps
+ resolve_deps(self.tempdir, self_update=False,
+ overrideroots={"hg": os.path.dirname(self.config.repository)},
+ skipdependencies={"buildtools"})
+ finally:
+ logging.disable(logging.NOTSET)
+
def writeChangelog(self, changes):
"""
write the changelog file into the cloned repository
"""
baseDir = os.path.join(self.config.nightliesDirectory, self.basename)
if not os.path.exists(baseDir):
os.makedirs(baseDir)
changelogFile = "%s-%s.changelog.xhtml" % (self.basename, self.version)
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld