Debian Bug report logs - #834957
flashrom: please make the build reproducible

version graph

Package: src:flashrom; Maintainer for src:flashrom is Debian EFI <debian-efi@lists.debian.org>;

Reported by: Chris Lamb <lamby@debian.org>

Date: Sat, 20 Aug 2016 23:39:02 UTC

Severity: wishlist

Tags: patch

Found in version flashrom/0.9.9+r1954-1

Done: Gürkan Myczko <gurkan@phys.ethz.ch>

Bug is archived. No further changes may be made.

Toggle useless messages

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to debian-bugs-dist@lists.debian.org, reproducible-builds@lists.alioth.debian.org, Uwe Hermann <uwe@debian.org>:
Bug#834957; Package src:flashrom. (Sat, 20 Aug 2016 23:39:05 GMT) (full text, mbox, link).


Acknowledgement sent to Chris Lamb <lamby@debian.org>:
New Bug report received and forwarded. Copy sent to reproducible-builds@lists.alioth.debian.org, Uwe Hermann <uwe@debian.org>. (Sat, 20 Aug 2016 23:39:05 GMT) (full text, mbox, link).


Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Chris Lamb <lamby@debian.org>
To: submit@bugs.debian.org
Subject: flashrom: please make the build reproducible
Date: Sun, 21 Aug 2016 00:34:08 +0100
[Message part 1 (text/plain, inline)]
Source: flashrom
Version: 0.9.9+r1954-1
Severity: wishlist
Tags: patch
User: reproducible-builds@lists.alioth.debian.org
Usertags: timestamps
X-Debbugs-Cc: reproducible-builds@lists.alioth.debian.org

Hi,

Whilst working on the Reproducible Builds effort [0], I noticed
that flashrom could not be built reproducibly.

Patch attached.

 [0] https://reproducible-builds.org/


Regards,

-- 
      ,''`.
     : :'  :     Chris Lamb
     `. `'`      lamby@debian.org / chris-lamb.co.uk
       `-
[flashrom.diff.txt (text/plain, attachment)]

Information forwarded to debian-bugs-dist@lists.debian.org, Uwe Hermann <uwe@debian.org>:
Bug#834957; Package src:flashrom. (Sun, 21 Aug 2016 08:54:06 GMT) (full text, mbox, link).


Acknowledgement sent to Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>:
Extra info received and forwarded to list. Copy sent to Uwe Hermann <uwe@debian.org>. (Sun, 21 Aug 2016 08:54:06 GMT) (full text, mbox, link).


Message #10 received at 834957@bugs.debian.org (full text, mbox, reply):

From: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
To: Chris Lamb <lamby@debian.org>, 834957@bugs.debian.org
Subject: Re: Bug#834957: flashrom: please make the build reproducible
Date: Sun, 21 Aug 2016 10:37:05 +0200
On Sun, 21 Aug 2016 00:34:08 +0100
Chris Lamb <lamby@debian.org> wrote:

> Source: flashrom
> Version: 0.9.9+r1954-1
> Severity: wishlist
> Tags: patch
> User: reproducible-builds@lists.alioth.debian.org
> Usertags: timestamps
> X-Debbugs-Cc: reproducible-builds@lists.alioth.debian.org
> 
> Hi,
> 
> Whilst working on the Reproducible Builds effort [0], I noticed
> that flashrom could not be built reproducibly.

Hi Chris,

if I read the specs right then the date formatting should not happen at
build time but at runtime to catch user locales.

"Formatting MUST be deferred until runtime if an end user should
observe the value in their own locale or timezone."

That would make your patch not fully complying.
I'll try to improve on that upstream for the next release. Thanks for
the pointer.

-- 
Kind regards/Mit freundlichen Grüßen, Stefan Tauner



Information forwarded to debian-bugs-dist@lists.debian.org, Uwe Hermann <uwe@debian.org>:
Bug#834957; Package src:flashrom. (Sun, 21 Aug 2016 09:57:08 GMT) (full text, mbox, link).


Acknowledgement sent to Chris Lamb <lamby@debian.org>:
Extra info received and forwarded to list. Copy sent to Uwe Hermann <uwe@debian.org>. (Sun, 21 Aug 2016 09:57:08 GMT) (full text, mbox, link).


Message #15 received at 834957@bugs.debian.org (full text, mbox, reply):

From: Chris Lamb <lamby@debian.org>
To: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>, 834957@bugs.debian.org
Subject: Re: Bug#834957: flashrom: please make the build reproducible
Date: Sun, 21 Aug 2016 10:56:34 +0100
Stefan Tauner wrote:

> "Formatting MUST be deferred until runtime if an end user should
> observe the value in their own locale or timezone."

Personally, I don't think this should be a MUST especially because — 
as you are noticing — it requires a more-invasive patch for not much
value.

I have already proposed this be relaxed.

In the meantime, I would focus on the conditional clause, ie.:

  … if an end user should observe the value in their own locale
  or timezone.

I feel I can interpret this as "the user should not observe a *build*
timestamp in their own locale timezone as that defeats the point of
it being a quasi-unique identifier. Therefore, there is no need to
defer formatting until runtime.

If it helps, I am not sure I have seen any package implement the
SOURCE_DATE_EPOCH variable by deferring. :)


Regards,

-- 
      ,''`.
     : :'  :     Chris Lamb
     `. `'`      lamby@debian.org / chris-lamb.co.uk
       `-



Information forwarded to debian-bugs-dist@lists.debian.org, Uwe Hermann <uwe@debian.org>:
Bug#834957; Package src:flashrom. (Sun, 05 Mar 2017 09:34:13 GMT) (full text, mbox, link).


Acknowledgement sent to Chris Lamb <lamby@debian.org>:
Extra info received and forwarded to list. Copy sent to Uwe Hermann <uwe@debian.org>. (Sun, 05 Mar 2017 09:34:13 GMT) (full text, mbox, link).


Message #20 received at 834957@bugs.debian.org (full text, mbox, reply):

From: Chris Lamb <lamby@debian.org>
To: 834957@bugs.debian.org
Subject: Re: Bug#834957: flashrom: please make the build reproducible
Date: Sun, 05 Mar 2017 09:29:40 +0000
> Would you consider applying this patch and uploading?

Friendly ping on this :)


Best wishes,

-- 
      ,''`.
     : :'  :     Chris Lamb
     `. `'`      lamby@debian.org / chris-lamb.co.uk
       `-



Reply sent to Gürkan Myczko <gurkan@phys.ethz.ch>:
You have taken responsibility. (Thu, 21 Feb 2019 09:27:06 GMT) (full text, mbox, link).


Notification sent to Chris Lamb <lamby@debian.org>:
Bug acknowledged by developer. (Thu, 21 Feb 2019 09:27:06 GMT) (full text, mbox, link).


Message #25 received at 834957-done@bugs.debian.org (full text, mbox, reply):

From: Gürkan Myczko <gurkan@phys.ethz.ch>
To: 834957-done@bugs.debian.org
Subject: flashrom 1.0-1
Date: Thu, 21 Feb 2019 10:22:32 +0100
I just checked, this is already fixed upstream, without your patch.
Closing...



Bug archived. Request was from Debbugs Internal Request <owner@bugs.debian.org> to internal_control@bugs.debian.org. (Fri, 22 Mar 2019 07:27:00 GMT) (full text, mbox, link).


Send a report that this bug log contains spam.


Debian bug tracking system administrator <owner@bugs.debian.org>. Last modified: Wed May 17 14:00:39 2023; Machine Name: buxtehude

Debian Bug tracking system

Debbugs is free software and licensed under the terms of the GNU Public License version 2. The current version can be obtained from https://bugs.debian.org/debbugs-source/.

Copyright © 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson, 2005-2017 Don Armstrong, and many other contributors.