Simplify module information with python configuration

review363001

MANIFEST.in

@@ -4,6 +4,7 @@ include TODO
 include COPYRIGHT
 include CHANGELOG
 include LICENSE
+include tryton.cfg
 include *.xml
 include *.odt
 include locale/*.po

README

@@ -2,7 +2,6 @@ trytond_carrier
 =============
 
 The carrier module of the Tryton application platform.
-See __tryton__.py
 
 Installing
 ----------

__tryton__.py

@@ -1,44 +0,0 @@
-# -*- coding: utf-8 -*-
-#This file is part of Tryton.  The COPYRIGHT file at the top level of
-#this repository contains the full copyright notices and license terms.
-{
-    'name': 'Carrier',
-    'name_bg_BG': 'Управление на превозвачи',
-    'name_es_AR': 'Transportistas',
-    'name_es_ES': 'Transportista',
-    'name_de_DE': 'Frachtführer',
-    'name_fr_FR': 'Transporteur',
-    'version': '2.5.0',
-    'author': 'B2CK',
-    'email': 'info@b2ck.com',
-    'website': 'http://www.tryton.org/',
-    'description': 'Define carriers.',
-    'description_bg_BG': 'Задаване на превозвачи.',
-    'description_es_ES': 'Define transportistas.',
-    'description_de_DE': '''
-    - Ermöglicht die Definition von Frachtführern.
-    ''',
-    'description_es_AR': 'Define transportistas.',
-    'description_fr_FR': 'Définit des transporteurs.',
-    'depends': [
-        'ir',
-        'res',
-        'party',
-        'product',
-    ],
-    'xml': [
-        'carrier.xml',
-    ],
-    'translation': [
-        'locale/bg_BG.po',
-        'locale/ca_ES.po',
-        'locale/cs_CZ.po',
-        'locale/es_AR.po',
-        'locale/es_ES.po',
-        'locale/es_CO.po',
-        'locale/de_DE.po',
-        'locale/fr_FR.po',
-        'locale/nl_NL.po',
-        'locale/ru_RU.po',
-    ],
-}

setup.py

@@ -4,8 +4,19 @@
 
 from setuptools import setup
 import re
+import os
+import ConfigParser
 
-info = eval(open('__tryton__.py').read())
+
+def read(fname):
+    return open(os.path.join(os.path.dirname(__file__), fname)).read()
+
+config = ConfigParser.ConfigParser()
+config.readfp(open('tryton.cfg'))
+info = dict(config.items('tryton'))
+for key in ('depends', 'extras_depend', 'xml'):
+    if key in info:
+        info[key] = info[key].strip().splitlines()
 major_version, minor_version, _ = info.get('version', '0.0.1').split('.', 2)
 major_version = int(major_version)
 minor_version = int(minor_version)
@@ -21,21 +32,21 @@ requires.append('trytond >= %s.%s, < %s.%s' %
 
 setup(name='trytond_carrier',
     version=info.get('version', '0.0.1'),
-    description=info.get('description', ''),
-    author=info.get('author', ''),
-    author_email=info.get('email', ''),
-    url=info.get('website', ''),
+    description='Tryton module with carriers',
+    long_description=read('README'),
+    author='Tryton',
+    url='http://www.tryton.org/',
     download_url="http://downloads.tryton.org/" + \
-            info.get('version', '0.0.1').rsplit('.', 1)[0] + '/',
+        info.get('version', '0.0.1').rsplit('.', 1)[0] + '/',
     package_dir={'trytond.modules.carrier': '.'},
     packages=[
         'trytond.modules.carrier',
         'trytond.modules.carrier.tests',
-    ],
+        ],
     package_data={
         'trytond.modules.carrier': info.get('xml', []) \
-                + info.get('translation', []),
-    },
+            + ['tryton.cfg', 'locale/*.po'],
+        },
     classifiers=[
         'Development Status :: 5 - Production/Stable',
         'Environment :: Plugins',
@@ -56,11 +67,9 @@ setup(name='trytond_carrier',
         'Programming Language :: Python :: 2.6',
         'Programming Language :: Python :: 2.7',
         'Topic :: Office/Business',
-    ],
+        ],
     license='GPL-3',
     install_requires=requires,
-    extras_require={
-    },
     zip_safe=False,
     entry_points="""
     [trytond.modules]
@@ -68,4 +77,4 @@ setup(name='trytond_carrier',
     """,
     test_suite='tests',
     test_loader='trytond.test_loader:Loader',
-)
+    )

tryton.cfg

@@ -0,0 +1,9 @@
+[tryton]
+version=2.5.0
+depends:
+    ir
+    party
+    product
+    res
+xml:
+    carrier.xml