[Reportbug-maint] Bug#861153: Bug#861153: reportbug: Architecture field split into two lines

Nis Martensen nis.martensen at web.de
Sun May 28 10:53:26 UTC 2017


Hey Sandro!

> should we revert to the previous behavior of get_command_output before
> introducing other regressions?

Having re-read the code again, I can't see any risk of introducing other
regressions.

Only lsb_release_info() and get_arch() directly return the result of
get_command_output(), all other callers do some parsing and do not care
about newlines at the end of the output.

The alternative you have in mind is changing get_command_output like
this, right? :

-    return subprocess.run([...]).stdout.decode([...])
+    return subprocess.run([...]).stdout.decode([...]).strip()

This will work as well. I slightly prefer my other patch, but it does
not matter much.



More information about the Reportbug-maint mailing list