vincenthsu / systemd-ngrok

Automatically start ngrok by systemd
The Unlicense
475 stars 170 forks source link

Why git clone again in install.sh #6

Closed rajanski closed 4 years ago

rajanski commented 4 years ago

https://github.com/vincenthsu/systemd-ngrok/blob/master/install.sh#L13 doesnt make sense to me, since I assume I have the repo already cloned when executing install.sh

JoaquimLey commented 4 years ago

You could simply

touch ngrok_install.sh

And then nano raw contents, also this would ensure you have the latest version, not the one living on your machine.

This is just my $0.02 but better to wait for the maintainer's answer.

JoaquimLey commented 4 years ago

Hey @rajanski after seeing this issue and having to re-install multiple times on my raspis I decided to write a self-sufficient script so you don't need to re-clone:

The whole package is open source here:

I've added detailed instructions in the README.MD file.

But you can simply copy-paste the raw script:

vincenthsu commented 4 years ago

Hi @rajanski,

Because I usually use the install.sh before cloning the repo:

curl -O  https://raw.githubusercontent.com/vincenthsu/systemd-ngrok/master/install.sh && chmod +x install.sh && sudo ./install.sh 
 <your_authtoken>

Thanks for your opinion. I will add some condition to avoid double clones.