Merge pull request #892 from GLolol/notify-incompatible-plugins-on-load

Owner: catch ImportError in a less confusing, friendlier way (Closes #861, Really closes #850)
This commit is contained in:
Valentin Lorentz 2014-11-01 19:29:18 +01:00
commit e925069ae9

View File

@ -194,6 +194,16 @@ class Owner(callbacks.Plugin):
s = 'Failed to load %s: no suitable database(%s).' % (name, e) s = 'Failed to load %s: no suitable database(%s).' % (name, e)
log.warning(s) log.warning(s)
except ImportError as e: except ImportError as e:
e = str(e)
if e.endswith(name):
s = 'Failed to load {0}: No plugin named {0} exists.'.format(
utils.str.dqrepr(name))
elif "No module named 'config'" in e:
s = ("Failed to load %s: This plugin may be incompatible "
"with your current Python version. If this error is appearing "
"with stock Supybot plugins, remove the stock plugins directory "
"(usually ~/Limnoria/plugins) from 'config directories.plugins'." % name)
else:
s = 'Failed to load %s: import error (%s).' % (name, e) s = 'Failed to load %s: import error (%s).' % (name, e)
log.warning(s) log.warning(s)
except Exception as e: except Exception as e:
@ -418,6 +428,9 @@ class Owner(callbacks.Plugin):
except ImportError as e: except ImportError as e:
if str(e).endswith(name): if str(e).endswith(name):
irc.error('No plugin named %s exists.' % utils.str.dqrepr(name)) irc.error('No plugin named %s exists.' % utils.str.dqrepr(name))
elif "No module named 'config'" in str(e):
irc.error('This plugin may be incompatible with your current Python '
'version. Try running 2to3 on it.')
else: else:
irc.error(str(e)) irc.error(str(e))
return return