Skip to content

Add configuration file for multi-user support#7

Open
twillard wants to merge 3 commits intosunmockyang:masterfrom
twillard:master
Open

Add configuration file for multi-user support#7
twillard wants to merge 3 commits intosunmockyang:masterfrom
twillard:master

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 notifications@github.com 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

Comments