Message157269
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. |
|
Date |
User |
Action |
Args |
2012-04-01 10:06:50 | asvetlov | set | recipients:
+ 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:50 | asvetlov | set | messageid: <1333274810.53.0.552288872847.issue7839@psf.upfronthosting.co.za> |
2012-04-01 10:06:49 | asvetlov | link | issue7839 messages |
2012-04-01 10:06:49 | asvetlov | create | |
|