id,summary,reporter,owner,description,type,status,priority,milestone,component,resolution,keywords,cc,branch,branch_author,launchpad_bug
4085,tap2rpm should not use os.system for running other commands,TimAllen,,"Currently tap2rpm uses os.system to call various commands including tar and rpmbuild, which has two issues:

 * Little attempt has been made at quoting, which means a single `$` or `""` could make the whole thing blow up.
 * The entire, horribly verbose rpm build process gets dumped to stdout.

tap2rpm should be updated to use Python's `popen2` or `subprocess` modules, or even Twisted's own t.i.utils.getProcessOutputAndValue(). If the return value is zero, the process has probably been successful and the output can be discarded; otherwise the output probably contains useful error messages and should be printed.",enhancement,closed,normal,,core,fixed,,,,,