< bitcoin-git> [bitcoin] GChuf opened pull request #16732: [trivial] Add spacing to command (master...patch-1) https://github.com/bitcoin/bitcoin/pull/16732
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/efe1ee0d8d7f...215eec3bb532
< bitcoin-git> bitcoin/master a64dbef Hennadii Stepanov: doc: Update labels in CONTRIBUTING.md
< bitcoin-git> bitcoin/master 215eec3 fanquake: Merge #16723: doc: Update labels in CONTRIBUTING.md
< bitcoin-git> [bitcoin] fanquake merged pull request #16723: doc: Update labels in CONTRIBUTING.md (master...20190825-build-prefix) https://github.com/bitcoin/bitcoin/pull/16723
< bitcoin-git> [bitcoin] fanquake closed pull request #16731: doc: remove "experimental" from README.md (master...master) https://github.com/bitcoin/bitcoin/pull/16731
< bitcoin-git> [bitcoin] fanquake closed pull request #16492: rpc: Add feeRate argument to bumpFee RPC (master...feerate_bumpfee) https://github.com/bitcoin/bitcoin/pull/16492
< meshcollider> luke-jr: so you mean keep the signature constant, change message, derive address, then sign a second message with the same address?
< meshcollider> No that shouldn't be possible
< bitcoin-git> [bitcoin] fanquake pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/215eec3bb532...d21e8aa0a946
< bitcoin-git> bitcoin/master eb2d64b Chuf: doc: Add spacing to command in translation_process.md
< bitcoin-git> bitcoin/master d21e8aa fanquake: Merge #16732: doc: Add spacing to command in translation_process.md
< bitcoin-git> [bitcoin] fanquake merged pull request #16732: doc: Add spacing to command in translation_process.md (master...patch-1) https://github.com/bitcoin/bitcoin/pull/16732
< instagibbs> jeremyrubin, what is a "really bad hash" re:robinhood_hash, specifically?
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/d21e8aa0a946...a7be1cc92be4
< bitcoin-git> bitcoin/master 8a3b2eb James O'Beirne: move-only: move coins statistics utils out of RPC
< bitcoin-git> bitcoin/master a7be1cc MarcoFalke: Merge #16728: move-only: move coins statistics utils out of RPC
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #16728: move-only: move coins statistics utils out of RPC (master...2019-08-au-coinstats) https://github.com/bitcoin/bitcoin/pull/16728
< bitcoin-git> [bitcoin] GChuf opened pull request #16735: GUI: Remove unused menu items for Windows and Linux (master...menu-items) https://github.com/bitcoin/bitcoin/pull/16735
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16737: test: Establish only one connection between nodes in rpc_invalidateblock (master...1908-testConnectOnce) https://github.com/bitcoin/bitcoin/pull/16737
< bitcoin-git> [bitcoin] hebasto opened pull request #16738: build: Remove needless GCC diagnostic pragmas (master...20190827-gcc-pragmas) https://github.com/bitcoin/bitcoin/pull/16738
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #16739: ci: Pass down $MAKEJOBS to test_runner.py, other improvements (master...1908-ciScratch) https://github.com/bitcoin/bitcoin/pull/16739
< emilengler> Does SaveBlockToDisk works with buffers?
< sipa> what are buffers?
< emilengler> I added a usleep to it but it saves multiple blocks at once
< emilengler> I added a 10 seconds usleep function at the end of SaveBlockToDisk
< emilengler> But the log prints lots of them every 10 seconds
< emilengler> Like not one UpdateTip per 10 seconds, around 50 UpdateTip per second
< sipa> what is the code you added?
< emilengler> Correction, I didn't used a usleep I used this_thread::sleep_for
< emilengler> std::this_thread::sleep_for(std::chrono::milliseconds(10000));
< emilengler> #include <chrono>;#include <thread>
< sipa> are you sure the sleep is executed?
< emilengler> sipa: Yes in the log it takes pauses for around 10 seconds
< emilengler> Before something else happens
< sipa> emilengler: oh
< sipa> block activation is not the same as block accepting
< sipa> blocks are stored on disk when they're syntactically valid, and the headers of their parents are known etc
< sipa> full validation only happens when that block becomes part of the best chain
< sipa> due to out-of-order downloading that may be at a different point in time than when it was downloaded
< warren> Checking if this is still true: since sha1 collisions are now possible signed git tags are unsafe since they sign only the sha1 githash, but a signed git commit is OK because it signs a lot more data?
< sipa> warren: the hash used in a signature doesn't need to be collision resistant, only preimage resistant
< sipa> i take that back
< sipa> i'm not sure
< emilengler> sipa: Sorry for the late response but what do you mean with out-of-order downloading? Or more a general question, where does the download actually happens or are there any good graphics for symbolization?
< sipa> emilengler: net_processing mostly
< emilengler> Thanks
< sipa> we don't download blocks in order
< emilengler> Why?
< sipa> emilengler: bitcoin.stackexchange.com
< sipa> search there, and if you don't find an answer ask it there; i'll answer it personally if needed
< emilengler> FWIW, I got the answer. It downloads parallel for a fast-as-possible download if I got it correctly
< harding> emilengler: this is old, but I think it's still roughly accurate: https://btcinformation.org/en/developer-guide#headers-first
< sipa> harding: yeah, not much changed fundamentally
< jeremyrubin> instagibbs: one that throws std::overflow_error would be *really* bad /s
< jeremyrubin> instagibbs: for instance something that can't be distinguished from urandom('8') << 10, because then the tail 10 bits are always 0
< jeremyrubin> instagibbs: so even though the top bits are solid, the bottom bits always being 0 makes it fail.
< bitcoin-git> [bitcoin] promag opened pull request #16740: qa: Relax so that the subscriber is ready before publishing zmq messages (master...2019-08-slowjoiner) https://github.com/bitcoin/bitcoin/pull/16740
< meshcollider> sipa: you mean second preimage right
< meshcollider> warren: annotated tags have more info than just the commit ID, you must be thinking of lightweight tags