On Wed, Sep 21, 2016 at 01:13:42PM +0000, Jérémy Bobbio wrote:
> Mattia Rizzolo:
> > The reason we like to have a static string in there (and in jenkins we
> > use the host architecture), is that otherwise diffoscope would be noisy
> > about the difference in the name if we compared the .changes files […].
> 
> Would it still be the case after Satyam's work which resulted in
> https://anonscm.debian.org/cgit/reproducible/diffoscope.git/commit/?id=c61f41c5128094508d6fff8b746cecc3841f6b27
> ?
> 
> This change has been shipped as part of diffoscope/60 from what I can see.

yes:

mattia@chase ~/pbuilder/result/sid/amd64 % diffoscope 
b1/libjs-jquery-scrollto_2.1.2+dfsg-3_all.changes 
b2/libjs-jquery-scrollto_2.1.2+dfsg-3_all.changes
--- b1/libjs-jquery-scrollto_2.1.2+dfsg-3_all.changes
+++ b2/libjs-jquery-scrollto_2.1.2+dfsg-3_all.changes
├── Files
│ @@ -1,3 +1,3 @@
│  
│ - 20e7077ef60cd2df6d4b6a21139f6c3d 4492 web extra 
libjs-jquery-scrollto_2.1.2+dfsg-3_20160921T143043z-20e7077e.buildinfo
│ + 20e7077ef60cd2df6d4b6a21139f6c3d 4492 web extra 
libjs-jquery-scrollto_2.1.2+dfsg-3_20160921T143138z-20e7077e.buildinfo
│   28bd68174b41c58b73d56cd8746d3c88 8308 web extra 
libjs-jquery-scrollto_2.1.2+dfsg-3_all.deb
│   --- libjs-jquery-scrollto_2.1.2+dfsg-3_20160921T143043z-20e7077e.buildinfo
├── +++ libjs-jquery-scrollto_2.1.2+dfsg-3_20160921T143138z-20e7077e.buildinfo
│┄ Files similar despite different names (difference score: 0)
╵
1 mattia@chase ~/pbuilder/result/sid/amd64 %                                    
                                                                                
     :(


And this is a case for a reproducible build.
I also tried to stick in an extra field in b2/.buildinfo and (still with
the different filename) it reported that difference too (which is not
supposed to do).

And btw, I also tried to stick in an additional file in the second
.buildinfo, and renaming them, which is supposedly enough to have
diffoscope skip that difference, but:

mattia@chase ~/pbuilder/result/sid/amd64 % diffoscope 
b1/libjs-jquery-scrollto_2.1.2+dfsg-3_all.changes 
b2/libjs-jquery-scrollto_2.1.2+dfsg-3_all.changes
--- b1/libjs-jquery-scrollto_2.1.2+dfsg-3_all.changes
+++ b2/libjs-jquery-scrollto_2.1.2+dfsg-3_all.changes
├── Files
│ @@ -1,3 +1,3 @@
│  
│ - 20e7077ef60cd2df6d4b6a21139f6c3d 4492 web extra 
libjs-jquery-scrollto_2.1.2+dfsg-3_amd64.buildinfo
│ + 845e31a2b41665575b4c3af404a3f130 4492 web extra 
libjs-jquery-scrollto_2.1.2+dfsg-3_amd64.buildinfo
│   28bd68174b41c58b73d56cd8746d3c88 8308 web extra 
libjs-jquery-scrollto_2.1.2+dfsg-3_all.deb
├── libjs-jquery-scrollto_2.1.2+dfsg-3_amd64.buildinfo
│   ├── Env
│   │ @@ -0,0 +1 @@
│   │ +ciao
│   ╵
╵


now, I'm not sure whether I should take care of also updating all
hashes, etc while messing around with .changes for diffoscope to work
correctly, but it is not working as I would expect.  I remember also
trying the day I saw the commit coming, but noticed something was fishy.
I always meant to check what was going on, but I'm already quite limited
on my time available.

Not to mention that this is not totally applicable right away as we can't
compare .changes until it stops recursing over files twice (one through
.changes and one trhough .changes->.buildinfo).  I should probably file
a bug for this though.  Until now it was really "oral knowledge" of very
few (and maybe of even fewer who remembers seeing such diffs nearly an
year ago).


-- 
regards,
                        Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540      .''`.
more about me:  https://mapreri.org                             : :'  :
Launchpad user: https://launchpad.net/~mapreri                  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Reproducible-builds mailing list
Reproducible-builds@lists.alioth.debian.org
http://lists.alioth.debian.org/cgi-bin/mailman/listinfo/reproducible-builds

Reply via email to