py3: use pycompat.fsdecode() to pass to imp.* functions
authorPulkit Goyal <7895pulkit@gmail.com>
Mon, 05 Dec 2016 06:46:51 +0530
changeset 30575 5ffbaba9acac
parent 30574 f8c9a7d2bbbf
child 30576 541949a10a68
py3: use pycompat.fsdecode() to pass to imp.* functions When we try to pass a bytes argument to a function from imp library, it returns TypeError as it deals with unicodes internally. So we can't use bytes with imp.* functions. Hunting through this, I found we were returning bytes path variable to loadpath() on Python 3.5 (yes most of our codebase is dealing with bytes on Python 3 especially the path variables). Passing unicode does not fails the purpose of loding the extensions and a module object is returned.
mercurial/extensions.py
--- a/mercurial/extensions.py	Tue Dec 06 17:06:39 2016 +0000
+++ b/mercurial/extensions.py	Mon Dec 05 06:46:51 2016 +0530
@@ -60,6 +60,8 @@
 def loadpath(path, module_name):
     module_name = module_name.replace('.', '_')
     path = util.normpath(util.expandpath(path))
+    module_name = pycompat.fsdecode(module_name)
+    path = pycompat.fsdecode(path)
     if os.path.isdir(path):
         # module/__init__.py style
         d, f = os.path.split(path)