Skip to content
This repository has been archived by the owner on Jan 25, 2023. It is now read-only.

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Yevgeniy Brikman <[email protected]>
  • Loading branch information
izzyleung and brikis98 authored May 5, 2021
1 parent 9e29444 commit 73ab718
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions modules/install-consul/install-consul
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,8 @@ function fetch_binary {
local -r version="$1"
local download_url="$2"

local -r cpu_arch="$(uname -m)"
local cpu_arch
cpu_arch="$(uname -m)"
local binary_arch=""
case "$cpu_arch" in
x86_64)
Expand All @@ -191,7 +192,8 @@ function fetch_binary {
# $ uname -m
# $ readelf -a /proc/self/exe | grep -q -c Tag_ABI_VFP_args && echo "armhf" || echo "armel"
#
local -r vfp_tag="$(readelf -a /proc/self/exe | grep -q -c Tag_ABI_VFP_args)"
local vfp_tag
vfp_tag="$(readelf -a /proc/self/exe | grep -q -c Tag_ABI_VFP_args)"
if [[ ! $vfp_tag ]]; then
binary_arch="armelv5"
else
Expand Down

0 comments on commit 73ab718

Please sign in to comment.