[Adding reproducible-b...@lists.alioth.debian.org to CC]

Hi Hilmar,

> The provided patch disables time stamping in general. I don't think this
> is a good idea, as users might complain about changed program behavior.

Is that likely? Genuine question. :)

> Could you extend the patch so the program obeys the SOURCE_DATE_EPOCH
> variable and changes its behavior only if that variable is set?

Yes, but is this really worth any potential "regression" when compared
to the admittedly-minor additionally code complexity you are proposing?

Note that if we do change the behaviour based on SOURCE_DATE_EPOCH we
have two options. First, we simply hide the date if this variable is
exported or secondly we do some ugly parsing and use the value of this
variable instead.

(Naturally, the former is somewhat simpler.)


Regards,

-- 
      ,''`.
     : :'  :     Chris Lamb
     `. `'`      la...@debian.org 🍥 chris-lamb.co.uk
       `-

Reply via email to