-
-
Notifications
You must be signed in to change notification settings - Fork 15.5k
Comparing changes
Open a pull request
base repository: NixOS/nixpkgs
base: 30bfbe7e81b6
head repository: NixOS/nixpkgs
compare: 6ea79d2707e0
- 12 commits
- 11 files changed
- 1 contributor
Commits on Jan 30, 2020
-
Configuration menu - View commit details
-
Copy full SHA for 547b91b - Browse repository at this point
Copy the full SHA 547b91bView commit details -
nixos/mailman: siteOwner default -> example
A default of example.com is useful to nobody. The correct value of this depends on the system.
Configuration menu - View commit details
-
Copy full SHA for 112fa07 - Browse repository at this point
Copy the full SHA 112fa07View commit details -
nixos/mailman: don't keep secrets in the Nix store
This replaces all Mailman secrets with ones that are generated the first time the service is run. This replaces the hyperkittyApiKey option, which would lead to a secret in the world-readable store. Even worse were the secrets hard-coded into mailman-web, which are not just world-readable, but identical for all users! services.mailman.hyperkittyApiKey has been removed, and so can no longer be used to determine whether to enable Hyperkitty. In its place, there is a new option, services.mailman.hyperkitty.enable. For consistency, services.mailman.hyperkittyBaseUrl has been renamed to services.mailman.hyperkitty.baseUrl.
Configuration menu - View commit details
-
Copy full SHA for c397d19 - Browse repository at this point
Copy the full SHA c397d19View commit details -
Configuration menu - View commit details
-
Copy full SHA for f254fb9 - Browse repository at this point
Copy the full SHA f254fb9View commit details -
Configuration menu - View commit details
-
Copy full SHA for bc8eece - Browse repository at this point
Copy the full SHA bc8eeceView commit details -
Configuration menu - View commit details
-
Copy full SHA for db0a371 - Browse repository at this point
Copy the full SHA db0a371View commit details -
nixos/mailman: don't set Postfix hashes
It's likely that a user might want to set multiple values for relay_domains, transport_maps, and local_recipient_maps, and the order is significant. This means that there's no good way to set these across multiple NixOS modules, and they should probably all be set together in the user's Postfix configuration. So, rather than setting these in the Mailman module, just make the Mailman module check that the values it needs to occur somewhere, and advise the user on what to set if not.
Configuration menu - View commit details
-
Copy full SHA for 8d9636e - Browse repository at this point
Copy the full SHA 8d9636eView commit details -
mailman: init package for Mailman CLI
We already had python3Packages.mailman, but that's only really usable as a library. The only other option was to create a whole Python environment, which was undesirable to install as a system-wide package.
Configuration menu - View commit details
-
Copy full SHA for a8538a7 - Browse repository at this point
Copy the full SHA a8538a7View commit details -
mailman-web: use upstream, improve NixOS module
Previously, some files were copied into the Nixpkgs tree, which meant we wouldn't easily be able to update them, and was also just messy. The reason it was done that way before was so that a few NixOS options could be substituted in. Some problems with doing it this way were that the _package_ changed depending on the values of the settings, which is pretty strange, and also that it only allowed those few settings to be set. In the new model, mailman-web is a usable package without needing to override, and I've implemented the NixOS options in a much more flexible way. NixOS' mailman-web config file first reads the mailman-web settings to use as defaults, but then it loads another configuration file generated from the new services.mailman.webSettings option, so _any_ mailman-web Django setting can be customised by the user, rather than just the three that were supported before. I've kept the old options, but there might not really be any good reason to keep them.
Configuration menu - View commit details
-
Copy full SHA for 881dd99 - Browse repository at this point
Copy the full SHA 881dd99View commit details -
Configuration menu - View commit details
-
Copy full SHA for 8f4fd4d - Browse repository at this point
Copy the full SHA 8f4fd4dView commit details -
nixos/mailman: make mailman package configurable
This will allow users to provide other archiver plugins than the default mailman-hyperkitty.
Configuration menu - View commit details
-
Copy full SHA for 0167eb3 - Browse repository at this point
Copy the full SHA 0167eb3View commit details -
Configuration menu - View commit details
-
Copy full SHA for 6ea79d2 - Browse repository at this point
Copy the full SHA 6ea79d2View commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff 30bfbe7e81b6...6ea79d2707e0