Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: NixOS/nixpkgs
base: ce12576dd0cc
Choose a base ref
...
head repository: NixOS/nixpkgs
compare: fc316f7b319a
Choose a head ref
  • 1 commit
  • 1 file changed
  • 1 contributor

Commits on Mar 22, 2020

  1. nixos/ssmtp: declare all option renames manually

    While renaming `networking.defaultMailServer` directly to
    `services.ssmtp` is shorter and probably clearer, it causes eval errors
    due to the second rename (directDelivery -> enable) when using e.g. `lib.mkForce`.
    
    For instance,
    
    ``` nix
    { lib, ... }: {
      networking.defaultMailServer = {
        hostName = "localhost";
        directDelivery = lib.mkForce true;
        domain = "example.org";
      };
    }
    ```
    
    would break with the following (rather confusing) error:
    
    ```
    error: The option value `services.ssmtp.enable' in `/home/ma27/Projects/nixpkgs/nixos/modules/programs/ssmtp.nix' is not of type `boolean'.
    (use '--show-trace' to show detailed location information)
    ```
    Ma27 committed Mar 22, 2020
    Copy the full SHA
    fc316f7 View commit details
    Browse the repository at this point in the history