[Nix-dev] Re: [Nix-commits] SVN commit: nix - 17259 - NicolasPierron - nixos/trunk

[Nicolas Pierron]

On Fri, Sep 18, 2009 at 18:01, Eelco Dolstra <e.dolstra at tudelft.nl> wrote:
> Nicolas Pierron wrote:
>
>> -        revision = if rev == 1234 then "HEAD" else toString rev;
>> +        revision = with nixosSrc;
>> +          if rev == 1234 then "HEAD" else toString rev;
>
> Why not just: revision = toString nixosSrc.rev ?
>
> The only significance of the "1234" is to allow a test build of a release from
> nix-build.

because you can test the manual by building it with the following command:

nix-build ./release.nix -A manual

which does not fetch the revision number.  Thus the closest behavior
is to replace the revision number by "HEAD".

-- 
Nicolas Pierron
http://www.linkedin.com/in/nicolasbpierron