neithere / argh

An argparse wrapper that doesn't make you say "argh" each time you deal with it.
http://argh.rtfd.org
GNU Lesser General Public License v3.0
369 stars 56 forks source link

Argparse dependency #41

Closed neithere closed 11 years ago

neithere commented 11 years ago

Hi,

thanks for making argh, it is really great ! I am using it for sigal but I have an issue when packaging it for Archlinux. When calling sigal I get the following error:

#!python

pkg_resources.DistributionNotFound: argparse>=1.1

This is because you are adding argparse as a dependency in your setup.py even for python 2.7. So when installing argh with the system package I have a file /usr/lib/python2.7/site-packages/argh-0.23.0-py2.7.egg-info/requires.txt which contains argparse. And when calling my command with the entry_point system, it checks this dependency and fails.

Could you change your setup.py to add argparse only for python 2.6 ?

Note: This issue has been automatically migrated from Bitbucket Created by @saimn on 2013-03-10 23:20:21+00:00, last updated: 2013-03-11 13:34:37+00:00

neithere commented 11 years ago

Thanks for the report, I'll try to fix the problem.

By the way, does your sigal package for ArchLinux depend on python-argh (or python2-argh) from AUR?

Note: This comment has been automatically migrated from Bitbucket Created by @neithere on 2013-03-11 12:55:51+00:00

neithere commented 11 years ago

yes, it depends on python2-argh. The PKGBUILD is here (I will push it to AUR when this issue with argh/argparse is resolved) : https://gist.github.com/saimn/5130765

If I add python2-argparse to the dependencies, then it works fine, but adding this dependency should not be needed for python 2.7. I can reproduce the same error with the python2-watchdog AUR package which also depends on python2-argh,

Note: This comment has been automatically migrated from Bitbucket Created by @saimn on 2013-03-11 13:06:18+00:00

neithere commented 11 years ago

Thanks again for the informative bug report. I believe b900869 fixes it.

Note: This comment has been automatically migrated from Bitbucket Created by @neithere on 2013-03-11 13:34:37+00:00

neithere commented 11 years ago

Thanks for the quick fix and release :-)

Note: This comment has been automatically migrated from Bitbucket Created by @saimn on 2013-03-11 13:46:51+00:00