Skip to content

Commit

Permalink
Make master-side source steps respect timeout option.
Browse files Browse the repository at this point in the history
Fixes #2248, #2203.
  • Loading branch information
tomprince committed May 28, 2012
1 parent 25fff52 commit 5a18dc6
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 1 deletion.
3 changes: 2 additions & 1 deletion master/buildbot/steps/source/base.py
Expand Up @@ -149,7 +149,6 @@ def __init__(self, workdir=None, mode='update', alwaysUseLatest=False,
assert isinstance(repeats, int)
assert repeats > 0
self.args = {'mode': mode,
'timeout': timeout,
'retry': retry,
'patch': None, # set during .start
}
Expand All @@ -166,6 +165,7 @@ def __init__(self, workdir=None, mode='update', alwaysUseLatest=False,

self.logEnviron = logEnviron
self.env = env
self.timeout = timeout

descriptions_for_mode = {
"clobber": "checkout",
Expand Down Expand Up @@ -285,6 +285,7 @@ def start(self):

self.args['logEnviron'] = self.logEnviron
self.args['env'] = self.env
self.args['timeout'] = self.timeout
self.startVC(branch, revision, patch)

def commandComplete(self, cmd):
Expand Down
1 change: 1 addition & 0 deletions master/buildbot/steps/source/bzr.py
Expand Up @@ -196,6 +196,7 @@ def _dovccmd(self, command, abandonOnFailure=True, collectStdout=False):
cmd = buildstep.RemoteShellCommand(self.workdir, ['bzr'] + command,
env=self.env,
logEnviron=self.logEnviron,
timeout=self.timeout,
collectStdout=collectStdout)
cmd.useLog(self.stdio_log, False)
d = self.runCommand(cmd)
Expand Down
1 change: 1 addition & 0 deletions master/buildbot/steps/source/cvs.py
Expand Up @@ -211,6 +211,7 @@ def _dovccmd(self, command, workdir=None):
cmd = buildstep.RemoteShellCommand(workdir, ['cvs'] +
command,
env=self.env,
timeout=self.timeout,
logEnviron=self.logEnviron)
cmd.useLog(self.stdio_log, False)
d = self.runCommand(cmd)
Expand Down
1 change: 1 addition & 0 deletions master/buildbot/steps/source/git.py
Expand Up @@ -294,6 +294,7 @@ def _dovccmd(self, command, abandonOnFailure=True, collectStdout=False, initialS
cmd = buildstep.RemoteShellCommand(self.workdir, ['git'] + command,
env=self.env,
logEnviron=self.logEnviron,
timeout=self.timeout,
collectStdout=collectStdout,
initialStdin=initialStdin)
cmd.useLog(self.stdio_log, False)
Expand Down
1 change: 1 addition & 0 deletions master/buildbot/steps/source/mercurial.py
Expand Up @@ -224,6 +224,7 @@ def _dovccmd(self, command, collectStdout=False):
cmd = buildstep.RemoteShellCommand(self.workdir, ['hg', '--verbose'] + command,
env=self.env,
logEnviron=self.logEnviron,
timeout=self.timeout,
collectStdout=collectStdout)
cmd.useLog(self.stdio_log, False)
log.msg("Starting mercurial command : hg %s" % (" ".join(command), ))
Expand Down
1 change: 1 addition & 0 deletions master/buildbot/steps/source/svn.py
Expand Up @@ -228,6 +228,7 @@ def _dovccmd(self, command, collectStdout=False):
cmd = buildstep.RemoteShellCommand(self.workdir, ['svn'] + command,
env=self.env,
logEnviron=self.logEnviron,
timeout=self.timeout,
collectStdout=collectStdout)
cmd.useLog(self.stdio_log, False)
log.msg("Starting SVN command : svn %s" % (" ".join(command), ))
Expand Down

0 comments on commit 5a18dc6

Please sign in to comment.