Skip to content

Commit

Permalink
remove now-unused db.examine_buildset
Browse files Browse the repository at this point in the history
  • Loading branch information
djmitche committed May 2, 2011
1 parent 24398dd commit d5ccf1d
Showing 1 changed file with 0 additions and 26 deletions.
26 changes: 0 additions & 26 deletions master/buildbot/db/connector.py
Expand Up @@ -398,32 +398,6 @@ def _txn_get_buildrequestids_for_buildset(self, t, bsid):
(bsid,))
return dict(t.fetchall())

# use by Status.getBuildSets
def examine_buildset(self, bsid):
return self.runInteractionNow(self._txn_examine_buildset, bsid)
def _txn_examine_buildset(self, t, bsid):
# "finished" means complete=1 for all builds. Return False until
# all builds are complete, then True.
# "successful" means complete=1 and results!=FAILURE for all builds.
# Returns None until the last success or the first failure. Returns
# False if there is at least one failure. Returns True if all are
# successful.
q = self.quoteq("SELECT br.complete,br.results"
" FROM buildsets AS bs, buildrequests AS br"
" WHERE br.buildsetid=bs.id AND bs.id=?")
t.execute(q, (bsid,))
results = t.fetchall()
finished = True
successful = None
for (c,r) in results:
if not c:
finished = False
if c and r not in (SUCCESS, WARNINGS):
successful = False
if finished and successful is None:
successful = True
return (successful, finished)

# used by getSourceStamp
def getChangeNumberedNow(self, changeid, t=None):
# this is a synchronous/blocking version of getChangeByNumber
Expand Down

0 comments on commit d5ccf1d

Please sign in to comment.