`glob_with` case insensitivity seems broken
Open
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 595
- Forks
- 93
- PR merge metrics
- No merged PRs in 30d
Description
In developing globtest (crate here), I seem to have run into an issue where the case_sensitive paremeter of MatchOptions is completely ignored in the match_with function.
Here's an example:
mqudsi@ZBook /m/c/U/M/g/globtest> globtest -d "*"
options {
case_sensitive: true,
require_literal_leading_dot: true,
require_literal_separator: true
};
Cargo.lock
Cargo.toml
LICENSE
README.md
src
target
mqudsi@ZBook /m/c/U/M/g/globtest> globtest -d "target"
options {
case_sensitive: true,
require_literal_leading_dot: true,
require_literal_separator: true
};
target
mqudsi@ZBook /m/c/U/M/g/globtest> globtest -id "Target"
options {
case_sensitive: false,
require_literal_leading_dot: true,
require_literal_separator: true
};
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with MatchOptions and the match_with function, then reproduce the report using the globtest commands and the target/Target examples. Trace how case_sensitive is handled and add coverage for both true and false settings; done means case-sensitive matching distinguishes the names while case-insensitive matching finds target for Target.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- tooling
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100