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

root permissions #6

Open
mathieujobin opened this issue May 10, 2017 · 4 comments
Open

root permissions #6

mathieujobin opened this issue May 10, 2017 · 4 comments

Comments

@mathieujobin
Copy link

the install requires root as it install it under /usr/local/bin/

but then using the command also requires root as it writes to /usr/local/bin/.jsonv/

this should not be....

@frederickjh
Copy link

This is also true in my experience of installing, testing and uninstalling.

@McNinja
Copy link

McNinja commented Sep 6, 2017

+1

@yang-dev
Copy link

This is very broken. Why write temp (garbage) into /usr/local/bin/?

@mathieujobin
Copy link
Author

this piece is what seems to require replacement

https://github.com/archan937/jsonv.sh/blob/master/jsonv#L50-L62

if some of these are required for the script to work, maybe it would fit under /usr/local/share
for anything temporary, it should be /tmp/ or /var/run/tmp

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

No branches or pull requests

4 participants