-
Notifications
You must be signed in to change notification settings - Fork 5
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
Provide a simple download link for iofogctl
#135
Comments
@ctron I understand the reticency to run a script using sudo. However this is standard practice if you want to use your linux package manager to install iofogctl and be able to manage updates. We haven't written this script ourselves, it is provided by our cloud package provider (https://packagecloud.io/) Here are the manual installation steps: https://packagecloud.io/iofog/iofogctl/install#manual-deb We should probably add those as an option in our documention |
"Standard practice" is a bit far fetched. Yes, a package manager like RPM is standard practice. "curling" and running scripts as sudo however is not. Maybe take some inspiration from the
Also other tools, simply provide a download link (no root access required):
So if you want to make it easy, provide a simple download link. Additional options maybe helpful, but might also cause more problems. |
@ctron currently working on it. If you want to download a dev binary in the meantime: https://storage.googleapis.com/iofogctl/linux/dev/3.0/iofogctl |
Staging iofog.org is updated with instructions to install release binary: http://iofog.staging.edgeworx.io/docs/3/iofogctl/download.html#install-iofogctl-on-linux |
This is definitely an improvement. But what speaks against using plain and simple GitHub releases downloads? |
In the installation instructions of
iofogctl
I can't find any simple download link for the Linux binary.See: https://iofog.org/docs/2/iofogctl/download.html
And I don't want to run some shell script with sudo access directly from curl.
As you provided the link for Windows, I think it makes sense to do the same for Linux and Mac OS X.
Just like others do it as well: https://github.com/helm/helm/releases
The text was updated successfully, but these errors were encountered: