1 2017-02-18 00:17:59	0|luke-jr|"Syncing Headers (%1%)…" <-- is this valid?
  2 2017-02-18 00:36:18	0|luke-jr|wumpus: is update-translations.py not giving you an assert in check_format_specifiers for zh_TW?
  3 2017-02-18 00:38:00	0|cfields|achow101: we've got a mismatch on osx. rebuilding mine.
  4 2017-02-18 00:40:48	0|achow101|cfields: ok. I'll run it again too. Are we still using the osx 10.11 sdk?
  5 2017-02-18 00:41:51	0|cfields|achow101: yes. I suspect that we probably have different versions of the sdk.
  6 2017-02-18 00:42:15	0|achow101|I've been using the same version of the sdk for a long time now
  7 2017-02-18 00:43:08	0|cfields|achow101: ah, I was thinking it was bumped from 0.13, but it's not.
  8 2017-02-18 00:43:49	0|achow101|so I don't need to re-run?
  9 2017-02-18 00:44:12	0|cfields|achow101: let's both re-run and see what happens
 10 2017-02-18 00:44:18	0|achow101|alright
 11 2017-02-18 00:44:28	0|cfields|(unless it's inconvenient for you)
 12 2017-02-18 00:45:06	0|achow101|no, it's fine, I can re-run it
 13 2017-02-18 00:55:42	0|achow101|cfields: I got the same results
 14 2017-02-18 00:57:13	0|cfields|achow101: me too :(
 15 2017-02-18 00:57:32	0|achow101|uh oh
 16 2017-02-18 00:57:57	0|cfields|we need a 3rd builder...
 17 2017-02-18 00:58:14	0|cfields|achow101: can you drop your bitcoin-0.14.0-osx64.tar.gz somewhere?
 18 2017-02-18 00:58:21	0|achow101|sure
 19 2017-02-18 01:00:20	0|achow101|https://drive.google.com/file/d/0Bxw3ip9QfNOUeVFSa0dyWTdHckU/view?usp=sharing
 20 2017-02-18 01:02:36	0|cfields|thank
 21 2017-02-18 01:02:37	0|cfields|s
 22 2017-02-18 01:06:09	0|cfields|achow101: http://pastebin.com/raw/ZTA5Hahi
 23 2017-02-18 01:06:22	0|cfields|looks like there may be non-determinism there
 24 2017-02-18 01:08:21	0|achow101|maybe it's caches?
 25 2017-02-18 01:10:29	0|cfields|we both had empty caches this time
 26 2017-02-18 01:10:47	0|cfields|Binary files bitcoin-0.14.0-me/bin/bitcoin-qt and bitcoin-0.14.0-achow/bin/bitcoin-qt differ
 27 2017-02-18 01:11:15	0|cfields|will see if i can track it down. maybe something new in qt 5.7
 28 2017-02-18 01:20:22	0|bitcoin-git|[13bitcoin] 15MarcoFalke pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/12f46fa7d87d...50a226563cd8
 29 2017-02-18 01:20:23	0|bitcoin-git|13bitcoin/06master 1450a2265 15MarcoFalke: Merge #9788: gitian: bump descriptors for master...
 30 2017-02-18 01:20:23	0|bitcoin-git|13bitcoin/06master 148e5cca0 15Cory Fields: gitian: bump descriptors for master...
 31 2017-02-18 01:20:44	0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #9788: gitian: bump descriptors for master (06master...06gitian-bump-0.15) 02https://github.com/bitcoin/bitcoin/pull/9788
 32 2017-02-18 01:34:19	0|cfields|achow101: can you: find MacOSX10.11.sdk/ -type f -exec sha256sum {} >> sdk.sha256 \;
 33 2017-02-18 01:34:49	0|cfields|in particular, i think we might have differing libobjc.A.tbd
 34 2017-02-18 01:36:18	0|achow101|where should this file/folder be?
 35 2017-02-18 01:37:21	0|cfields|achow101: don't worry about it, i'll just grab a fresh sdk and rule it out
 36 2017-02-18 01:37:50	0|achow101|found it (i've never extracted the sdk tar before)
 37 2017-02-18 01:37:56	0|luke-jr|isn't the SDK always different?
 38 2017-02-18 01:38:39	0|luke-jr|I guess that's probably just the tar format
 39 2017-02-18 01:39:00	0|achow101|this is the hash for libobjc.A.tbd: ef9438dc380ce92cf82538ef483fdbf96743b5f8907c752def4a6d922f822ec6
 40 2017-02-18 01:39:56	0|cfields|achow101: damn, same
 41 2017-02-18 01:40:09	0|cfields|luke-jr: yea, contents should be the same
 42 2017-02-18 01:41:35	0|achow101|cfields: here's the link to the entire sdk.sha256 file: https://drive.google.com/file/d/0Bxw3ip9QfNOUWWtNb2Z6VC1hRXc/view?usp=sharing
 43 2017-02-18 01:46:11	0|cfields|achow101: hmm, they match up
 44 2017-02-18 01:46:49	0|cfields|ok, time to dig into hex/disasm
 45 2017-02-18 02:13:05	0|sipa|cfields: are there any locks protecting CAddrDB and CBanDB?
 46 2017-02-18 02:13:29	0|sipa|DumpBanList or any of its callees never grab any locks
 47 2017-02-18 02:14:41	0|sipa|ah, but it writes a copy
 48 2017-02-18 02:14:49	0|cfields|sipa: it's only grabbed in Start() iirc
 49 2017-02-18 02:16:03	0|sipa|less sure about DumpAddresses
 50 2017-02-18 02:17:51	0|sipa|ah, but addrman has its own lock
 51 2017-02-18 02:35:15	0|achow101|cfields: any luck?
 52 2017-02-18 02:36:39	0|cfields|achow101: not yet
 53 2017-02-18 03:09:08	0|bitcoin-git|[13bitcoin] 15sipa opened pull request #9792: FastRandomContext improvements and switch to ChaCha20 (06master...06chacha) 02https://github.com/bitcoin/bitcoin/pull/9792
 54 2017-02-18 03:37:10	0|cfields|achow101: any chance you could grab the .o files from your gitian environment?
 55 2017-02-18 03:38:05	0|gmaxwell|sipa: re random, seeding is a lot more of an interesting issue... that patch feels like sideways motion to me. sounds fine and all but doesn't obviously get rid of openssl.
 56 2017-02-18 03:42:26	0|sipa|gmaxwell: sure... 1) make fastrandomcontext stronger 2) move some uses that now use getrandbytes to fastrandomcontexts 3) move everything else to a single secure RNG 4) make that single secure RNG not be OpenSSL
 57 2017-02-18 03:43:03	0|sipa|when every use of the strong RNG reads from /dev/urandom etc, i feel we can avoid a lot of the complexity in it (gathering entropy etc)
 58 2017-02-18 03:44:03	0|sipa|s/gathering/background/
 59 2017-02-18 03:47:11	0|achow101|cfields: I don't think so
 60 2017-02-18 03:47:26	0|cfields|achow101: ok
 61 2017-02-18 03:55:23	0|gmaxwell|Making fastrandomcontext robust agaist VM restore, forward/backward prediction will make it slow.
 62 2017-02-18 03:56:06	0|sipa|gmaxwell: don't use fastrandomcontext in places where it needs to be robust
 63 2017-02-18 03:57:02	0|sipa|we don't need VM robustness for determining the filename we temporarily write the addrman db to
 64 2017-02-18 03:57:11	0|sipa|(which currently uses GetRandBytes)
 65 2017-02-18 04:00:14	0|gmaxwell|We don't, I agree. But review what you just said above!
 66 2017-02-18 04:00:35	0|sipa|2) move *some* uses that now use getrandbytes to fastrandomcontexts
 67 2017-02-18 04:03:40	0|gmaxwell|okay I looked through the usage of GetRand* and I'm less skeptical.
 68 2017-02-18 04:03:57	0|gmaxwell|The ones where we need better properties we can use something fantastically slow.
 69 2017-02-18 04:05:13	0|sipa|right, anything that constructs an actual secret key usually just runs once during the whole application (cache blinding keys, wallet keys, ...)
 70 2017-02-18 04:05:45	0|sipa|for ping nonces i don't think that VM restores are that much of a deal
 71 2017-02-18 04:26:07	0|cfields|sigh, best i can tell the linker is reordering some symbols
 72 2017-02-18 04:26:30	0|cfields|don't know how we could be running into that now and not before, though
 73 2017-02-18 04:32:47	0|achow101|it would be great if someone else could also do a gitian build :/
 74 2017-02-18 04:33:05	0|cfields|yea :(
 75 2017-02-18 04:33:39	0|cfields|I'm going to have to throw in the towel for tonight. I'll pick it up again tomorrow. Will be much easier with a third data point.
 76 2017-02-18 04:51:06	0|bitcoin-git|[13bitcoin] 15kobake opened pull request #9793: Vs2015 wip (06master...06vs2015-wip) 02https://github.com/bitcoin/bitcoin/pull/9793
 77 2017-02-18 05:03:22	0|bitcoin-git|[13bitcoin] 15kobake closed pull request #9793: Vs2015 wip (06master...06vs2015-wip) 02https://github.com/bitcoin/bitcoin/pull/9793
 78 2017-02-18 05:13:48	0|luke-jr|achow101: pushed my sigs
 79 2017-02-18 05:14:10	0|luke-jr|which match cfields's
 80 2017-02-18 05:14:24	0|achow101|:(
 81 2017-02-18 05:14:31	0|cfields|woohoo!
 82 2017-02-18 05:16:01	0|achow101|nuked cache and input (except for sdk). trying again
 83 2017-02-18 05:16:21	0|cfields|achow101: double-check your sdk? The only thing I can come up with other than ld64 constructing the indirect symbol table in a non-deterministic order, is that your sdk has a different stub which leads to a different layout
 84 2017-02-18 05:16:44	0|luke-jr|my guess would be something with his VM
 85 2017-02-18 05:16:48	0|cfields|luke-jr: thanks, btw
 86 2017-02-18 05:16:54	0|luke-jr|np, sorry I'm late
 87 2017-02-18 05:17:17	0|cfields|luke-jr: i'm not sure what could matter, other than filesystem or locale
 88 2017-02-18 05:17:20	0|achow101|how do I check the sdk? I don't really have a way to get another one
 89 2017-02-18 05:17:27	0|cfields|but i'd think those would cause many issues, not just these
 90 2017-02-18 05:17:34	0|achow101|(short of someone sending me one)
 91 2017-02-18 05:17:41	0|cfields|achow101: the dump you gave me matches mine. Just make sure that's really the one you're using
 92 2017-02-18 05:18:01	0|cfields|achow101: lxc or kvm?
 93 2017-02-18 05:18:11	0|achow101|kvm
 94 2017-02-18 05:18:16	0|cfields|ok
 95 2017-02-18 05:18:23	0|cfields|yes, same
 96 2017-02-18 05:18:44	0|achow101|lxc stopped working for me a long time ago. I still have an issue open somewhere about it
 97 2017-02-18 05:20:04	0|achow101|the sdk is definitely the same. my original and the one in inputs have the same sha256sum
 98 2017-02-18 05:20:15	0|fanquake|achow101 can you post the sha here?
 99 2017-02-18 05:20:22	0|achow101|9f950cd715b58b0ae002ec47bf2fe7ed76857d4f7b0b3520ae2820082b3e41ce
100 2017-02-18 05:22:13	0|fanquake|That's the sha256 of MacOSX10.11.sdk.tar.gz ?
101 2017-02-18 05:22:17	0|achow101|yep
102 2017-02-18 05:22:26	0|fanquake|Weirdly mine does not match.
103 2017-02-18 05:22:55	0|achow101|should it?
104 2017-02-18 05:22:57	0|fanquake|I see 57a69e69bec23aa190b9ca50726df1ca8bc3d68d70094868211dcbeeb477db91
105 2017-02-18 05:23:48	0|fanquake|I'll wait until my builds done. Should only be a few more minutes.
106 2017-02-18 05:28:46	0|luke-jr|fanquake: it doesn't normally match
107 2017-02-18 05:29:00	0|luke-jr|MacOSX10.11.sdk.tar.gz is not itself typically deterministic because we make it on our own
108 2017-02-18 05:34:00	0|achow101|build finished, got the exact same results
109 2017-02-18 05:40:49	0|achow101|fanquake: your build done yet?
110 2017-02-18 05:41:25	0|fanquake|achow101 yep, one sec.
111 2017-02-18 05:42:20	0|fanquake|achow101 heh my osx sigs match yours.
112 2017-02-18 05:42:28	0|fanquake|8c42c1f996935fc0f2a13616d8b0f557594b23588e7781f72c9ebe53ef097cbd  bitcoin-0.14.0-osx-unsigned.dmg
113 2017-02-18 05:42:41	0|achow101|well shit
114 2017-02-18 05:45:23	0|luke-jr|O.o
115 2017-02-18 05:45:51	0|luke-jr|timezone?
116 2017-02-18 05:46:07	0|achow101|how would that matter?
117 2017-02-18 05:46:46	0|luke-jr|dunno, just guessing cfields and I are probably in the same one and you two might not be
118 2017-02-18 05:48:01	0|achow101|could the version of gitian matter?
119 2017-02-18 05:48:15	0|fanquake|Just PR'd my sigs, if you want to take look.
120 2017-02-18 05:48:34	0|fanquake|Not sure about timezone. I'm using latest gitian.
121 2017-02-18 05:49:09	0|achow101|I think I'm using the latest. hash: ad3f9cc4c2c8c0899961a366f5b9fbd1483b0ee3
122 2017-02-18 05:49:25	0|fanquake|Building with VirtualBox on OS X. Basically using the build-guide in doc.
123 2017-02-18 05:49:43	0|fanquake|Yep, same. "Merge #135: Fixes Debian jessie lxc-init path issue".
124 2017-02-18 05:49:45	0|gribble|https://github.com/bitcoin/bitcoin/issues/135 | [mac os x] Crash on exit · Issue #135 · bitcoin/bitcoin · GitHub
125 2017-02-18 05:53:09	0|luke-jr|I'm on ee1b69d Merge #122: Allow build to use sudo without a password, part deux
126 2017-02-18 05:53:11	0|gribble|https://github.com/bitcoin/bitcoin/issues/122 | Spent per txout by sipa · Pull Request #122 · bitcoin/bitcoin · GitHub
127 2017-02-18 05:54:24	0|achow101|oh well. I'm going to sleep now
128 2017-02-18 05:59:28	0|luke-jr|night
129 2017-02-18 07:43:14	0|bitcoin-git|[13bitcoin] 15mitchellcash opened pull request #9794: Minor update to qrencode package builder (06master...06minor_depends_fix) 02https://github.com/bitcoin/bitcoin/pull/9794
130 2017-02-18 09:22:30	0|wumpus|luke-jr: I fixed that assertion problem recently
131 2017-02-18 09:22:48	0|luke-jr|?
132 2017-02-18 09:22:53	0|luke-jr|oh
133 2017-02-18 09:23:23	0|wumpus|https://github.com/bitcoin/bitcoin/pull/9735
134 2017-02-18 09:24:18	0|luke-jr|thanks
135 2017-02-18 09:26:37	0|wumpus|apparently my for gitian output for linux mismatches
136 2017-02-18 09:27:01	0|wumpus|windows matches, and osx too (except for achow101)
137 2017-02-18 09:30:42	0|wumpus|linux: looks like only the x86_64 output mismatches - aarch64, arm, i686 are the same
138 2017-02-18 09:32:21	0|wumpus|cfields: your gitian assert package overview has "Reading package lists..." ""Building dependency tree..." apt command output in it
139 2017-02-18 09:34:14	0|wumpus|my assert has *three* versions of g++-4.8 in it:g++-4.8_4.8.2-19ubuntu1_amd64.deb, g++-4.8_4.8.4-2ubuntu1~14.04.1_amd64.deb, g++-4.8_4.8.4-2ubuntu1~14.04.3_amd64.deb  ... that seems overkill
140 2017-02-18 09:35:25	0|wumpus|that almost must be the problem, will try to clean it up
141 2017-02-18 09:35:37	0|wumpus|the base image that is, not the assert file :)
142 2017-02-18 10:32:31	0|bitcoin-git|13bitcoin/06master 14a432aa0 15Takashi Mitsuta: Remove unused module from rpc-tests
143 2017-02-18 10:32:31	0|bitcoin-git|[13bitcoin] 15MarcoFalke pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/50a226563cd8...8efd1c820b9a
144 2017-02-18 10:32:32	0|bitcoin-git|13bitcoin/06master 148efd1c8 15MarcoFalke: Merge #9744: Remove unused module from rpc-tests...
145 2017-02-18 10:32:50	0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #9744: Remove unused module from rpc-tests (06master...06remove-unused-modules-from-rpc-tests) 02https://github.com/bitcoin/bitcoin/pull/9744
146 2017-02-18 11:00:03	0|bitcoin-git|[13bitcoin] 15MarcoFalke opened pull request #9795: doc: Update manpages for master (laanwj) (06master...06Mf1602-docManMaster) 02https://github.com/bitcoin/bitcoin/pull/9795
147 2017-02-18 11:14:07	0|bitcoin-git|[13bitcoin] 15MarcoFalke pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/8efd1c820b9a...aa5fa642b0e7
148 2017-02-18 11:14:08	0|bitcoin-git|13bitcoin/06master 140c9b9b7 15practicalswift: [trivial] Fix recently introduced typos in comments
149 2017-02-18 11:14:08	0|bitcoin-git|13bitcoin/06master 14aa5fa64 15MarcoFalke: Merge #9696: [trivial] Fix recently introduced typos in comments...
150 2017-02-18 11:14:31	0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #9696: [trivial] Fix recently introduced typos in comments (06master...06fix-recently-introduced-typos) 02https://github.com/bitcoin/bitcoin/pull/9696
151 2017-02-18 13:09:37	0|wumpus|ok, above gitian issue was solved by rebuildling with a new base image
152 2017-02-18 14:30:38	0|bitcoin-git|[13bitcoin] 15MarcoFalke pushed 6 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/aa5fa642b0e7...7ff4a538a868
153 2017-02-18 14:30:39	0|bitcoin-git|13bitcoin/06master 141581ecb 15John Newbery: Use configparser in rpc-tests.py...
154 2017-02-18 14:30:40	0|bitcoin-git|13bitcoin/06master 1491bffff 15John Newbery: Use argparse in rpc_tests.py...
155 2017-02-18 14:30:40	0|bitcoin-git|13bitcoin/06master 14afd38e7 15John Newbery: Improve rpc-tests.py arguments...
156 2017-02-18 14:30:52	0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #9657: Improve rpc-tests.py (06master...06improvepytests2) 02https://github.com/bitcoin/bitcoin/pull/9657
157 2017-02-18 14:54:04	0|cfields|wumpus: so your sigs match mine now?
158 2017-02-18 14:54:50	0|achow101|I'm going to be away from my build computer today so I won't be able to help
159 2017-02-18 14:56:16	0|cfields|ok
160 2017-02-18 14:56:37	0|cfields|i guess if we have 3 matches we can go with that one, but we really need to figure out why we're getting 2 different results
161 2017-02-18 14:57:43	0|achow101|I tried remaking the base vm and rebuilding before I left but I was having issues with making it
162 2017-02-18 14:59:43	0|cfields|achow101: thanks for trying
163 2017-02-18 15:02:34	0|cfields|wumpus: please confirm if you'd like me to go ahead and push the sigs for the one we've got. the fact that achow101 and fanquake have matching mismatches for osx is a bit worrisome
164 2017-02-18 15:28:02	0|MarcoFalke|"matching mismatches", heh.
165 2017-02-18 15:28:17	0|MarcoFalke|going to run the osx gitian, lets see what my result is...
166 2017-02-18 16:41:00	0|cfields|ok, i'm going to go ahead and push
167 2017-02-18 16:41:41	0|cfields|we need to try to figure out what the discrepancy is before rc2, though
168 2017-02-18 16:59:15	0|cfields|gitian builders: detached osx/win sigs for 0.14.0rc1 pushed.
169 2017-02-18 17:00:03	0|cfields|achow101 / fanquake: not sure what to tell you yet, i'll keep trying to get it pinned down
170 2017-02-18 17:29:18	0|wumpus|cfields: yes, let's try to continue. It's strange that macosx apparently isn't determinstic, but that's no reason to not go on with the other platforms for rc1
171 2017-02-18 17:29:34	0|wumpus|would be nice to sort this out before final, of course
172 2017-02-18 21:50:39	0|luke-jr|[13:09:35] <wumpus> ok, above gitian issue was solved by rebuildling with a new base image <-- eh, not sure this is a good resolution :|
173 2017-02-18 23:02:56	0|luke-jr|thoughts on a GUI screen showing what #bitcoin-watch used to? https://en.bitcoin.it/wiki/Bitcoin-Watch
174 2017-02-18 23:03:04	0|luke-jr|(minus the trade info I guess)