yudaike/openSUSE-release-tools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Just findall linked tag to instead of search sourceinfo

Browse Source 
In this case, sourceinfo is not sub-element but root.
This commit is contained in:
Max Lin 2016-11-30 18:21:04 +08:00
parent 92716f4671
commit 982bd86c6b
1 changed files with 13 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
osc-check_source.py
View File

@ -370,20 +370,19 @@ def _checker_one_request(self, rq, opts):
return
# decline the delete request against linked package
for si in root.findall('sourceinfo'):
links = si.findall('linked')
if links is None or len(links) == 0:
self._checker_change_review_state(opts, id_, 'accepted',
by_group='factory-auto',
message="Unchecked request type %s" % _type)
else:
linked = links[0]
lprj = linked.get('project')
lpkg = linked.get('package')
msg = "This is an incorrect request, it's a linked package to %s/%s" % (lprj, lpkg)
self._checker_change_review_state(opts, id_, 'declined',
by_group='factory-auto',
message=msg)
links = root.findall('linked')
if links is None or len(links) == 0:
self._checker_change_review_state(opts, id_, 'accepted',
by_group='factory-auto',
message="Unchecked request type %s" % _type)
else:
linked = links[0]
lprj = linked.get('project')
lpkg = linked.get('package')
msg = "This is an incorrect request, it's a linked package to %s/%s" % (lprj, lpkg)
self._checker_change_review_state(opts, id_, 'declined',
by_group='factory-auto',
message=msg)
else:
self._checker_change_review_state(opts, id_, 'accepted',
by_group='factory-auto',