zipimport: Remove deprecated find_loader() and find_module() methods, use find_spec() instead #94379

Activity

1. vstinner

   added

   type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

   on Jun 28, 2022
2. vstinner

   added a commit that references this issue on Jul 5, 2022

   gh-94379: Remove zipimport find_loader() and find_module() methods (#…

   ...

   92bcb26
3. warsaw commented on Oct 7, 2022

   

   warsaw

   on Oct 7, 2022

   Member

   More actions

   I'm looking at removing find_loader() everywhere. @brettcannon @ericsnowcurrently
4. warsaw commented on Oct 7, 2022

   

   warsaw

   on Oct 7, 2022

   Member

   More actions

   @vstinner Shouldn't this ticket be closed given that the PR you referenced was merged?
5. vstinner commented on Oct 10, 2022

   

   vstinner

   on Oct 10, 2022

   MemberAuthor

   More actions

   @vstinner Shouldn't this ticket be closed given that the PR you referenced was merged?

   First my PR #94380 also tried to remove load_module() but I got issues, so 92bcb26 only removes find_loader() and find_module() methods.

   This issue title is only "zipimport: Remove deprecated find_loader() and find_module() methods", so yep, I'm fine with closing it.
6. vstinner

   closed this as completedon Oct 10, 2022
7. warsaw commented on Oct 10, 2022

   

   warsaw

   on Oct 10, 2022

   Member

   More actions

   Thanks @vstinner. When @brettcannon and I talked about it, we noticed that load_module() only raised ImportWarning in 3.10, so in 3.12 it will raise DeprecationWarning and will be removed in 3.14.
8. vstinner commented on Oct 10, 2022

   

   vstinner

   on Oct 10, 2022

   MemberAuthor

   More actions

   Ah? I only looked at zipimport.py code:

       # Load and return the module named by 'fullname'.
       def load_module(self, fullname):
           """load_module(fullname) -> module.
   
           Load the module specified by 'fullname'. 'fullname' must be the
           fully qualified (dotted) module name. It returns the imported
           module, or raises ZipImportError if it could not be imported.
   
           Deprecated since Python 3.10. Use exec_module() instead.
           """
           msg = ("zipimport.zipimporter.load_module() is deprecated and slated for "
                  "removal in Python 3.12; use exec_module() instead")
           _warnings.warn(msg, DeprecationWarning)
           (...)