< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #18300: fuzz: Add option to merge input dir to test runner (master...Mf1902-fuzzMerge) https://github.com/bitcoin/bitcoin/pull/18300
< bitcoin-git>
[bitcoin] MarcoFalke merged pull request #18299: tests: Update FUZZERS_MISSING_CORPORA to enable regression fuzzing for all harnesses in master (master...FUZZERS_MISSING_CORPORA) https://github.com/bitcoin/bitcoin/pull/18299
< bitcoin-git>
[bitcoin] practicalswift opened pull request #18299: tests: Update FUZZERS_MISSING_CORPORA to enable regression fuzzing for all harnesses in master (master...FUZZERS_MISSING_CORPORA) https://github.com/bitcoin/bitcoin/pull/18299
< bitcoin-git>
[bitcoin] hebasto closed pull request #18117: build: Fix Qt link issue for macOS target with DEBUG=1 (master...20200211-macos-debug) https://github.com/bitcoin/bitcoin/pull/18117
< bitcoin-git>
[bitcoin] hebasto opened pull request #18298: build: Fix Qt processing of configure script for depends with DEBUG=1 (master...20200309-depends-debug) https://github.com/bitcoin/bitcoin/pull/18298
< bitcoin-git>
[bitcoin] fanquake merged pull request #18274: rpc/wallet: initialize nFeeRequired to avoid using garbage value on failure (master...2003-reset-nfeeret) https://github.com/bitcoin/bitcoin/pull/18274
< bitcoin-git>
bitcoin/master 6ddf435 fanquake: Merge #18274: rpc/wallet: initialize nFeeRequired to avoid using garbage v...
< bitcoin-git>
bitcoin/master a652ba6 Karl-Johan Alm: rpc/wallet: initialize nFeeRequired to avoid using garbage value on failure
< bitcoin-git>
[bitcoin] hebasto opened pull request #18297: build: Use pkg-config in BITCOIN_QT_CONFIGURE for all hosts including Windows (master...20200308-pkgconfig-mingw) https://github.com/bitcoin/bitcoin/pull/18297
< MrSquanchee>
As i can see on the issues page that there aren't much good first issues available. I want to ask whether there is any mechanism in bitcoin dev community to assign specific issues to someone or it's chaos !!!
< bitcoin-git>
[bitcoin] practicalswift closed pull request #18293: tests: Remove exclusion of the feature_block test in the Travis ThreadSanitizer (TSan) job (master...tsan-fix) https://github.com/bitcoin/bitcoin/pull/18293
< bitcoin-git>
[bitcoin] fanquake opened pull request #18295: build: teach ld64 to actually insert MH_BINDATLOAD (master...does_noone_care_about_MH_BINDATLOAD_anymore) https://github.com/bitcoin/bitcoin/pull/18295
< bitcoin-git>
[bitcoin] fanquake closed pull request #16381: Set minimum required Boost to 1.53.0 (master...20190713-bump-minimum-boost) https://github.com/bitcoin/bitcoin/pull/16381
< bitcoin-git>
[bitcoin] practicalswift opened pull request #18293: tests: Remove exclusion of the feature_block test in the Travis ThreadSanitizer (TSan) job (master...tsan-fix) https://github.com/bitcoin/bitcoin/pull/18293
< bitcoin-git>
[bitcoin] dannmat opened pull request #18291: Changed the date format on Block Syncing overlay (master...mar-20-date-format-string) https://github.com/bitcoin/bitcoin/pull/18291
< bitcoin-git>
[bitcoin] hebasto opened pull request #18290: build: Set minimum Automake version to 1.13 (master...20200307-automake-version) https://github.com/bitcoin/bitcoin/pull/18290
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #18289: refactor: Make scheduler methods type safe (master...2003-schedTypeChrono) https://github.com/bitcoin/bitcoin/pull/18289
< bitcoin-git>
[bitcoin] practicalswift opened pull request #18288: build: Add MemorySanitizer (MSan) in Travis to detect use of uninitialized memory (master...msan-in-travis) https://github.com/bitcoin/bitcoin/pull/18288
< bitcoin-git>
[bitcoin] luke-jr opened pull request #18287: depends: Patch libevent build to fix IPv6 -rpcbind on Windows (master...fix_libevent_win_ipv6) https://github.com/bitcoin/bitcoin/pull/18287
< bitcoin-git>
[bitcoin] practicalswift closed pull request #17828: net: Use log categories when logging events that P2P peers can trigger arbitrarily (master...log-categories) https://github.com/bitcoin/bitcoin/pull/17828
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #18285: test: Check that wait_until returns if time point is in the past (master...2002-debugBoost) https://github.com/bitcoin/bitcoin/pull/18285
< bitcoin-git>
[bitcoin] luke-jr opened pull request #18284: [0.19] scheduler: Workaround negative nsecs bug in boost's wait_until (0.19...wrkarnd_boost_wait_until) https://github.com/bitcoin/bitcoin/pull/18284
< aqua42>
you should head over to #bitcoin though (unless you want to improve the rpc :P), this is for bitcoin-core development
< Stealthy>
does bitcoin-qt?
< kallewoof>
Stealthy: zmq works reasonably well, but i hear you. I wouldn't be surprised if there was a nodejs lib out there that does what you want, but most of us here are focused on bitcoin core, as the name of the channel implies.
< bitcoin-git>
[bitcoin] fanquake closed pull request #18276: doc: rpcdoc remove redundant type from description#18258 (master...issue#18258) https://github.com/bitcoin/bitcoin/pull/18276
< bitcoin-git>
[bitcoin] yashpatel20 opened pull request #18276: doc: rpcdoc remove redundant type from description#18258 (master...issue#18258) https://github.com/bitcoin/bitcoin/pull/18276
< bitcoin-git>
[bitcoin] kallewoof opened pull request #18275: wallet: error if an explicit fee rate was given but the needed fee rate differed (master...2003-wallet-error-on-feechange) https://github.com/bitcoin/bitcoin/pull/18275
< bitcoin-git>
[bitcoin] kallewoof opened pull request #18274: wallet: set nFeeRet to 0 to avoid garbage value upon error (master...2003-reset-nfeeret) https://github.com/bitcoin/bitcoin/pull/18274
< bitcoin-git>
[bitcoin] practicalswift opened pull request #18270: util: Fail to parse space-only strings in ParseMoney(...) (instead of parsing as zero) (master...parsemoney-followup) https://github.com/bitcoin/bitcoin/pull/18270
< bitcoin-git>
[bitcoin] MarcoFalke merged pull request #17996: tests: Add fuzzing harness for serialization/deserialization of floating-points and integrals (master...fuzzers-float) https://github.com/bitcoin/bitcoin/pull/17996
< bitcoin-git>
bitcoin/master 8f6fb0a practicalswift: tests: Add serialization/deserialization fuzzing for integral types
< bitcoin-git>
bitcoin/master 3c82b92 practicalswift: tests: Add fuzzing harness for functions taking floating-point types as in...
< bitcoin-git>
bitcoin/master c2bd588 practicalswift: Add missing includes
< BlueMatt>
luke-jr: bitcoin core, as a project, maintains packaging scripts for fetching official release binaries and suggests users run those. if you disagree and want to start suggesting users run packages linked against os libs that have a long history of only half-working, its probably something that needs to be discussed as a project, not the two of us.
< BlueMatt>
luke-jr: if someone wants to replace the ppa with something that checks gitian signatures and downloads official binaries, as I've been saying needs to happen ~forever, then maybe its worth maintinaing again. but no one has ever shown a desire to do that, and the ppa has had many issues due to ubuntu insanity. its definitely a bad idea to use those build scripts for bitcoin nodes.
< luke-jr>
BlueMatt: there's been some interest in keeping the PPA using the bitcoin/bitcoin URI; can you add me to the org?
< moneyball>
For the purpose of this discussion, let's say it is a very high probability that Bitcoin 2020 conference may be postponed.
< moneyball>
Coinbase, Stripe, Microsoft, Twitter all strongly encouraging working from home. Lastly, there is a high probability Bitcoin 2020 will be postponed.
< gribble>
https://github.com/bitcoin/bitcoin/issues/15600 | lockedpool: When possible, use madvise to avoid including sensitive information in core dumps by luke-jr · Pull Request #15600 · bitcoin/bitcoin · GitHub
< wumpus>
the 0.20.0 feature freeze is in roughly a week, see https://github.com/bitcoin/bitcoin/issues/17432, I think it makes sense to focus review on the feature PRs that have a chance to still make it in
< bitcoin-git>
[bitcoin] dangershony closed pull request #18223: Add new filter type v0 for segwit only Scripts to blockfilterindex (master...nutrino-p2wpkh-filters) https://github.com/bitcoin/bitcoin/pull/18223
< mantoshelis>
It's good idea but unfortunately we don't have enough time resources to implement this. As a result, we choose to perform tests on official Bitcoin testnet. :(
< provoostenator>
mantoshelis: testnet is pretty terrible in general. It might be worth looking into adding Signet support to BitcoinJ. Should be pretty easy: https://en.bitcoin.it/wiki/Signet
< kallewoof>
If anyone has topics they wanna talk about, we could gather those and bring them up after the next PR review, assuming people are up for doing more. I'm not actually sure how #bitcoin-core-dev maintains meeting topic proposals tho.
< bitcoin-git>
[bitcoin] luke-jr reopened pull request #15600: lockedpool: When possible, use madvise to avoid including sensitive information in core dumps or forked process memory spaces (master...lockedpool_dontdump) https://github.com/bitcoin/bitcoin/pull/15600
< MarcoFalke>
And if the exception is handled in the scheduler, it seems overkill to modify Bitcoin Core code for purly (unit) testing needs
< bitcoin-git>
[bitcoin] MarcoFalke closed pull request #18232: WIP test: Check that wait_until returns if time point is in the past (master...2002-debugBoost) https://github.com/bitcoin/bitcoin/pull/18232
< MarcoFalke>
Well, it might if you change your time zone or system time. But I think Bitcoin Core does not support that anyway?!
< gribble>
https://github.com/bitcoin/bitcoin/issues/18232 | test: Check that wait_until returns if time point is in the past by MarcoFalke · Pull Request #18232 · bitcoin/bitcoin · GitHub
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #18249: test: Bump timeouts to accomodate really slow disks (master...2003-testTimeout) https://github.com/bitcoin/bitcoin/pull/18249
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #18247: test: Wait for both veracks in add_p2p_connection (master...2003-qaMininodeVerackRace) https://github.com/bitcoin/bitcoin/pull/18247
< bitcoin-git>
[bitcoin] promag opened pull request #18246: gui: Drop connectSlotsByName usage (master...2020-03-drop-connectslotsbyName) https://github.com/bitcoin/bitcoin/pull/18246
< bitcoin-git>
[bitcoin] instagibbs opened pull request #18245: Test some transaction creation with non-empty fee estimator (master...test_est_txn) https://github.com/bitcoin/bitcoin/pull/18245
< bitcoin-git>
[bitcoin] Sjors opened pull request #18244: rpc: have lockUnspents also lock manually selected coins (master...2020/03/rpc_coin_locks) https://github.com/bitcoin/bitcoin/pull/18244
< bitcoin-git>
[bitcoin] jonasschnelli opened pull request #18242: Add BIP324 encrypted p2p transport de-/serializer (only used in tests) (master...2020/03/net_v2) https://github.com/bitcoin/bitcoin/pull/18242
< bitcoin-git>
[bitcoin] kallewoof opened pull request #18241: wallet/refactor: refer to CWallet immutably when possible (master...2002-const-fixes) https://github.com/bitcoin/bitcoin/pull/18241
< kallewoof>
sipa, aj, luke-jr: for the record, changing to immutable-CWallet-pointers in places where it made sense (many places did non-const stuff) resulted in 68 changes; see https://github.com/bitcoin/bitcoin/compare/master...kallewoof:2002-const-fixes ... is this something people would concept ack on a PR?
< bitcoin-git>
[bitcoin] meshcollider merged pull request #18224: Make AnalyzePSBT next role calculation simple, correct (master...analyze_psbt_role_simple) https://github.com/bitcoin/bitcoin/pull/18224
< bitcoin-git>
bitcoin/master 1f88624 Samuel Dobson: Merge #18224: Make AnalyzePSBT next role calculation simple, correct
< bitcoin-git>
bitcoin/master 1ef28b4 Gregory Sanders: Make AnalyzePSBT next role calculation simple, correct
< bitcoin-git>
[bitcoin] promag opened pull request #18239: gui: Refactor to drop client and wallet models setters (master...2020-03-drop-setmodel) https://github.com/bitcoin/bitcoin/pull/18239
< bitcoin-git>
[bitcoin] ajtowns opened pull request #18238: net_processing: Retry notfounds with more urgency (master...202002-bump-notfound) https://github.com/bitcoin/bitcoin/pull/18238
< kallewoof>
So, bitcoin core right now seems to lean towards constant func/method parameters to pointers in the form "type* const" (55 instances of "CWallet* const pwallet") over "const type*" (15 instances of "const CWallet*"), but from what I understand, "type* const" means the pointer itself is const (i.e. "pwallet = blarf" is illegal) but the pointed to instance is not const (i.e. pwallet->MutableThing() is legal). I tested this
< bitcoin-git>
[bitcoin] ajtowns opened pull request #18234: scheduler.cpp: Replace boost::mutex,condition_var,chrono with std equivalents (master...202002-scheduler-deboost) https://github.com/bitcoin/bitcoin/pull/18234
2020-02-29
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #18232: test: Check that wait_until returns if time point is in the past (master...2002-debugBoost) https://github.com/bitcoin/bitcoin/pull/18232