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

Commit

Permalink
Merge pull request #480 from adelyi/patch-1
Browse files Browse the repository at this point in the history
Update setup.sh
  • Loading branch information
ChrisTruncer authored Oct 14, 2022
2 parents caaf626 + 354a709 commit d8acd4c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions config/setup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
## Global variables
os="$( awk -F '=' '/^ID=/ {print $2}' /etc/os-release 2>&- )"

if [ "${os}" == "arch" ] \
if [ "${os}" == "aarch64" ] \
|| [ "${os}" == "manjaro" ]\
|| [ "${os}" == "blackarch" ] \
|| [ "${os}" == "debian" ] \
Expand Down Expand Up @@ -90,7 +90,7 @@ func_title(){
echo " os = ${os}"
echo " osversion = ${osversion}"
echo " osmajversion = ${osmajversion}"
echo " arch = ${arch}"
echo " arch = {aarch64}"
echo " trueuser = ${trueuser}"
echo " userprimarygroup = ${userprimarygroup}"
echo " userhomedir = ${userhomedir}"
Expand Down

1 comment on commit d8acd4c

@thesamesam
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't look right. It's replaced all "arch" with "aarch64". One instance of it meant "Arch Linux" and the other seemingly meant "architecture".

I don't see what this change was trying to achieve.

Please sign in to comment.