diff options
| author | Scott Bahling <sbahling@suse.de> | 2018-08-03 16:43:53 +0200 |
|---|---|---|
| committer | Scott Bahling <sbahling@suse.de> | 2018-08-24 18:34:14 +0200 |
| commit | 802a888933274702247866e1f7204581f0d860ea (patch) | |
| tree | 66a0c41fa2792e42d9d6fc9cbc49fc6c05ab11b1 | |
| parent | b15e41a14f03d66f54353a7185cc57319c16ee17 (diff) | |
| download | obsapi-802a888933274702247866e1f7204581f0d860ea.tar.gz obsapi-802a888933274702247866e1f7204581f0d860ea.tar.xz obsapi-802a888933274702247866e1f7204581f0d860ea.zip | |
Generic API get now returns None on failure. Handle appropriately
| -rw-r--r-- | obsapi/core.py | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/obsapi/core.py b/obsapi/core.py index 090c953..261970e 100644 --- a/obsapi/core.py +++ b/obsapi/core.py @@ -162,6 +162,10 @@ class ObsApi(object): def package_ls(self, prj, pkg): xml = self.source.get(prj, pkg=pkg) + + if xml is None: + return (Directory(None, None, None, None), []) + d = etree.fromstring(xml) directory = Directory(d.get('name'), d.get('rev'), @@ -194,12 +198,14 @@ class ObsApi(object): return sinfo def binaries_ls(self, prj, pkg, repo, arch): + binaries = [] xml = self.build.get(prj, repo, arch, pkg) - blist = etree.fromstring(xml) - binaries = [Binary(filename=i.get('filename'), - size=i.get('size'), - mtime=i.get('mtime')) - for i in blist.findall('binary')] + if xml: + blist = etree.fromstring(xml) + binaries = [Binary(filename=i.get('filename'), + size=i.get('size'), + mtime=i.get('mtime')) + for i in blist.findall('binary')] return binaries |