Merge remote-tracking branch 'pmp-p/patch-14' into pmpp_test

ports/javascript/modules/imp.py

@@ -2,6 +2,9 @@ import sys
 import builtins
 import types
 
+# keep the builtin function accessible in this module and via imp.__import__
+__import__ = __import__
+
 # Deprecated since version 3.4: Use types.ModuleType instead.
 # but micropython aims toward full 3.4
 
@@ -10,50 +13,16 @@ def new_module(name):
     return types.ModuleType(name)
 
 
-
-def load_module(module, *argv):
-    m = new_module(module)
-    if m:
-        import embed
-        file = '/assets/%s.py' % module
-        mroot = module.split('.')[0]
-        m = sys.modules[mroot]
-        runf(file, module=embed.vars(m), patch='\n\n__file__=%r\n' % file )
-        globals()[mroot] = m
-        return m
-
-try:
-    vars
-except:
-    #wasm port
-    import embed
-    builtins.vars = embed.vars
-
-try:
-    vars
-except:
-    print(""" This could provide a workaround for :\r
-    https://github.com/pmp-p/micropython-ports-wasm/issues/5\r
-use imp.load_module(modulename) to load modules from /assets/*.py
-""")
-    raise ImportError("ERROR: this build has no vars() support at all")
-
-
-# keep the builtin function accessible in this module and from imp.__import__
-__import__ = __import__
-
-def importer(name,*argv):
+# not spaghetti
+def importer(name,*argv,**kw):
     global __import__
-    if sys.modules.get(name,None) is None:
-        print("import %s" % name,argv)
     try:
         return __import__(name,*argv)
     except ImportError:
         pass
 
-
     file = ':{0}.py'.format(name)
-    print("trying to go online for",file)
+    print("INFO: getting online version of",file)
     # todo open the file via open() or raise importerror
     try:
         code = open(file,'r').read()
@@ -73,7 +42,12 @@ def importer(name,*argv):
         raise
 
     # execute it in its own empty namespace.
-    ns = vars(mod)
+    try:
+        ns = vars(mod)
+    except:
+        print("WARNING: this python implementation lacks vars()")
+        ns = mod.__dict__
+
 
     try:
         exec( code, ns, ns)
@@ -86,14 +60,6 @@ def importer(name,*argv):
     sys.modules[name] = mod
     return mod
 
-
-
-def reload(name):
-    if sys.modules.get(name,None):
-        del sys.modules[name]
-    return importError(name)
-
-
 # install hook
 builtins.__import__ = importer
 print("__import__ is now", importer)

ports/javascript/modules/imp_empty_pivot_module.py

@@ -0,0 +1 @@
+__dict__ = globals()

ports/javascript/modules/imp_pivot.py

@@ -0,0 +1,5 @@
+__name__ = imp.pivot_name
+__file__ = imp.pivot_file
+print('pivot',__name__,__file__, globals())
+
+exec( compile( imp.pivot_code, __file__, 'exec') , globals(), globals() )

ports/javascript/modules/site.py

@@ -0,0 +1,10 @@
+import sys
+import builtins
+
+for name in ("sys", "__main__", "builtins"):
+    sys.modules[name] = __import__(name)
+
+import imp
+
+builtins.imp = imp
+

ports/javascript/modules/types.py

@@ -1,4 +1,5 @@
 import sys
+import imp
 
 
 # not thread safe
@@ -9,12 +10,14 @@ def ModuleType(name):
         return sys.modules[name]
 
     # get a new fresh module
-    import imp_empty_pivot_module as pivot
+    # be sure to use the builtin func
+    pivot = imp.__import__('imp_empty_pivot_module') #
     # low risk, who would call his module like that ?
     del sys.modules['imp_empty_pivot_module']
 
     #still unknown at this time
-    del pivot.__file__
+    if hasattr(pivot,'__file__'):
+        del pivot.__file__
 
     pivot.__name__ = name