Message121764
Looks good to me, with one exception:
if temp_dir_abspath.startswith('./'):
Wouldn’t this be better:
if not _os.path.isabs(temp_dir_abspath)
(P.S. file is not a builtin anymore in 3.x, it’s used for example as an argument to the print function.) |
|
Date |
User |
Action |
Args |
2010-11-20 20:43:47 | eric.araujo | set | recipients:
+ eric.araujo, brett.cannon, roysmith, pitrou, dstanek, r.david.murray, jesstess, thomas.holmes, BreamoreBoy, elesbom |
2010-11-20 20:43:47 | eric.araujo | set | messageid: <1290285827.54.0.337966582533.issue7325@psf.upfronthosting.co.za> |
2010-11-20 20:43:41 | eric.araujo | link | issue7325 messages |
2010-11-20 20:43:41 | eric.araujo | create | |
|