Skip to content
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

jekyll doctor should print a warning if two permalinks only differ by case #3035

Closed
akoeplinger opened this issue Oct 28, 2014 · 6 comments · Fixed by #3171
Closed

jekyll doctor should print a warning if two permalinks only differ by case #3035

akoeplinger opened this issue Oct 28, 2014 · 6 comments · Fixed by #3171

Comments

@akoeplinger
Copy link
Contributor

(This is a continuation from #3034)

If I'm on a case-insensitive file system and create two pages with permalinks that only differ by case, the site generation overwrites one of the pages. This should result in a jekyll doctor warning.

@parkr
Copy link
Member

parkr commented Oct 28, 2014

Case insensitive file system? What kind of tom-foolery is this?!

I don't actually know how Ruby would know whether you're on a case-insensitive filesystem... so not sure how we would show this. But you can give it a shot if you want!

@akoeplinger
Copy link
Contributor Author

I don't think it'd actually need to know if I'm on a case-insensitive FS right now, as someone else might be and run into a problem (as I showed in jekyll/jekyll-redirect-from#51). So I'd say actively discourage two permalinks whose strings are equal-ignore-case.

@parkr
Copy link
Member

parkr commented Oct 28, 2014

If you open up _site on that system, do you see download and Download directories? If so, it's a WEBrick problem. URL's are case-insensitive, I think.

@akoeplinger
Copy link
Contributor Author

On a case-insensitive FS there's no way to have a download and a Download folder, you'd get an exception when trying to create the second one. WEBrick just serves what it sees on the FS, so if you're on a case-insensitive FS the URLs are case-insensitive too, otherwise they are not, like on Linux.

@parkr
Copy link
Member

parkr commented Nov 29, 2014

Want to code this up?

akoeplinger added a commit to akoeplinger/jekyll that referenced this issue Nov 29, 2014
Those URLs are problematic on case-insensitive file systems because one of the URLs is overwritten by the other.
Fixes jekyll#3035
akoeplinger added a commit to akoeplinger/jekyll that referenced this issue Nov 29, 2014
Those URLs are problematic on case-insensitive file systems because one of the URLs is overwritten by the other.
Fixes jekyll#3035
@akoeplinger
Copy link
Contributor Author

@parkr Here you go: #3171

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants