Message205779
Thanks for the review, Antoine! Here is the updated patch. I haven't tested it on Windows yet because I want to clarify one thing.
Let's say we have this valid directory:
/tmp/@test123 <= directory
And this directory only has one valid file:
/tmp/@test123/cutecat <= file
We agree that pathlib.Path('/tmp/@test123/foo').resolve(False) => '/tmp/@test123/foo'.
But what about this case: pathlib.Path('/tmp/@test123/cutecat/foo').resolve(False)?
It should be "/tmp/@test123/cutecat" or "/tmp/@test123/cutecat/foo"? |
|
Date |
User |
Action |
Args |
2013-12-10 08:58:46 | vajrasky | set | recipients:
+ vajrasky, gvanrossum, pitrou, neologix, serhiy.storchaka |
2013-12-10 08:58:46 | vajrasky | set | messageid: <1386665926.05.0.548812691022.issue19717@psf.upfronthosting.co.za> |
2013-12-10 08:58:46 | vajrasky | link | issue19717 messages |
2013-12-10 08:58:45 | vajrasky | create | |
|