From d20cb23998c672fea3ef344d836102a781517c80 Mon Sep 17 00:00:00 2001
From: Tom Ryder <tom@sanctum.geek.nz>
Date: Sat, 30 May 2015 23:56:54 +1200
Subject: Overhaul grep(1) and ls(1) options and wrappers

---
 bash/bashrc.d/grep.bash | 51 ++++++++++++-------------------------------------
 bash/bashrc.d/ls.bash   | 37 ++++++++++++-----------------------
 2 files changed, 24 insertions(+), 64 deletions(-)

(limited to 'bash/bashrc.d')

diff --git a/bash/bashrc.d/grep.bash b/bash/bashrc.d/grep.bash
index 4560c4ac..559b5e09 100644
--- a/bash/bashrc.d/grep.bash
+++ b/bash/bashrc.d/grep.bash
@@ -1,40 +1,13 @@
-# Return appropriate options for grep
-grepopts() {
-
-    # Start with ignoring binary files
-    local -a grepopts=(-I)
-
-    # Snarf the output of `grep --help` into a variable
-    local grephelp=$(grep --help 2>/dev/null)
-
-    # If the --exclude option is available, exclude some VCS files
-    if [[ $grephelp == *--exclude* ]] ; then
-        for exclude_file in .gitignore .gitmodules ; do
-            grepopts=("${grepopts[@]}" --exclude="$exclude_file")
-        done
-    fi
-
-    # If the --exclude-dir option is available, exclude some VCS dirs
-    if [[ $grephelp == *--exclude-dir* ]] ; then
-        for exclude_dir in .cvs .git .hg .svn ; do
-            grepopts=("${grepopts[@]}" --exclude-dir="$exclude_dir")
-        done
-    fi
-
-    # If the --color option is available and we have a terminal that supports
-    # at least eight colors, add --color=auto to the options
-    local colors=$(tput colors 2>/dev/null)
-    if [[ $grephelp == *--color* ]] && ((colors >= 8)) ; then
-        grepopts=("${grepopts[@]}" --color=auto)
-    fi
-
-    # Print the options as a single string, space-delimited
-    printf %s "${grepopts[*]}"
-}
-
-# Alias grep with those options
-alias grep="grep $(grepopts)"
-
-# Unset helper function
-unset -f grepopts
+# Define function wrapper for grep(1) with --color option if GREP_COLORS is
+# set; checks that color is available in the terminal within the function
+if [[ $GREP_COLORS ]] ; then
+    grep() {
+        local -i colors=$(tput colors 2>/dev/null)
+        if ((colors >= 8)) ; then
+            command grep --color "$@"
+        else
+            command grep "$@"
+        fi
+    }
+fi
 
diff --git a/bash/bashrc.d/ls.bash b/bash/bashrc.d/ls.bash
index a559a44a..322644c5 100644
--- a/bash/bashrc.d/ls.bash
+++ b/bash/bashrc.d/ls.bash
@@ -1,26 +1,13 @@
-# Return appropriate options for ls
-lsopts() {
-
-    # Snarf the output of `ls --help` into a variable
-    local lshelp=$(ls --help 2>/dev/null)
-
-    # Start collecting available options
-    local -a lsopts
-
-    # If the --color option is available and we have a terminal that supports
-    # at least eight colors, add --color=auto to the options
-    local colors=$(tput colors 2>/dev/null)
-    if [[ $lshelp == *--color* ]] && ((colors >= 8)) ; then
-        lsopts=("${lsopts[@]}" --color=auto)
-    fi
-
-    # Print the options as a single string, space-delimited
-    printf %s "${lsopts[*]}"
-}
-
-# Alias ls with these options
-alias ls="ls $(lsopts)"
-
-# Unset helper function
-unset -f lsopts
+# Define function wrapper for ls(1) with --color option if LS_COLORS is set;
+# checks that color is available in the terminal within the function
+if [[ $LS_COLORS ]] ; then
+    ls() {
+        local colors=$(tput colors 2>/dev/null)
+        if ((colors >= 8)) ; then
+            command ls --color=auto "$@"
+        else
+            command ls "$@"
+        fi
+    }
+fi
 
-- 
cgit v1.2.3