Skip to content

fnmatch not matching glob syntax#7947

Merged
headius merged 1 commit intojruby:masterfrom
ahorek:fnmatch
Oct 5, 2023
Merged

fnmatch not matching glob syntax#7947
headius merged 1 commit intojruby:masterfrom
ahorek:fnmatch

Conversation

@ahorek
Copy link
Contributor

@ahorek ahorek commented Oct 5, 2023

closes #7942

@ahorek ahorek changed the title WIP: fnmatch not matching glob syntax fnmatch not matching glob syntax Oct 5, 2023
@headius
Copy link
Member

headius commented Oct 5, 2023

@ahorek This does look like the right fix. The failed path can be followed normally from the various break in the switch, but it needs an additional trigger at the top of the loop.

I'll merge and add some comments about why this code is duplicated.

@headius headius added this to the JRuby 9.4.4.0 milestone Oct 5, 2023
@headius headius merged commit bcdf196 into jruby:master Oct 5, 2023
headius added a commit that referenced this pull request Oct 5, 2023
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.

fnmatch not matching glob syntax in 9.3.8 but it did in 9.2.20.1

2 participants