update: prefix rm call with command in trap (#9107)

* Suppress the problematic trap output in check_upg The newly added trap, in systems where `rm` is aliased to `rm="rm -v"`, shows a message stating that "update.lock" has been removed each time `zsh` is called. I simply suppressed it with directing the output to `/dev/null`. * Use `command` instead of >/dev/null to suppress

update: prefix rm call with command in trap (#9107)
5c1a5c6c · M. Yas. Davoodeh · GitHub · e3131d98 · 5c1a5c6c
Unverified Commit 5c1a5c6c authored Jul 13, 2020 by M. Yas. Davoodeh Committed by GitHub Jul 13, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

check_for_upgrade.sh tools/check_for_upgrade.sh +1 -1

No files found.
--- a/tools/check_for_upgrade.sh
+++ b/tools/check_for_upgrade.sh
@@ -51,7 +51,7 @@ function update_ohmyzsh() {
    #  The return status from the function is handled specially. If it is zero, the signal is
    #  assumed to have been handled, and execution continues normally. Otherwise, the shell
    #  will behave as interrupted except that the return status of the trap is retained.
-    trap "rm -rf '$ZSH/log/update.lock'; return 1" EXIT INT QUIT
+    trap "command rm -rf '$ZSH/log/update.lock'; return 1" EXIT INT QUIT

    # Create or update .zsh-update file if missing or malformed
    if ! source "${ZSH_CACHE_DIR}/.zsh-update" 2>/dev/null || [[ -z "$LAST_EPOCH" ]]; then