GNU bug report logs - #25916
[PATCH] gnu: mumble: Make builds bit-reproducible.

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: ludo <at> gnu.org (Ludovic Courtès)
Cc: tracker <at> debbugs.gnu.org
Subject: bug#25916: closed ([PATCH] gnu: mumble: Make builds
 bit-reproducible.)
Date: Tue, 04 Apr 2017 21:43:01 +0000

[Message part 1 (text/plain, inline)]

Your message dated Tue, 04 Apr 2017 23:42:01 +0200
with message-id <87vaqjn93q.fsf <at> gnu.org>
and subject line Re: bug#25916: [PATCH] gnu: mumble: Make builds bit-reproducible.
has caused the debbugs.gnu.org bug report #25916,
regarding [PATCH] gnu: mumble: Make builds bit-reproducible.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
25916: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=25916
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Kei Kebreau <kei <at> openmailbox.org>
To: guix-patches <at> gnu.org
Cc: Kei Kebreau <kei <at> openmailbox.org>
Subject: [PATCH] gnu: mumble: Make builds bit-reproducible.
Date: Wed,  1 Mar 2017 11:35:30 -0500

* gnu/packages/telephony.scm (mumble)[source]: Add 'substitute*' calls
to 'snippet' for src/Version.h and src/mumble/VersionCheck.cpp.
---
 gnu/packages/telephony.scm | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/gnu/packages/telephony.scm b/gnu/packages/telephony.scm
index 87261825c..4ff2acd25 100644
--- a/gnu/packages/telephony.scm
+++ b/gnu/packages/telephony.scm
@@ -330,6 +330,12 @@ address of one of the participants.")
               (modules '((guix build utils)))
               (snippet
                `(begin
+                  ;; Remove timestamps for reproducibility.
+                  (substitute* "src/Version.h"
+                    (("\"Compiled \" __DATE__ \" \" __TIME__") "\"\""))
+                  (substitute* "src/mumble/VersionCheck.cpp"
+                    (("^.*__DATE__.*$") "")
+                    (("^.*__TIME__.*$") ""))
                   ;; Remove bundled software.
                   (for-each delete-file-recursively '("3rdparty"
                                                       "speex"
-- 
2.12.0

[Message part 3 (message/rfc822, inline)]

From: ludo <at> gnu.org (Ludovic Courtès)
To: Marius Bakke <mbakke <at> fastmail.com>
Cc: 25916-done <at> debbugs.gnu.org, Kei Kebreau <kei <at> openmailbox.org>,
 Leo Famulari <leo <at> famulari.name>
Subject: Re: bug#25916: [PATCH] gnu: mumble: Make builds bit-reproducible.
Date: Tue, 04 Apr 2017 23:42:01 +0200

Hello!

Marius Bakke <mbakke <at> fastmail.com> skribis:

> Leo Famulari <leo <at> famulari.name> writes:
>
>> On Wed, Mar 01, 2017 at 12:20:41PM -0500, Kei Kebreau wrote:
>>> Marius Bakke <mbakke <at> fastmail.com> writes:
>>> 
>>> > Kei Kebreau <kei <at> openmailbox.org> writes:
>>> >
>>> >> * gnu/packages/telephony.scm (mumble)[source]: Add 'substitute*' calls
>>> >> to 'snippet' for src/Version.h and src/mumble/VersionCheck.cpp.
>>> >
>>> > Thanks a lot for this! The patch LGTM, but FYI won't be necessary once
>>> > 'core-updates' is merged. See this commit, which backports a gcc7 patch.
>>> >
>>> > d71d6fe85c4db778be6a2081cbbec6a3a494e4f9
>>> >
>>> 
>>> Yay, what a useful patch!
>>> 
>>> Aren't we pretty close to merging core-updates? If that's the case, I
>>> won't worry about pushing this to master.
>>
>> Someone should try checking if they can reproduce mumble on
>> core-updates, to make sure the backported GCC 7 patch is working
>> correctly :)
>
> That's a good test. Unfortunately, mumble fails to build on
> 'core-updates'.

It builds with current master (post ‘core-updates’ merge).

I’ve done a --check build and can confirm that our GCC now honors
‘SOURCE_DATE_EPOCH’, so the patch isn’t needed.  \o/

Closing this bug!

Ludo’.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.