Skip to content

deps: cherry-pick libuv/libuv@e640dc9 - #65118

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
ulofiai:fix/issue-63638
Aug 14, 2026
Merged

deps: cherry-pick libuv/libuv@e640dc9#65118
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
ulofiai:fix/issue-63638

Conversation

@ulofiai

@ulofiai ulofiai commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Cherry-picks the upstream libuv fix for Windows fs-event watchers using 8.3 short paths.

When the watched directory is supplied in short-path form, resolving an event path to its long form can make it no longer share the stored directory prefix. Instead of asserting (or calculating an invalid relative path in release builds), fall back to the filename returned by ReadDirectoryChangesW, which is already relative to the watched directory. The regression test now creates a real directory with distinct long and short forms and tries both the Windows temporary directory and the current directory.

Fixes: #63638
Refs: libuv/libuv#5152
Refs: libuv/libuv@e640dc9

Signed-off-by: ulofiai <monsterking@tutamail.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/security-wg

@nodejs-github-bot nodejs-github-bot added libuv Issues and PRs related to the libuv dependency or the uv binding. needs-ci PRs that need a full CI run. labels Aug 7, 2026
@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels Aug 11, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 11, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@aduh95 aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 11, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 38e3955 into nodejs:main Aug 14, 2026
72 of 73 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 38e3955

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. libuv Issues and PRs related to the libuv dependency or the uv binding. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Potential regression with libuv in Node 24.16.0

3 participants