Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

extUri.relativePath returns wrong value if the path has UTF characters #174693

Closed
sandy081 opened this issue Feb 17, 2023 · 3 comments
Closed

extUri.relativePath returns wrong value if the path has UTF characters #174693

sandy081 opened this issue Feb 17, 2023 · 3 comments
Assignees
Labels
nodejs NodeJS support issues upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-linked This is an upstream issue that has been reported upstream

Comments

@sandy081
Copy link
Member

const actual = path.win32.relative('C:\\Users\\İ\\.vscode-insiders\\extensions', 'C:\\Users\\İ\\.vscode-insiders\\extensions\\ms-vscode.cpptools-1.14.3-win32-x64');
		assert.strictEqual(actual, 'ms-vscode.cpptools-1.14.3-win32-x64');
@bpasero
Copy link
Member

bpasero commented Feb 17, 2023

Relevant upstream issue: nodejs/node#27534
And suggested PR: nodejs/node#27662

I am not entirely convinced we should take a change here that was not accepted in node.js. We try to stay identical to node.js behaviour with our path library for good reason so that there is identical interop.

I would close this "as designed".

@bpasero bpasero added upstream Issue identified as 'upstream' component related (exists outside of VS Code) nodejs NodeJS support issues upstream-issue-linked This is an upstream issue that has been reported upstream labels Feb 17, 2023
@BridgeAR
Copy link

BridgeAR commented Aug 6, 2024

It just got fixed in Node.js and should be released soon in newer versions

@bpasero
Copy link
Member

bpasero commented Dec 16, 2024

We will monitor upstream and pull it in once available for us.

@bpasero bpasero closed this as not planned Won't fix, can't repro, duplicate, stale Dec 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
nodejs NodeJS support issues upstream Issue identified as 'upstream' component related (exists outside of VS Code) upstream-issue-linked This is an upstream issue that has been reported upstream
Projects
None yet
Development

No branches or pull requests

5 participants