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 configuration file for multi-user support #7

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

twillard
Copy link

This small change moves the PUUSH_API_KEY configuration item into a per-user config file called $HOME/.puush.

This allows multiple users on the same system to use this awesome puush script, and also allows you to put restrictive permissions on your .puush config file so other people can't see your API key.

Also added a small install script to simplify the process of installing onto the system

…sh.png so it's more specific when installed in the system, and add a simple install script to put the stuff in /usr/local/bin
… helpfully autogenerate a default config file if it doesn't exist
@sunmockyang
Copy link
Owner

Hey, I don't have a chance to view the diff, but can you also do me a favour and rename puush into puush.sh?

Sent from my Cyanogen phone

On 28 Oct 2015 12:07 am, Travis Willard [email protected] wrote:

This small change moves the PUUSH_API_KEY configuration item into a per-user config file called $HOME/.puush.

This allows multiple users on the same system to use this awesome puush script, and also allows you to put restrictive permissions on your .puush config file so other people can't see your API key.

Also added a small install script to simplify the process of installing onto the system


You can view, comment on, or merge this pull request online at:

#7

Commit Summary

Make PUUSH_API_KEY multi-account configurable, rename icon.png to puush.png so it's more specific when installed in the system, and add a simple install script to put the stuff in /usr/local/binSome further cleanup, updating README.md for the new config file, and helpfully autogenerate a default config file if it doesn't exist

File Changes

M README.md (8) A install.sh (6) M puush (28) R puush.png (0)

Patch Links:

https://github.com/sunmockyang/puush-linux/pull/7.patchhttps://github.com/sunmockyang/puush-linux/pull/7.diff


Reply to this email directly or view it on GitHub.

@twillard
Copy link
Author

I think this should do it?

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.

2 participants