fix(brew): Prevent loading Homebrew if user is root (thanks to @HastD)

fiftydinar · web-flow · commit e66445df325f · 2025-03-10T08:32:01.000+01:00
diff --git a/modules/brew/brew-bash-completions.sh b/modules/brew/brew-bash-completions.sh
@@ -3,7 +3,7 @@
 # Completion is in sh to account for the zsh syntax & when zsh tries to source scripts from /etc/profiles
 
 # Check for interactive bash and that we haven't already been sourced.
-if [ "x${BASH_VERSION-}" != x -a "x${PS1-}" != x -a "x${BREW_BASH_COMPLETION-}" = x ]; then
+if [ "x${BASH_VERSION-}" != x -a "x${PS1-}" != x -a "x${BREW_BASH_COMPLETION-}" = x -a "$(/usr/bin/id -u)" != 0 ]; then
 
     # Check for recent enough version of bash.
     if [ "${BASH_VERSINFO[0]}" -gt 4 ] ||
diff --git a/modules/brew/brew-fish-completions.fish b/modules/brew/brew-fish-completions.fish
@@ -1,6 +1,6 @@
 #!/usr/bin/env fish
 #shellcheck disable=all
-if status --is-interactive
+if status --is-interactive and if test $(/usr/bin/id -u) -ne 0
     if [ -d /home/linuxbrew/.linuxbrew ]
         eval "$(/home/linuxbrew/.linuxbrew/bin/brew shellenv)"
         if test -d (brew --prefix)/share/fish/completions
diff --git a/modules/brew/brew.sh b/modules/brew/brew.sh
@@ -215,7 +215,9 @@ if [[ ! -f "/etc/profile.d/brew.sh" ]]; then
   echo "Apply brew path export fix, to solve path conflicts between system & brew programs with same name"
   cat > /etc/profile.d/brew.sh <<EOF
 #!/usr/bin/env bash
-[[ -d /home/linuxbrew/.linuxbrew && \$- == *i* ]] && eval "\$(/home/linuxbrew/.linuxbrew/bin/brew shellenv)"
+if [[ -d /home/linuxbrew/.linuxbrew && \$- == *i* && "\$(/usr/bin/id -u)" != 0 ]]; then
+  eval "\$(/home/linuxbrew/.linuxbrew/bin/brew shellenv)"
+fi
 EOF
 fi
 
