[Nix-dev] Re: [Nix-commits] SVN commit: nix - 15723 - eelco - nixos/branches/modular-nixos/system

[Nicolas Pierron]

Hi,

On Mon, May 25, 2009 at 15:52, Eelco Dolstra <e.dolstra at tudelft.nl> wrote:
> Log:
> * I'm pretty sure the i18n module shouldn't be conditional on
>  pulseaudio :-)
>
>
> Changes:
>
> Modified: nixos/branches/modular-nixos/system/i18n.nix
> ===================================================================
> --- nixos/branches/modular-nixos/system/i18n.nix        2009-05-25 13:42:46 UTC (rev 15722)
> +++ nixos/branches/modular-nixos/system/i18n.nix        2009-05-25 13:52:18 UTC (rev 15723)
> @@ -39,9 +39,8 @@
>
>  ###### implementation
>
> -mkIf config.services.pulseaudio.enable {
> +{
>   require = [
>     options
>   ];
> -
>  }

I would even say that you don't need a require stament in this case.
You can just in-line the options instead of the " require = [ options
]; ".

-{
-  require = [
-    options
-  ];
-}
+options

A set of option declarations is a correct modules.

-- 
Nicolas Pierron
http://www.linkedin.com/in/nicolasbpierron
- If you are doing something twice then you should try to do it once.