Message212629
Ah, I think I see what may have happened - when we finally switched all the stdlib importers over to PEP 302 (and consistently set __loader__), the pkgutil.get_loader fallback to pkgutil.find_loader stopped being exercised by the test suite (it's currently missing lower level unit tests), and so we missed that it was still calling a deprecated API. (And we don't currently have any CI set up to warn us when coverage of a module goes backwards)
I'll add some new tests to get coverage and change it to use importlib.util.find_spec instead. |
|
Date |
User |
Action |
Args |
2014-03-03 11:31:56 | ncoghlan | set | recipients:
+ ncoghlan, brett.cannon, larry, aronacher, Arfrever, eric.snow |
2014-03-03 11:31:56 | ncoghlan | set | messageid: <1393846316.39.0.47591282014.issue20839@psf.upfronthosting.co.za> |
2014-03-03 11:31:56 | ncoghlan | link | issue20839 messages |
2014-03-03 11:31:55 | ncoghlan | create | |
|