Re: [PATCH] zlib: bump to 1.2.8.

[Ludovic Courtès]

Cyril Roelandt <address@hidden> skribis:

> On 05/23/2013 02:03 PM, Ludovic Courtès wrote:
>> Cyril Roelandt<address@hidden>  skribis:
>>
>>> --- a/gnu/packages/compression.scm
>>> +++ b/gnu/packages/compression.scm
>>> @@ -26,7 +26,7 @@
>>>   (define-public zlib
>>>     (package
>>>       (name "zlib")
>>> -    (version "1.2.7")
>>> +    (version "1.2.8")
>>
>> Unless I’m mistaken, this triggers a complete rebuild now that GCC
>> depends on it.  Thus it would need to go into ‘core-updates’.
>>
>
> The problem is that the 1.2.7 tarball is no longer available, and Guix
> therefore fails to install zlib.

This is not a problem when using the substituter, because hydra.gnu.org
mirrors it:

--8<---------------cut here---------------start------------->8---
$ ./pre-inst-env guix build -S zlib
The following file will be downloaded:
   /nix/store/5b910ka58b3h4x7n3vbdgndf6zfxnpwf-zlib-1.2.7.tar.gz
@ substituter-started 
/nix/store/5b910ka58b3h4x7n3vbdgndf6zfxnpwf-zlib-1.2.7.tar.gz 
/home/ludo/src/guix/nix/scripts/substitute-binary
downloading `/nix/store/5b910ka58b3h4x7n3vbdgndf6zfxnpwf-zlib-1.2.7.tar.gz' 
from 
`http://hydra.gnu.org/nar/5b910ka58b3h4x7n3vbdgndf6zfxnpwf-zlib-1.2.7.tar.gz'...
@ substituter-succeeded 
/nix/store/5b910ka58b3h4x7n3vbdgndf6zfxnpwf-zlib-1.2.7.tar.gz
/nix/store/5b910ka58b3h4x7n3vbdgndf6zfxnpwf-zlib-1.2.7.tar.gz
--8<---------------cut here---------------end--------------->8---

Did you use --no-substitutes?

> This is a blocking bug for anyone who would like to try Guix from the
> git repository, so I'd like to have this fixed in master. WDYT ?

It can still be found at mirror://sourceforge/libpng/zlib-1.2.7.tar.gz.
So for the moment I would suggest changing the ‘uri’ field to a list:

     (origin
      (method url-fetch)
      (uri (list (string-append "http://zlib.net/zlib-";
                                 version ".tar.gz")
                 (string-append "mirror://sourceforge/libpng/zlib-"
                                 version ".tar.gz")))
      (sha256
       (base32
        "1i96gsdvxqb6skp9a58bacf1wxamwi9m9pg4yn7cpf7g7239r77s")))

Problem solved.  :-)

And the upgrade can be scheduled for later, in a ‘core-updates’ branch.

WDYT?

Ludo’.