-
-
Notifications
You must be signed in to change notification settings - Fork 61
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
Script ends with cancelled by user #2
Comments
Hi! sorry for the late response! Humm... You should be asked if you want to install the updates at that point. |
Hi again! Just made a test piping it to I would ask you to just download the script and then run it locally instead of directly piping it from curl or wget. So, you can use curl or wget to download as described on the readme, or even use git to clone the repo. Then you can just call it with or without parameters. If you still want to download and run at once, I would recommend using something like this:
This will download, set to executable and run (on the current directory). Cheers! |
Ok, so this seems to be normal to happen in this case.
Because there is not stdin for |
Thanks for your response, I have not tried to run it locally but I will try the script you just posted thank you. Update: |
Sweet! |
I will read the links you posted soon. I was hoping it would work the same as it does in Ubuntu, I use a DNS server called Pihole and that's how it's installed. |
You should also put that command on your Readme page as long as people have curl installed it works great.
DONE |
As far as I can tell the problem is only with The download call is basically the same I have on the readme, but without using variables and this one runs the script after downloading. I think it is less intrusive that way, but who knows. The better way is always to use git in any case, but not really needed. I also have an explanation for the Cheers! |
Just a side note. This would also happen in any other linux distro (including ubuntu). Pihole works because it does not ask anything for the user (or he would have the same problem with read not receiving the answer I think). I would need a separate installer script for you to run like that. Same approach is used by get.acme.sh (LetsEncrypt SH script). He has an installer on http://get.acme.sh which install the acme.sh script itself. So, in any case, running the installer like that would not run the script itself. |
Also, he uses a variable to pass a parameter, as in:
That |
I tried your script but it does not ask me if I want to install updates it just say's cancelled by user, Any guidence would be appreciated.
This is the command I used:
curl -k -sSL https://raw.githubusercontent.com/tavinus/opkg-upgrade/master/opkg-upgrade.sh | sh
A little bit about my install and my router:
LEDE/OpenWRT
Model | TP-Link TL-WDR4300 v1
Firmware Version | LEDE Reboot 17.01.2 r3435-65eec8bd5f / LuCI lede-17.01 branch (git-17.152.82987-7f6fc16)
Kernel Version | 4.4.71
Running your script using curl:
The text was updated successfully, but these errors were encountered: