[Nix-dev] Re: [Nix-commits] SVN commit: nix - 15668 - MarcWeber - in nixos/branches/modular-nixos: etc system

[Nicolas Pierron]

Hi,

I don't understand why you have removed all " activateLib.* ".  This
was made to have a central repository of functionalities provided to
the current snippet of code.  By removing it, you reduce the error
messages which should tell you where things started to go wrong (bad
naming -> attribute does not exists).  Even if the solution sounds a
bit stupid:

activateLib.systemConfig = "systemConfig"

should do the trick and wouldn't have require such modifications.

On Wed, May 20, 2009 at 03:35, Marc Weber <marco-oweber at gmx.de> wrote:
> Modified: nixos/branches/modular-nixos/etc/default.nix
> ===================================================================
> --- nixos/branches/modular-nixos/etc/default.nix        2009-05-19 23:51:13 UTC (rev 15667)
> +++ nixos/branches/modular-nixos/etc/default.nix        2009-05-20 01:35:46 UTC (rev 15668)
> @@ -229,8 +229,6 @@
>  let
>   inherit (pkgs.stringsWithDeps) noDepEntry fullDepEntry packEntry;
>
> -  activateLib = config.system.activationScripts.lib;
> -
>   copyScript = {source, target, mode ? "644", own ? "root.root"}:
>     assert target != "nixos"; ''
>     source="${source}"
> @@ -295,9 +293,9 @@
>             fi
>         done
>       '' [
> -        activateLib.systemConfig
> -        activateLib.defaultPath # path to cp, chmod, chown
> -        activateLib.stdio
> +        "systemConfig"
> +        "defaultPath" # path to cp, chmod, chown
> +        "stdio"
>       ];
>     };
>   };




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