core/modules: get rid of set_repo uses, it was just complicating everythin

my/cfg.py

@@ -27,3 +27,5 @@ def set_repo(name: str, repo: Union[Path, str]) -> None:
 
 
 # TODO set_repo is still useful, but perhaps move this thing away to core?
+
+# TODO ok, I need to get rid of this, better to rely on regular imports

my/core/common.py

@@ -34,9 +34,9 @@ def import_from(path: PathIsh, name: str) -> types.ModuleType:
         sys.path.remove(path)
 
 
-def import_dir(path: PathIsh) -> types.ModuleType:
+def import_dir(path: PathIsh, extra: str='') -> types.ModuleType:
     p = Path(path)
-    return import_from(p.parent, p.name)
+    return import_from(p.parent, p.name + extra)
 
 
 T = TypeVar('T')

my/hypothesis.py

@@ -26,9 +26,9 @@ class hypothesis(user_config):
     def dal_module(self):
         rpath = self.hypexport
         if rpath is not None:
-            from .cfg import set_repo
-            set_repo('hypexport', rpath)
-
+            from .core.common import import_dir
+            return import_dir(rpath, '.dal')
+        else:
             import my.config.repos.hypexport.dal as dal
             return dal
 

my/instapaper.py

@@ -25,9 +25,9 @@ class instapaper(user_config):
     def dal_module(self):
         rpath = self.instapexport
         if rpath is not None:
-            from .cfg import set_repo
-            set_repo('instapexport', rpath)
-
+            from .core.common import import_dir
+            return import_dir(rpath, '.dal')
+        else:
             import my.config.repos.instapexport.dal as dal
             return dal
 

my/reddit.py

@@ -26,9 +26,9 @@ class reddit(uconfig):
     def dal_module(self) -> ModuleType:
         rpath = self.rexport
         if rpath is not None:
-            from .cfg import set_repo
-            set_repo('rexport', rpath)
-
+            from .core.common import import_dir
+            return import_dir(rpath, '.dal')
+        else:
             import my.config.repos.rexport.dal as dal
             return dal