Skip to content

More std::filesystem::path#14974

Draft
Ericson2314 wants to merge 4 commits intomasterfrom
more-mingw-fixes
Draft

More std::filesystem::path#14974
Ericson2314 wants to merge 4 commits intomasterfrom
more-mingw-fixes

Conversation

@Ericson2314
Copy link
Member

Motivation

Progress on #9205

Context


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 11, 2026 02:44
@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 11, 2026
@Ericson2314 Ericson2314 changed the title More mingw fixes More std::filesystem::path Jan 11, 2026
return std::find(normalized.begin(), normalized.end(), "profiles") == normalized.end()
&& std::filesystem::is_symlink(profileDir);
}())
profileDir = std::filesystem::read_symlink(profileDir);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this can throw, needs try except

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I will convert these back, we have the wrapper functions now

profileDir = readLink(profileDir.string());
while ([&] {
auto normalized = profileDir.lexically_normal();
return std::find(normalized.begin(), normalized.end(), "profiles") == normalized.end()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe throws?

@Ericson2314 Ericson2314 marked this pull request as draft January 13, 2026 18:59
@Ericson2314
Copy link
Member Author

Draft because #14973 should be merged first.

@Ericson2314 Ericson2314 force-pushed the more-mingw-fixes branch 6 times, most recently from 1b9bf1f to 5d1fd3f Compare January 27, 2026 04:15
@github-actions github-actions bot added the c api Nix as a C library with a stable interface label Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c api Nix as a C library with a stable interface 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.

3 participants