< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #10537: Few Minor per-utxo assert-semantics re-adds and tweak (master...2017-06-per-utxo-fixes) https://github.com/bitcoin/bitcoin/pull/10537
< da2ce7_> Good Morning -core-dev. I have a draft-mailing list post that I would like some pre-posting review.
< da2ce7_> Wallet and Exchange Proposal Best Practices for Schism Forks
< da2ce7_> * Proposed Wallet and Exchange Best Practices for the case of Schism Forks
< luke-jr> da2ce7_: I disagree that softforks can be the *cause* of a schism fork. In the case of BIP148, for example, the split is caused by miners violating the protocol rules. This is really no different than a split when the rule isn't newly added.
< luke-jr> and if nodes wilfully choose to follow the invalid chain, that removal of the rule is an attempted hardfork (and probably contentious)
< da2ce7_> Yes, miners can choose to cause a schism fork at any time if they wish to use checkpoints, the soft-fork is just an excuse.
< luke-jr> "pre-schism tokens" should be defined. Does it mean a UTXO valid on all chains? (or better, for fungibility, a claim to the same amount on all chains)
< luke-jr> da2ce7_: checkpoints are not needed either; they could for example choose to mine a larger-than-allowed subsidy
< da2ce7_> If we had evil-softfork implemented by the miners, then users could choose to create a schism to to reject this soft-forkl
< da2ce7_> Yes. I agree that the pre-schism tokens should be more-well-defined.
< luke-jr> evilforks aren't softforks :p
< da2ce7_> Maybe the virtual tokens should be something like 'spendable balance' on both chains. (utxo outputs, minus fees required to spend). So if somebody deposits 10000x small outputs is doesn't unbalance the pair.
< luke-jr> da2ce7_: what if Joe Schmo makes a schism fork of his own and doesn't tell exchanges? they owe JoeSchmocoins?
< da2ce7_> yes, we need some significance clause in the document.
< luke-jr> "Make sure that transaction replay protection has been implemented." I disagree this should be the standard case. If people want to withdraw "pre-schism tokens", replay should be intentionally enabled if anything.
< da2ce7_> no. you are forbidden to withdrawal pre-schism virtual tokens.
< luke-jr> that's dumb :P
< luke-jr> "Withdrawals MUST only be enabled until transaction relay protection has been added for both sides of the schism." <-- doesn't make sense
< da2ce7_> they don't exist. They are only virtually exist for accounting purposes.
< luke-jr> da2ce7_: before the schism, all withdrawls are pre-schism tokens
< da2ce7_> yup. post-schism pre-schism tokens don't exist. Only their virtual counterparts exist.
< luke-jr> the ideal for a neutral merchant/user is to wait for receives to confirm on both chains; this is only possible if replay is standard
< da2ce7_> Ok... I'll rephrase that part.
< luke-jr> preventing replay should only be done when the sender and recipient are taking a position
< da2ce7_> luke-jr no, because there are all sorts of attacks that may happen. For example, it may be not the same transaction that confirms because of transaction malleability.
< luke-jr> I don't see how that's a concern
< luke-jr> if I'm expecting a payment, and I have no bias as to which side of the schism, I simply wait for it to confirm on both sides. txids don't matter
< luke-jr> otoh, I guess if we assume market values on each side will differ, perhaps this neutral ideal is impractical
< da2ce7_> only if you are dealing with pre-schism tokens exclusively. However I wish make it clear to the users that there is now indeed two distinct coins, if they love that fact or not is besides the point.
< wumpus> btcdrak: restarted
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/9fec4da0bec9...323a46e03414
< bitcoin-git> bitcoin/master 29c0719 shaolinfry: Rename -bip9params to -vbparams
< bitcoin-git> bitcoin/master b463bc9 Jorge Timón: scripted-diff: s/BIP9DeploymentInfo/VBDeploymentInfo/...
< bitcoin-git> bitcoin/master 323a46e Wladimir J. van der Laan: Merge #10463: Names: BIP9 vs versionbits...
< bitcoin-git> [bitcoin] laanwj closed pull request #10463: Names: BIP9 vs versionbits (master...b15-bip8-cherry-reanme) https://github.com/bitcoin/bitcoin/pull/10463
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/323a46e03414...16f6c98fa8ee
< bitcoin-git> bitcoin/master 8b22af3 Pieter Wuille: Replace bytes_serialized with bogosize
< bitcoin-git> bitcoin/master 16f6c98 Wladimir J. van der Laan: Merge #10426: Replace bytes_serialized with bogosize...
< bitcoin-git> [bitcoin] laanwj closed pull request #10426: Replace bytes_serialized with bogosize (master...bogosize) https://github.com/bitcoin/bitcoin/pull/10426
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/16f6c98fa8ee...67700b39241d
< bitcoin-git> bitcoin/master e265200 Suhas Daftuar: Delay parallel block download until chain has sufficient work...
< bitcoin-git> bitcoin/master 76f7481 Suhas Daftuar: Add timeout for headers sync...
< bitcoin-git> bitcoin/master 67700b3 Wladimir J. van der Laan: Merge #10345: [P2P] Timeout for headers sync...
< bitcoin-git> [bitcoin] laanwj closed pull request #10345: [P2P] Timeout for headers sync (master...2017-05-timeout-headers-sync) https://github.com/bitcoin/bitcoin/pull/10345
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/67700b39241d...30853e16d332
< bitcoin-git> bitcoin/master fa62dad MarcoFalke: [qa] fundrawtx: Fix shutdown race
< bitcoin-git> bitcoin/master 30853e1 MarcoFalke: Merge #10535: [qa] fundrawtx: Fix shutdown race...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10535: [qa] fundrawtx: Fix shutdown race (master...Mf1706-qaFundrawRace) https://github.com/bitcoin/bitcoin/pull/10535
< CypherBlock> What would people think of a bip to start rejecting segwit signaling blocks, starting on Aug 1? This is a safety mechanism to prevent reorg and segwit will not activate naturally anyway. So it is a forced unarming of segwit. Thoughts?
< wumpus> I'm going to tag 0.14.2rc2 in a bit - anything that needs to get in?
< instagibbs> wumpus, a version bump :)
< wumpus> I think that went in on the branch?
< wumpus> let's do a translations update at least
< bitcoin-git> [bitcoin] practicalswift opened pull request #10538: [trivial] Fix typo: "occurrences" (misspelled as "occurrances") (master...occurrences) https://github.com/bitcoin/bitcoin/pull/10538
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.14: https://github.com/bitcoin/bitcoin/commit/cc23438af017da8d477e2dbe58a5ba374a691a03
< bitcoin-git> bitcoin/0.14 cc23438 Wladimir J. van der Laan: qt: 0.14.2 pre-rc2 translations update
< wumpus> there's also the nMinimumChainWork/defaultAssumeValid updates mentioned in the release process ("for every minor and major release")
< wumpus> but that might be a bit on the late side, better to leave it for 0.14.3 I guess
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/30853e16d332...980deaf0b1cc
< bitcoin-git> bitcoin/master 870824e Luke Dashjr: RPC/Mining: Restore API compatibility for prioritisetransaction...
< bitcoin-git> bitcoin/master 980deaf Wladimir J. van der Laan: Merge #10252: RPC/Mining: Restore API compatibility for prioritisetransaction...
< bitcoin-git> [bitcoin] laanwj closed pull request #10252: RPC/Mining: Restore API compatibility for prioritisetransaction (master...rpc_prioritise_api) https://github.com/bitcoin/bitcoin/pull/10252
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/980deaf0b1cc...300f8e7a828f
< bitcoin-git> bitcoin/master cf66f04 Russell Yanofsky: Add FindEarliestAtLeast test for edge cases...
< bitcoin-git> bitcoin/master 300f8e7 Wladimir J. van der Laan: Merge #9909: tests: Add FindEarliestAtLeast test for edge cases...
< bitcoin-git> [bitcoin] laanwj closed pull request #9909: tests: Add FindEarliestAtLeast test for edge cases (master...pr/faltest) https://github.com/bitcoin/bitcoin/pull/9909
< fanquake> wumpus don't think so.
< MarcoFalke> wumpus: The version bump and doc update for the man pages?
< wumpus> MarcoFalke: good idea
< wumpus> I don't think that one is even in the release process
< wumpus> fanquake: ok
< BlueMatt> whats our policy on static-global-var-in-file?
< BlueMatt> does it get a g_?
< BlueMatt> its not really "global" if its just for a single file
< wumpus> BlueMatt: it's still global in the file scope I'd say
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to 0.14: https://github.com/bitcoin/bitcoin/commit/5e408d99a219f80e3dab06d3ba03f102d404b886
< bitcoin-git> bitcoin/0.14 5e408d9 Wladimir J. van der Laan: doc: Update manpages for 0.14.2
< BlueMatt> k
< wumpus> (I mean the prefixes are introduced to distinguish it from locals, or class variables, etc, which is just as valid for something static)
< BlueMatt> makes sense
< bitcoin-git> [bitcoin] laanwj opened pull request #10539: doc: Mention update manpages in release process (master...2017_06_release_process_manpage_update) https://github.com/bitcoin/bitcoin/pull/10539
< wumpus> ok, going to tag 0.14.2rc2 then
< wumpus> * [new tag] v0.14.2rc2 -> v0.14.2rc2
< bitcoin-git> [bitcoin] laanwj closed pull request #10539: doc: Mention update manpages in release process (master...2017_06_release_process_manpage_update) https://github.com/bitcoin/bitcoin/pull/10539
< bitcoin-git> [bitcoin] jnewbery opened pull request #10540: Salvage wallet should not set the aggressive flag on Db::verify() (master...fixsalvage) https://github.com/bitcoin/bitcoin/pull/10540
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/300f8e7a828f...1b708f2cf3e6
< bitcoin-git> bitcoin/master 24980a3 Mario Dian: Make functions in validation.cpp static and pass chainparams...
< bitcoin-git> bitcoin/master 1b708f2 Wladimir J. van der Laan: Merge #10201: pass Consensus::Params& to functions in validation.cpp and make them static...
< bitcoin-git> [bitcoin] laanwj closed pull request #10201: pass Consensus::Params& to functions in validation.cpp and make them static (master...consensusparams-receivedblocktransactions) https://github.com/bitcoin/bitcoin/pull/10201
< bitcoin-git> [bitcoin] ABISprotocol opened pull request #10541: Docs: Update INSTALL.md to add link to dev notes (master...patch-1) https://github.com/bitcoin/bitcoin/pull/10541
< bitcoin-git> [bitcoin] morcos opened pull request #10543: Change API to estimaterawfee (master...estimaterawapi) https://github.com/bitcoin/bitcoin/pull/10543
< bitcoin-git> [bitcoin] sipa opened pull request #10544: Update to LevelDB 1.20 (master...leveldb120) https://github.com/bitcoin/bitcoin/pull/10544
< bitcoin-git> [bitcoin] practicalswift opened pull request #10545: Use list initialization (C++11) for maps/vectors instead of boost::assign::map_list_of/list_of (master...list_of) https://github.com/bitcoin/bitcoin/pull/10545
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1b708f2cf3e6...32751807c9c0
< bitcoin-git> bitcoin/master b083db6 practicalswift: [trivial] Fix typo: "occurrences" (misspelled as "occurrances")...
< bitcoin-git> bitcoin/master 3275180 MarcoFalke: Merge #10538: [trivial] Fix typo: "occurrences" (misspelled as "occurrances")...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10538: [trivial] Fix typo: "occurrences" (misspelled as "occurrances") (master...occurrences) https://github.com/bitcoin/bitcoin/pull/10538
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/32751807c9c0...2805d606bc46
< bitcoin-git> bitcoin/master a8c09af practicalswift: Remove unused variables
< bitcoin-git> bitcoin/master 2805d60 MarcoFalke: Merge #10522: [wallet] Remove unused variables...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10522: [wallet] Remove unused variables (master...unused-strings) https://github.com/bitcoin/bitcoin/pull/10522
< bitcoin-git> [bitcoin] MarcoFalke pushed 6 new commits to master: https://github.com/bitcoin/bitcoin/compare/2805d606bc46...75e898c094ee
< bitcoin-git> bitcoin/master ce58e93 John Newbery: Change bitcoin-util-test.py to use Python3
< bitcoin-git> bitcoin/master e9265df John Newbery: Change help_text in bitcoin-util-test.py to a docstring.
< bitcoin-git> bitcoin/master 89fcd35 John Newbery: Use an .ini config file for environment vars in bitcoin-util-test.py
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #10331: Share config between util and functional tests (master...shared_util_function_test_config) https://github.com/bitcoin/bitcoin/pull/10331
< Lightsword> hmm, github is down