`glob.translate("a/**/c", recursive=True, include_hidden=True)` does not match "a//c"

[ap--]

Bug report

Bug description:

The pattern returned by glob.translate("a/**/c", recursive=True, include_hidden=True) currently does not match paths that have two slashes "//" between two parts:

import glob

pattern = glob.translate("a/**/c", recursive=True, include_hidden=True)
# '(?s:a/(?:.+/)?c)\\Z'

assert re.match(pattern, "a/c")    # match
assert re.match(pattern, "a/b/c")  # match
assert re.match(pattern, "a//c")   # None
assert re.match(pattern, "a///c")  # match

It looks like the fix should be simple and requires replacing .+ with .* in

cpython/Lib/glob.py

Line 280 in d2f1b0e

any_segments = f'(?:.+{any_sep})?'

I'd be happy to add tests and apply the fix in a PR.

Cheers,
Andreas

CPython versions tested on:

CPython main branch

Operating systems tested on:

macOS

Linked PRs

• gh-116393: glob.translate match empty path parts with "**" when recursive=True #116565

[barneygale]

The include_hidden=False case will also need a fix I think -- specifically any_segment and any_last_segment.