Skip to content

Commit

Permalink
Merge pull request #5480 from planetscale/morgo-fix-arch
Browse files Browse the repository at this point in the history
Don't use arch in shell scripts
  • Loading branch information
sougou authored Dec 1, 2019
2 parents e8f9f88 + 3196204 commit 031d50e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 6 deletions.
9 changes: 4 additions & 5 deletions bootstrap.sh
Original file line number Diff line number Diff line change
Expand Up @@ -127,12 +127,11 @@ function install_dep() {
# 1. Installation of dependencies.
#

# Wrapper around the `arch` command which plays nice with OS X
# We should not use the arch command, since it is not reliably
# available on macOS or some linuxes:
# https://www.gnu.org/software/coreutils/manual/html_node/arch-invocation.html
function get_arch() {
case $(uname) in
Linux) arch;;
Darwin) uname -m;;
esac
uname -m
}


Expand Down
2 changes: 1 addition & 1 deletion dev.env
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ export PKG_CONFIG_PATH

# According to https://github.com/etcd-io/etcd/blob/a621d807f061e1dd635033a8d6bc261461429e27/Documentation/op-guide/supported-platform.md,
# currently, etcd is unstable on arm64, so ETCD_UNSUPPORTED_ARCH should be set.
if [ "$(arch)" == aarch64 ]; then
if [ "$(uname -m)" == aarch64 ]; then
export ETCD_UNSUPPORTED_ARCH=arm64
fi

Expand Down

0 comments on commit 031d50e

Please sign in to comment.