Message262775
Hi Brett,
I don't think that the patch from that issue is relevant anymore.
I did take the test case that was proposed in that issue and I am attaching a fixed version here.
I did realize from the discussion that my patch probably doesn't work on Windows (I think the change itself is fine - It's just won't have the test module), I will try to get a working Windows environment and make the appropriate changes tomorrow.
Also, the other issue was also asking for built-in packages and not just built-in submodules. I already have a note about that in my original message.
Can we move forward as-is or do you want me to add support for built-in packages as well? |
|
Date |
User |
Action |
Args |
2016-04-01 21:58:08 | Daniel Shaulov | set | recipients:
+ Daniel Shaulov, brett.cannon, ncoghlan, eric.snow |
2016-04-01 21:58:08 | Daniel Shaulov | set | messageid: <1459547888.75.0.714915570448.issue26646@psf.upfronthosting.co.za> |
2016-04-01 21:58:08 | Daniel Shaulov | link | issue26646 messages |
2016-04-01 21:58:08 | Daniel Shaulov | create | |
|