1. Tarek Ziadé
  2. distribute
  3. Issues
Issue #187 invalid

Distribute distributes itself with win32 .exe executables on all platforms

Anonymous created an issue

Distribute-0.6.14 distributes itself with win32 executables on all platforms.

patch: Index: distribute-0.6.14/setup.py =================================================================== --- distribute-0.6.14.orig/setup.py 2011-01-23 13:56:43.000000000 +0000 +++ distribute-0.6.14/setup.py 2011-01-23 14:02:20.000000000 +0000 @@ -131,6 +131,10 @@ from distribute_setup import _before_install _before_install()

+from distutils import util +package_data = {} +if (util.get_platform().startswith("win") or util.get_platform().startswith("cygwin")): + package_data = {'setuptools':['*.exe']},

dist = setup( name="distribute", @@ -146,7 +150,7 @@ test_suite = 'setuptools.tests', src_root = src_root, packages = find_packages(), - package_data = {'setuptools':['*.exe']}, + package_data = package_data,

 py_modules = ['pkg_resources', 'easy_install', 'site'],

Comments (2)

  1. Anonymous
    Index: distribute-0.6.14/setup.py
    ===================================================================
    --- distribute-0.6.14.orig/setup.py	2011-01-23 13:56:43.000000000 +0000
    +++ distribute-0.6.14/setup.py	2011-01-23 14:02:20.000000000 +0000
    @@ -131,6 +131,10 @@
         from distribute_setup import _before_install
         _before_install()
     
    +from distutils import util
    +package_data = {}
    +if (util.get_platform().startswith("win") or util.get_platform().startswith("cygwin")):
    +    package_data = {'setuptools':['*.exe']},
     
     dist = setup(
         name="distribute",
    @@ -146,7 +150,7 @@
         test_suite = 'setuptools.tests',
         src_root = src_root,
         packages = find_packages(),
    -    package_data = {'setuptools':['*.exe']},
    +    package_data = package_data,
     
         py_modules = ['pkg_resources', 'easy_install', 'site'],
    
  2. Tarek Ziadé repo owner

    We do source releases only, so I don't see the problem of including win32 exe when sdist is built under Linux. Applying your change would make source releases broken in fact

  3. Log in to comment