Hi everyone,
Windows 7; current master fails for me on simple/test-fs-realpath test with
the following:
assert.js:102
throw new assert.AssertionError({
^
AssertionError: realpathSync("..") expected: c:\Users\Dima\Documents\GitHub
actual:C:\Users\Dima\Documents\GitHub
at Object.<anonymous>
(C:\Users\Dima\Documents\GitHub\node\test\simple\test-fs-realpath.js:396:8)
at Module._compile (module.js:456:26)
at Object.Module._extensions..js (module.js:474:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:312:12)
at Function.Module.runMain (module.js:497:10)
at startup (node.js:119:16)
at node.js:903:3
The problem is that path.join (actually, path.normalize inside it) converts
drive letter to lower case. But this is not always correct for Windows (it
depends on the command line prompt you run from); I always have upper-case
C drive, for example. Windows is not consistent with drive letter case and
is actually drive-letter agnostic.
I'm not sure what to do with this, thus didn't yet post this to Github
issues. Worth making this test case-agnostic for Windows, maybe?
Regards,
Dmitriy.
--
--
Job Board: http://jobs.nodejs.org/
Posting guidelines:
https://github.com/joyent/node/wiki/Mailing-List-Posting-Guidelines
You received this message because you are subscribed to the Google
Groups "nodejs" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/nodejs?hl=en?hl=en
---
You received this message because you are subscribed to the Google Groups
"nodejs" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
