[issue13645] import machinery vulnerable to timestamp collisions
report at bugs.python.org
Thu Jan 12 18:03:38 CET 2012
Brett Cannon <brett at python.org> added the comment:
LGTM (although I didn't run the unit tests and focused mainly on the importlib._bootstrap and abc changes). Only two things I would change. One is possibly deprecating path_mtime() so people don't waste time implementing it (we actually never need to remove it thanks to the ABC; otherwise we need to make sure the docs strongly state to only bother with path_stats()). The other is to say the mtime key should contain a value that is a real number (ie. float and any other numeric type that can cast to an integer). That way you get your higher resolution in the dict while still being able to use the value in writing the bytecode.
And is there any efficient way to get the stat info on a file AND its contents in a single call? If so we might want a method to support that (eg. add a 'source_bytes' key or something), but I can't think of any low-level call that supports that kind of thing.
assignee: -> pitrou
Python tracker <report at bugs.python.org>
More information about the Python-bugs-list