search

archan937 / jsonv.sh

A Bash command line tool for converting JSON to CSV
MIT License
217 stars 45 forks source link

root permissions #6

Open mathieujobin opened 7 years ago

mathieujobin commented 7 years ago

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 commented 7 years ago

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

McNinja commented 7 years ago

+1

yang-dev commented 6 years ago

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

mathieujobin commented 6 years ago

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