Skip to content
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

Improved setup.py and potentially fixed issue 6 #12

Merged
merged 1 commit into from
Feb 28, 2013

Conversation

fenekku
Copy link

@fenekku fenekku commented Feb 27, 2013

Hello,

Zenlib still had some issues building on Ubuntu so I worked on the setup.py script (see below) and added installation instructions in the repository's readme.md so that interested parties can install the library quickly.

The original issue: zenlib depends on networkx but networkx wasn't listed in the install instructions nor was setup.py fetching any dependencies. I hypothesize that the commenting out of the install_requires was originally due to setuptools and cython's incompatibility which led to the use of distutils.core as a stop-gap. To fix this I've clearly indicated the dependency on distribute which fixes this issue. Dependencies can now be listed again! It seems like distribute is getting traction so I don't think it is such a far-fetched solution.

I also believe this fixes #6 as the problem there was due to missing .c files which would not have been built by the setup.py script using regular setuptools.

Hopefully this makes sense and helps make zenlib easier to install.

@druths druths merged commit 7d44ec6 into networkdynamics:master Feb 28, 2013
@druths
Copy link
Member

druths commented Feb 28, 2013

Thanks so much for your help with this! You've achieved precisely what I wanted the setup.py to do. This is a big help. Thanks again for your help!

@druths
Copy link
Member

druths commented Feb 28, 2013

Hmm... after an attempt to use the new setup.py file on a Mac 10.7.4 machine, I discovered that the setup.py has the old problem due to Distribute not being configured properly on my system. While I could change this, my inclination is to stick with a distribution system that is going to work on the majority of systems without any additional effort. I'll slate incorporating Distribute by 1.0, but I don't want to switch things on the userbase right now. In the meantime, I'm commenting out the modification you made for later incorporation - since this certainly is the only way of getting cython and dependencies working right now.

Thanks so much for your contributions - please keep contributing!

@fenekku
Copy link
Author

fenekku commented Feb 28, 2013

Hmm, I would have thought that the setup_requires='distribute' would handle the distribute dependency on Macs... I am not familiar at all with the configuration process on OS X :(.

Perhaps a 1.0-dev branch might be useful in this case? This way further developments can be made independently and newcomers can be pointed to that branch to get an install procedure that has the desired features (if they are using a version of Mac OS X > 10.7.4 or if they are ready to make more configurations).

Thanks for the support!

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.

zen fails to build under python 2.7.1 on OS X and Linux
2 participants