< gmaxwell> sipa: POWER9 has a random instruction... it's called "darn" ... Deliver A Random Number.
< sipa> haha
< achow101> fucking git deleted all my worktrees..
< sipa> achow101: define deleted?
< achow101> pruned all of the worktrees. the actual directories were untouched. it was mid rebase though so I thought a lost a bunch of commits
< midnightmagic> Just for future reference: since I was told that the vmbuilder thingy is broken on modern systems I attempted to manually upgrade the gitian base image for bionic -- from the old trusty that I was using, via lxc-execute in the base image. however, the grab-packages script attempts to force a re-download of all installed packages but will fail in an upgrade scenario due to the "deinstall
< midnightmagic> ok" possible state for some of the
< midnightmagic> upgraded packages (cgmanager in particular.) Additionally, the query issued by grab-packages will return installed packages which are leftovers from the upgrade -- libapt-inst1.5 for example. These have to be manually apt-get remove'd. gsign on the newer platform now (sigh) gets to deal with a pinentry timeout, but otherwise I think this is about it for doing an upgrade rather than a new
< midnightmagic> install
< provoostenator> meshcollider: you're right, it's a seperate field now, but sill in the comment section, so we could move that to the left.
< meshcollider> I thought thats what Luke's PR did
< provoostenator> #11803 does the formatting work too, we only to move it to the left
< gribble> https://github.com/bitcoin/bitcoin/issues/11803 | Bugfix: RPC/Wallet: Include HD key metadata in dumpwallet by luke-jr · Pull Request #11803 · bitcoin/bitcoin · GitHub
< warren> midnightmagic: vmbuilder has been broken for me for years now, but fortunately dongcarl is close to making a much better deterministic environment to replace gitian.
< warren> midnightmagic: https://github.com/bitcoin/bitcoin/pull/15277 it needs more work
< fanquake> meshcollider can you have a look at 15392
< meshcollider> fanquake: yep, almost finished with reviewing #15368 then I'll take a look
< gribble> https://github.com/bitcoin/bitcoin/issues/15368 | Descriptor checksums by sipa · Pull Request #15368 · bitcoin/bitcoin · GitHub
< fanquake> np
< midnightmagic> warren: oh, is he using rbm?
< midnightmagic> (for the deterministic builds?)
< midnightmagic> I think the tor people are using rbm.
< IZooo> Hello :)
< bitcoin-git> [bitcoin] luke-jr reopened pull request #11803: Bugfix: RPC/Wallet: Include HD key metadata in dumpwallet (master...bugfix_dumpwallet_hdkeypath) https://github.com/bitcoin/bitcoin/pull/11803
< wumpus> achow101: looks like the psbt tests are not passing on #13932
< gribble> https://github.com/bitcoin/bitcoin/issues/13932 | Additional utility RPCs for PSBT by achow101 · Pull Request #13932 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] luke-jr opened pull request #15423: torcontrol: Query Tor for correct -onion configuration (master...tor_socks_port) https://github.com/bitcoin/bitcoin/pull/15423
< achow101> wumpus: should be fixed now
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 commits to master: https://github.com/bitcoin/bitcoin/compare/743c2f461c0e...f9d50e83e290
< bitcoin-git> bitcoin/master fab6b07 MarcoFalke: test: txindex: interrupt threadGroup before calling destructor
< bitcoin-git> bitcoin/master f9d50e8 MarcoFalke: Merge #15410: test: txindex: interrupt threadGroup before calling destruct...
< bitcoin-git> [bitcoin] MarcoFalke merged pull request #15410: test: txindex: interrupt threadGroup before calling destructor (master...Mf1902-testIndexInterr) https://github.com/bitcoin/bitcoin/pull/15410
< RC> hi
< wumpus> that's beautiful
< wumpus> eh re gmaxwell | sipa: POWER9 has a random instruction... it's called "darn" ... Deliver A Random Number.
< wumpus> achow101: ok
< harding> achow101: my git worktrees for the bitcoin repository also broke inexplicably two days ago in between my uses of them. I thought maybe something on my system broke them (I use Qubes 4 + Debian 9), e.g. some symlink was lost, but now I'm wondering if it was something else.
< achow101> harding: i'm also using qubes 4 with debian 9. Maybe a qubes problem..
< harding> achow101: do you use worktrees with any other projects? I do, and I just checked a few and didn't find any of them broken (even ones on the same qube as my bitcoin repository). I don't know what would distinguish bitcoin from other repositories I use; the only thing I can think of is that it's the largest (size and object count) that I use with worktrees.
< achow101> harding: i do. I think it's because bitcoin has a lot of objects which the garbage collector cleans up every so often.
< harding> achow101: interesting. I have another repository on a separate qube that has lots of objects and gets routinely compacted. I don't have a worktree of it, but I'll create one now and check back in on it in a week or so.
< wumpus> no problems with my worktrees here (ubuntu 18.04)
< wumpus> (on that system I have a seperate worktree for every branch)
< bitcoin-git> [bitcoin] laanwj pushed 6 commits to master: https://github.com/bitcoin/bitcoin/compare/f9d50e83e290...d5b929c813ff
< bitcoin-git> bitcoin/master 7344a7b Andrew Chow: Implement utxoupdatepsbt RPC and tests
< bitcoin-git> bitcoin/master 08f749c Andrew Chow: Implement joinpsbts RPC and tests
< bitcoin-git> bitcoin/master cb40b3a Andrew Chow: Figure out what is missing during signing
< bitcoin-git> [bitcoin] laanwj merged pull request #13932: Additional utility RPCs for PSBT (master...psbt-util-rpcs) https://github.com/bitcoin/bitcoin/pull/13932
< bitcoin-git> [bitcoin] Sjors opened pull request #15424: [wallet] wallet-tool: command to remove key metadata (master...2019/02/wallet_tool_remove_metadata) https://github.com/bitcoin/bitcoin/pull/15424
< bitcoin-git> [bitcoin] laanwj pushed 5 commits to master: https://github.com/bitcoin/bitcoin/compare/d5b929c813ff...f60d029a2a76
< bitcoin-git> bitcoin/master 3b40bff Pieter Wuille: Descriptor checksum
< bitcoin-git> bitcoin/master b52cb63 Pieter Wuille: Add getdescriptorinfo to compute checksum
< bitcoin-git> bitcoin/master be62903 Pieter Wuille: Make descriptor checksums mandatory in deriveaddresses and importmulti
< bitcoin-git> [bitcoin] laanwj merged pull request #15368: Descriptor checksums (master...201902_descsum) https://github.com/bitcoin/bitcoin/pull/15368
< wumpus> I think that concludes the feature-merging for 0.18
< wumpus> the rest in the milestone I'd consider fixes
< sipa> \o/
< bitcoin-git> [bitcoin] harding opened pull request #15425: [Doc] add missing newline to listunspent help for witnessScript (master...2019-02-help-listunspent-missing-newline) https://github.com/bitcoin/bitcoin/pull/15425
< meshcollider> \o/
< bitcoin-git> [bitcoin] harding opened pull request #15426: [Doc] importmulti: add missing description of keypool option (master...2019-02-importmulti-keypool) https://github.com/bitcoin/bitcoin/pull/15426
< achow101> \o/