Require with an uppercase make a new instance of module on mac osx

[samrm111]

Given a file named test.js,
Using require in those two way will works, but node will create a new instance for each:
require('./test.js') and require('./Test.js')
Tested on Mac osx 10.11.1 with node 4.2.4

[bnoordhuis]

That's expected. HFS is (by default) case insensitive but node is not. You get the same behavior with NTFS on Windows.

[hgwood]

@bnoordhuis The documentation about this is a little bit hard to understand:

Modules are cached after the first time they are loaded. This means (among other things) that every call to require('foo') will get exactly the same object returned, if it would resolve to the same file.

Modules are cached based on their resolved filename. Since modules may resolve to a different filename based on the location of the calling module (loading from node_modules folders), it is not a guarantee that require('foo') will always return the exact same object, if it would resolve to different files.

In @samrm111's context, ./test.js and ./Test.js do resolve to the same file, so I would expect the same object to be returned. There's quite a long discussion about this in the 0.x archive.

If this issue is not something that will be fixed, could the documentation be updated to make the problem clear?

[bnoordhuis]

The documentation should probably clarify the distinction between file paths and file objects. If you open a pull request, I'll review it.

[hgwood]

I'm afraid I do not have the skills to do that :(.

[bnoordhuis]

I'll be happy to walk you through the process. The file you want to edit is doc/api/modules.markdown. Make sure you read CONTRIBUTING.md in the top-level directory as well. Let me know if you have questions.

[hgwood]

Oh I was not talking about how to contribute, that's fine, but about how to improve the doc and how the require function actually works. For example:

every call to require('foo') will get exactly the same object returned, if it would resolve to the same file.

I have no idea if that is 100% false or if it is true in some definitions of "resolve" and "file" that I don't understand.

Since modules may resolve to a different filename based on the location of the calling module

Same thing. What does "resolve" mean here? I'm not sure.

But, anyway, that's not an excuse isn't it? I'll run some experiments and come back with sufficient knowledge and/or questions. :)

[bnoordhuis]

You can find the logic for the module loader in lib/module.js, the logic in src/node.js is for built-in modules only.

'Resolve' means 'look up relative to the file calling require()` (with an exception for built-in modules, those always take precedence.)

The path to the included file is turned into an absolute path, symbolic links are followed and the result is used as the key in the module cache.

[hgwood]

@bnoordhuis

The path to the included file is turned into an absolute path

If that is so, why would the documentation say

Since modules may resolve to a different filename based on the location of the calling module

How can the same module be "resolved" to 2 different absolute paths without symlinks?

[hgwood]

Just opened a PR that doesn't modify any of the existing content but adds to it to explain this particular case.

[samrm111]

👍 thanks for the PR!