Mention require's case-sensitivity in documentation #17249
Conversation
| @@ -1489,7 +1489,7 @@ This function is typically used to load library code, and is implicitly called b | |||
| load packages. | |||
|
|
|||
| When searching for files, `require` first looks for package code under `Pkg.dir()`, then tries | |||
| paths in the global array `LOAD_PATH`. | |||
| paths in the global array `LOAD_PATH`. `require` is case-sensitive on all platforms including those with case-insensitive filesystems like macOS and Windows. | |||
There was a problem hiding this comment.
Apple renamed it to macOS officially.
There was a problem hiding this comment.
Also, I don't think Windows is case-insensitive, is it?
EDIT: I see it's complicated, the filesystem is case-sensitive but the OS generally isn't.
There was a problem hiding this comment.
Apple renamed it to macOS officially.
OK, my bad.
There was a problem hiding this comment.
NTFS is case-sensitive! I did not know that.
From Wikipedia
Current Windows file systems, like NTFS, are case-sensitive; that is a readme.txt and a Readme.txt can exist in the same directory. However, the API for file access in Windows applications is case-insensitive, which makes filenames case-insensitive from the application's point of view.
I am not sure whether I should keep windows.
There was a problem hiding this comment.
IIUC, NTFS can be case-sensitive, but Windows doesn't use that feature, so it's good idea to keep it.
There was a problem hiding this comment.
You can change a setting to make your filesystem case sensitive, and Cygwin can use things that way, but Win32 generally doesn't differentiate.
Please wrap docstring lines.
There was a problem hiding this comment.
wrapped lines now.
kshyatt
changed the title
| @@ -1489,7 +1489,9 @@ This function is typically used to load library code, and is implicitly called b | |||
| load packages. | |||
|
|
|||
| When searching for files, `require` first looks for package code under `Pkg.dir()`, then tries | |||
| paths in the global array `LOAD_PATH`. | |||
| paths in the global array `LOAD_PATH`. `require` is case-sensitive on all | |||
| platforms including those with case-insensitive filesystems like macOS and | |||
There was a problem hiding this comment.
You need a comma before the word "including", and "MacOS" is capitalized.
There was a problem hiding this comment.
"MacOS" is capitalized.
Not anymore....
There was a problem hiding this comment.
Oh wow, I just noticed Apple's latest rebranding as "macOS".
Mention require's case-sensitivity in documentation
fixes #13579