compare_pkglist: fix return code handling introduced by 2e339acffb03b6786c902c2ebd250d1c3720e315
This commit is contained in:
parent
af2471ef6c
commit
9c852461d4
@ -80,10 +80,13 @@ class CompareList(object):
|
|||||||
u = makeurl(self.apiurl, ['source', project, package], query=query)
|
u = makeurl(self.apiurl, ['source', project, package], query=query)
|
||||||
root = ET.parse(http_GET(u)).getroot()
|
root = ET.parse(http_GET(u)).getroot()
|
||||||
links = root.findall('linkinfo/linked')
|
links = root.findall('linkinfo/linked')
|
||||||
|
if links is None:
|
||||||
|
return False
|
||||||
|
|
||||||
for linked in links:
|
for linked in links:
|
||||||
if linked.get('project') == project and linked.get('package').startswith("%s." % package):
|
if linked.get('project') == project and linked.get('package').startswith("%s." % package):
|
||||||
return False
|
return False
|
||||||
return links
|
return True
|
||||||
|
|
||||||
def check_diff(self, package, old_prj, new_prj):
|
def check_diff(self, package, old_prj, new_prj):
|
||||||
logging.debug('checking %s ...' % package)
|
logging.debug('checking %s ...' % package)
|
||||||
@ -158,7 +161,7 @@ class CompareList(object):
|
|||||||
if pkg not in target:
|
if pkg not in target:
|
||||||
# ignore the second specfile package
|
# ignore the second specfile package
|
||||||
linked = self.is_linked_package(self.old_prj, pkg)
|
linked = self.is_linked_package(self.old_prj, pkg)
|
||||||
if linked is not None:
|
if linked:
|
||||||
continue
|
continue
|
||||||
|
|
||||||
if self.existin:
|
if self.existin:
|
||||||
|
Loading…
x
Reference in New Issue
Block a user