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. _______________________________________________ Reportbug-maint mailing list Reportbug-maint@lists.alioth.debian.org http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/reportbug-maint
