[PATCH] reproducible Debian: Reverse Python 2.x dict order in the second build.

[Chris Lamb]

> Umh, could you please set that variable in the pbuilderrc of the second
> build instead of using the hooks (which I don't understand how that
> would ever work

I just grepped for existing changes (eg. "I capture the environment"). If
these things don't work as you claim, please could you remove them so they
are not confusing? Perhaps I'm missing something as that particular one
isn't even in pbuilderrc

> 1 line change is better than changing dozens of files anyway, I suppose.

Exactly ^  :)


> So just dump it in reproducible_build.sh, line 682 onwards

Sure:

  commit 086d2938f25331c0635537d3445db9f1ba29774f
  Author: Chris Lamb <lamby at debian.org>
  Date:   Sun Feb 12 10:03:02 2017 +1300
  
      reproducible Debian: Reverse Python 2.x dict order in the second build.
      
      Signed-off-by: Chris Lamb <lamby at debian.org>
  
   bin/reproducible_build.sh  | 1 +
   bin/reproducible_common.sh | 2 +-
   2 files changed, 2 insertions(+), 1 deletion(-)

You can also merge from the "export-PYTHONREVERSEDICTKEYORDER-pbuilderrc" branch of
https://github.com/lamby/jenkins.debian.net if that is more convenient.

> PS: was it you that triggered a admin password reset?

Yes. (The button is misleading, I didn't think it would actually reset
the password but rather give me immediate ML admin access.)


Regards,

-- 
      ,''`.
     : :'  :     Chris Lamb
     `. `'`      lamby at debian.org / chris-lamb.co.uk
       `-
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-reproducible-Debian-Reverse-Python-2.x-dict-order-in.patch
Type: text/x-patch
Size: 1748 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/reproducible-builds/attachments/20170212/cd4e7745/attachment.bin>