< jtimon> I tried force pushing to #8994 several times but got timeouts all times, I don't think the branch is to blame, but perhaps I'm missing something. see https://travis-ci.org/bitcoin/bitcoin/builds/246986373
< gribble> https://github.com/bitcoin/bitcoin/issues/8994 | Testchains: Introduce custom chain whose constructor... by jtimon · Pull Request #8994 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] hans-liu opened pull request #10675: Update INSTALL.md (master...master) https://github.com/bitcoin/bitcoin/pull/10675
< bitcoin-git> [bitcoin] instagibbs opened pull request #10676: document script-based return fields for validateaddress (master...validatata) https://github.com/bitcoin/bitcoin/pull/10676
< bitcoin-git> [bitcoin] instagibbs opened pull request #10677: RPC Docs: addmultisigaddress is intended for non-watchonly addresses (master...importmultiwatch) https://github.com/bitcoin/bitcoin/pull/10677
< sipa> important new feature: you can now sort PRs in github by most smileys!
< luke-jr> lol
< luke-jr> might actually be handy..
< wumpus> sort:reactions-heart-desc
< wumpus> #10636 seems to be causing some travis false positives, e.g. https://travis-ci.org/bitcoin/bitcoin/jobs/247360729 in zapwallettxes.py
< gribble> https://github.com/bitcoin/bitcoin/issues/10636 | [qa] util: Check return code after closing bitcoind proc by MarcoFalke · Pull Request #10636 · bitcoin/bitcoin · GitHub
< wumpus> not been able to reproduce it locally (as usual :/)
< bitcoin-git> [bitcoin] schildbach opened pull request #10679: Document the non-DER-conformance of one test in tx_valid.json. (master...tx-valid-comment) https://github.com/bitcoin/bitcoin/pull/10679
< jtimon> or not? not sure
< luke-jr> I wonder if it would fly to PR https://github.com/BitcoinHardfork/bitcoin/pull/3 on Core, but with a date 7 years in the future..
< luke-jr> (with an accompanying BIP ofc)
< bitcoin-git> [bitcoin] MeshCollider opened pull request #10680: Fix inconsistencies and grammar in various files (master...fix_typos_and_inconsistencies) https://github.com/bitcoin/bitcoin/pull/10680
< wumpus> for a hardfork you certainly need a BIP first
< wumpus> which needs to be discussed, and there needs to be wide agreement on it, only then it makes sense to PR it
< luke-jr> right, I just mean whether it could get done, or would encounter politics for some reason
< wumpus> i'm sure it will encounter politics, but that shouldn't affect whether to do a proposal
< luke-jr> no point proposing something doomed to fail
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/234ffc677ee5...78783531b7cc
< bitcoin-git> bitcoin/master 21180ff Pieter Wuille: Simplify return values of GetCoin/HaveCoin(InCache)...
< bitcoin-git> bitcoin/master 7878353 Wladimir J. van der Laan: Merge #10581: Simplify return values of GetCoin/HaveCoin(InCache)...
< bitcoin-git> [bitcoin] laanwj closed pull request #10581: Simplify return values of GetCoin/HaveCoin(InCache) (master...simplehavecoin) https://github.com/bitcoin/bitcoin/pull/10581
< bitcoin-git> [bitcoin] laanwj closed pull request #10675: Update INSTALL.md (master...master) https://github.com/bitcoin/bitcoin/pull/10675
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/78783531b7cc...1680ee0edf46
< bitcoin-git> bitcoin/master c074752 Ricardo Velhote: [RPC] Add an uptime command that displays the amount of time that bitcoind has been running
< bitcoin-git> bitcoin/master 1680ee0 Wladimir J. van der Laan: Merge #10400: [RPC] Add an uptime command that displays the amount of time (in seconds) bitcoind has been running...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/1680ee0edf46...7c87a9c748be
< bitcoin-git> bitcoin/master 76859e6 John Newbery: [tests] Update functional tests documentation
< bitcoin-git> bitcoin/master e7ba6c1 John Newbery: [tests] add example test
< bitcoin-git> bitcoin/master 7c87a9c Wladimir J. van der Laan: Merge #10612: The young person's guide to the test_framework...
< bitcoin-git> [bitcoin] laanwj closed pull request #10612: The young person's guide to the test_framework (master...templatefunctionaltest) https://github.com/bitcoin/bitcoin/pull/10612
< jnewbery> jtimon: I've been away for the last week but it looks like no-one gave you the most obvious answer to your question
< jnewbery> > what's the simplest way to find out whoch commit deleted a given variable, say, pwalletMain
< jnewbery> git log -S<string>
< jnewbery> wumpus jcorgan : I think this is easier than git bisect/grep ^^
< bitcoin-git> [bitcoin] instagibbs opened pull request #10681: add gdb attach process to test README (master...gdbattach) https://github.com/bitcoin/bitcoin/pull/10681
< instagibbs> jnewbery, oh right, certain string searching, even better
< Chris_Stewart_5> Is it a consensus rule that the miner's reward is at coinbaseTx.vout[0]?
< instagibbs> no
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/7c87a9c748be...ac52492cd227
< bitcoin-git> bitcoin/master 506b700 Jorge Timón: Util: Remove redundant calls to gArgs.IsArgSet()...
< bitcoin-git> bitcoin/master ed866ab Jorge Timón: Indentation after 'Remove redundant calls to gArgs.IsArgSet()'
< bitcoin-git> bitcoin/master ac52492 Wladimir J. van der Laan: Merge #10118: Util: Remove redundant calls to argsGlobal.IsArgSet()...
< bitcoin-git> [bitcoin] laanwj closed pull request #10118: Util: Remove redundant calls to argsGlobal.IsArgSet() (master...0.14-args-class2) https://github.com/bitcoin/bitcoin/pull/10118
< bitcoin-git> [bitcoin] esneider opened pull request #10682: Trivial: Move the AreInputsStandard documentation next to its implementation (master...move-doc) https://github.com/bitcoin/bitcoin/pull/10682
< bitcoin-git> [bitcoin] laanwj opened pull request #10683: rpc: Move the `generate` RPC call to rpcwallet (master...2017_06_wallet_mining) https://github.com/bitcoin/bitcoin/pull/10683
< wumpus> Chris_Stewart_5: no, the coinbase tx can have arbitrary outputs like any transaction
< wumpus> (the amount just needs to be <= blockreward+fees)
< wumpus> jnewbery: forgot about -S
< Chris_Stewart_5> Is there any strong feelings on constraining the miner output to be at index 0? I'm working on a drivechain proposal BIP that would do this... and fix the witness commitment at index 1
< wumpus> some mining pools pay out to multiple parties in the coinbase transaction, I think that would be broken then?
< sipa> jnewbery: wah, i didn't know about -S
< jnewbery> sipa: yeah, it's a useful trick!
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/ac52492cd227...acb11535cb84
< bitcoin-git> bitcoin/master fab1fb7 MarcoFalke: [qa] blockchain: Pass on closed connection during generate call
< bitcoin-git> bitcoin/master acb1153 Wladimir J. van der Laan: Merge #10659: [qa] blockchain: Pass on closed connection during generate call...
< bitcoin-git> [bitcoin] laanwj closed pull request #10659: [qa] blockchain: Pass on closed connection during generate call (master...Mf1706-qaStopPass) https://github.com/bitcoin/bitcoin/pull/10659
< bitcoin-git> [bitcoin] morcos opened pull request #10684: Remove no longer used mempool.exists(outpoint) (master...lessHaveCoin) https://github.com/bitcoin/bitcoin/pull/10684
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10685: Clarify CCoinsViewMemPool documentation. (master...2017-06-ccoinsviewmempool-doc-cleanup) https://github.com/bitcoin/bitcoin/pull/10685
< bitcoin-git> [bitcoin] morcos closed pull request #10586: More economical fee estimates for opt-in-RBF transactions (master...aggressiveEstimates) https://github.com/bitcoin/bitcoin/pull/10586
< BlueMatt> wumpus: when you get a chance, can you respond to sipa's question at https://github.com/bitcoin/bitcoin/pull/10179#issuecomment-310548915 ?
< BlueMatt> uhhh, we dont parse options with "--", right?
< BlueMatt> oh, maybe we do
< sipa> we do
< sipa> I thought?
< BlueMatt> yes
< BlueMatt> i believe so
< cfields> yea, there's some magic in there to add or remove a -
< PRab> I'm having trouble with my gitian build for 0.14.2. http://sebsauvage.net/paste/?df3f6f7b71515e7b#Zwec/18rxhOtXoI13s1CRQaII2dtsfBxEzZ3ACjJGwE=
< PRab> Any ideas?
< PRab> I already tried running gbuild --upgrade like it says to.
< sipa> sdaftuar: ugh, i was not aware that DISCOURAGE_UPGRADABLE_NOPS was permitted without CLTV/CSV