Merge pull request #5898 from MikeMcQuaid/no-autoupdate-no-params

brew.sh: don’t always autoupdate with no params.

Library/Homebrew/brew.sh

@@ -416,8 +416,22 @@ update-preinstall() {
   [[ -z "$HOMEBREW_HELP" ]] || return
   [[ -z "$HOMEBREW_NO_AUTO_UPDATE" ]] || return
   [[ -z "$HOMEBREW_AUTO_UPDATING" ]] || return
-  [[ -z "$HOMEBREW_AUTO_UPDATE_CHECKED" ]] || return
   [[ -z "$HOMEBREW_UPDATE_PREINSTALL" ]] || return
+  [[ -z "$HOMEBREW_AUTO_UPDATE_CHECKED" ]] || return
+
+  # If we've checked for updates, we don't need to check again.
+  export HOMEBREW_AUTO_UPDATE_CHECKED="1"
+
+  if [[ "$HOMEBREW_COMMAND" = "cask" ]]
+  then
+    if [[ "$HOMEBREW_CASK_COMMAND" != "upgrade" && $HOMEBREW_ARG_COUNT -lt 3 ]]
+    then
+      return
+    fi
+  elif [[ "$HOMEBREW_COMMAND" != "upgrade" && $HOMEBREW_ARG_COUNT -lt 2 ]]
+  then
+    return
+  fi
 
   if [[ "$HOMEBREW_COMMAND" = "install" || "$HOMEBREW_COMMAND" = "upgrade" ||
         "$HOMEBREW_COMMAND" = "bump-formula-pr" ||
@@ -432,9 +446,6 @@ update-preinstall() {
       timer_pid=$!
     fi
 
-    # Allow auto-update migration now we have a fix in place (below in this function).
-    export HOMEBREW_ENABLE_AUTO_UPDATE_MIGRATION="1"
-
     brew update --preinstall
 
     if [[ -n "$timer_pid" ]]
@@ -445,15 +456,9 @@ update-preinstall() {
 
     unset HOMEBREW_AUTO_UPDATING
 
-    # If we've checked for updates, we don't need to check again.
-    export HOMEBREW_AUTO_UPDATE_CHECKED="1"
-
     # exec a new process to set any new environment variables.
     exec "$HOMEBREW_BREW_FILE" "$@"
   fi
-
-  # If we've checked for updates, we don't need to check again.
-  export HOMEBREW_AUTO_UPDATE_CHECKED="1"
 }
 
 if [[ -n "$HOMEBREW_BASH_COMMAND" ]]