Skip to content

Conversation

@xuhdev
Copy link

@xuhdev xuhdev commented Oct 19, 2016

Close #19.


If LS_COLORS is not defined, eval $(dircolors) should define a good default LS_COLORS. We can also add that when LS_COLORS is not defined.

@xuhdev xuhdev force-pushed the gnu-ls-color branch 2 times, most recently from 4ef41b3 to 066793c Compare October 19, 2016 08:56
@xuhdev
Copy link
Author

xuhdev commented Oct 19, 2016

It's totally possible to avoid calling ls by parsing $LS_COLORS, but can the shell script version do faster than leaving the task to a fast program in native code?

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

منيكاتي

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

مستر منيكاتي

# But we don't want to quote '.'; so instead we escape the escape manually and use q-
NAME="${${NAME##*/}//$'\e'/\\e}" # also propagate changes to SYMLINK_TARGET below

if [[ $IS_DIRECTORY == 1 ]]; then
Copy link

@ghost ghost Dec 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need help

Copy link

@ghost ghost Dec 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need help

@ghost
Copy link

ghost commented Dec 9, 2025

Help

@ghost
Copy link

ghost commented Dec 9, 2025

منيكاتي

heelp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve color handling

1 participant