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

Add entry_point to setup.py #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fawkesley
Copy link

This makes pip install pgpdump to install the pgpdump script
into the user's $PATH (or .local/bin/)

See
https://chriswarrick.com/blog/2014/09/15/python-apps-the-right-way-entry_points-and-scripts/

This makes `pip install pgpdump` to install the `pgpdump` script
into the user's $PATH (or `.local/bin/`)

See
https://chriswarrick.com/blog/2014/09/15/python-apps-the-right-way-entry_points-and-scripts/
@fawkesley
Copy link
Author

Hi @toofishes is this repo still alive?

@toofishes
Copy link
Owner

Yes it is! I've been super busy with other projects though so have been neglecting pull requests here. I'll try and take a look at this sometime this week.

On Mar 30, 2015, at 5:12 PM, Paul M Furley notifications@github.com wrote:

Hi @toofishes is this repo still alive?


Reply to this email directly or view it on GitHub.

@fawkesley
Copy link
Author

Totally understand!

You can test locally by doing cd python-pgpdump; pip install --user -e . - you should see the pgpdump script appear in ~/.local/bin/ :)

@cicku
Copy link

cicku commented Jul 28, 2015

I'm against this change.

This will conflict with the pgpdump from http://www.mew.org/~kazu/proj/pgpdump/en/ in /usr/bin. I maintain both packages in Fedora.

@cicku
Copy link

cicku commented Jul 28, 2015

Also, this is just a "parser library", I found no reason of having executable.

@fawkesley
Copy link
Author

@cicku Fair enough about conflicting namespace.

I can't agree that it's just a library though - else why would it offer a main function at all?

@cicku
Copy link

cicku commented Jul 28, 2015

My understanding of representing main in the code is for tests ONLY, pgpdump imports and execute tests without side effects.

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.

3 participants