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/firefox: init #174975
nixos/firefox: init #174975
Conversation
@mweinelt @lovesegfault I'm requesting review since you are maintainers of the existing Firefox package. Would you like to be added to the maintainers list of this module, too, if it is merged? |
I really think home-manager is the better place for a firefox module. https://github.com/nix-community/home-manager/blob/master/modules/programs/firefox.nix |
HM has its own problem and it's not official. I don't want to use HM but I want to config Firefox in my nixos config. For single-user system the module system just works and it can also be used as a system-wide default config for a multi-user system. |
Also, this creates a policy which prevents the settings from being overridden, which could be useful to an organisation. There is already a module like this for Chromium so I think it makes sense to add the Firefox equivalent. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/any-chance-to-have-a-firefox-module-in-nixos/23125/1 |
@danth ping |
For a single user, HM can be used as documented in the Wiki: https://nixos.wiki/wiki/Home_Manager#Usage_as_a_NixOS_module |
Is anyone interested on fixing the git conflicts? |
@AndersonTorres Conflicts are fixed now. |
Now I need someone to test it and someone to fix ofborg's complaint :) |
Converted the docs to Markdown. That should have fixed the errors regarding the manual |
God bless programs.firefox! |
Description of changes
Added a module which allows Firefox preferences and policies to be set in the NixOS configuration.
I know this can already be done by overriding the package, however:
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes