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

Install command line scripts #5

Closed
cdeil opened this issue May 27, 2015 · 5 comments
Closed

Install command line scripts #5

cdeil opened this issue May 27, 2015 · 5 comments

Comments

@cdeil
Copy link
Contributor

cdeil commented May 27, 2015

It would be convenient if pythons setup.py install installed the command line scripts.

The recommended way to do this is with setuptools entry_points:
https://pythonhosted.org/setuptools/setuptools.html#automatic-script-creation
http://astropy.readthedocs.org/en/latest/development/scripts.html

@telegraphic
Copy link
Owner

Thanks & agreed, I've marked it as an enhancement and will put it on the todo list.

@telegraphic
Copy link
Owner

This probably motivates moving to argparse from optparse, so will have to do that first. Looks relatively straightforward after that.

@telegraphic
Copy link
Owner

OK, seems to be working. I probably need to do a bit more testing as it's the first time I've used entry points, but looking good.

@cdeil
Copy link
Contributor Author

cdeil commented May 30, 2015

@telegraphic – Thanks!

Is it useful to keep the top-level scripts in the repo?
I don't think it's common to do this for Python packages that have command line scripts, and generally it's bad to give users two ways to do the same thing?

@telegraphic
Copy link
Owner

Looks like I did remove the top-level scripts at some stage, so closing this one too.

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

No branches or pull requests

2 participants