Re: [PATCH 1/4] gnu: Add perl-net-psyc.

[ng0]

Leo Famulari <address@hidden> writes:

> On Sun, Oct 02, 2016 at 10:35:43AM +0000, ng0 wrote:
>> Leo Famulari <address@hidden> writes:
>> 
>> > On Thu, Sep 29, 2016 at 10:38:07AM +0000, ng0 wrote:
>> >> * gnu/packages/psyc.scm: New file.
>> >> * gnu/local.mk (GNU_SYSTEM_MODULES): Add it.
>> >
>> >> +    (source
>> >> +     (origin
>> >> +       (method url-fetch)
>> >> +       (uri (string-append "http://perl.psyc.eu/";
>> >> +                           "perlpsyc-" version ".zip"))
>> >> +       (file-name (string-append name "-" version ".zip"))
>> >> +       (sha256
>> >> +        (base32
>> >> +         "1lw6807qrbmvzbrjn1rna1dhir2k70xpcjvyjn45y35hav333a42"))
>> >
>> > This is the wrong hash.
>> > http://lists.gnu.org/archive/html/guix-devel/2016-09/msg02076.html
>> 
>> No, it's not. Our server recently had to be moved and I don't know what
>> happened during that time, but to make sure I checked the download
>> before I commited the new series and repeated that check just now:
>> 
>> address@hidden ~/re-src$ wget http://perlpsyc.psyc.eu/perlpsyc-1.1.zip
>> --2016-10-02 10:34:12--  http://perlpsyc.psyc.eu/perlpsyc-1.1.zip
>> Resolving perlpsyc.psyc.eu (perlpsyc.psyc.eu)... 188.40.42.221
>> Connecting to perlpsyc.psyc.eu (perlpsyc.psyc.eu)|188.40.42.221|:80... 
>> connected.
>> HTTP request sent, awaiting response... 200 OK
>> Length: 444423 (434K) [application/zip]
>> Saving to: ‘perlpsyc-1.1.zip’
>> 
>> perlpsyc-1.1.zip           100%[=====================================>] 
>> 434.01K   927KB/s    in 0.5s    
>> 
>> 2016-10-02 10:34:12 (927 KB/s) - ‘perlpsyc-1.1.zip’ saved [444423/444423]
>> 
>> address@hidden ~/re-src$ guix hash perlpsyc-1.1.zip 
>> 1lw6807qrbmvzbrjn1rna1dhir2k70xpcjvyjn45y35hav333a42
>
> The URL you used with wget is different from the URL used in the patch.
>
> The download from perlpsyc.psyc.eu is a Git repo whereas the download
> from perl.psyc.eu is a plain directory.
>
> The patch uses the URL for the plain directory while the hash is of the
> Git repo.
>
> Can you clarify which download the package should use?
>

Okay so

+     (origin
+       (method url-fetch)
+       (uri (string-append "http://perl.psyc.eu/";
+                           "perlpsyc-" version ".zip"))

is wrong. it hsould've been

+     (origin
+       (method url-fetch)
+       (uri (string-append "http://perlpsyc.eu/";
+                           "perlpsyc-" version ".zip"))

The new patch series which will be coming as the next messages fixes
this. This is the only location of the perlpsyc downloads, my mistake,
Sorry.
The archive is indeed just a zipped git repository, that's correct.
It did not affect building and installing for me. I'll address it in one
of our TODO lists if it is a problem for you.
--