-
-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
Upper case letters in path causing match to fail #7686
Comments
I accidentally duped this bug (as jmooring noticed) - but it still happened 29 days ago on the most recent hugo back then:
|
Still suffering this bug in the latest Hugo version:
|
This issue has been automatically marked as stale because it has not had recent activity. The resources of the Hugo team are limited, and so we are asking for your help. |
Still reproducible with v0.100.1 MRE git clone --single-branch -b hugo-github-issue-7686 https://github.com/jmooring/hugo-testing hugo-github-issue-7686
cd hugo-github-issue-7686
hugo server |
@jmooring have you tested it with ... Hugo 0.104.0? |
Yes, still fails. |
That is certainly odd, oh well. What I wish for my next life is that every OS implemented a file system with forward slashes and filenames limited to lower case ascii. |
#10305 did not consider case-unmatched patterns. However, it is unexpected that Seemingly, we need more test and fixations. |
- add integration tests - fix not found error in `walk.go` Fix gohugoio#7686 Fix gohugoio#10112 - ref - gohugoio#10305 - gohugoio@281554e
- add integration tests - fix not found error in `walk.go` Fix gohugoio#7686 Fix gohugoio#10112 - ref - gohugoio#10305 - gohugoio@281554e
- add integration tests - fix not found error in `walk.go` Fix gohugoio#7686 Fix gohugoio#10112 - ref - gohugoio#10305 - gohugoio@281554e
- add integration tests - fix not found error in `walk.go` Fix gohugoio#7686 Fix gohugoio#10112 - ref - gohugoio#10305 - gohugoio@281554e
- add integration tests - use canonical path for glog - fix cache key for resources Fix gohugoio#7686 Fix gohugoio#10112 - ref - gohugoio#10305 - gohugoio@281554e
I wish for operating system APIs that handle path components more like
objects than strings, and for all APIs to just automatically do the right
thing for UTF8, and that every programmer wonders every time they touch a
string, "will this work for UTF8?"
But I have screwed it up myself plenty of times so I know that's a pipe
dream.
…On Fri, Sep 23, 2022, 11:12 Bjørn Erik Pedersen ***@***.***> wrote:
Yes, still fails.
That is certainly odd, oh well.
What I wish for my next life is that very OS implemented a file system
with forward slashes and filenames limited to lower case ascii.
—
Reply to this email directly, view it on GitHub
<#7686 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AALGLJIMKDBGHFIEPCEJQKDV7XJF3ANCNFSM4RLJMUGQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
What version of Hugo are you using (
hugo version
)?Does this issue reproduce with the latest release?
yes
Probably easiest to read jmoorings comment on the forum:
https://discourse.gohugo.io/t/upper-case-letters-in-path-causing-match-to-fail/28069
Basically there seems to be an issue when matching folder names that have upper case letters in. jmooring suggests it is because case normalisation is being done on the pattern, rather than on the directory path.
The text was updated successfully, but these errors were encountered: