* Re: 01/06: gnu: guile-redis: Update to 1.0.0.
[not found] ` <20180820143227.CAE172052C@vcs0.savannah.gnu.org>
@ 2018-08-23 6:44 ` Mark H Weaver
2018-08-24 10:33 ` Ludovic Courtès
0 siblings, 1 reply; 2+ messages in thread
From: Mark H Weaver @ 2018-08-23 6:44 UTC (permalink / raw)
To: Ludovic Courtès; +Cc: guix-devel
Hi Ludovic,
ludo@gnu.org (Ludovic Courtès) writes:
> civodul pushed a commit to branch master
> in repository guix.
>
> commit a50eed201bf470b3bd124a2983bcea3453ec698f
> Author: Ludovic Courtès <ludo@gnu.org>
> Date: Mon Aug 20 14:35:33 2018 +0200
>
> gnu: guile-redis: Update to 1.0.0.
>
> * gnu/packages/guile.scm (guile-redis): Update to 1.0.0.
> [source]: Fetch from github.com. Remove snippet.
> [native-inputs]: Add AUTOCONF, AUTOMAKE, and PKG-CONFIG.
> ---
> gnu/packages/guile.scm | 27 ++++++++-------------------
> 1 file changed, 8 insertions(+), 19 deletions(-)
>
> diff --git a/gnu/packages/guile.scm b/gnu/packages/guile.scm
> index f179f29..358613b 100644
> --- a/gnu/packages/guile.scm
> +++ b/gnu/packages/guile.scm
> @@ -1241,31 +1241,20 @@ above command-line parameters.")
> (define-public guile-redis
> (package
> (name "guile-redis")
> - (version "0.1.0")
> + (version "1.0.0")
> + (home-page "https://github.com/aconchillo/guile-redis")
> (source (origin
> (method url-fetch)
> - (uri (string-append "mirror://savannah/guile-redis/guile-redis-"
> - version ".tar.gz"))
> + (uri (string-append home-page "/archive/" version ".tar.gz"))
[...]
> (home-page "https://savannah.nongnu.org/projects/guile-redis/")
> (synopsis "Redis client library for Guile")
> (description "Guile-redis provides a Scheme interface to the Redis
This commit introduced a duplicate 'home-page' field. You made the same
mistake in your recent commit adding 'guile-gcrypt'. I detected these
problems quickly because I include the "records: detect duplicate field
initializers" commit on my private branch.
Anyway, in the case of 'guile-gcrypt', both of the home-page
initializers were the same. In the case of 'guile-redis', they are
different. They are:
> + (home-page "https://github.com/aconchillo/guile-redis")
[...]
> (home-page "https://savannah.nongnu.org/projects/guile-redis/")
Which one is correct? I deleted the second one, because the first one
is referenced in the source URI field, and initially I didn't notice
that they were different. Did guile-redis move from Savannah to GitHub?
Mark
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: 01/06: gnu: guile-redis: Update to 1.0.0.
2018-08-23 6:44 ` 01/06: gnu: guile-redis: Update to 1.0.0 Mark H Weaver
@ 2018-08-24 10:33 ` Ludovic Courtès
0 siblings, 0 replies; 2+ messages in thread
From: Ludovic Courtès @ 2018-08-24 10:33 UTC (permalink / raw)
To: Mark H Weaver; +Cc: guix-devel
Mark H Weaver <mhw@netris.org> skribis:
> ludo@gnu.org (Ludovic Courtès) writes:
>
>> civodul pushed a commit to branch master
>> in repository guix.
>>
>> commit a50eed201bf470b3bd124a2983bcea3453ec698f
>> Author: Ludovic Courtès <ludo@gnu.org>
>> Date: Mon Aug 20 14:35:33 2018 +0200
>>
>> gnu: guile-redis: Update to 1.0.0.
>>
>> * gnu/packages/guile.scm (guile-redis): Update to 1.0.0.
>> [source]: Fetch from github.com. Remove snippet.
>> [native-inputs]: Add AUTOCONF, AUTOMAKE, and PKG-CONFIG.
>> ---
>> gnu/packages/guile.scm | 27 ++++++++-------------------
>> 1 file changed, 8 insertions(+), 19 deletions(-)
>>
>> diff --git a/gnu/packages/guile.scm b/gnu/packages/guile.scm
>> index f179f29..358613b 100644
>> --- a/gnu/packages/guile.scm
>> +++ b/gnu/packages/guile.scm
>> @@ -1241,31 +1241,20 @@ above command-line parameters.")
>> (define-public guile-redis
>> (package
>> (name "guile-redis")
>> - (version "0.1.0")
>> + (version "1.0.0")
>> + (home-page "https://github.com/aconchillo/guile-redis")
>> (source (origin
>> (method url-fetch)
>> - (uri (string-append "mirror://savannah/guile-redis/guile-redis-"
>> - version ".tar.gz"))
>> + (uri (string-append home-page "/archive/" version ".tar.gz"))
>
> [...]
>
>> (home-page "https://savannah.nongnu.org/projects/guile-redis/")
>> (synopsis "Redis client library for Guile")
>> (description "Guile-redis provides a Scheme interface to the Redis
>
> This commit introduced a duplicate 'home-page' field. You made the same
> mistake in your recent commit adding 'guile-gcrypt'. I detected these
> problems quickly because I include the "records: detect duplicate field
> initializers" commit on my private branch.
Oops!
> Anyway, in the case of 'guile-gcrypt', both of the home-page
> initializers were the same. In the case of 'guile-redis', they are
> different. They are:
>
>> + (home-page "https://github.com/aconchillo/guile-redis")
> [...]
>> (home-page "https://savannah.nongnu.org/projects/guile-redis/")
>
> Which one is correct? I deleted the second one, because the first one
> is referenced in the source URI field, and initially I didn't notice
> that they were different. Did guile-redis move from Savannah to GitHub?
Yes, it moved to GitHub.
Thanks for fixing these!
Ludo’.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-08-24 10:33 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20180820143226.22345.8646@vcs0.savannah.gnu.org>
[not found] ` <20180820143227.CAE172052C@vcs0.savannah.gnu.org>
2018-08-23 6:44 ` 01/06: gnu: guile-redis: Update to 1.0.0 Mark H Weaver
2018-08-24 10:33 ` Ludovic Courtès
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).