Additional checks for -F support

This commit is contained in:
Nicholas Rodrigues Lordello 2023-11-02 13:00:58 +01:00 committed by Fabian Boehm
parent 5cf36bf3f8
commit 93b3a0c1f5

View file

@ -7,16 +7,18 @@ function ls --description "List contents of directory"
# Solaris 11's ls command takes a --color flag.
# OpenBSD requires the separate colorls program for color support.
# Also test -F because we'll want to define this function even with an ls that can't do colors (like NetBSD).
if not set -q __fish_ls_color_opt
set -g __fish_ls_color_opt
if not set -q __fish_ls_command
set -g __fish_ls_command ls
set -g __fish_ls_color_opt
set -g __fish_ls_indicators_opt
# OpenBSD ships a command called "colorls" that takes "-G" and "-F",
# but there's also a ruby implementation that doesn't understand "-F".
# Since that one's quite different, don't use it.
if command -sq colorls
and command colorls -GF >/dev/null 2>/dev/null
set -g __fish_ls_color_opt -G
set -g __fish_ls_command colorls
set -g __fish_ls_color_opt -G
set -g __fish_ls_indicators_opt -F
else
for opt in --color=auto -G --color
if command ls $opt / >/dev/null 2>/dev/null
@ -24,12 +26,16 @@ function ls --description "List contents of directory"
break
end
end
if command ls -F / >/dev/null 2>/dev/null
set -g __fish_ls_indicators_opt -F
end
end
end
set -l opt
set -l indicators_opt
isatty stdout
and set -a opt -F
and set -a indicators_opt $__fish_ls_indicators_opt
# Terminal.app doesn't set $COLORTERM or $CLICOLOR,
# but the new FreeBSD ls requires either to be set,
@ -40,5 +46,5 @@ function ls --description "List contents of directory"
test "$TERM_PROGRAM" = Apple_Terminal
and set -lx CLICOLOR 1
command $__fish_ls_command $__fish_ls_color_opt $opt $argv
command $__fish_ls_command $__fish_ls_color_opt $indicators_opt $argv
end