From: SourceForge.net <no...@so...> - 2007-08-28 18:05:52
|
Patches item #1783554, was opened at 2007-08-28 19:05 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=312867&aid=1783554&group_id=12867 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Philip Jenvey (pjenvey) Assigned to: Nobody/Anonymous (nobody) Summary: CPython compatible zipimporter Initial Comment: This is a full implementation of CPython's zipimporter. It replaces our ZipFileImporter, which actually predates CPython's zipimporter that stemmed from PEP 302 Notes: o ZipFileImporter was removed. It's possible we could remove SyspathArchive in the future, if we can make SyspathJavaLoader not rely on it (SyspathArchive is pretty lame) o ZipImportException is initialized during exceptions.classDictInit when built in exceptions are initialized during startup. It has to be initialized as early as possible, but has to be done with a current PyFrame (which exceptions.classDictInit fakes to create its own exceptions) o zipimporter creates the PyCode, then sets __loader__ and __path__ on its PyModule, then executes the PyCode. This is to match CPython's behavior. ZipFileImporter was lazier about this (resulting in some potential bugs). A couple imp methods were made public/split into two methods from one to support this ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=312867&aid=1783554&group_id=12867 |
From: SourceForge.net <no...@so...> - 2007-08-28 23:48:23
|
Patches item #1783554, was opened at 2007-08-28 19:05 Message generated for change (Comment added) made by pjenvey You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=312867&aid=1783554&group_id=12867 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Philip Jenvey (pjenvey) Assigned to: Nobody/Anonymous (nobody) Summary: CPython compatible zipimporter Initial Comment: This is a full implementation of CPython's zipimporter. It replaces our ZipFileImporter, which actually predates CPython's zipimporter that stemmed from PEP 302 Notes: o ZipFileImporter was removed. It's possible we could remove SyspathArchive in the future, if we can make SyspathJavaLoader not rely on it (SyspathArchive is pretty lame) o ZipImportException is initialized during exceptions.classDictInit when built in exceptions are initialized during startup. It has to be initialized as early as possible, but has to be done with a current PyFrame (which exceptions.classDictInit fakes to create its own exceptions) o zipimporter creates the PyCode, then sets __loader__ and __path__ on its PyModule, then executes the PyCode. This is to match CPython's behavior. ZipFileImporter was lazier about this (resulting in some potential bugs). A couple imp methods were made public/split into two methods from one to support this ---------------------------------------------------------------------- >Comment By: Philip Jenvey (pjenvey) Date: 2007-08-29 00:48 Message: Logged In: YES user_id=145787 Originator: YES test_zipimport.py needs some work (e.g. it should test loading $py.class instead of .pycs). I haven't fixed it yet because it's relying on the marshal module, which is broken ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=312867&aid=1783554&group_id=12867 |
From: SourceForge.net <no...@so...> - 2007-08-31 22:21:48
|
Patches item #1783554, was opened at 2007-08-28 19:05 Message generated for change (Comment added) made by pjenvey You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=312867&aid=1783554&group_id=12867 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None >Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Philip Jenvey (pjenvey) Assigned to: Nobody/Anonymous (nobody) Summary: CPython compatible zipimporter Initial Comment: This is a full implementation of CPython's zipimporter. It replaces our ZipFileImporter, which actually predates CPython's zipimporter that stemmed from PEP 302 Notes: o ZipFileImporter was removed. It's possible we could remove SyspathArchive in the future, if we can make SyspathJavaLoader not rely on it (SyspathArchive is pretty lame) o ZipImportException is initialized during exceptions.classDictInit when built in exceptions are initialized during startup. It has to be initialized as early as possible, but has to be done with a current PyFrame (which exceptions.classDictInit fakes to create its own exceptions) o zipimporter creates the PyCode, then sets __loader__ and __path__ on its PyModule, then executes the PyCode. This is to match CPython's behavior. ZipFileImporter was lazier about this (resulting in some potential bugs). A couple imp methods were made public/split into two methods from one to support this ---------------------------------------------------------------------- >Comment By: Philip Jenvey (pjenvey) Date: 2007-08-31 23:21 Message: Logged In: YES user_id=145787 Originator: YES applied in r3462-3 ---------------------------------------------------------------------- Comment By: Philip Jenvey (pjenvey) Date: 2007-08-29 00:48 Message: Logged In: YES user_id=145787 Originator: YES test_zipimport.py needs some work (e.g. it should test loading $py.class instead of .pycs). I haven't fixed it yet because it's relying on the marshal module, which is broken ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=312867&aid=1783554&group_id=12867 |