2009/4/6 Ludovic Courtès <[email protected]>:
> Saluton,
>
> Llus Batlle <[email protected]> writes:
>
>> + src = fetchsvn {
>> + url =
>> https://panotools.svn.sourceforge.net/svnroot/panotools/trunk/libpano;
>> + rev = 955;
>> + };
>
> Two remarks:
>
> 1. It'd be better to provide `sha256', for improved reproducibility.
How can I know the sha256 of a repository checkout?
>
> 2. Why not use a release version? Often, upstream developers know
> better than we do what's "releasable".
When they advised for libpano13 to be checked out, they told to
checkout the trunk. I supposed they maintain a trunk stable. But I
decided to add the revision at least.
>
>> + meta = {
>> + homepage = http://panotools.sourceforge.net/;
>> + description = "Panorama Tools";
>> + license = "GPL";
>> + };
>
> Please use "GPLv2+", or "GPLv3+", etc.
You can read a later patch about that.
>
> Also, the description is a bit terse and `longDescription' is missing.
I never wrote a longDescription. I will update the description.
Regards,
Lluís.
_______________________________________________
nix-dev mailing list
[email protected]
https://mail.cs.uu.nl/mailman/listinfo/nix-dev