mirrors/github.com_openSUSE_osc
1
0
Fork 0
mirror of https://github.com/openSUSE/osc.git synced 2024-12-27 02:16:12 +01:00
Code

Do not crash when running "osc search --binary --verbose foo"

Browse Source 
The old code does not support the --binary option in combination
with the --verbose option. Specifying --binary and --verbose at
the same time results in a crash (because the binary listing
contains no <title>...</title> element).
In order to fix this, do not try to access a <title>...</title>
element when --binary and --verbose are both specified. Instead,
in this case, include information about the repo, arch, version,
and release of the corresponding binary element.

Fixes: #933 ("osc se -v -B crash")
This commit is contained in:
Marcus Huewe 2021-07-30 21:05:05 +02:00
parent e5dda8337c
commit d7dbd1bc0b
1 changed files with 15 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
osc/commandline.py
View File

@ -7960,6 +7960,12 @@ Please submit there instead, or use --nodevelproject to force direct submission.
result.append(s)
if opts.verbose:
if opts.binary:
result.append(node.get('repository') or '-')
result.append(node.get('arch') or '-')
result.append(node.get('version') or '-')
result.append(node.get('release') or '-')
else:
title = node.findtext('title').strip()
if len(title) > 60:
title = title[:61] + '...'
@ -7995,6 +8001,10 @@ Please submit there instead, or use --nodevelproject to force direct submission.
headline.append('Rev')
headline.append('Srcmd5')
if opts.verbose:
if opts.binary:
headline.extend(['# Repository', '# Arch', '# Version',
'# Release'])
else:
headline.append('# Title')
if opts.repos_baseurl:
headline.append('# URL')