1
0
mirror of https://github.com/openSUSE/osc.git synced 2024-11-10 06:46:15 +01:00

Removed bogus check in get_package_results (fixes #224)

The bogus check is twofold: the first error is due to a wrong
refactoring (repostate was intended to be the "code" attribute
and "code" was supposed to the "code" attribute of the status
node) (see commit f3a1d12a). The second error is a logic error
in the original code, because the package state "succeeded" and
the repostate "unpublished" is a valid combination. Consequently,
the check didn't make any sense...
This commit is contained in:
Marcus Huewe 2016-08-18 01:24:34 +02:00
parent 556b101076
commit e7dc0c740a

View File

@ -5525,10 +5525,6 @@ def get_package_results(apiurl, project, package, wait=False, *args, **kwargs):
elif result.get('code') in waiting_states:
waiting = True
break
elif (result.get('code') == 'succeeded'
and result.get('repostate') != 'published'):
waiting = True
break
if not wait or not waiting:
break