From 56a6799deb0fdcd77434baf85d8629450df783fc Mon Sep 17 00:00:00 2001 From: James McCoy Date: Thu, 10 Oct 2013 18:54:50 -0400 Subject: [PATCH 01/17] commands: Consolidate state.channel checks into getChannel Signed-off-by: James McCoy --- src/commands.py | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/src/commands.py b/src/commands.py index ca079d7a0..ef3d2c043 100644 --- a/src/commands.py +++ b/src/commands.py @@ -303,8 +303,7 @@ def getNetworkIrc(irc, msg, args, state, errorIfNoMatch=False): state.args.append(irc) def getHaveOp(irc, msg, args, state, action='do that'): - if not state.channel: - getChannel(irc, msg, args, state) + getChannel(irc, msg, args, state) if state.channel not in irc.state.channels: state.error('I\'m not even in %s.' % state.channel, Raise=True) if not irc.state.channels[state.channel].isOp(irc.nick): @@ -329,8 +328,7 @@ def getHostmask(irc, msg, args, state): def getBanmask(irc, msg, args, state): getHostmask(irc, msg, args, state) - if not state.channel: - getChannel(irc, msg, args, state) + getChannel(irc, msg, args, state) channel = state.channel banmaskstyle = conf.supybot.protocols.irc.banmask state.args[-1] = banmaskstyle.makeBanmask(state.args[-1]) @@ -406,6 +404,8 @@ def getSeenNick(irc, msg, args, state, errmsg=None): state.error(errmsg, Raise=True) def getChannel(irc, msg, args, state): + if state.channel: + return if args and irc.isChannel(args[0]): channel = args.pop(0) elif irc.isChannel(msg.args[0]): @@ -437,8 +437,7 @@ def getChannelDb(irc, msg, args, state, **kwargs): state.args.append(channel) def inChannel(irc, msg, args, state): - if not state.channel: - getChannel(irc, msg, args, state) + getChannel(irc, msg, args, state) if state.channel not in irc.state.channels: state.error('I\'m not in %s.' % state.channel, Raise=True) @@ -478,8 +477,7 @@ def getChannelOrNone(irc, msg, args, state): state.args.append(None) def checkChannelCapability(irc, msg, args, state, cap): - if not state.channel: - getChannel(irc, msg, args, state) + getChannel(irc, msg, args, state) cap = ircdb.canonicalCapability(cap) cap = ircdb.makeChannelCapability(state.channel, cap) if not ircdb.checkCapability(msg.prefix, cap): From bf48d521f26daa3b1523d5a3af24a5e255d615fb Mon Sep 17 00:00:00 2001 From: James McCoy Date: Thu, 10 Oct 2013 20:37:12 -0400 Subject: [PATCH 02/17] Use generic setuptools instead of requiring distribute Signed-off-by: James McCoy --- INSTALL | 6 +- distribute_setup.py | 515 -------------------------------------------- setup.py | 15 +- src/version.py | 2 +- 4 files changed, 10 insertions(+), 528 deletions(-) delete mode 100644 distribute_setup.py diff --git a/INSTALL b/INSTALL index 5017e4c68..b3bd5d224 100644 --- a/INSTALL +++ b/INSTALL @@ -1,7 +1,9 @@ Common - First things first: Supybot *requires* at least Python 2.6. There -ain't no getting around it. You can get it from http://www.python.org/. + First things first: Supybot *requires* at least Python 2.6 and +setuptools. There ain't no getting around it. You can get Python from +http://www.python.org/ and setuptools from +https://pypi.python.org/pypi/setuptools. Recommended Software diff --git a/distribute_setup.py b/distribute_setup.py deleted file mode 100644 index 8f5b0637b..000000000 --- a/distribute_setup.py +++ /dev/null @@ -1,515 +0,0 @@ -#!python -"""Bootstrap distribute installation - -If you want to use setuptools in your package's setup.py, just include this -file in the same directory with it, and add this to the top of your setup.py:: - - from distribute_setup import use_setuptools - use_setuptools() - -If you want to require a specific version of setuptools, set a download -mirror, or use an alternate download directory, you can do so by supplying -the appropriate options to ``use_setuptools()``. - -This file can also be run as a script to install or upgrade setuptools. -""" -import os -import sys -import time -import fnmatch -import tempfile -import tarfile -from distutils import log - -try: - from site import USER_SITE -except ImportError: - USER_SITE = None - -try: - import subprocess - - def _python_cmd(*args): - args = (sys.executable,) + args - return subprocess.call(args) == 0 - -except ImportError: - # will be used for python 2.3 - def _python_cmd(*args): - args = (sys.executable,) + args - # quoting arguments if windows - if sys.platform == 'win32': - def quote(arg): - if ' ' in arg: - return '"%s"' % arg - return arg - args = [quote(arg) for arg in args] - return os.spawnl(os.P_WAIT, sys.executable, *args) == 0 - -DEFAULT_VERSION = "0.6.28" -DEFAULT_URL = "http://pypi.python.org/packages/source/d/distribute/" -SETUPTOOLS_FAKED_VERSION = "0.6c11" - -SETUPTOOLS_PKG_INFO = """\ -Metadata-Version: 1.0 -Name: setuptools -Version: %s -Summary: xxxx -Home-page: xxx -Author: xxx -Author-email: xxx -License: xxx -Description: xxx -""" % SETUPTOOLS_FAKED_VERSION - - -def _install(tarball, install_args=()): - # extracting the tarball - tmpdir = tempfile.mkdtemp() - log.warn('Extracting in %s', tmpdir) - old_wd = os.getcwd() - try: - os.chdir(tmpdir) - tar = tarfile.open(tarball) - _extractall(tar) - tar.close() - - # going in the directory - subdir = os.path.join(tmpdir, os.listdir(tmpdir)[0]) - os.chdir(subdir) - log.warn('Now working in %s', subdir) - - # installing - log.warn('Installing Distribute') - if not _python_cmd('setup.py', 'install', *install_args): - log.warn('Something went wrong during the installation.') - log.warn('See the error message above.') - finally: - os.chdir(old_wd) - - -def _build_egg(egg, tarball, to_dir): - # extracting the tarball - tmpdir = tempfile.mkdtemp() - log.warn('Extracting in %s', tmpdir) - old_wd = os.getcwd() - try: - os.chdir(tmpdir) - tar = tarfile.open(tarball) - _extractall(tar) - tar.close() - - # going in the directory - subdir = os.path.join(tmpdir, os.listdir(tmpdir)[0]) - os.chdir(subdir) - log.warn('Now working in %s', subdir) - - # building an egg - log.warn('Building a Distribute egg in %s', to_dir) - _python_cmd('setup.py', '-q', 'bdist_egg', '--dist-dir', to_dir) - - finally: - os.chdir(old_wd) - # returning the result - log.warn(egg) - if not os.path.exists(egg): - raise IOError('Could not build the egg.') - - -def _do_download(version, download_base, to_dir, download_delay): - egg = os.path.join(to_dir, 'distribute-%s-py%d.%d.egg' - % (version, sys.version_info[0], sys.version_info[1])) - if not os.path.exists(egg): - tarball = download_setuptools(version, download_base, - to_dir, download_delay) - _build_egg(egg, tarball, to_dir) - sys.path.insert(0, egg) - import setuptools - setuptools.bootstrap_install_from = egg - - -def use_setuptools(version=DEFAULT_VERSION, download_base=DEFAULT_URL, - to_dir=os.curdir, download_delay=15, no_fake=True): - # making sure we use the absolute path - to_dir = os.path.abspath(to_dir) - was_imported = 'pkg_resources' in sys.modules or \ - 'setuptools' in sys.modules - try: - try: - import pkg_resources - if not hasattr(pkg_resources, '_distribute'): - if not no_fake: - _fake_setuptools() - raise ImportError - except ImportError: - return _do_download(version, download_base, to_dir, download_delay) - try: - pkg_resources.require("distribute>=" + version) - return - except pkg_resources.VersionConflict: - e = sys.exc_info()[1] - if was_imported: - sys.stderr.write( - "The required version of distribute (>=%s) is not available,\n" - "and can't be installed while this script is running. Please\n" - "install a more recent version first, using\n" - "'easy_install -U distribute'." - "\n\n(Currently using %r)\n" % (version, e.args[0])) - sys.exit(2) - else: - del pkg_resources, sys.modules['pkg_resources'] # reload ok - return _do_download(version, download_base, to_dir, - download_delay) - except pkg_resources.DistributionNotFound: - return _do_download(version, download_base, to_dir, - download_delay) - finally: - if not no_fake: - _create_fake_setuptools_pkg_info(to_dir) - - -def download_setuptools(version=DEFAULT_VERSION, download_base=DEFAULT_URL, - to_dir=os.curdir, delay=15): - """Download distribute from a specified location and return its filename - - `version` should be a valid distribute version number that is available - as an egg for download under the `download_base` URL (which should end - with a '/'). `to_dir` is the directory where the egg will be downloaded. - `delay` is the number of seconds to pause before an actual download - attempt. - """ - # making sure we use the absolute path - to_dir = os.path.abspath(to_dir) - try: - from urllib.request import urlopen - except ImportError: - from urllib2 import urlopen - tgz_name = "distribute-%s.tar.gz" % version - url = download_base + tgz_name - saveto = os.path.join(to_dir, tgz_name) - src = dst = None - if not os.path.exists(saveto): # Avoid repeated downloads - try: - log.warn("Downloading %s", url) - src = urlopen(url) - # Read/write all in one block, so we don't create a corrupt file - # if the download is interrupted. - data = src.read() - dst = open(saveto, "wb") - dst.write(data) - finally: - if src: - src.close() - if dst: - dst.close() - return os.path.realpath(saveto) - - -def _no_sandbox(function): - def __no_sandbox(*args, **kw): - try: - from setuptools.sandbox import DirectorySandbox - if not hasattr(DirectorySandbox, '_old'): - def violation(*args): - pass - DirectorySandbox._old = DirectorySandbox._violation - DirectorySandbox._violation = violation - patched = True - else: - patched = False - except ImportError: - patched = False - - try: - return function(*args, **kw) - finally: - if patched: - DirectorySandbox._violation = DirectorySandbox._old - del DirectorySandbox._old - - return __no_sandbox - - -def _patch_file(path, content): - """Will backup the file then patch it""" - existing_content = open(path).read() - if existing_content == content: - # already patched - log.warn('Already patched.') - return False - log.warn('Patching...') - _rename_path(path) - f = open(path, 'w') - try: - f.write(content) - finally: - f.close() - return True - -_patch_file = _no_sandbox(_patch_file) - - -def _same_content(path, content): - return open(path).read() == content - - -def _rename_path(path): - new_name = path + '.OLD.%s' % time.time() - log.warn('Renaming %s into %s', path, new_name) - os.rename(path, new_name) - return new_name - - -def _remove_flat_installation(placeholder): - if not os.path.isdir(placeholder): - log.warn('Unkown installation at %s', placeholder) - return False - found = False - for file in os.listdir(placeholder): - if fnmatch.fnmatch(file, 'setuptools*.egg-info'): - found = True - break - if not found: - log.warn('Could not locate setuptools*.egg-info') - return - - log.warn('Removing elements out of the way...') - pkg_info = os.path.join(placeholder, file) - if os.path.isdir(pkg_info): - patched = _patch_egg_dir(pkg_info) - else: - patched = _patch_file(pkg_info, SETUPTOOLS_PKG_INFO) - - if not patched: - log.warn('%s already patched.', pkg_info) - return False - # now let's move the files out of the way - for element in ('setuptools', 'pkg_resources.py', 'site.py'): - element = os.path.join(placeholder, element) - if os.path.exists(element): - _rename_path(element) - else: - log.warn('Could not find the %s element of the ' - 'Setuptools distribution', element) - return True - -_remove_flat_installation = _no_sandbox(_remove_flat_installation) - - -def _after_install(dist): - log.warn('After install bootstrap.') - placeholder = dist.get_command_obj('install').install_purelib - _create_fake_setuptools_pkg_info(placeholder) - - -def _create_fake_setuptools_pkg_info(placeholder): - if not placeholder or not os.path.exists(placeholder): - log.warn('Could not find the install location') - return - pyver = '%s.%s' % (sys.version_info[0], sys.version_info[1]) - setuptools_file = 'setuptools-%s-py%s.egg-info' % \ - (SETUPTOOLS_FAKED_VERSION, pyver) - pkg_info = os.path.join(placeholder, setuptools_file) - if os.path.exists(pkg_info): - log.warn('%s already exists', pkg_info) - return - - if not os.access(pkg_info, os.W_OK): - log.warn("Don't have permissions to write %s, skipping", pkg_info) - - log.warn('Creating %s', pkg_info) - f = open(pkg_info, 'w') - try: - f.write(SETUPTOOLS_PKG_INFO) - finally: - f.close() - - pth_file = os.path.join(placeholder, 'setuptools.pth') - log.warn('Creating %s', pth_file) - f = open(pth_file, 'w') - try: - f.write(os.path.join(os.curdir, setuptools_file)) - finally: - f.close() - -_create_fake_setuptools_pkg_info = _no_sandbox( - _create_fake_setuptools_pkg_info -) - - -def _patch_egg_dir(path): - # let's check if it's already patched - pkg_info = os.path.join(path, 'EGG-INFO', 'PKG-INFO') - if os.path.exists(pkg_info): - if _same_content(pkg_info, SETUPTOOLS_PKG_INFO): - log.warn('%s already patched.', pkg_info) - return False - _rename_path(path) - os.mkdir(path) - os.mkdir(os.path.join(path, 'EGG-INFO')) - pkg_info = os.path.join(path, 'EGG-INFO', 'PKG-INFO') - f = open(pkg_info, 'w') - try: - f.write(SETUPTOOLS_PKG_INFO) - finally: - f.close() - return True - -_patch_egg_dir = _no_sandbox(_patch_egg_dir) - - -def _before_install(): - log.warn('Before install bootstrap.') - _fake_setuptools() - - -def _under_prefix(location): - if 'install' not in sys.argv: - return True - args = sys.argv[sys.argv.index('install') + 1:] - for index, arg in enumerate(args): - for option in ('--root', '--prefix'): - if arg.startswith('%s=' % option): - top_dir = arg.split('root=')[-1] - return location.startswith(top_dir) - elif arg == option: - if len(args) > index: - top_dir = args[index + 1] - return location.startswith(top_dir) - if arg == '--user' and USER_SITE is not None: - return location.startswith(USER_SITE) - return True - - -def _fake_setuptools(): - log.warn('Scanning installed packages') - try: - import pkg_resources - except ImportError: - # we're cool - log.warn('Setuptools or Distribute does not seem to be installed.') - return - ws = pkg_resources.working_set - try: - setuptools_dist = ws.find( - pkg_resources.Requirement.parse('setuptools', replacement=False) - ) - except TypeError: - # old distribute API - setuptools_dist = ws.find( - pkg_resources.Requirement.parse('setuptools') - ) - - if setuptools_dist is None: - log.warn('No setuptools distribution found') - return - # detecting if it was already faked - setuptools_location = setuptools_dist.location - log.warn('Setuptools installation detected at %s', setuptools_location) - - # if --root or --preix was provided, and if - # setuptools is not located in them, we don't patch it - if not _under_prefix(setuptools_location): - log.warn('Not patching, --root or --prefix is installing Distribute' - ' in another location') - return - - # let's see if its an egg - if not setuptools_location.endswith('.egg'): - log.warn('Non-egg installation') - res = _remove_flat_installation(setuptools_location) - if not res: - return - else: - log.warn('Egg installation') - pkg_info = os.path.join(setuptools_location, 'EGG-INFO', 'PKG-INFO') - if (os.path.exists(pkg_info) and - _same_content(pkg_info, SETUPTOOLS_PKG_INFO)): - log.warn('Already patched.') - return - log.warn('Patching...') - # let's create a fake egg replacing setuptools one - res = _patch_egg_dir(setuptools_location) - if not res: - return - log.warn('Patched done.') - _relaunch() - - -def _relaunch(): - log.warn('Relaunching...') - # we have to relaunch the process - # pip marker to avoid a relaunch bug - _cmd = ['-c', 'install', '--single-version-externally-managed'] - if sys.argv[:3] == _cmd: - sys.argv[0] = 'setup.py' - args = [sys.executable] + sys.argv - sys.exit(subprocess.call(args)) - - -def _extractall(self, path=".", members=None): - """Extract all members from the archive to the current working - directory and set owner, modification time and permissions on - directories afterwards. `path' specifies a different directory - to extract to. `members' is optional and must be a subset of the - list returned by getmembers(). - """ - import copy - import operator - from tarfile import ExtractError - directories = [] - - if members is None: - members = self - - for tarinfo in members: - if tarinfo.isdir(): - # Extract directories with a safe mode. - directories.append(tarinfo) - tarinfo = copy.copy(tarinfo) - tarinfo.mode = 448 # decimal for oct 0700 - self.extract(tarinfo, path) - - # Reverse sort directories. - if sys.version_info < (2, 4): - def sorter(dir1, dir2): - return cmp(dir1.name, dir2.name) - directories.sort(sorter) - directories.reverse() - else: - directories.sort(key=operator.attrgetter('name'), reverse=True) - - # Set correct owner, mtime and filemode on directories. - for tarinfo in directories: - dirpath = os.path.join(path, tarinfo.name) - try: - self.chown(tarinfo, dirpath) - self.utime(tarinfo, dirpath) - self.chmod(tarinfo, dirpath) - except ExtractError: - e = sys.exc_info()[1] - if self.errorlevel > 1: - raise - else: - self._dbg(1, "tarfile: %s" % e) - - -def _build_install_args(argv): - install_args = [] - user_install = '--user' in argv - if user_install and sys.version_info < (2, 6): - log.warn("--user requires Python 2.6 or later") - raise SystemExit(1) - if user_install: - install_args.append('--user') - return install_args - - -def main(argv, version=DEFAULT_VERSION): - """Install or upgrade setuptools and EasyInstall""" - tarball = download_setuptools() - _install(tarball, _build_install_args(argv)) - - -if __name__ == '__main__': - main(sys.argv[1:]) diff --git a/setup.py b/setup.py index 590a493d7..7c6addd0e 100644 --- a/setup.py +++ b/setup.py @@ -42,14 +42,6 @@ import os.path import textwrap from src.version import version - -try: - from distribute_setup import use_setuptools -except ImportError: - pass -else: - use_setuptools(version='0.6c9') - from setuptools import setup def normalizeWhitespace(s): @@ -84,9 +76,12 @@ setup( name='supybot', version=version, author='Jeremy Fincher', - url='http://sourceforge.net/projects/supybot/', author_email='jemfinch@supybot.com', - download_url='http://www.sf.net/project/showfiles.php?group_id=58965', + maintainer='James McCoy', + maintainer_email='jamessan@users.sourceforge.net', + url='https://sourceforge.net/projects/supybot/', + download_url='https://sourceforge.net/projects/supybot/files/', + platforms=['linux', 'linux2', 'win32', 'cygwin', 'darwin'], description='A flexible and extensible Python IRC bot and framework.', long_description=normalizeWhitespace("""A robust, full-featured Python IRC bot with a clean and flexible plugin API. Equipped with a complete ACL diff --git a/src/version.py b/src/version.py index df82bea75..2f7a1082d 100644 --- a/src/version.py +++ b/src/version.py @@ -1,3 +1,3 @@ """stick the various versioning attributes in here, so we only have to change them once.""" -version = '0.83.4.1+git' +version = '0.84.0-dev' From cfe71c4f55951857f6c0e69ecfee19d3a5692788 Mon Sep 17 00:00:00 2001 From: James McCoy Date: Thu, 10 Oct 2013 20:56:19 -0400 Subject: [PATCH 03/17] Add a .mailmap to coerce my old name to my new --- .mailmap | 1 + 1 file changed, 1 insertion(+) create mode 100644 .mailmap diff --git a/.mailmap b/.mailmap new file mode 100644 index 000000000..cfb26288b --- /dev/null +++ b/.mailmap @@ -0,0 +1 @@ +James McCoy From 4d677dbcf5f2f4a74219eccd97362a12d98ecb62 Mon Sep 17 00:00:00 2001 From: James McCoy Date: Sun, 13 Oct 2013 10:00:33 -0400 Subject: [PATCH 04/17] Seen: Fix any to work with wildcards when protocols.irc.strictRfc is true Signed-off-by: James McCoy --- plugins/Seen/plugin.py | 4 ++-- plugins/Seen/test.py | 25 +++++++++++++++++++++---- 2 files changed, 23 insertions(+), 6 deletions(-) diff --git a/plugins/Seen/plugin.py b/plugins/Seen/plugin.py index d330d45c4..08f97baa4 100644 --- a/plugins/Seen/plugin.py +++ b/plugins/Seen/plugin.py @@ -1,6 +1,6 @@ ### # Copyright (c) 2002-2004, Jeremiah Fincher -# Copyright (c) 2010-2011, James McCoy +# Copyright (c) 2010-2011, 2013, James McCoy # All rights reserved. # # Redistribution and use in source and binary forms, with or without @@ -243,7 +243,7 @@ class Seen(callbacks.Plugin): else: self._last(irc, channel, any=True) any = wrap(any, ['channel', getopts({'user': 'otherUser'}), - additional('nick')]) + additional('something')]) def _last(self, irc, channel, any=False): if any: diff --git a/plugins/Seen/test.py b/plugins/Seen/test.py index 77eaeb37d..f27af30c5 100644 --- a/plugins/Seen/test.py +++ b/plugins/Seen/test.py @@ -1,5 +1,6 @@ ### # Copyright (c) 2002-2004, Jeremiah Fincher +# Copyright (c) 2013, James McCoy # All rights reserved. # # Redistribution and use in source and binary forms, with or without @@ -55,6 +56,16 @@ class ChannelDBTestCase(ChannelPluginTestCase): prefix=self.prefix)) self.assertRegexp('seen any %s' % self.nick, '^%s was last seen' % self.nick) + orig = conf.supybot.protocols.irc.strictRfc() + try: + for state in (True, False): + conf.supybot.protocols.irc.strictRfc.setValue(state) + for wildcard in self.wildcardTest: + self.assertRegexp('seen any %s' % wildcard, + '^%s was last seen' % self.nick) + self.assertRegexp('seen any bar*', '^I haven\'t seen anyone matching') + finally: + conf.supybot.protocols.irc.strictRfc.setValue(orig) def testSeen(self): self.assertNotError('seen last') @@ -64,10 +75,16 @@ class ChannelDBTestCase(ChannelPluginTestCase): self.failUnless(self.nick.upper() in m.args[1]) self.assertRegexp('seen user %s' % self.nick, '^%s was last seen' % self.nick) - for wildcard in self.wildcardTest: - self.assertRegexp('seen %s' % wildcard, - '^%s was last seen' % self.nick) - self.assertRegexp('seen bar*', '^I haven\'t seen anyone matching') + orig = conf.supybot.protocols.irc.strictRfc() + try: + for state in (True, False): + conf.supybot.protocols.irc.strictRfc.setValue(state) + for wildcard in self.wildcardTest: + self.assertRegexp('seen %s' % wildcard, + '^%s was last seen' % self.nick) + self.assertRegexp('seen bar*', '^I haven\'t seen anyone matching') + finally: + conf.supybot.protocols.irc.strictRfc.setValue(orig) def testSeenNoUser(self): self.assertNotRegexp('seen user alsdkfjalsdfkj', 'KeyError') From 949c093952912795cc43188278e59bba62854ff0 Mon Sep 17 00:00:00 2001 From: James McCoy Date: Sun, 5 Jan 2014 22:35:28 -0500 Subject: [PATCH 05/17] Correct the URL reported by Misc.source Signed-off-by: James McCoy --- plugins/Misc/plugin.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/Misc/plugin.py b/plugins/Misc/plugin.py index c53c71bce..93fb8f645 100644 --- a/plugins/Misc/plugin.py +++ b/plugins/Misc/plugin.py @@ -227,7 +227,7 @@ class Misc(callbacks.Plugin): Returns a URL saying where to get Supybot. """ - irc.reply('My source is at http://supybot.com/') + irc.reply('My source is at http://sourceforge.net/projects/supybot/') source = wrap(source) def more(self, irc, msg, args, nick): From ad3deb7c21ba156f5c19da32fa642de6a31605ad Mon Sep 17 00:00:00 2001 From: Sergio Conde Date: Tue, 1 Apr 2014 20:16:20 +0200 Subject: [PATCH 06/17] Fix AttributeError exception in log.py with python3.4 --- src/log.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/log.py b/src/log.py index b5425e4e9..63abafeba 100644 --- a/src/log.py +++ b/src/log.py @@ -209,7 +209,10 @@ class ValidLogLevel(registry.String): def set(self, s): s = s.upper() try: - level = logging._levelNames[s] + try: + level = logging._levelNames[s] + except AttributeError: + level = logging._nameToLevel[s] except KeyError: try: level = int(s) From 048c1e77f43f4b6c09103ba3bfe7ed314621dda2 Mon Sep 17 00:00:00 2001 From: Valentin Lorentz Date: Wed, 2 Apr 2014 17:18:20 +0000 Subject: [PATCH 07/17] httpserver: Fix favicon handling. --- src/httpserver.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/httpserver.py b/src/httpserver.py index b07d25e03..6eb81618c 100644 --- a/src/httpserver.py +++ b/src/httpserver.py @@ -361,15 +361,15 @@ class Favicon(SupyHTTPServerCallback): file_path = conf.supybot.servers.http.favicon() found = False if file_path: + response = None try: icon = open(file_path, 'r') - found = True + response = icon.read() except IOError: pass finally: icon.close() - if found: - response = icon.read() + if response is not None: filename = file_path.rsplit(os.sep, 1)[1] if '.' in filename: ext = filename.rsplit('.', 1)[1] From 4d9a8a1408c6cbcb17b9f6892cad35d6fe03d3f8 Mon Sep 17 00:00:00 2001 From: Valentin Lorentz Date: Wed, 2 Apr 2014 17:34:04 +0000 Subject: [PATCH 08/17] httpserver: Open icon in binary mode. --- src/httpserver.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/httpserver.py b/src/httpserver.py index 6eb81618c..688349417 100644 --- a/src/httpserver.py +++ b/src/httpserver.py @@ -363,7 +363,7 @@ class Favicon(SupyHTTPServerCallback): if file_path: response = None try: - icon = open(file_path, 'r') + icon = open(file_path, 'rb') response = icon.read() except IOError: pass From 36568aa849fa03eb45dd69614cee7c2a3a1d05c8 Mon Sep 17 00:00:00 2001 From: Valentin Lorentz Date: Thu, 3 Apr 2014 11:47:24 +0000 Subject: [PATCH 09/17] Config: Also tell about current channel value in @help if it is different from the global one. Closes GH-581. --- plugins/Config/plugin.py | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/plugins/Config/plugin.py b/plugins/Config/plugin.py index d7fa3e435..128731ad6 100644 --- a/plugins/Config/plugin.py +++ b/plugins/Config/plugin.py @@ -239,7 +239,18 @@ class Config(callbacks.Plugin): s = group.help() if s: if hasattr(group, 'value') and not group._private: - s += _(' (Current value: %s)') % group + channel = msg.args[0] + if irc.isChannel(channel): + globvalue = str(group) + chanvalue = str(group.get(channel)) + if chanvalue != globvalue: + s += _(' (Current global value: %s; ' + 'current channel value: %s)') % \ + (globvalue, chanvalue) + else: + s += _(' (Current value: %s)') % group + else: + s += _(' (Current value: %s)') % group irc.reply(s) else: irc.reply(_('That configuration group exists, but seems to ' From de1c01f47b9daec7ba303de5c52b4b7c2f1b3367 Mon Sep 17 00:00:00 2001 From: Valentin Lorentz Date: Thu, 3 Apr 2014 11:51:13 +0000 Subject: [PATCH 10/17] Fix previous commit for config variables that are not channel-specific. --- plugins/Config/plugin.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/plugins/Config/plugin.py b/plugins/Config/plugin.py index 128731ad6..f065474d2 100644 --- a/plugins/Config/plugin.py +++ b/plugins/Config/plugin.py @@ -240,7 +240,8 @@ class Config(callbacks.Plugin): if s: if hasattr(group, 'value') and not group._private: channel = msg.args[0] - if irc.isChannel(channel): + if irc.isChannel(channel) and \ + channel in group._children: globvalue = str(group) chanvalue = str(group.get(channel)) if chanvalue != globvalue: From 3bb183cf6d1261916347ff94712032e027ce8248 Mon Sep 17 00:00:00 2001 From: Ken Spencer Date: Thu, 3 Apr 2014 12:01:41 -0400 Subject: [PATCH 11/17] Add my own repo (IotaSpencer --- plugins/PluginDownloader/plugin.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/plugins/PluginDownloader/plugin.py b/plugins/PluginDownloader/plugin.py index b2232146b..be3beb57a 100644 --- a/plugins/PluginDownloader/plugin.py +++ b/plugins/PluginDownloader/plugin.py @@ -297,6 +297,10 @@ repositories = { 'GLolol', 'SupyPlugins', ), + 'Iota': GithubRepository( + 'IotaSpencer', + 'supyplugins', + ), } class PluginDownloader(callbacks.Plugin): From e5825a4d37dc4596d38cf050f492c19e3b845479 Mon Sep 17 00:00:00 2001 From: Dave Woodfall Date: Sat, 5 Apr 2014 06:54:36 +0000 Subject: [PATCH 12/17] RSS 'initialAnnounceHeadlines' variable changed to Integer. Zero can be used for 'initialAnnounceHeadlines' in RSS plugin. --- plugins/RSS/config.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/RSS/config.py b/plugins/RSS/config.py index 53c5ff7b6..26665a827 100644 --- a/plugins/RSS/config.py +++ b/plugins/RSS/config.py @@ -97,7 +97,7 @@ conf.registerGlobalValue(RSS, 'defaultNumberOfHeadlines', registry.PositiveInteger(1, _("""Indicates how many headlines an rss feed will output by default, if no number is provided."""))) conf.registerChannelValue(RSS, 'initialAnnounceHeadlines', - registry.PositiveInteger(5, _("""Indicates how many headlines an rss feed + registry.Integer(5, _("""Indicates how many headlines an rss feed will output when it is first added to announce for a channel."""))) conf.registerChannelValue(RSS, 'keywordWhitelist', registry.SpaceSeparatedSetOfStrings([], _("""Space separated list of From d89e4b5635b4bad52a4f75050577d57d96993dc7 Mon Sep 17 00:00:00 2001 From: Valentin Lorentz Date: Sat, 5 Apr 2014 22:56:40 +0200 Subject: [PATCH 13/17] Fix previous merge. --- .gitignore | 1 + src/commands.py | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitignore b/.gitignore index 17dd40d60..9cc03ecd6 100644 --- a/.gitignore +++ b/.gitignore @@ -23,3 +23,4 @@ supybot.egg-info/ test-conf/ test-data/ test-logs/ +src/version.py diff --git a/src/commands.py b/src/commands.py index a2282b850..793023843 100644 --- a/src/commands.py +++ b/src/commands.py @@ -335,7 +335,6 @@ def getNetworkIrc(irc, msg, args, state, errorIfNoMatch=False): else: state.args.append(irc) -<<<<<<< HEAD def getHaveVoice(irc, msg, args, state, action=_('do that')): getChannel(irc, msg, args, state) if state.channel not in irc.state.channels: From f569af62303d8e953afc2f40d643d43874741c11 Mon Sep 17 00:00:00 2001 From: Valentin Lorentz Date: Sat, 5 Apr 2014 23:08:33 +0200 Subject: [PATCH 14/17] remove generated src/version.py. --- src/version.py | 1 - 1 file changed, 1 deletion(-) delete mode 100644 src/version.py diff --git a/src/version.py b/src/version.py deleted file mode 100644 index ac9150dda..000000000 --- a/src/version.py +++ /dev/null @@ -1 +0,0 @@ -version = '0.83.4.1+limnoria 2014.03.10' From da003279e495ee401e565a9f8264138ee5b9508a Mon Sep 17 00:00:00 2001 From: Valentin Lorentz Date: Sat, 5 Apr 2014 23:08:41 +0200 Subject: [PATCH 15/17] Seen: Fix tests. --- plugins/Seen/test.py | 2 ++ 1 file changed, 2 insertions(+) diff --git a/plugins/Seen/test.py b/plugins/Seen/test.py index 435cd95fe..23027940c 100644 --- a/plugins/Seen/test.py +++ b/plugins/Seen/test.py @@ -64,6 +64,7 @@ class ChannelDBTestCase(ChannelPluginTestCase): with conf.supybot.plugins.seen.showLastMessage.context(False): self.assertRegexp('seen any %s' % self.nick, '^%s was last seen[^:]*' % self.nick) + self.assertNotError('config plugins.Seen.minimumNonWildcard 0') orig = conf.supybot.protocols.irc.strictRfc() try: for state in (True, False): @@ -80,6 +81,7 @@ class ChannelDBTestCase(ChannelPluginTestCase): prefix=self.prefix)) self.assertNotError('seen last') self.assertNotError('list') + self.assertNotError('config plugins.Seen.minimumNonWildcard 2') self.assertError('seen *') self.assertNotError('seen %s' % self.nick) m = self.assertNotError('seen %s' % self.nick.upper()) From b8f31a3fcad3bc1b2feb21dbe274bee32c63f24c Mon Sep 17 00:00:00 2001 From: Valentin Lorentz Date: Sun, 6 Apr 2014 14:05:40 +0000 Subject: [PATCH 16/17] =?UTF-8?q?Web:=20disable=20threading=20in=20command?= =?UTF-8?q?s.=20(They=20are=20run=20in=20separated=20processes=20anyway?= =?UTF-8?q?=E2=80=A6)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- plugins/Web/plugin.py | 1 - 1 file changed, 1 deletion(-) diff --git a/plugins/Web/plugin.py b/plugins/Web/plugin.py index 20bb233b8..b95f97494 100644 --- a/plugins/Web/plugin.py +++ b/plugins/Web/plugin.py @@ -114,7 +114,6 @@ def catch_web_errors(f): class Web(callbacks.PluginRegexp): """Add the help for "@help Web" here.""" - threaded = True regexps = ['titleSnarfer'] @fetch_sandbox From 7d9845c7e99b584ee526f8bd6203bc2877193508 Mon Sep 17 00:00:00 2001 From: Valentin Lorentz Date: Tue, 8 Apr 2014 20:00:07 +0000 Subject: [PATCH 17/17] ChannelLogger: Fix KeyError on some quits. (Why?) --- plugins/ChannelLogger/plugin.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/plugins/ChannelLogger/plugin.py b/plugins/ChannelLogger/plugin.py index 8faaca62a..c2d2488a7 100644 --- a/plugins/ChannelLogger/plugin.py +++ b/plugins/ChannelLogger/plugin.py @@ -220,7 +220,7 @@ class ChannelLogger(callbacks.Plugin): def doNick(self, irc, msg): oldNick = msg.nick newNick = msg.args[0] - for (channel, c) in irc.state.channels.iteritems(): + for (channel, c) in irc.state.channels.items(): if newNick in c.users: self.doLog(irc, channel, '*** %s is now known as %s\n', oldNick, newNick) @@ -278,7 +278,9 @@ class ChannelLogger(callbacks.Plugin): reason = "" if not isinstance(irc, irclib.Irc): irc = irc.getRealIrc() - for (channel, chan) in self.lastStates[irc].channels.iteritems(): + if irc not in self.lastStates: + return + for (channel, chan) in self.lastStates[irc].channels.items(): if(self.registryValue('showJoinParts', channel)): if msg.nick in chan.users: self.doLog(irc, channel,