Message138038
Here is an updated patch with the tests refactored even further. The patch seems correct to me, and almost all the comment tests fail before the patch and pass after.
Benjamin, this patch fixes a regression relative to 2.7.1 and 3.1.3, so I'm setting it to release blocker so you can decide if it is worth putting in. netrc is perhaps not a heavily used module, but the fact that we've had several bugs reports that have seen attention and fixes from users recently indicates it *is* used, and is probably worth fixing. Up to you, though. Sorry I didn't get this in before the RCs :(
The patch is for 2.7. |
|
Date |
User |
Action |
Args |
2011-06-09 23:27:46 | r.david.murray | set | recipients:
+ r.david.murray, georg.brandl, benjamin.peterson, eric.araujo, rmstoi |
2011-06-09 23:27:46 | r.david.murray | set | messageid: <1307662066.05.0.965105086399.issue12009@psf.upfronthosting.co.za> |
2011-06-09 23:27:45 | r.david.murray | link | issue12009 messages |
2011-06-09 23:27:45 | r.david.murray | create | |
|