X-Git-Url: https://jfr.im/git/erebus.git/blobdiff_plain/9221e6f6b1e364c61812d39c73ad6ea41fd2c651..52c80cff91dce5fa14ae903c6c2bf20533ccca46:/ctlmod.py

diff --git a/ctlmod.py b/ctlmod.py
index 09ce8fa..3e30892 100644
--- a/ctlmod.py
+++ b/ctlmod.py
@@ -1,15 +1,16 @@
 # Erebus IRC bot - Author: John Runyon
+# vim: fileencoding=utf-8
 # module loading/unloading/tracking code
 
 from __future__ import print_function
 
-import sys, time, importlib
+import sys, time, importlib, traceback
 import modlib
 
 if sys.version_info.major >= 3:
-	from importlib import reload
+	from importlib import reload # reload is only available as a global in Py2, only in importlib in Py3
 else:
-	importlib.invalidate_caches = lambda: None
+	importlib.invalidate_caches = lambda: None # invalidate_caches doesn't exist in Py2
 
 modules = {}
 dependents = {}
@@ -30,6 +31,8 @@ def load(parent, modname, dependent=False):
 			print("failed: %s)" % (modstatus), end=' ')
 		else:
 			print("failed: %s." % (modstatus))
+			if isinstance(modstatus, modlib.error) and isinstance(modstatus.errormsg, BaseException):
+				traceback.print_exception(modstatus.errormsg)
 	elif modstatus == True:
 		if dependent:
 			print("OK)", end=' ')
@@ -77,15 +80,17 @@ def _load(parent, modname, dependent=False):
 			if bool(int(parent.cfg.get('autoloads', dep, default=1))):
 				if dep not in modules:
 					depret = load(parent, dep, dependent=True)
-					if depret is not None:
-						if not depret:
-							successstatus.append("softdep %s failed" % (dep))
+					if depret is not None and not depret:
+						successstatus.append("softdep %s failed" % (dep))
 			else:
 				successstatus.append("softdep %s disabled" % (dep))
 			#swallow errors loading - softdeps are preferred, not required
 
 
-		ret = mod.modstart(parent)
+		try:
+			ret = mod.modstart(parent)
+		except Exception as e:
+			return modlib.error(e)
 		if ret is None:
 			ret = True
 		if not ret: