src: expose Windows-only fs open flags by PickBas · Pull Request #64775 · nodejs/node · GitHub
Skip to content

src: expose Windows-only fs open flags - #64775

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
JaneaSystems:win-new-constants
Aug 7, 2026
Merged

src: expose Windows-only fs open flags#64775
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
JaneaSystems:win-new-constants

Conversation

@PickBas

@PickBas PickBas commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

src: expose Windows-only fs open flags

libuv honors several Windows-only open flags but Node never exported the constants, so they were unreachable from fs.open(). Expose UV_FS_O_TEMPORARY, UV_FS_O_SHORT_LIVED, UV_FS_O_SEQUENTIAL and UV_FS_O_RANDOM under the UV_FS_O_ prefix, matching the existing UV_FS_O_FILEMAP. They are 0 on non-Windows platforms.

UV_FS_O_TEMPORARY deletes the file when the last handle closes, UV_FS_O_SHORT_LIVED avoids flushing a throwaway file to disk, and UV_FS_O_SEQUENTIAL/UV_FS_O_RANDOM hint the access pattern to optimize caching.

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. labels Jul 27, 2026
@PickBas
PickBas force-pushed the win-new-constants branch from 3fb62df to 23c32f5 Compare July 27, 2026 07:43
@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

@StefanStojanovic StefanStojanovic added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Jul 28, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Jul 28, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@StefanStojanovic StefanStojanovic added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Aug 3, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Aug 3, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@StefanStojanovic StefanStojanovic added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Aug 4, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Aug 4, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Signed-off-by: Kirill Saied <sayed.kirill@gmail.com>
@PickBas
PickBas force-pushed the win-new-constants branch from 23c32f5 to cee62bb Compare August 6, 2026 11:36
@StefanStojanovic StefanStojanovic added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Aug 6, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Aug 6, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@StefanStojanovic StefanStojanovic added the commit-queue PRs queued for automated landing through the Commit Queue. label Aug 7, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Aug 7, 2026
@nodejs-github-bot
nodejs-github-bot merged commit 985aa07 into nodejs:main Aug 7, 2026
71 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

aduh95 pushed a commit that referenced this pull request Aug 13, 2026
Signed-off-by: Kirill Saied <sayed.kirill@gmail.com>
PR-URL: #64775
Reviewed-By: Aviv Keller <me@aviv.sh>
aduh95 pushed a commit that referenced this pull request Aug 25, 2026
Signed-off-by: Kirill Saied <sayed.kirill@gmail.com>
PR-URL: #64775
Reviewed-By: Aviv Keller <me@aviv.sh>
aduh95 pushed a commit that referenced this pull request Aug 27, 2026
Signed-off-by: Kirill Saied <sayed.kirill@gmail.com>
PR-URL: #64775
Reviewed-By: Aviv Keller <me@aviv.sh>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants