Merge pull request #6 from mrymtsk/mac-notification-fix

Escape backslash in notification message for macOS
This commit is contained in:
Michael Aquilina 2019-07-17 13:38:32 +01:00 committed by GitHub
commit d6d00f5cc7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -18,13 +18,14 @@ function _auto_notify_message() {
local command="$1" local command="$1"
local elapsed="$2" local elapsed="$2"
# Run using echo -e in order to make sure notify-send picks up new line # Run using echo -e in order to make sure notify-send picks up new line
text="$(echo -e "\"$command\" has completed\n(Total time: $elapsed seconds)")" text_linux="$(echo -e "\"$command\" has completed\n(Total time: $elapsed seconds)")"
text_darwin="$(echo -e "\\\"$command\\\" has completed\n(Total time: $elapsed seconds)")"
platform="$(uname)" platform="$(uname)"
if [[ "$platform" == "Linux" ]]; then if [[ "$platform" == "Linux" ]]; then
notify-send "$text" notify-send "$text_linux"
elif [[ "$platform" == "Darwin" ]]; then elif [[ "$platform" == "Darwin" ]]; then
osascript -e "display notification \"$text\" with title \"Command Completed\"" osascript -e "display notification \"$text_darwin\" with title \"Command Completed\""
else else
printf "Unknown platform for sending notifications: $platform\n" printf "Unknown platform for sending notifications: $platform\n"
printf "Please post an issue on gitub.com/MichaelAquilina/zsh-auto-notify/issues/\n" printf "Please post an issue on gitub.com/MichaelAquilina/zsh-auto-notify/issues/\n"