guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] gnu: moc: Update to 2.5.2.


From: Marius Bakke
Subject: Re: [PATCH] gnu: moc: Update to 2.5.2.
Date: Wed, 11 Jan 2017 02:00:06 +0100
User-agent: Notmuch/0.23.4 (https://notmuchmail.org) Emacs/25.1.1 (x86_64-unknown-linux-gnu)

Kei Kebreau <address@hidden> writes:

> Overdue update to moc 2.5.2. If it's reproducible for others as well, I'll
> commit it to master.
> From 389a7e8f02b800bc937a5f7e191d1d090c980401 Mon Sep 17 00:00:00 2001
> From: Kei Kebreau <address@hidden>
> Date: Tue, 10 Jan 2017 19:18:42 -0500
> Subject: [PATCH] gnu: moc: Update to 2.5.2.
>
> * gnu/packages/music.scm (moc): Update to 2.5.2.
> ---
>  gnu/packages/music.scm | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/music.scm b/gnu/packages/music.scm
> index 59fd508f6..803e0a6e5 100644
> --- a/gnu/packages/music.scm
> +++ b/gnu/packages/music.scm
> @@ -2179,7 +2179,7 @@ with a number of bugfixes and changes to improve IT 
> playback.")
>  (define-public moc
>    (package
>      (name "moc")
> -    (version "2.5.1")
> +    (version "2.5.2")
>      (source (origin
>                (method url-fetch)
>                (uri (string-append "http://ftp.daper.net/pub/soft/";
> @@ -2187,7 +2187,13 @@ with a number of bugfixes and changes to improve IT 
> playback.")
>                                    name "-" version ".tar.bz2"))
>                (sha256
>                 (base32
> -                "1wn4za08z64bhsgfhr9c0crfyvy8c3b6a337wx7gz19am5srqh8v"))))
> +                "026v977kwb0wbmlmf6mnik328plxg8wykfx9ryvqhirac0aq39pk"))
> +              (modules '((guix build utils)))
> +              (snippet
> +               ;; Remove use of __DATE__ and __TIME__ for reproducibility.
> +               '(substitute* "main.c"
> +                  (("printf \\(\"            Built : %s\", __DATE__\\);") "")
> +                  (("printf \\(\" %s\", __TIME__\\);") "")))))

The commit message should mention the change to [source]. I would prefer
if this was a separate patch, unless the __DATE__ and __TIME__ macros
were introduced in this version, but no strong opinion.

LGTM!

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]