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
freedesktop modules: disable by default #45920
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.
I feel like this makes sense especially in the spirit of not polluting headless systems.
Maybe they can be disabled by default but auto-enabled if the X server is enabled? |
These only have effect when you add packages to systemPackages though. I think it’s safe to have it on by default IMO. |
That's exactly the case - |
@teto Could you please explain the issues you see with this? |
I am a supporter of the XDG spec that - has now outlived the X ecosystem since non X programs support it: ncmpcpp/neovim/khal/khard/... so I am kind of wary as to anything that makes it harder to use. Also I had missed the part |
@teto I'm not sure, I've never ever used mime stuff on a headless machine. I think we can set it to |
Any news on this? |
I still think we should do this. We now have some precedent in that |
That's mostly because it's not quite there yet to being able to be used by everyone, and it brings in rather large dependencies. |
Sure, but:
It's true that it's probably less annoying to unexpectedly have |
Thank you for your contributions.
|
Closing due to inactivity. Feel free to reopen the discussion. |
Motivation for this change
In the spirit of not asking for things unless they're actually needed, this sets the options introduced in #45058 to false by default.
This should only affect headless systems, since they're set in
xserver.nix
. I think it's plausible that you don't need most of these on a headless system. Maybe some people use MIME info, but I wouldn't have thought so.This should only be merged after the 18.09 branch off.
@jtojnar @samueldr
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)