Message194031
Hi Catherine,
I saw your patch. It looks good, except you are trying to copy the directory onto its subdirectory.
src_dir = tempfile.mkdtemp()
dst_dir = os.path.join(tempfile.mkdtemp(), 'destination')
I prefer it if you copy it to somewhere else. This would be better:
tmp_dir = self.mkdtemp()
src = os.path.join(tmp_dir, 'foo')
dst = os.path.join(tmp_dir, 'bar') |
|
Date |
User |
Action |
Args |
2013-08-01 04:35:31 | vajrasky | set | recipients:
+ vajrasky, georg.brandl, jlgijsbers, thomaswaldmann, pitrou, catherinedevlin, christian.heimes, ajaksu2, tarek, eric.araujo, daniel.ugra, mcjeff |
2013-08-01 04:35:31 | vajrasky | set | messageid: <1375331731.86.0.121049235479.issue1666318@psf.upfronthosting.co.za> |
2013-08-01 04:35:31 | vajrasky | link | issue1666318 messages |
2013-08-01 04:35:31 | vajrasky | create | |
|