Skip to content

🐛 fix(#172): capital letters with wildcard matching#245

Draft
IGLOU-EU wants to merge 1 commit into
muesli:masterfrom
IGLOU-EU:issue172
Draft

🐛 fix(#172): capital letters with wildcard matching#245
IGLOU-EU wants to merge 1 commit into
muesli:masterfrom
IGLOU-EU:issue172

Conversation

@IGLOU-EU

@IGLOU-EU IGLOU-EU commented May 8, 2023

Copy link
Copy Markdown
Contributor

fix #172

Signed-off-by: Adrien Kara <adrien@iglou.eu>
@muesli

muesli commented Mar 11, 2024

Copy link
Copy Markdown
Owner

Nice one, but I wonder if this should be an option and/or really the default.

@IGLOU-EU
IGLOU-EU marked this pull request as draft September 16, 2024 22:38
imabug added a commit to imabug/duf that referenced this pull request Mar 29, 2025
Comment thread man.go
$ duf --hide-mp /,/home,/dev

Wildcards inside quotes work:
Wildcards inside quotes work and are case-sensitive:

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

With the code change this should say 'case-insensitive', should it not?

@DrHyde

DrHyde commented May 13, 2026

Copy link
Copy Markdown

Nice one, but I wonder if this should be an option and/or really the default.

Maybe it should be the default on Macs only, as that's the only major platform that suffers from this filesystem stupidity?

@JonathanDoughty

Copy link
Copy Markdown

I have avoided this problem for years by commenting out

v = strings.ToLower(v)

in main.go (now line 112)

Maybe I'm missing something but I don't see why duf should be lower-casing any information that users input to match what their systems' other utilities use. I hide a whole bunch of repetitive stuff that macOS otherwise produces with that one line source change and the other capabilities that duf provides as part of a shell function that executes:

duf --only local --hide-mp '*/.timemachine/*,/System/**,/private/var/folders/*,/Library/Application Support/**'

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.

Matching fail with maj key

4 participants