Message272677
Thanks for the review! I'll probably update the patch next week based on your feedback (which I agree with).
As for error messages and tests, they exist in my patches on issues dependent on this one (e.g. the tests included in issues #27524 and #27182 which pull in Jelle's test from his original patch); I have been doing all of my os package-related changes in a single checkout and so this one isn't wholly written in isolation. |
|
Date |
User |
Action |
Args |
2016-08-14 16:00:35 | brett.cannon | set | recipients:
+ brett.cannon, ethan.furman, serhiy.storchaka, JelleZijlstra |
2016-08-14 16:00:35 | brett.cannon | set | messageid: <1471190435.62.0.776283293333.issue26027@psf.upfronthosting.co.za> |
2016-08-14 16:00:35 | brett.cannon | link | issue26027 messages |
2016-08-14 16:00:35 | brett.cannon | create | |
|