Author asvetlov
Recipients Alexander.Belopolsky, Arfrever, asvetlov, brian.curtin, csernazs, cvrebert, dstanek, eric.araujo, eric.smith, giampaolo.rodola, holdenweb, larry, ncoghlan, r.david.murray, vstinner
Date 2012-04-01.10:06:49
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1333274810.53.0.552288872847.issue7839@psf.upfronthosting.co.za>
In-reply-to
Content
After trying to make a patch I found what current test suite itself has calls like (str, shell=False), (bytes, shell=True) and (['shell command'], shell=True).

We can:
1. Implement Eric's suggestion with fixing/removing broken tests.
2. Add Pope.shell and Popen.exec as Nick propose.

I don't like former because it changes already existing contracts fixed and published by explicit unittests without strong reason. Also Nick pointed reasons why user may need existing functionality. Adding new 'safe' shell and exec classmethods as prime entrypoints for Popen make sense.
History
Date User Action Args
2012-04-01 10:06:50asvetlovsetrecipients: + asvetlov, holdenweb, csernazs, ncoghlan, vstinner, dstanek, larry, eric.smith, giampaolo.rodola, eric.araujo, Arfrever, r.david.murray, brian.curtin, cvrebert, Alexander.Belopolsky
2012-04-01 10:06:50asvetlovsetmessageid: <1333274810.53.0.552288872847.issue7839@psf.upfronthosting.co.za>
2012-04-01 10:06:49asvetlovlinkissue7839 messages
2012-04-01 10:06:49asvetlovcreate