From 329db6f6de93772a56922657076661c98bf99f34 Mon Sep 17 00:00:00 2001
From: Eryn Wells <eryn@erynwells.me>
Date: Tue, 12 Nov 2024 13:47:20 -0800
Subject: [PATCH] [zsh] Configure ls(1) colors with environment variables
 instead of aliases

ls(1) on macOS and FreeBSD is the BSD version of the utility. As such it can be
configured to display directory contents in color using a command line argument
(-G) or environment variables (LSCOLORS). Using environment variables is nice
because it'll work regardless of the arguments you pass, so there's no need to
configure aliases.
---
 zsh/func/init-rc-ls | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

diff --git a/zsh/func/init-rc-ls b/zsh/func/init-rc-ls
index 0ae4e3b..5d2bcbd 100644
--- a/zsh/func/init-rc-ls
+++ b/zsh/func/init-rc-ls
@@ -3,23 +3,23 @@
 
 function init-rc-ls
 {
-    local ls_options='--color=auto'
-    alias ls="ls $ls_options"
     alias la="ls -A $ls_options"
     alias ll="ls -l $ls_options"
     alias l.="ls -d $ls_options .*"
 
-    local dircolors_bin=$(whence -p dircolors || whence -p gdircolors)
-    if [[ -n "$dircolors_bin" ]]; then
-        local dircolors_config
-        if [[ -f "$HOME/.dircolors/$SYS.cfg" ]]; then
-            dircolors_file="$HOME/.dircolors/$SYS.cfg"
-        else
-            dircolors_file="$HOME/.dircolors/default.cfg"
-        fi
+    # Enable ls colors
+    export CLICOLOR=1 COLORTERM=1
 
-        if [[ -f "$dircolors_config" ]]; then
-            eval $($dircolors_bin $dircolors_config)
+    # Define colors for ls. See the LSCOLORS documentation in ls(1).
+    # The default is "exfxcxdxbxegedabagacadah".
+    export LSCOLORS=Exdxcxfxbxegedabagacadah
+
+    local dircolors_bin=$(whence -p dircolors || whence -p gdircolors)
+    if [[ -x "$dircolors_bin" ]]; then
+        if [[ -f "$HOME/.dircolors/$SYS.cfg" ]]; then
+            eval $dircolors_bin "$HOME/.dircolors/$SYS.cfg"
+        elif [[ -f "$HOME/.dircolors/default.cfg" ]]; then
+            eval $dircolors_bin "$HOME/.dircolors/default.cfg"
         fi
     fi
 }