New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fileSystems: strip trailing slash of mountPoints #95308
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like a simple change that looks fine to me... but I'm going to defer to someone who might know better if there are any implications to this change.
👍
What do you think about this solution: #95406 |
Hm, I'd prefer if trivial things like these were just corrected automatically. Preventing duplicates would be a desirable property though. Would it perhaps be possible to strip the slash earlier, during the merge? I'm not too familiar how these submodules with custom names work and how they're merged unfortunately. |
I think there is an |
7c88420
to
a3394ac
Compare
There is indeed and it's a cleaner solution to the problem. I could not figure out how to make it work with submodule names though, so duplicates are still possible. The purpose of this PR isn't to sanitise bad user configs but rather work around a bug in our handling of essential mountPoints, so that's fine. |
61f8a30
to
12d0004
Compare
12d0004
to
86f71da
Compare
They are semantically the same as the non-slash version and therefore are potential source of duplicates. Also fixes NixOS#78951 Alternative to NixOS#95308
I marked this as stale due to inactivity. → More info |
since #95406 is merged i assume this can be closed |
Motivation for this change
Fixes #78951
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)