[Nix-dev] Re: [Nix-commits] SVN commit: nix - 15748 - eelco - in nixos/branches/modular-nixos: modules modules/system modules/system/upstart upstart-jobs

[Nicolas Pierron]

On Wed, May 27, 2009 at 12:32, Eelco Dolstra <e.dolstra at tudelft.nl> wrote:
> Copied: nixos/branches/modular-nixos/modules/system/upstart/make-job.nix (from rev 15738, nixos/branches/modular-nixos/upstart-jobs/make-job.nix)
> ===================================================================
> --- nixos/branches/modular-nixos/modules/system/upstart/make-job.nix                            (rev 0)
> +++ nixos/branches/modular-nixos/modules/system/upstart/make-job.nix    2009-05-27 10:32:30 UTC (rev 15748)
> @@ -0,0 +1,38 @@
> +{runCommand}: job:
> +

This file is not a module.  We have two possibilities:
- have a detection system based on the nix-expression syntax (we could
have false/possitive)
- have a specific directory naming or file naming:

nixos/branches/modular-nixos/modules/system/upstart/upstart.impl/make-job.nix

then all non-module configurations are stored side-by-side close to
the module which is using them.

nixos/branches/modular-nixos/modules/system/upstart/upstart.nix
nixos/branches/modular-nixos/modules/system/upstart/upstart.impl/make-job.nix

or

nixos/branches/modular-nixos/modules/system/upstart/upstart.nix
nixos/branches/modular-nixos/modules/system/upstart/make-job.impl.nix

The idea is to remove from the generated list of modules without all
files path containing ".impl[/.]".

Another solution suggested by Eelco waas to have another extension for
modules or for other nix expressions.

nixos/branches/modular-nixos/modules/system/upstart/upstart.nmo
nixos/branches/modular-nixos/modules/system/upstart/make-job.impl.nix

nmo: nix module
mnix: modular nix expression
...

Any suggestions, remarks?

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