-
-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sensitive cases #69
base: master
Are you sure you want to change the base?
Sensitive cases #69
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm this is interesting. My understanding was that GNU ls was always ASCII-case-insensitive on matching extensions. But it looks like that only changed in 2018: coreutils/coreutils@24053fb.
As of March 2023 they now do a "smart case" type thing: coreutils/coreutils@47988fa
The test mentioned is also the one I worked on to succeed. btw, it wont since nu-ansi-term seems to not keep the color codes ordered. |
if there's a particular order they're supposed to be in, we at nu-ansi-term, would probably accept a PR for this ordering. |
New Feature: case sensitive parsing of LS_COLORS
disclaimer: it looks like a lot of changes, it is, but thats due to the fact that its based on another PR
...