2015-10-04

< CodeShark> rather than having to constrain our refactoring efforts to avoid pushback from people who long ago forked Bitcoin Core :p
< CodeShark> even alt coins could be built that can contribute useful ideas back to Bitcoin ;)
< gmaxwell> the structure of bitcoin core today is much more agreeable to that than it was in the past.
< gmaxwell> there was a begin a it in bitcoin core that was long since stripped out
< Luke-Jr> FYI: Someone in #bitcoin is actively trying to get a virus signature mined into the blockchain.

2015-10-03

< GitHub103> [bitcoin] ptschip opened pull request #6752: Changed rpc-tests.sh to rpc-tests.py in README.md (master...readme) https://github.com/bitcoin/bitcoin/pull/6752
< GitHub18> [bitcoin] petertodd opened pull request #6751: Document pull-req #6424 in release-notes (master...pull-req-6424-release-notes) https://github.com/bitcoin/bitcoin/pull/6751

2015-10-02

< BlueMatt> morcos: please make sure to fix https://github.com/bitcoin/bitcoin/issues/6531 when you do
< BlueMatt> the big miners I've heard from dont care about fees, and are perfectly willing to forgo them to make bitcoin work properly for users - if users' wallets are using cpfp to unstick transactions, then it becomes required for bitcoin to work properly for users
< btcdrak> this really is a discussion for #bitcoin-dev or #bitcoin
< jcorgan> guys, can you take it to #bitcoin-dev or #bitcoin?
< jgarzik> no it's not quite chichen-and-egg. users are incentivized against it because long chains of unconfirmed transactions do not work well in the bitcoin system. that's not specific to CPFP but relevant to evaluating CPFP.
< jgarzik> In theory sure. But's it's optimizing for the "last 0.0001%" Bitcoin isn't great for long chains of unconfirmed transactions. Sure miners should capture those, but users and services will not generate them.
< btcdrak> maaku: sdaftuar was asking if there was a resolution to this http://lists.linuxfoundation.org/pipermail/bitcoin-dev/2015-July/009344.html
< GitHub108> [bitcoin] petertodd opened pull request #6750: Recent rejects backport to v0.11 (0.11...recent-rejects-v0.11-backport) https://github.com/bitcoin/bitcoin/pull/6750
< CodeShark> are you saying the race conditions are in Bitcoin Core? or in the travis environment?
< wumpus> run your tests while running a parallel a build of bitcoin core with gccin the background?
< Luke-Jr> can someone throw a notice on BIP 62 that it is currently not deployable? http://blog.coinkite.com/post/130318407326/ongoing-bitcoin-malleability-attack-low-s-high
< wumpus> where bitcoin developers do their ritual tribal name-swapping dance
< gmaxwell> http://blog.coinkite.com/post/130318407326/ongoing-bitcoin-malleability-attack-low-s-high "BIP62 is not currently deployed and we encourage all miners to enforce it as soon as possible." 0_o
< GitHub176> [bitcoin] paveljanik opened pull request #6748: Rewrite help texts for features enabled by default (master...configure_disable) https://github.com/bitcoin/bitcoin/pull/6748
< GitHub54> [bitcoin] CodeShark opened pull request #6747: Softforks unit (master...softforks_unit) https://github.com/bitcoin/bitcoin/pull/6747
< CodeShark> to be clear, I don't want to sound like I'm complaining - this is not a complaint...I want to help Bitcoin Core be more efficient
< CodeShark> a good number of people doing real quantum leap type work on Bitcoin are waiting on changes that are taking forever
< CodeShark> rusty, basically https://github.com/bitcoin/bitcoin/compare/master...CodeShark:versionbits but without softforks.h/softforks.cpp
< phantomcircuit> (i bet that's mostly the reason why people who up in #bitcoin and ask why they have no inbound connections)
< PRab> https://github.com/bitcoin/bitcoin/blob/master/contrib/gitian-downloader/prab-key.pgp has expired, but I have created and rolled over to a new key. What is the "proper" way to update this?
< GitHub53> [bitcoin] pstratem opened pull request #6745: Net: Remove "Address refresh broadcast" logic. (master...addr_known_reset) https://github.com/bitcoin/bitcoin/pull/6745

2015-10-01

< gribble> variable was used before it was initialized by tuttleorbuttle · Pull Request #4096 · bitcoin/bitcoin · GitHub | https://github.com/bitcoin/bitcoin/pull/4096
< gribble> variable was used before it was initialized by tuttleorbuttle · Pull Request #4096 · bitcoin/bitcoin · GitHub | https://github.com/bitcoin/bitcoin/pull/4096
< gribble> Update block-tester by TheBlueMatt · Pull Request #5422 · bitcoin/bitcoin · GitHub | https://github.com/bitcoin/bitcoin/pull/5422
< kanzure> maaku: re: https://github.com/bitcoin/bitcoin/pull/6564 my concern was something about CSV-failing transaction rejection and then parent replacement if it was also in mempool, but nevermind i see why that's not important here
< GitHub142> [bitcoin] laanwj closed pull request #5422: Update block-tester (master...mempoolfix4) https://github.com/bitcoin/bitcoin/pull/5422
< GitHub91> [bitcoin] laanwj closed pull request #6616: Regression Tests: Migrated rpc-tests.sh to all Python rpc-tests.py (master...all_python) https://github.com/bitcoin/bitcoin/pull/6616
< GitHub196> bitcoin/master 5ab5dca Wladimir J. van der Laan: Merge pull request #6616...
< GitHub196> bitcoin/master 5467820 ptschip: Migrated rpc-tests.sh to all python rpc-tests.py...
< GitHub196> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/19c71864258f...5ab5dca6f1bb
< btcdrak> BIP113 Mempool-only Median time-past as endpoint for lock-time calculations https://github.com/bitcoin/bitcoin/pull/6566
< btcdrak> BIP112 Mempool-only CHECKSEQUENCEVERIFY https://github.com/bitcoin/bitcoin/pull/6564
< btcdrak> BIP68 Mempool-only sequence number constraint verification https://github.com/bitcoin/bitcoin/pull/6312
< GitHub173> [bitcoin] laanwj closed pull request #6739: Trivial: start the help texts with lowercase (master...patch-9) https://github.com/bitcoin/bitcoin/pull/6739
< GitHub65> bitcoin/master 19c7186 Wladimir J. van der Laan: Merge pull request #6739...
< GitHub65> bitcoin/master 96106f0 paveljanik: [Trivial] start the help texts with lowercase
< GitHub65> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cd78c2a421ae...19c71864258f
< GitHub63> [bitcoin] laanwj closed pull request #6424: Policy: Decouple Solver() from nMaxDatacarrierBytes (for free) (master...policy-datacarriersize-0.11.99) https://github.com/bitcoin/bitcoin/pull/6424
< GitHub54> bitcoin/master da894ab Peter Todd: Accept any sequence of PUSHDATAs in OP_RETURN outputs...
< GitHub54> bitcoin/master 6a07eb6 Peter Todd: Make TX_SCRIPTHASH clear vSolutionsRet first...
< GitHub54> bitcoin/master 5d8709c Peter Todd: Add IsPushOnly(const_iterator pc)...
< GitHub54> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/bb882d04e862...cd78c2a421ae
< paveljanik> can we have a bot here "expanding" PR numbers? E.g. after entering #6550, it could print: Obfuscate chainstate (https://github.com/bitcoin/bitcoin/pull/6650).
< sipa> no, in #bitcoin-dev
< wumpus> in #bitcoin-dev still
< GitHub70> [bitcoin] laanwj opened pull request #6744: build: disable -Wself-assign (master...2015_10_clang_self_assignment_warning) https://github.com/bitcoin/bitcoin/pull/6744
< GitHub35> [bitcoin] jmcorgan opened pull request #6743: autotools: move checking for zmq library to common area in configure.ac (master...fixes/6679) https://github.com/bitcoin/bitcoin/pull/6743
< Naphex> then just move back to #bitcoin-dev and get jgarzik too stop removing the bans
< jgarzik> ok, #bitcoin-dev as planned
< wumpus> but also bitcoin core specific stuff like the mempool management
< jgarzik> Is it a Bitcoin Core meeting or bitcoin protocol meeting? If the former, have the meeting in here, and keep an eye on #bitcoin-dev for any stragglers
< btcdrak> wumpus: meetbot is not setup for #bitcoin-dev, so better here.
< GitHub185> [bitcoin] laanwj closed pull request #5079: Accept any sequence of PUSHDATAs in OP_RETURN outputs (master...op-return-accept-multiple-pushdatas) https://github.com/bitcoin/bitcoin/pull/5079
< GitHub119> [bitcoin] laanwj closed pull request #6732: build: remove libressl check (proposal) (master...2015_09_remove_libressl_check) https://github.com/bitcoin/bitcoin/pull/6732
< GitHub186> bitcoin/master bb882d0 Wladimir J. van der Laan: Merge pull request #6732...
< GitHub186> bitcoin/master 5978388 Wladimir J. van der Laan: build: remove libressl check...
< GitHub186> bitcoin/master a3874c7 Wladimir J. van der Laan: doc: no longer require use of openssl in OpenBSD build guide
< GitHub186> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/f297042cae57...bb882d04e862
< sipa> ok, #bitcoin-dev
< wumpus> ok, I'd say keep it to #bitcoin-dev for this week, we could move it next week, and announce somewhat longer in before
< jonasschnelli> Maybe its better communication to do it on #bitcoin-dev unless there are clear infos about where the next irc meeting will be
< wumpus> sipa: it's still on the schedule as #bitcoin-dev. Could announce a new channel, don't know what makes sense.
< jonasschnelli> wumpus: yeah. Need to do that upstream now... that was one of the goals, ... to improve UniValue without disturbing bitcoin-core.
< GitHub171> [bitcoin] jonasschnelli closed pull request #6580: [UniValue] replace global function find_value() with UniValue::findValue() (master...2015/08/univalue_find_value) https://github.com/bitcoin/bitcoin/pull/6580
< jonasschnelli> https://github.com/bitcoin/bitcoin/pull/6215 <-- trivial review, code is only used in unit tests right now
< GitHub25> [bitcoin] laanwj closed pull request #6637: Add UniValue as subtree (master...2015/09/univalue_subtree) https://github.com/bitcoin/bitcoin/pull/6637
< GitHub50> bitcoin/master 6e16a41 Jonas Schnelli: Merge commit '2f9f082b5ef3c495c70598ef23383effef675f9a' as 'src/univalue'
< GitHub50> bitcoin/master 2f9f082 Jonas Schnelli: Squashed 'src/univalue/' content from commit 87d9045...
< GitHub50> bitcoin/master 0917306 Jonas Schnelli: remove univalue, prepare for subtree
< GitHub50> [bitcoin] laanwj pushed 6 new commits to master: https://github.com/bitcoin/bitcoin/compare/17d0e638b66b...f297042cae57
< GitHub83> [bitcoin] laanwj closed pull request #6686: [travis] add zmq python module, re-enable zmq rpc tests (master...2015/09/travis_zmq) https://github.com/bitcoin/bitcoin/pull/6686
< GitHub147> bitcoin/master a9c27cd Jonas Schnelli: [travis] add zmq python module
< GitHub147> bitcoin/master 745f909 Cory Fields: travis: install a recent libzmq and pyzmq for tests
< GitHub147> bitcoin/master 110a1fd Jonas Schnelli: enable zmq-test in rpc-tests.sh
< GitHub147> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/4899a04c24b9...17d0e638b66b
< wumpus> someone on #bitcoin also reporting malleated transactions
< jonasschnelli> btcdrak: better: https://github.com/bitcoin/bitcoin/pull/6637?
< jonasschnelli> https://github.com/bitcoin/bitcoin/pull/6637 <--- seeks for a final review
< GitHub44> [bitcoin] laanwj closed pull request #5987: Bugfix: Fix testnet-in-a-box use case (master...bugfix_tniab) https://github.com/bitcoin/bitcoin/pull/5987
< GitHub144> bitcoin/master 4899a04 Wladimir J. van der Laan: Merge pull request #5987...
< GitHub144> bitcoin/master e761d7a Luke Dashjr: Bugfix: Allow mining on top of old tip blocks for testnet (fixes testnet-in-a-box use case)
< GitHub144> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8a86d53bd570...4899a04c24b9
< TZander> anyway #bitcoin-dev is kinda empty right now
< wumpus> the list of issues to be resolved here is at: https://github.com/bitcoin/bitcoin/issues , there are 367 open, good luck
< wumpus> I have to be, to avoid this becoming #bitcoin-dev again
< wumpus> to avoid political meta discussion, and focus on develpoment of the Bitcoin Core project in github.com/bitcoin/bitcoin
< TZander> Aquentin: Channel #bitcoin-core-dev created on Mon Nov 17 2014
< btcdrak> Trolling for reviews, Mempool-only CHECKSEQUENCEVERIFY https://github.com/bitcoin/bitcoin/pull/6564
< btcdrak> Trolling for reviews, Mempool-only sequence number constraint verification https://github.com/bitcoin/bitcoin/pull/6312
< jgarzik> Trolling for reviews, Bugfix: Fix testnet-in-a-box use case #5987 https://github.com/bitcoin/bitcoin/pull/5987
< jgarzik> Trolling for reviews, add bip32 pub key serialization #6215 https://github.com/bitcoin/bitcoin/pull/6215
< GitHub54> [bitcoin] jgarzik closed pull request #6731: doc: Add build guide for OpenBSD 5.7 (master...2015_09_openbsd_build_guide) https://github.com/bitcoin/bitcoin/pull/6731
< GitHub178> [bitcoin] jgarzik pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cf9bb11f97db...8a86d53bd570
< GitHub178> bitcoin/master 9639ead Wladimir J. van der Laan: doc: Add build guide for OpenBSD 5.7...
< GitHub178> bitcoin/master 8a86d53 Jeff Garzik: Merge pull request #6731
< GitHub1> [bitcoin] jgarzik closed pull request #6588: In (strCommand == "tx"), return if AlreadyHave() (master...already_have) https://github.com/bitcoin/bitcoin/pull/6588
< GitHub55> bitcoin/master cf9bb11 Jeff Garzik: Merge pull request #6588
< GitHub55> bitcoin/master 9524c4d Tom Harding: In (strCommand == "tx"), return if AlreadyHave()...
< GitHub55> [bitcoin] jgarzik pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/12a7712abd49...cf9bb11f97db
< GitHub126> [bitcoin] laanwj closed pull request #5924: Clean up change computation in CreateTransaction. (master...change-cleanup) https://github.com/bitcoin/bitcoin/pull/5924
< GitHub193> bitcoin/master 12a7712 Wladimir J. van der Laan: Merge pull request #5924...
< GitHub193> bitcoin/master 835c122 Daniel Kraft: Clean up change computation in CreateTransaction....
< GitHub193> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1119cc3f5918...12a7712abd49
< wumpus> minutes of last meeting were posted to the mailing list by Daniel Stadulis ( http://lists.linuxfoundation.org/pipermail/bitcoin-dev/2015-September/011190.html )
< wumpus> it was created because people didn't want the github commit notifications on #bitcoin-dev

2015-09-30

< sipa> but for use in Bitcoin we can't just only have a strict DER one
< rusty> sipa: I stole the DER encode/decode from bitcoin.
< GitHub166> [bitcoin] arnuschky opened pull request #6742: Changed logging to make -logtimestamps to work also for -printtoconsole (master...feature-logtimestamps-toconsole) https://github.com/bitcoin/bitcoin/pull/6742
< GitHub156> bitcoin/master 1119cc3 Wladimir J. van der Laan: Merge pull request #6741...
< GitHub69> [bitcoin] laanwj closed pull request #6741: doc: Change #bitcoin-dev IRC channel to #bitcoin-core-dev (master...2015_09_channel_split) https://github.com/bitcoin/bitcoin/pull/6741
< GitHub156> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/1119cc3f5918575ca397518c9fd31a64704c7e4f
< GitHub21> [bitcoin] laanwj opened pull request #6741: doc: Change #bitcoin-dev IRC channel to #bitcoin-core-dev (master...2015_09_channel_split) https://github.com/bitcoin/bitcoin/pull/6741