Skip to content

Cleanup stat usage#15077

Merged
Ericson2314 merged 1 commit intomasterfrom
stat-wrapper
Jan 26, 2026
Merged

Cleanup stat usage#15077
Ericson2314 merged 1 commit intomasterfrom
stat-wrapper

Conversation

@Ericson2314
Copy link
Member

Motivation

Use wrappers to make error handling easier.

Context

On Windows we are using proper 64-bit time and size info.

We still have the problem of no lstat on Windows, but this will be dealt with in future PRs.


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

@Ericson2314 Ericson2314 requested a review from edolstra as a code owner January 24, 2026 23:18
@github-actions github-actions bot added new-cli Relating to the "nix" command store Issues and pull requests concerning the Nix store fetching Networking with the outside (non-Nix) world, input locking labels Jan 24, 2026
Use wrappers to make error handling easier.

On Windows we are using proper 64-bit time and size info.

We still have the problem of no `lstat` on Windows, but this will be
dealt with in future PRs.
@Ericson2314 Ericson2314 added this pull request to the merge queue Jan 26, 2026
Merged via the queue into master with commit 252e4ee Jan 26, 2026
18 checks passed
@Ericson2314 Ericson2314 deleted the stat-wrapper branch January 26, 2026 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fetching Networking with the outside (non-Nix) world, input locking new-cli Relating to the "nix" command store Issues and pull requests concerning the Nix store

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants