Merge pull request #400 from johannesgerer/master

Only use colors if stdout is a terminal.
8 years ago · 7ee214c9c2
parent f26a38e0f9 8fa8b06349
commit 7ee214c9c2
2 changed files with 10 additions and 2 deletions
--- a/bin/autojump.bash
+++ b/bin/autojump.bash
@ -61,7 +61,11 @@ j() {

    output="$(autojump ${@})"
    if [[ -d "${output}" ]]; then
-        echo -e "\\033[31m${output}\\033[0m"
+				if [ -t 1 ]; then  # if stdout is a terminal, use colors
+						echo -e "\\033[31m${output}\\033[0m"
+				else
+						echo -e "${output}"
+				fi
        cd "${output}"
    else
        echo "autojump: directory '${@}' not found"
--- a/bin/autojump.zsh
+++ b/bin/autojump.zsh
@ -52,7 +52,11 @@ j() {
    setopt localoptions noautonamedirs
    local output="$(autojump ${@})"
    if [[ -d "${output}" ]]; then
-        echo -e "\\033[31m${output}\\033[0m"
+				if [ -t 1 ]; then  # if stdout is a terminal, use colors
+						echo -e "\\033[31m${output}\\033[0m"
+				else
+						echo -e "${output}"
+				fi
        cd "${output}"
    else
        echo "autojump: directory '${@}' not found"