Skip to content

install script - trivial updates to fix io => org where possible, and remove broken link #3725

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.

Sign up for GitHub

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

install script - trivial updates to fix io => org where possible, and remove broken link #3725

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
b06c5c5
Clarify why an anonymous function is used during installation
neuroradiology Dec 23, 2024
9347ad9
Update install documentation link to sandstorm.org
neuroradiology Dec 23, 2024
21d825e
Update install.sh to remove ".io" from the "sandstorm.io team"
neuroradiology Dec 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 11 additions & 5 deletions install.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
#
# If `curl|bash` makes you uncomfortable, see other options here:
#
# https://docs.sandstorm.io/en/latest/install/
# https://docs.sandstorm.org/install/
#
# This script only modifies your system in the following ways:
# - Install Sandstorm into the directory you choose, typically /opt/sandstorm.
Expand Down Expand Up @@ -41,8 +41,14 @@ fi

# We wrap the entire script in a big function which we only call at the very end, in order to
# protect against the possibility of the connection dying mid-script. This protects us against
# the problem described in this blog post:
# http://blog.existentialize.com/dont-pipe-to-your-shell.html
# the problem described in a since-deleted blog post that is discussed on HN:
# https://news.ycombinator.com/item?id=6650987
# Briefly, if the install script has a line that says
#
# > rm -rf $somefolder
#
# and the network fails (so the script terminates) before the last argument is received, then
# piping it into the shell will wipe the filesystem.
_() {

set -euo pipefail
Expand Down Expand Up @@ -85,7 +91,7 @@ fail() {

# Users can export REPORT=no to avoid the error-reporting behavior, if they need to.
if [ "${REPORT:-yes}" = "yes" ] ; then
if USE_DEFAULTS=no prompt-yesno "Hmm, installation failed. Would it be OK to send an anonymous error report to the sandstorm.io team so we know something is wrong?
if USE_DEFAULTS=no prompt-yesno "Hmm, installation failed. Would it be OK to send an anonymous error report to the sandstorm team so we know something is wrong?
It would only contain this error code: $error_code" "yes" ; then
echo "Sending problem report..." >&2
local BEARER_TOKEN="4-Og3Ty2SPmpkZGnVc_8hnBGXK0JBBXDeBn_55FWixJ"
Expand Down Expand Up @@ -458,7 +464,7 @@ assert_linux_x86_64() {
fail "E_NON_LINUX" "Sandstorm requires Linux. If you want to run Sandstorm on a Windows or
Mac system, you can use Vagrant or another virtualization tool. See our install documentation:

- https://docs.sandstorm.io/en/latest/install/"
- https://docs.sandstorm.org/install"
fi

if [ "$(uname -m)" != x86_64 ]; then
Expand Down
Loading