< jonasschnelli>
jamesob: I think a way would be: start in your own personal account (or a new entity), find (a) capable maintainer(s), find contributors. If there is acceptable development performance, migrate it to the bitcoin-core github organization.
< jamesob>
jonasschnelli: yeah, I agree - just might be good to have one that's clearly maintained by the bitcoin github org
< jamesob>
Curious - is there a compelling reason against pulling out all the indexes into a separate codebase/runtime, perhaps also in a repo under the bitcoin org? Only exception I can think of would be that `getrawtransaction` rpc would need to be moved into that codebase for txindex=1, but is there any other instance where existing core behavior relies on the existence of those indexes?
< bitcoin-git>
[bitcoin] practicalswift opened pull request #19259: tests: Add fuzzing harness for LoadMempool(...) and DumpMempool(...) (master...fuzzers-mempool-io) https://github.com/bitcoin/bitcoin/pull/19259
< Kiminuo>
Hi, is there some consensus regarding migration from c++11 to c++17 in Bitcoin core? I mean it seems that c++17 is not as widespread as I thought. So the migration will happen in medium term horizon, right? Like maybe in a year or two, right?
< gribble>
https://github.com/bitcoin/bitcoin/issues/19216 | wallet: Remove first parameter to ScanForWalletTransactions start_hash by pstratem · Pull Request #19216 · bitcoin/bitcoin · GitHub
< wumpus>
yes before upgrading bitcoin core (definitely the major version) you need to make sure your other software is compatible
< wumpus>
no changes to bitcoin coren eeded, at all
< yevaud>
jeremyrubin: it turns out that breaking up bitcoin into different daemons needs a comical amount of state exposed, sadly.
< jeremyrubin>
and bitcoin-node is the just network process
< dongcarl>
Perhaps a good topic for some day where there aren't many topics is: what things can we remove from bitcoin-core that would have a net-positive impact?
< gribble>
https://github.com/bitcoin/bitcoin/issues/19251 | [Do Not Merge] ci: Temporary Test Case for negative capabilities in the Clang Thread Safety annotations by hebasto · Pull Request #19251 · bitcoin/bitcoin · GitHub
< bitcoin-git>
[bitcoin] hebasto opened pull request #19251: [Do Not Merge] ci: Temporary Test Case for negative capabilities in the Clang Thread Safety annotations (master...200611-dnm-test) https://github.com/bitcoin/bitcoin/pull/19251
< gribble>
https://github.com/bitcoin/bitcoin/issues/19249 | Add means to handle negative capabilities in the Clang Thread Safety annotations by hebasto · Pull Request #19249 · bitcoin/bitcoin · GitHub
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #19250: wallet: [move-only bugfix] Make RPC help compile-time static (master...2006-rpcWalletHelp) https://github.com/bitcoin/bitcoin/pull/19250
< bitcoin-git>
[bitcoin] hebasto opened pull request #19249: Add means to handle negative capabilities in the Clang Thread Safety annotations (master...200611-nc) https://github.com/bitcoin/bitcoin/pull/19249
< bitcoin-git>
[bitcoin] luke-jr opened pull request #19243: banman: Limit resources consumed by misbehaving node deprioitisation (master...misbehaving_limit) https://github.com/bitcoin/bitcoin/pull/19243
< bitcoin-git>
[bitcoin] hebasto reopened pull request #19238: refactor: Replace RecursiveMutex with Mutex in CAddrMan (master...200610-addrman-mx) https://github.com/bitcoin/bitcoin/pull/19238
< bitcoin-git>
[bitcoin] hebasto closed pull request #19238: refactor: Replace RecursiveMutex with Mutex in CAddrMan (master...200610-addrman-mx) https://github.com/bitcoin/bitcoin/pull/19238
< bitcoin-git>
[bitcoin] luke-jr opened pull request #19242: Add -uaappend option to append a literal string to user agent (master...uaappend) https://github.com/bitcoin/bitcoin/pull/19242
< bitcoin-git>
[bitcoin] dongcarl closed pull request #18072: Use `libc++` headers from macOS SDK instead of from clang (master...2020-01-macos-sdk-with-headers) https://github.com/bitcoin/bitcoin/pull/18072
< wumpus>
I like using sqlite, to be honest, I don't think rolling a new database just for bitcoin core's database is a good idea, yes a dumb store would be enough for key storage, but for transactions it's likely that some more advanced query functionality would be useful at some point
< Kiminuo>
But it's very hard for me to assess whether it's a good idea or not and if it sounds good whether there are corner cases (different platforms, different Bitcoin features,...)
< bitcoin-git>
[bitcoin] jnewbery opened pull request #19239: tests: move generate_wif_key to wallet_util.py (master...2020-06-generate-wif-key) https://github.com/bitcoin/bitcoin/pull/19239
< Kiminuo>
wumpus, Hi, I have been investigating std::filesystem in c++17 a bit (see https://github.com/bitcoin/bitcoin/pull/19183#issuecomment-641321318) and I would like to lay the groundwork for a worthy PR. I'm just wondering whether you would be willing to answer a few questions along the way - given that you know the old issues with boost::filesystem.
< bitcoin-git>
[bitcoin] hebasto opened pull request #19238: refactor: Replace RecursiveMutex with Mutex in CAddrMan (master...200610-addrman-mx) https://github.com/bitcoin/bitcoin/pull/19238
< bitcoin-git>
[bitcoin] Sjors closed pull request #13818: More intuitive GUI settings behavior when -proxy is set (master...2018/07/gui-proxy) https://github.com/bitcoin/bitcoin/pull/13818
< harding>
Yeah, I think the question is why bitcoin-cli -regtest deriveaddresses 'pk(0279be667ef9dcbbac55a06295ce870b07029bfcdb2dce28d959f2815b16f81798)#gn28ywm7' returns a P2PKH address.
< bitcoin-git>
[bitcoin] hebasto opened pull request #19233: Make SetMiscWarning() accept bilingual_str argument (master...200610-bi-warn) https://github.com/bitcoin/bitcoin/pull/19233
< bitcoin-git>
[bitcoin] fanquake merged pull request #19231: gui: add missing translation.h include to fix build (master...translations_gui_fixup) https://github.com/bitcoin/bitcoin/pull/19231
< bitcoin-git>
bitcoin/master 20e9531 fanquake: Merge #19231: gui: add missing translation.h include to fix build
< bitcoin-git>
bitcoin/master 948f113 fanquake: gui: add missing translation.h include to fix build
< sipa>
qt/bitcoin.cpp:460:35: error: ‘Untranslated’ was not declared in this scope
< sipa>
qt/bitcoin.cpp: In function ‘int GuiMain(int, char**)’:
< bitcoin-git>
[bitcoin] sipa opened pull request #19230: [TESTS] Move base58 to own module to break circular dependency (master...202006_addr_base58) https://github.com/bitcoin/bitcoin/pull/19230
2020-06-09
< stevenroose>
kallewoof: did signet copy the bitcoin blockchain??
< gribble>
https://github.com/bitcoin/bitcoin/issues/19216 | wallet: Remove first parameter to ScanForWalletTransactions start_hash by pstratem · Pull Request #19216 · bitcoin/bitcoin · GitHub
< bitcoin-git>
[bitcoin] pstratem opened pull request #19216: wallet: Remove first parameter to ScanForWalletTransactions start_hash (master...2020-06-07-wallet-scanforwallettransactions) https://github.com/bitcoin/bitcoin/pull/19216
2020-06-08
< bitcoin-git>
[bitcoin] achow101 opened pull request #19215: psbt: Include and allow both non_witness_utxo and witness_utxo for segwit inputs (master...psbt-segwit-fixes) https://github.com/bitcoin/bitcoin/pull/19215
< bitcoin-git>
[bitcoin] MarcoFalke merged pull request #18826: Expose txinwitness for coinbase in JSON form from RPC (master...rvagg/txinwitness-for-coinbase) https://github.com/bitcoin/bitcoin/pull/18826
< bitcoin-git>
bitcoin/master 297466b MarcoFalke: Merge #18826: Expose txinwitness for coinbase in JSON form from RPC
< bitcoin-git>
bitcoin/master 3e44210 Rod Vagg: Expose txinwitness for coinbase in JSON form
< bitcoin-git>
bitcoin/master 34645c4 Rod Vagg: Test txinwitness is accessible on coinbase vin
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #19198: test: Check that peers with forcerelay permission are not asked to feefilter (master...2006-testForcerelayFeefilter) https://github.com/bitcoin/bitcoin/pull/19198
< bitcoin-git>
[bitcoin] MarcoFalke merged pull request #19005: doc: Add documentation for 'checklevel' argument in 'verifychain' RPC… (master...add-documentation-for-verifychain) https://github.com/bitcoin/bitcoin/pull/19005
< bitcoin-git>
bitcoin/master 1b90a7b MarcoFalke: Merge #19005: doc: Add documentation for 'checklevel' argument in 'verifyc...
< bitcoin-git>
bitcoin/master 501e6ab Calvin Kim: doc: Add documentation for 'checklevel' argument in 'verifychain' RPC call
< bitcoin-git>
[bitcoin] fanquake opened pull request #19197: init: use std::thread for ThreadImport() (master...thread_import_no_boost) https://github.com/bitcoin/bitcoin/pull/19197
< sipa>
phantomcircuit: or you mean inside bitcoin core codebase? GetMappedAs in CNetAddr
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #19179: [WIP RFC DONOTMERGE] ci: Run ci configs on cirrus (master...2006-ciCirrus) https://github.com/bitcoin/bitcoin/pull/19179
< bitcoin-git>
[bitcoin] DrahtBot reopened pull request #19078: test: Add salvage test for wallet tool (master...2005-testWalletToolSalvage) https://github.com/bitcoin/bitcoin/pull/19078