> and is actually drive-letter agnostic
drive letter case-agnostic, I meant.
On Wednesday, March 20, 2013 9:07:35 PM UTC+4, Dmitriy Khudorozhkov wrote:
>
> 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.
