Make this a proper namespace package

GitHubMixin doesn't need to inherit from Component.
All of the classes utilizing the mixin inherit from
Component, and we don't need to support enabling/
disabling GitHubMixin.

Fix test failures due to config section already present.
The section is already present due to an `Option`
defined on a class that doesn't inherit from Component:
https://trac.edgewall.org/ticket/12457

Author: rjollos

README.md

@@ -514,6 +514,7 @@ Changelog
 
 ### 2.4 (not yet released)
 
+* Fix improperly configured namespace package. (#131)
 * Add configuration option for path prefix of login and logout. (#127)
 * Add `GitHubPolicy` permission policy to make `[timeline]`
   `changeset_show_file` option work correctly. (#126)

runtests.py

@@ -124,7 +124,8 @@ def createTracEnvironment(cls, **kwargs):
             conf.set('git', 'cached_repository', 'true')
             conf.set('git', 'persistent_cache', 'true')
 
-        conf.add_section('github')
+        if not conf.has_section('github'):
+            conf.add_section('github')
         client_id = '01234567890123456789'
         if 'client_id' in kwargs:
             client_id = kwargs['client_id']

setup.py

@@ -4,7 +4,7 @@
 # This software is licensed as described in the file LICENSE, which
 # you should have received as part of this distribution.
 
-from setuptools import setup
+from setuptools import setup, find_packages
 
 setup(
     name='trac-github',
@@ -14,7 +14,8 @@
     url='https://github.com/trac-hacks/trac-github',
     description='Trac - GitHub integration',
     download_url='https://pypi.python.org/pypi/trac-github',
-    packages=['tracext'],
+    packages=find_packages(),
+    namespace_packages=['tracext'],
     platforms='all',
     license='BSD',
     extras_require={'oauth': ['requests_oauthlib >= 0.5']},

tracext/__init__.py

@@ -0,0 +1 @@
+__import__('pkg_resources').declare_namespace(__name__)

tracext/github.py tracext/github/__init__.pytracext/github.py renamed to tracext/github/__init__.py

@@ -210,7 +210,7 @@ def _oauth_session(self, req, state=None):
 
 
 
-class GitHubMixin(Component):
+class GitHubMixin(object):
 
     webhook_secret = Option('github', 'webhook_secret', '',
                             doc="""GitHub webhook secret token.