Skip to content

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 [email protected] 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