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
nixos/locate: Fix sshfs exclusion #104727
Conversation
This makes merging less painful. This is nixfmt's preferred format.
I felt bad not checking the "Tested via one or more NixOS test(s)" box, so I made a locate test and verified this sshfs change. @SuperSandro2000 , who already reviewed this: Would you prefer that I bring the test into this PR, or create a separate PR for it? |
I think you can just add it to this PR. It fits topic wise and please don't feel bad. Having tests is nice and for some things a must have but I think I'm this case it is nice to have. |
e394d1a
to
3dd6ba3
Compare
Ok. Now with tests! |
@GrahamcOfBorg test locate |
hmmm the test probably needs an entry in nixos/tests/all-tests.nix |
The "fuse" and "sshfs" entries already present are not keeping this find invocation out of sshfs mounts, which present as fstype "fuse.sshfs"
3dd6ba3
to
86f0dc2
Compare
nixos/tests/all-tests.nix entry added. |
@GrahamcOfBorg test locate |
Motivation for this change
locate should not index sshfs mounts. These are already intended to be excluded. Fix the mechanism so they're actually excluded.
Things done
sandbox
innix.conf
on non-NixOS linux)