Message263705
The patch looks good to me. The function was originally written to be included in the “os” module, hence the NameError. The patch should probably be fine with Mercurial, but it looks like the Reitveld review system doesn’t like it :)
What platform do you get the AttributeError with? Perhaps the function is not well covered in the test suite. |
|
Date |
User |
Action |
Args |
2016-04-19 02:53:43 | martin.panter | set | recipients:
+ martin.panter, abarry |
2016-04-19 02:53:43 | martin.panter | set | messageid: <1461034423.47.0.159561437247.issue26801@psf.upfronthosting.co.za> |
2016-04-19 02:53:43 | martin.panter | link | issue26801 messages |
2016-04-19 02:53:43 | martin.panter | create | |
|