spesmilo
/
electrum
/
8603
65%
|
Ran
|
Jobs
2
|
Files
46
|
Run time
2min
|
Badge
Embed ▾
README BADGES
|
push
travis-ci
SomberNightbuild: make NSIS windows binary deterministic by changing the .ico file see bitcoin/bitcoin@217208a36 ----- A lot of time was wasted on this... over the years actually... Some notes and rant here, for future reference. During the initial effort to try to make binaries reproducible, out of the three windows binaries being distributed (standalone, portable, setup), only the first two were successfully made deterministic. Later, we started to use Docker-based builds. At that point ThomasV and I could reproducibly build the same setup/nsis exe but Travis kept building a different one. Recently I have noticed that if I do two subsequent builds of the setup exe on the same machine, adding a new file in contrib/build-wine/ between the builds, then I get different binaries. Playing around with this a bit, it seems: - other files that are in the same folder as contrib/build-wine/electrum.nsi affect the binary - only files that are in exactly the same folder matter (not recursively) - only filenames matter (not permission, owner, timestamps, or file contents) To see the difference in the binaries, use vbindiff, and disable the compression done by nsis (SetCompress off). There is a ~48 byte diff near the very beginning of the "Uninstaller" section. I am only guessing it is the uninstaller section based on the sizes of the sections printed by nsis during the build. I have downloaded the binary built by Travis, and the diff is consistent with this (i.e. it's the same kind of diff that manifests if I change the filename of one of the supposedly unrelated files). Commenting out the "WriteUninstaller" line in .nsi fixes the issue. i.e. if no uninstaller is created then the binary becomes deterministic. Commenting out the "!define MUI_ICON" line in .nsi also fixes the issue. At this point I remembered the above referenced commit by bluematt; which I had thought we had already followed up on... Replacing the .ico file fixes the issu... (continued)
7384 of 13712 relevant lines covered (53.85%)
1.08 hits per line
Jobs
| ID | Job ID | Ran | Files | Coverage | |
|---|---|---|---|---|---|
| 1 | 8603.1 | 0 |
53.85 |
Travis Job 8603.1 | |
| 2 | 8603.2 | 0 |
53.85 |
Travis Job 8603.2 |
Source Files on build 8603
Detailed source file information is not available for this build.