Skip to content

Integrating setuptools #17

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 5 commits into from
Closed

Integrating setuptools #17

wants to merge 5 commits into from

Conversation

phlax
Copy link

@phlax phlax commented Mar 14, 2012

In order to get this to work i had to adjust the hostpython path for packages that are capable of using setuptools. There may be a better way of setting up the paths and hostpython.

I have tested this to the extent that i have successfully installed 20+ packages that use setuptools

@tito
Copy link
Member

tito commented Mar 14, 2012

Hi,

Ok for not removing distutils for the distribution, but can you blacklist it by default in the blacklist.txt? distutils would be not needed for running an app on the final device.

And, could you change your patch to use $BUILD_hostpython ?

Otherwise, look ok to me.

Cool :)

@phlax
Copy link
Author

phlax commented Mar 14, 2012

setuptools requires distutils and many packages require setuptools to parse their namespace init.pys.

if i add it into blacklist.txt, we'll need to provide docs somewhere that it needs to be removed if using setuptools based packages

re: $BUILD_python, yep

@tito
Copy link
Member

tito commented Mar 14, 2012

Ok, understood. Then agreed to let it by default.

On 14/03/2012 17:45, phlax wrote:

setuptools requires distutils and many packages require setuptools to parse their namespace init.pys.

if i add it into blacklist.txt, we'll need to provide docs somewhere that it needs to be removed if using setuptools based packages

re: $BUILD_python, yep


Reply to this email directly or view it on GitHub:
#17 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants