Do not use datafiles in setup.py #280

 * and other setup/packaging improvements

Signed-off-by: Philippe Ombredanne <[email protected]>

Author: pombredanne

MANIFEST.in

@@ -1,10 +1,29 @@
-include *.ABOUT
-include about*
-include *LICENSE*
-include *NOTICE*
-include *README*
+graft src
+graft tests
+graft docs
+graft etc
+graft example
+graft thirdparty
+
+prune src/aboutcode_toolkit.egg-info
+
+include about.ABOUT
+include about
+include about.bat
+include about.cfg
+include apache-2.0.LICENSE
+include about.bat
+include apache-2.0.LICENSE
+include configure
+include configure.bat
+include MANIFEST.in
+include NOTICE
+include README.rst
+include setup.cfg
+include setup.py
+include SPEC
 include USAGE.rst
-include *SPEC*
+include .travis.yml
+include appveyor.yml
 
-recursive-include tests/testdata *
-recursive-include about_code_tool/templates *
+global-exclude *.py[co] __pycache__ *.*~

setup.cfg

@@ -5,7 +5,7 @@ universal = 1
 license_file = NOTICE
 
 [aliases]
-release = clean --all sdist --formats=bztar,zip bdist_wheel register upload 
+release = clean --all sdist --formats=bztar,zip bdist_wheel 
 
 [tool:pytest]
 norecursedirs =

setup.py

@@ -66,23 +66,6 @@ def read(*names, **kwargs):
         'license', 'about', 'metadata', 'package', 'copyright',
         'attribution', 'software', 'inventory',
     ],
-    # FIXME: we SHOULD NOT use datafiles!!!! which is a mine field but
-    # only package data or use a MANIFEST.in instead for a sdist
-    data_files=[('about',
-        [
-            'about.ABOUT',
-            'about.bat',
-            'about',
-            'configure.bat',
-            'configure',
-            'about.cfg',
-            'README.rst',
-            'apache-2.0.LICENSE',
-            'NOTICE',
-            'SPEC',
-            'USAGE.rst',
-        ]),
-    ],
     install_requires=[
         'jinja2 >= 2.9, < 3.0',
         'click >= 6.7, < 7.0',