Merge pull request #43 from aplanas/master

3th version of the fix. Map it to a fail state.
This commit is contained in:
Stephan Kulow 2013-11-11 05:02:49 -08:00
commit 0c28058c0a

View File

@ -601,7 +601,10 @@ def _check_repo_buildsuccess(self, p, opts):
p.build_excluded = True
if arch.attrib['result'] == 'disabled':
founddisabled = True
if arch.attrib['result'] == 'failed':
if arch.attrib['result'] == 'failed' or arch.attrib['result'] == 'unknown':
# Sometimes an unknown status is equivalent to
# disabled, but we map it as failed to have a human
# check (no autoreject)
r_foundfailed = repo.attrib['name']
if arch.attrib['result'] == 'building':
r_foundbuilding = repo.attrib['name']
@ -612,9 +615,6 @@ def _check_repo_buildsuccess(self, p, opts):
# Next line is not needed, but for documentation
p.updated = True
return False
if arch.attrib['result'] == 'unknown':
print 'UNKOWN state -- Stoping check for this package'
return False
r_missings = r_missings.keys()
for pkg in r_missings:
@ -836,7 +836,7 @@ def _get_builddepinfo_graph(self, opts, project='openSUSE:Factory', repository='
missing = [d for d in deps if not d.startswith(_IGNORE_PREFIX) and d not in subpkgs]
if missing:
if p.pkg not in _ignore_packages:
#print 'Ignoring package. Missing dependencies %s -> (%s) %s...' % (p.pkg, len(missing), missing[:5])
# print 'Ignoring package. Missing dependencies %s -> (%s) %s...' % (p.pkg, len(missing), missing[:5])
_ignore_packages.add(p.pkg)
continue