pull request to address issue #13: "setuptools packaging"

robdennis commented 12 years ago

Hello, I'm interested in using a subset of creepy's functionality in a project of mine, and it would be convenient from me to have an explicit dependency of creepy[twitter] (for example). (here's the relevant link to python's handling of 'features': http://packages.python.org/distribute/setuptools.html?highlight=extra#declaring-extras-optional-features-with-their-own-dependencies)

I saw issue #13 and am submitting a pull request where I:

added setup.py packaging with 4 features (flickr/twitter/exif/map) to support people depending on a subset of creepy functionality
modified urlanalyzer to not throw import error in the event of the exif feature not being present, as the module is used by this twitter/flickr modules
bumped version to 0.2.0 as adding packaging with setuptools extra features is at least a minor level change

testing steps I performed: fresh ubuntu VM (install items not on pypi or just not installable with distutils) apt-get install python-pyexiv2 libosmgpsmap-dev python-osmgpsmap python-gtk python-gobject python (>2.6) setup.py develop # installs all the base requirements easy_install creepy[all] # installs all the optional dependencies and verifies that the above items (non-pypi modules) are installed

if the apt-get step was skipped and the feature was specified (e.g. creepy[map] or creepy[all]), a warning would be displayed with a link to the projects download page

Thanks, appreciate the work and hopefully this is something that you find useful

ilektrojohn commented 12 years ago

Hey,

This looks very interesting, I'll go over the changes tomorrow and if everything is ok I will pull this into the branch . Thanks for your time and I'm sorry I didn't answer earlier .

robdennis commented 12 years ago

sounds great, and glad to do it. This change allows me to list creepy[twitter] as a dependency and import creepy into my code and get a lot of the same functionality without the map, which is a huge help in my data mining class project :)

On Wed, Oct 26, 2011 at 4:14 AM, Yiannis Kakavas < reply@reply.github.com>wrote:

Hey,

This looks very interesting, I'll go over the changes tomorrow and if everything is ok I will pull this into the branch . Thanks for your time and I'm sorry I didn't answer earlier .

Reply to this email directly or view it on GitHub: https://github.com/ilektrojohn/creepy/pull/18#issuecomment-2527784

ilektrojohn / creepy

pull request to address issue #13: "setuptools packaging" #18