Inconsistent driver letters in debugger and integrated terminal

[IgorNovozhilov]

Inconsistent driver letters in debugger and integrated terminal. see Steps to Reproduce
To work correctly, you need to bring both driver letters, to either lowercase or uppercase.
Otherwise, problems arise when importing modules in NodeJS debug mode, similar to this: webpack/webpack#2362, is when the same module is imported twice and receives different instances executable code

• VSCode Version:
  Версия 1.22.0-insider
  Фиксация a716373
  Дата 2018-03-14T05:15:41.910Z
  Оболочка 1.7.9
  Отрисовщик 58.0.3029.110
  Узел 7.9.0
  Архитектура x64

• OS Version: win7 pro

Steps to Reproduce:

1. run code console.log(require('path').resolve('./test')); in integrated terminal
   

2. run code console.log(require('path').resolve('./test')); in debugger
   

see:
nodejs/node#6624

[IgorNovozhilov]

Repeated in VSCode
Версия 1.21.0
Фиксация 9a199d7
Дата 2018-03-07T11:04:09.969Z
Оболочка 1.7.9
Отрисовщик 58.0.3029.110
Узел 7.9.0
Архитектура x64

[Tyriar]

I seem to recall it's this way in the terminal for a reason, because some tool was failing when lowercase was being used. @weinand any thoughts on this?

[weinand]

@Tyriar VS Code normalizes drive letters to lower case when converting paths into urls and vice versa.

And the VS Code debugger started to do the same in the last milestone (see #43959). So expanded variables use lower case drive letters.

But see my comment #43959 (comment) why this doesn't help in cases where the drive letter casing comes from outside of VS Code.

How do you control the drive letter case in the integrated terminal?

[Tyriar]

@weinand

https://github.com/Microsoft/vscode/blob/1d026bbff95990e9ac381a2be85f54d2c7ba32d8/src/vs/workbench/parts/terminal/electron-browser/terminalInstance.ts#L995-L1001

And I found the answer to why it's uppercase: #9448

[Tyriar]

/cc @roblourens

[IgorNovozhilov]

Here's another example where the behavior is different

[weinand]

@DonJayamanne please create a new issue for this.

[vscodebot]

This issue is being closed to keep the number of issues in our inbox on a manageable level, we are closing issues that are not going to be addressed in the foreseeable future: We look at the number of votes the issue has received and the number of duplicate issues filed. More details here. If you disagree and feel that this issue is crucial: We are happy to listen and to reconsider.

If you wonder what we are up to, please see our roadmap and issue reporting guidelines.

Thanks for your understanding and happy coding!