unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: ng0 <ngillmann@runbox.com>
To: Leo Famulari <leo@famulari.name>
Cc: guix-devel@gnu.org
Subject: Re: [PATCH 1/4] gnu: Add perl-net-psyc.
Date: Tue, 04 Oct 2016 08:23:08 +0000	[thread overview]
Message-ID: <871szw1q1f.fsf@we.make.ritual.n0.is> (raw)
In-Reply-To: <20161004022434.GA16890@jasmine>

Leo Famulari <leo@famulari.name> writes:

> On Sun, Oct 02, 2016 at 10:35:43AM +0000, ng0 wrote:
>> Leo Famulari <leo@famulari.name> 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:
>> 
>> ng0@shadowwalker ~/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]
>> 
>> ng0@shadowwalker ~/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.
-- 

  reply	other threads:[~2016-10-04  8:23 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-29 10:38 [PATCH 1/4] gnu: Add perl-net-psyc ng0
2016-09-29 10:38 ` [PATCH 2/4] gnu: Add libpsyc ng0
2016-10-30  5:57   ` Ricardo Wurmus
2016-09-29 10:38 ` [PATCH 3/4] gnu: Add psyclpc ng0
2016-10-30  6:03   ` Ricardo Wurmus
2016-10-30 12:46     ` ng0
2016-09-29 10:38 ` [PATCH 4/4] gnu: Add psyced ng0
2016-09-29 10:44   ` ng0
2016-10-30  5:42     ` Ricardo Wurmus
2016-10-30  5:48   ` Ricardo Wurmus
2016-10-02  1:46 ` [PATCH 1/4] gnu: Add perl-net-psyc Leo Famulari
2016-10-02 10:35   ` ng0
2016-10-04  2:24     ` Leo Famulari
2016-10-04  8:23       ` ng0 [this message]
2016-10-04  8:34         ` psyc.scm: new patch series ng0
2016-10-04  8:34           ` [PATCH 1/4] gnu: Add perl-net-psyc ng0
2016-10-04  8:34           ` [PATCH 2/4] gnu: Add libpsyc ng0
2016-10-04  8:34           ` [PATCH 3/4] gnu: Add psyclpc ng0
2016-10-04  8:34           ` [PATCH 4/4] gnu: Add psyced ng0
2016-10-26 11:37             ` Efraim Flashner
2016-10-28 10:04               ` ng0
2016-10-30  5:54 ` [PATCH 1/4] gnu: Add perl-net-psyc Ricardo Wurmus

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://guix.gnu.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=871szw1q1f.fsf@we.make.ritual.n0.is \
    --to=ngillmann@runbox.com \
    --cc=guix-devel@gnu.org \
    --cc=leo@famulari.name \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).