Commit 2d87b6f
win: fix fs.realpath.native for long paths
Unlike other fs.js functions that work with paths, realpath.native isn't
using pathModule.toNamespacedPath prior to calling libuv function. This
is causing issues on windows.
Windows long path test is also improved to cover the mentioned issue.
Fixes: #39721
PR-URL: #44536
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>1 parent 8b256a3 commit 2d87b6fCopy full SHA for 2d87b6f
File tree
Expand file treeCollapse file tree
2 files changed
+5
-2
lines changedOpen diff view settings
Filter options
- lib
- test/parallel
Expand file treeCollapse file tree
2 files changed
+5
-2
lines changedOpen diff view settings
Collapse file
+2-2Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
2612 | 2612 | |
2613 | 2613 | |
2614 | 2614 | |
2615 | | - |
| 2615 | + |
2616 | 2616 | |
2617 | 2617 | |
2618 | 2618 | |
| ||
2772 | 2772 | |
2773 | 2773 | |
2774 | 2774 | |
2775 | | - |
| 2775 | + |
2776 | 2776 | |
2777 | 2777 | |
2778 | 2778 | |
|
Collapse file
test/parallel/test-fs-long-path.js
Copy file name to clipboardExpand all lines: test/parallel/test-fs-long-path.js+3Lines changed: 3 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
43 | 43 | |
44 | 44 | |
45 | 45 | |
| 46 | + |
| 47 | + |
| 48 | + |
46 | 49 | |
0 commit comments