< jnewbery>
luke-jr: not trolling. If a PR is opened where the goal is primarily to make things easier for downstream projects, then (1) that should be explicitly declared in the PR description (2) it should be judged for inclusion in Bitcoin Core solely on whether it's a benefit to the users of this project.
< bitcoin-git>
[bitcoin] sdaftuar opened pull request #20561: p2p: periodically clear m_addr_known, and choose new peers to receive relayed addresses (master...2020-12-moar-addrz) https://github.com/bitcoin/bitcoin/pull/20561
< luke-jr>
then people who want to switch between versions regularly can just put strict=0 in bitcoin.conf
< fanquake>
Yes that's possible, and probably quite likely. I'd imagine there are many people running bitcoin nodes, that have no interest in it supporting UPNP, ZMQ, or even having the wallet compiled into it.
< miketwenty1>
in the future / even with guix .. would you imagine gitian sigs for multiple common options for each minor/major release of bitcoin? like one with everything enabled.. another build is just the node.. another build with zmq and maybe a few other things .. etc..
< miketwenty1>
Question about bitcoin builds and gitian builds, when you build from source you can put in build options/flags to exclude/include certain things like whether or not the wallet, UPnP, zmq.. are these build options baked into gitian builds to enable everything? If I for example wanted to do a gitian build and not build someone or include something do I have this flexibility?
< bitcoin-git>
[bitcoin] laanwj merged pull request #20520: depends: Do not force Precompiled Headers (PCH) for building Qt on Linux (master...201127-pch) https://github.com/bitcoin/bitcoin/pull/20520
< bitcoin-git>
bitcoin/master a3186b6 Wladimir J. van der Laan: Merge #20520: depends: Do not force Precompiled Headers (PCH) for building...
< bitcoin-git>
bitcoin/master c82d15b Hennadii Stepanov: depends: Do not force Precompiled Headers (PCH) for building Qt on Linux
< aj>
#proposedmeetingtopic bitcoin-util cli utility for 19937 (see also 14671 and 18573)
2020-12-02
< bitcoin-git>
[bitcoin] luke-jr opened pull request #20551: RPC/Net: Allow changing the connection_type for addnode onetry (master...rpc_onetry_conntype) https://github.com/bitcoin/bitcoin/pull/20551
< bitcoin-git>
[bitcoin] luke-jr closed pull request #12674: RPC: Support addnode onetry without making the connection priviliged (master...rpc_onetry_nonpriv) https://github.com/bitcoin/bitcoin/pull/12674
< bitcoin-git>
[bitcoin] hebasto opened pull request #20544: build: Do not repeat warning names in -Werror=... options (master...201202-werror) https://github.com/bitcoin/bitcoin/pull/20544
< bitcoin-git>
[bitcoin] MarcoFalke merged pull request #20507: sync: print proper lock order location when double lock is detected (master...double_lock_print_location) https://github.com/bitcoin/bitcoin/pull/20507
< bitcoin-git>
bitcoin/master db058ef Vasil Dimov: sync: use HasReason() in double lock tests
< bitcoin-git>
bitcoin/master a21dc46 Vasil Dimov: sync: const-qualify the argument of double_lock_detected()
< bitcoin-git>
bitcoin/master 6d3689f Vasil Dimov: sync: print proper lock order location when double lock is detected
< jnewbery>
vasild: if you're still around, I have a question about addrv2 support negotiation. There seems to be a discrepency between the bip and the implementation in Bitcoin Core
< gleb>
So I switched the code to do sketch extensions instead. It's much less code, and the code complexity is now more aligned with general Bitcoin project complexity.
< gleb>
In practice, the implementation turned out to be too complicated on the Bitcoin Core p2p protocol side.
< cdecker>
If there's interest from miners to get access I can write up a faux-bitcoin-node that acts as a bridge between bitcoin p2p and the ln overlay relay network
< sipa>
i was hoping LN, because it's an inherently different situation than bitcoin's identityless P2P
< ariard>
sipa: define general ? if you mean for any bitcoin protocol without care of protocol designer I likely agree with you
< ariard>
michaelfolkson: note also that pinning doesn't only affect LN but also bitcoin protocol like vaults, DLC
< bitcoin-git>
[bitcoin] naumenkogs opened pull request #20539: Avoid rebucketing on restart when it's not necessary (master...2020-12-01-rebucket-asmap-fix) https://github.com/bitcoin/bitcoin/pull/20539
< bitcoin-git>
[bitcoin] MarcoFalke merged pull request #20519: Handle rename failure in DumpMempool(...) by using the RenameOver(...) return value. Add [[nodiscard]] to RenameOver(...). (master...renameover-nodiscard) https://github.com/bitcoin/bitcoin/pull/20519
< bitcoin-git>
bitcoin/master ffd5e7a MarcoFalke: Merge #20519: Handle rename failure in DumpMempool(...) by using the Renam...
< bitcoin-git>
bitcoin/master ce9dd45 practicalswift: Add [[nodiscard]] to RenameOver(...)
< bitcoin-git>
bitcoin/master 9429a39 practicalswift: Handle rename failure in DumpMempool(...) by using RenameOver(...) return ...
< bitcoin-git>
[bitcoin] theStack opened pull request #20537: refactor: replace manual Satoshis-to-BTC conversions with FormatMoney() (master...20201129-use-formatmoney) https://github.com/bitcoin/bitcoin/pull/20537
2020-11-30
< bitcoin-git>
[bitcoin] achow101 opened pull request #20536: wallet: Error with "Transaction too large" if the funded tx will end up being too large after signing (master...fundtx-error-large-tx) https://github.com/bitcoin/bitcoin/pull/20536
< sipa>
seed.bitcoin.sipa.be may not even *have* a full node
< pinheadmz>
oh I see, so it still may not be the full node at bitcoin.sipa.be that we conenct to, but any one of the nodes returned in A records to the exit node, which thinks we are just resolving a single service by name
< sipa>
yes, because you can't do DNS resolving over Tor (that'd require a DNS implementation in bitcoin core, and route it over tor)
< sipa>
but from bitcoin core's perspective there is no such thing as A or AAAA or CNAME or whatever records or DNS involved
< pinheadmz>
actually im not getting anything from seed.bitcoin.sipa.be but I am from `dig dnsseed.bluematt.me`
< sipa>
doing so likely makes your OS's/ISP's resolver connect to the DNS seed server, but that's transparent from bitcoin core's perspective
< pinheadmz>
ah, TIL so there is no `dig bitcoin.sipa.de` or whatever. I always thought we use A records from the seed to bootstrap
< willcl_ark>
Agreed. I imagined Luke might be trying to grep bitcoin and bitcoin-gui repos at the same time, and using some small script to bolt together two git searches? Not familiar with git-grep-all tbh :)
< sipa>
i've never had a situation where "git grep" in the bitcoin repo wasn't fast enough
< willcl_ark>
luke-jr: I've been trying out ripgrep recently. It can handle multiple directories and seems to be pretty fast. e.g. `rg CTxDestination bitcoin*` will search all directories in current dir starting with bitcoin.
< miketwenty1>
hebasto: then maybe im not understanding something or something is failing silently on my side after the build is done.. in your inputs dir does it create the version file and the non version file? -> gitian-builder/inputs/bitcoin-win-unsigned.tar.gz
< bitcoin-git>
[bitcoin] hebasto opened pull request #20520: ci, depends: Do not force Precompiled Headers (PCH) when building Qt (master...201127-pch) https://github.com/bitcoin/bitcoin/pull/20520
< bitcoin-git>
[bitcoin] luke-jr closed pull request #20484: RPC: Migrate estimatesmartfee return value from "feerate" (BTC/kvB) to "fee_rate" (sat/vB) (master...estimatefee_fee_rate) https://github.com/bitcoin/bitcoin/pull/20484
< bitcoin-git>
[bitcoin] practicalswift opened pull request #20519: Handle rename failure in DumpMempool(...) by using the RenameOver(...) return value. Add [[nodiscard]] to RenameOver(...). (master...renameover-nodiscard) https://github.com/bitcoin/bitcoin/pull/20519
< RickMortir22>
Anyone Can Help? - My Bitcoin Core dont Work Anymore, it stoped syng 19 hours ago and dont sync anymore, its shows "19 hours ago" forever, and status bar show "connecting..."
< RickMortir22>
Anyone Can Help? - My Bitcoin Core dont Work Anymore, it stoped syng 19 hours ago and dont sync anymore, its shows "19 hours ago" forever, and status bar show "connecting..."
< aj>
wumpus: any thoughts on "bitcoin-util grind" ? another option might be to add "bitcoin-cli -grind=HEADER" with the idea being to move it into bitcoin-util in a later PR that also includes other useful stuff, like psbt commands?
< bitcoin-git>
[bitcoin] bitcoinhodler opened pull request #20513: Release notes: remove mention of default wallet creation (master...no-default-wallet) https://github.com/bitcoin/bitcoin/pull/20513
< bitcoin-git>
[bitcoin] emilengler opened pull request #20512: doc: Add bash as an OpenBSD dependency (master...2020-11-doc-build-openbsd-bash-dependency) https://github.com/bitcoin/bitcoin/pull/20512
< ajonas>
In January, jnewbery sent out a survey that he planned to present the results of at the Coredev in March. Given that the meeting never happened, I put together a presentation/post summarizing the responses. Both can be found at https://adamjonas.com/bitcoin/coredev/retro/coredev-2019-retro/.
< glozow>
sipa: bitcoin gandalf, roams Middle Earth with the hobbitcoins
< RickMortir22>
Anyone Can Help? - My Bitcoin Core dont Work Anymore, it stoped syng 19 hours ago and dont sync anymore, its shows "19 hours ago" forever, and status bar show "connecting..."
< bitcoin-git>
[bitcoin] jonatack opened pull request #20510: [backport] wallet: allow zero-fee fundrawtransaction/walletcreatefundedpsbt and other fixes (0.21...backport-fee_rate-follow-ups) https://github.com/bitcoin/bitcoin/pull/20510
< bitcoin-git>
[bitcoin] vasild opened pull request #20509: net: CAddress deser: use stream's version, not what's coming from disk (master...caddress_deser_version) https://github.com/bitcoin/bitcoin/pull/20509
< bitcoin-git>
[bitcoin] sipsorcery opened pull request #20508: IGNORE: Testing appveyor CI build with pre-built dependencies (master...msvc-vcpkg-prebuilt) https://github.com/bitcoin/bitcoin/pull/20508
< bitcoin-git>
[bitcoin] vasild opened pull request #20507: sync: print proper lock order location when double lock is detected (master...double_lock_print_location) https://github.com/bitcoin/bitcoin/pull/20507
< bitcoin-git>
[bitcoin] sipsorcery opened pull request #20506: WIP: AppVeyor CI fixes in preparation for next image bump (master...msvc-no-optimise) https://github.com/bitcoin/bitcoin/pull/20506
< bitcoin-git>
[bitcoin] hebasto closed pull request #19832: p2p: Put disconnecting logs into BCLog::NET category (master...200829-log) https://github.com/bitcoin/bitcoin/pull/19832
< wumpus>
oh would have been more on-topic in #bitcoin-builds i guess sorry
< bitcoin-git>
[bitcoin] dergoegge opened pull request #20505: [backport] build: Avoid secp256k1.h include from system (0.21...21_backport) https://github.com/bitcoin/bitcoin/pull/20505
< bitcoin-git>
[bitcoin] fanquake opened pull request #20504: build: use more legible (q)make commands in qt package (master...legible_qt_config_cmds) https://github.com/bitcoin/bitcoin/pull/20504
< sipa>
there is no functionality for automatically detecting other bitcoin node in the local network
2020-11-25
< bitcoin-git>
[bitcoin] practicalswift opened pull request #20499: Remove obsolete NODISCARD ifdef forest. Use [[nodiscard]] (C++17). (master...remove-nodiscard-cruft) https://github.com/bitcoin/bitcoin/pull/20499
< bitcoin-git>
[bitcoin] sanket1729 opened pull request #20497: [Refactor] Add MAX_STANDARD_SCRIPTSIG_SIZE to policy (master...policy) https://github.com/bitcoin/bitcoin/pull/20497
< bitcoin-git>
[bitcoin] dongcarl opened pull request #20495: sync: Use decltype(auto) return type for WITH_LOCK (master...2020-11-decltype-auto) https://github.com/bitcoin/bitcoin/pull/20495