< midnightmagic> fanquake: I'm more curious about why my system is so different from everyone else's that my deterministic build resulted in different binaries.
< midnightmagic> fanquake: No OS/X updates means there's something else messed up in my case, or else I'd be making OS/X bins the same as everyone else's..
< midnightmagic> So I'm deciding how much I actually care or not.
< fanquake> midnightmagic: I didn’t realise that was for 0.18.0, not 0.18.1.. the difference should be the base image change then.
< elichai2> midnightmagic: what do deterministic builds help in a closed source OS?
< elichai2> (sorry, off topic 😅)
< sipa> elichai2: it's not built _on_ a closed source OS
< sipa> and yes, off topic
< elichai2> sipa: (well in the end the code is going to run *on Mac*, and if it's gitian then he probably uses a VM *on Mac* to use gitian)
< luke-jr> midnightmagic: when did you last let the base images update? before 0.18.0 matched for me, I had to let mine uupdate :/
< luke-jr> (which sadly means building pre-0.18.0 probably won't match now!)
< fanquake> elichai2: the macOS binaries are compiled on Linux using Clang. The only somewhat closed source component is extracting the macOS SDK to build against. As it requires downloading Xcode.
< fanquake> Gitian also runs on macOS using one small patch, details here: https://github.com/fanquake/core-review/blob/master/gitian-building/README.md
< sipa> well the SDK stuff is not closed source afaik; just not freepy licensed
< sipa> *freely
< fanquake> True. Also need to signup for a Apple dev account if you want to download.
< luke-jr> s/not freely licensed/ not licensed for distribution at all/
< elichai2> fanquake: ha. that's actuall nice. my experience with cross compilation was pretty bad in the past
< midnightmagic> 0.18.1rc1 works so far.. so I'll say "not enough." :-) so.. yeah looks like an update. Okay fair enough. I can deal with that.
< bitcoin-git> [bitcoin] fanquake opened pull request #16477: build: skip deploying plugins we dont use in macdeployqtplus (master...slim_macdeployqtplus) https://github.com/bitcoin/bitcoin/pull/16477
< mryandao> how can users residing in sanctioned countries raise issues on github now that github is blocking US-sacntioned countries?
< luke-jr> mryandao: why do they need to raise issues on github?
< midnightmagic> mryandao: there's good contact methods entirely outside of censored github.
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/febf3a856bcf...94df084f2ac5
< bitcoin-git> bitcoin/master a339367 João Barbosa: Exclude depends/Makefile in .gitignore
< bitcoin-git> bitcoin/master 94df084 fanquake: Merge #16468: Exclude depends/Makefile in .gitignore
< bitcoin-git> [bitcoin] fanquake merged pull request #16468: Exclude depends/Makefile in .gitignore (master...2019-07-gitignore) https://github.com/bitcoin/bitcoin/pull/16468
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/94df084f2ac5...3489b715120b
< bitcoin-git> bitcoin/master bf3be52 Suhas Daftuar: [qa] Ensure we don't generate a too-big block in p2sh sigops test
< bitcoin-git> bitcoin/master 3489b71 MarcoFalke: Merge #16464: [qa] Ensure we don't generate a too-big block in p2sh sigops...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16464: [qa] Ensure we don't generate a too-big block in p2sh sigops test (master...2019-07-fix-feature-block) https://github.com/bitcoin/bitcoin/pull/16464
< bitcoin-git> [bitcoin] promag closed pull request #15974: refactor: Avoid UniValue copy constructor (master...2019-05-copy-univalue) https://github.com/bitcoin/bitcoin/pull/15974
< bitcoin-git> [bitcoin] harding opened pull request #16481: Trivial: add missing space (master...2019-07-init-missing-space) https://github.com/bitcoin/bitcoin/pull/16481
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/3489b715120b...5c2885f9b2e3
< bitcoin-git> bitcoin/master 16b3748 David A. Harding: Trivial: add missing space
< bitcoin-git> bitcoin/master 5c2885f fanquake: Merge #16481: Trivial: add missing space
< bitcoin-git> [bitcoin] fanquake merged pull request #16481: logs: add missing space (master...2019-07-init-missing-space) https://github.com/bitcoin/bitcoin/pull/16481