northstack / northstack-client

NorthStack Client
https://northstack.com
GNU General Public License v3.0
7 stars 1 forks source link

Update install script so it doesn't need to be run with sudo. #14

Closed jeichorn closed 6 years ago

jeichorn commented 6 years ago

We only need sudo for installing the wrapper. In install.sh check if we need permissions and make the copy call with sudo if needed (make sure to message what we doing before calling sudo since it might prompt a login.

mlteal commented 6 years ago

ran install.sh without sudo (freshly pulled) and still getting cp: /usr/local/bin/northstack: Permission denied (ran with sudo after and it was happy)

mm-pagely commented 6 years ago

@mlteal Can you run the install script with bash in debug mode and with the DEBUG env var set and give us the output?

$ DEBUG=1 bash -x ./bin/install.sh
mlteal commented 6 years ago

Fixed now!

mm-pagely commented 6 years ago

ae97752874bba1c9ba8557fc1842f5a0356199c7 FTW