< cfields>
luke-jr: if bitcoin is as fickle as a default setting, the value of that setting isn't the issue...
< gmaxwell>
luke-jr: what is the security model of a bitcoin with no nodes? But with the assumevalid approach I don't know that it's really a meaningful change in the security model.
< luke-jr>
morcos: yes, I think we shouldn't recommend things that harm Bitcoin.
< gmaxwell>
luke-jr: I'm concerned that you seem to be fixated on size rather than on the greater goal of keeping bitcoin decenteralized, secure, usable. Size is increasingly not an interesting magic metric for that, moreover, single miners reducing their size does nothing to help here-- if it did there would be much less reason for a blocksize limit.
< bitcoin-git>
[bitcoin] laanwj closed pull request #10997: RPC: Add option -stdinrpcpass to bitcoin-cli to allow RPC password to be read from standard input (master...stdinrpcpass) https://github.com/bitcoin/bitcoin/pull/10997
< bitcoin-git>
bitcoin/master affe927 Wladimir J. van der Laan: Merge #10997: RPC: Add option -stdinrpcpass to bitcoin-cli to allow RPC password to be read from standard input...
< bitcoin-git>
bitcoin/master 79191f5 Joe Harvell: Add option -stdinrpcpass to allow RPC password to be read from standard input
< bitcoin-git>
[bitcoin] instagibbs opened pull request #11120: verifytxoutproof returns object including blockhash (master...verifytxoutproof-blockhash) https://github.com/bitcoin/bitcoin/pull/11120
< bitcoin-git>
[bitcoin] MarcoFalke opened pull request #11119: [doc] build-windows: Mention that only trusty works (master...Mf1708-docBuildWinTrusty) https://github.com/bitcoin/bitcoin/pull/11119
< bitcoin-git>
[bitcoin] laanwj closed 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
< bitcoin-git>
bitcoin/master 31b2612 Wladimir J. van der Laan: Merge #10679: Document the non-DER-conformance of one test in tx_valid.json....
< bitcoin-git>
bitcoin/master ecb11f5 Andreas Schildbach: Document the non-strict-DER-conformance of one test in tx_valid.json....
< bitcoin-git>
[bitcoin] laanwj closed pull request #10923: travis: Build with --enable-werror under OS X (master...thread-safety-analysis) https://github.com/bitcoin/bitcoin/pull/10923
< bitcoin-git>
bitcoin/master 2c9f5ec Wladimir J. van der Laan: Merge #10923: travis: Build with --enable-werror under OS X...
< bitcoin-git>
bitcoin/master a65e028 practicalswift: Build with --enable-werror under OS X
< bitcoin-git>
[bitcoin] jimpo opened pull request #11116: [script] Unit tests for script/standard and IsMine functions. (master...script-standard-tests) https://github.com/bitcoin/bitcoin/pull/11116
2017-08-22
< bitcoin-git>
[bitcoin] instagibbs closed pull request #11049: coincontrol can filter for segwit inputs, expose fundraw option (master...segwitfundraw) https://github.com/bitcoin/bitcoin/pull/11049
< bitcoin-git>
[bitcoin] jimpo opened pull request #11113: [net] Ignore getheaders requests for very old side blocks. (master...net-getheaders-fingerprint) https://github.com/bitcoin/bitcoin/pull/11113
< bitcoin-git>
[bitcoin] practicalswift opened pull request #11112: developer-notes: By default, declare single-argument constructors `explicit`. (master...declare-single-argument-constructors-explicit) https://github.com/bitcoin/bitcoin/pull/11112
< bitcoin-git>
bitcoin/master fc5c237 Wladimir J. van der Laan: Merge #11007: wallet: Fix potential memory leak when loading a corrupted wallet file...
< bitcoin-git>
bitcoin/master c06755f practicalswift: wallet: Fix memory leak when loading a corrupted wallet file
< bitcoin-git>
[bitcoin] laanwj closed pull request #10843: Add attribute [[noreturn]] (C++11) to functions that will not return (master...noreturn) https://github.com/bitcoin/bitcoin/pull/10843
< bitcoin-git>
bitcoin/master 2ab7c63 Wladimir J. van der Laan: Merge #10843: Add attribute [[noreturn]] (C++11) to functions that will not return...
< bitcoin-git>
bitcoin/master b82c55a practicalswift: Add attribute [[noreturn]] (C++11) to functions that will not return...
< bitcoin-git>
[bitcoin] lemzwerg opened pull request #11101: bitcoin_qt.m4: Minor fixes, improve code orthogonality (master...master) https://github.com/bitcoin/bitcoin/pull/11101
< bitcoin-git>
[bitcoin] laanwj closed pull request #11027: [RPC] Only return hex field once in getrawtransaction (master...fix-getrawtx) https://github.com/bitcoin/bitcoin/pull/11027
< bitcoin-git>
bitcoin/master 820ddd4 Wladimir J. van der Laan: Merge #11027: [RPC] Only return hex field once in getrawtransaction...
< bitcoin-git>
bitcoin/master 6bbdafc Andrew Chow: Pass serialization flags and whether to include hex to TxToUniv
< bitcoin-git>
bitcoin/master e029c6e Andrew Chow: Only return hex field once in getrawtransaction...
< bitcoin-git>
[bitcoin] TheBlueMatt opened pull request #11100: Use a sensible default for blockmax{size,weight} (master...2017-08-sane-default-limits) https://github.com/bitcoin/bitcoin/pull/11100
< bitcoinreminder>
no sorry, I was talking about bitcoin-core
< bitcoinreminder>
sorry, I meant bitcoin-core.org
< luke-jr>
bitcoinreminder: bitcoin.org isn't even Bitcoin Core..
< bitcoinreminder>
does anyone has a bitcoin-org email address? I created a impersonation-request on twitter for the fake https://twitter.com/BcoreProject account, but they told me that someone with a bitcoin-org email address has to create the request
< bitcoin-git>
[bitcoin] greenaddress opened pull request #11099: [RPC][mempool]: add rpc command to dump the mempool to disk (master...dump_mempool_rpc) https://github.com/bitcoin/bitcoin/pull/11099
< bitcoin-git>
[bitcoin] theuni opened pull request #11097: gitian: quick hack to fix version string in releases (master...gitian-version-fix) https://github.com/bitcoin/bitcoin/pull/11097
< bitcoin-git>
[bitcoin] mdaiter opened pull request #11095: Consensus: add const to temp int in base58.cpp (master...base58NuancedConst) https://github.com/bitcoin/bitcoin/pull/11095
< mryandao>
hey guys, when executing a estimatefee RPC command on a node that is not fully sync-ed, shouldn't bitcoin-cli make a note that said node is not fully sync-ed and user should be cautioned to not use the estimated fee?
< bitcoin-git>
[bitcoin] laanwj closed pull request #10952: [wallet] Remove vchDefaultKey and have better first run detection (master...remove-defaultkey) https://github.com/bitcoin/bitcoin/pull/10952
< bitcoin-git>
bitcoin/master 2621673 Wladimir J. van der Laan: Merge #10952: [wallet] Remove vchDefaultKey and have better first run detection...
< bitcoin-git>
bitcoin/master e53615b Andrew Chow: Remove vchDefaultKey and have better first run detection...
< bitcoin-git>
[bitcoin] laanwj closed pull request #11066: Document the preference of nullptr over NULL or (void*)0 (master...document-nullptr-preference) https://github.com/bitcoin/bitcoin/pull/11066
< bitcoin-git>
bitcoin/master 9e00a62 Wladimir J. van der Laan: Merge #11066: Document the preference of nullptr over NULL or (void*)0...
< bitcoin-git>
bitcoin/master bea8e9e practicalswift: Document the preference of nullptr over NULL or (void*)0
< bitcoin-git>
bitcoin/master 07685d1 Jonas Schnelli: Add length check for CExtKey deserialization
< bitcoin-git>
bitcoin/master 9f60b37 Wladimir J. van der Laan: Merge #11081: Add length check for CExtKey deserialization (jonasschnelli, guidovranken)...
< bitcoin-git>
[bitcoin] dooglus opened pull request #11085: Add 'sethdseed' RPC to initialize or replace HD seed. (master...set_hd_seed) https://github.com/bitcoin/bitcoin/pull/11085
< bitcoin-git>
bitcoin/0.15 1c4b9b3 Wladimir J. van der Laan: Merge #11076: 0.15 release-notes nits: fix redundancy, remove accidental parenthesis & fix range style...
< bitcoin-git>
[bitcoin] laanwj closed pull request #11071: Use static_assert(…, …) (C++11) instead of assert(…) where appropriate (master...static_assert) https://github.com/bitcoin/bitcoin/pull/11071
< bitcoin-git>
bitcoin/master dbf6bd6 Wladimir J. van der Laan: Merge #11071: Use static_assert(…, …) (C++11) instead of assert(…) where appropriate...
< praxeology>
Does bitcoin's networking code operate on polling or interrupt?
< bitcoin-git>
[bitcoin] jonasnick opened pull request #11083: Fix combinerawtransaction RPC help result section (master...fix-combinerawtransaction-help) https://github.com/bitcoin/bitcoin/pull/11083
< praxeology>
kallewoof: does your bitcoin process have enough memory to hold the entire chainstate?
< kallewoof>
gmaxwell: there is some amount of overhead as I am doing the profiling on my own. Maybe that's the cause for the high portion of time spent there, but it still seems like a lot of LOCK calls, regardless of actual CPU cycle count. Would be cool if the mempool could be copied once and then not lock cs at all. Code is here btw: https://github.com/kallewoof/bitcoin/tree/profile-resources
< praxeology>
Like... I know that BitPay and Coinbase are declaring that they will support Segwit2x... has any exchange declared that they will continue to support Bitcoin (Bitcoin Core's rules)?
< kallewoof>
I'm running a modified Bitcoin Core node to do some profiling on where resources are spent (CPU cycles and bandwidth in particular) and am seeing some really weird stuff. E.g:
< luke-jr>
in other news, Texas Bitcoin conference is promoting 2X as if it's Bitcoin, so I think that makes the decision to go simple (ie, not to)
< gmaxwell>
if they said "[compatible fully validating nodes] btc1 \n [compatible wallet software] bitcoin classic\n" it would n... oh okay, well I suppose because it's not a lie in every possible sense it's okay. :P
< Cryptocide>
Abra|BitClub Network|Bitcoin.com|BitFury|BitGo|Bitmain|BitPay Blockchain|Bloq|BTCC|Circle|Ledger|RSK Labs|Xapo, no thanks
< gmaxwell>
Wow, this is super dishonest https://segwit2x.github.io/segwit2x-announce.html ... "Bitcoin Upgrade" is untrue... it claims Bitcoin "Classic" and unlimited are compatible "Compatible Fully-Validating Node Software" but they don't implement the S2X rules and don't even implement segwit!
< sipa>
i find it amazing that all of bitcoin core's history, is less than 100 MB
< gmaxwell>
And in particular, the people that the users of bitcoin are generally reseting a fair amount of trust to create and maintain the software the network is using, for the most part (or completely though we can't be sure) don't agree with the narative they're being sold.
< BlueMatt>
I mean can you seriously claim that almost the entirety of your rather short blog posts is disagreed with by almost any contributor to bitcoin core
< gmaxwell>
(also, "here is a really interesting view you should read and consider" is not morally equivilent to /pretending to be us/ or faking that s2x is just an uncontroversial and low risk bitcoin upgrade...)
< bitcoin-git>
[bitcoin] luke-jr opened pull request #11082: Add new bitcoin_rw.conf file that is used for settings modified by this software itself (master...rwconf) https://github.com/bitcoin/bitcoin/pull/11082
< instagibbs>
i think there's the two issues: 1) claiming to be core 2) claiming to offer bitcoin upgrades
< BlueMatt>
morcos: see greg's comments - people are claiming to "be" bitcoin core saying otherwise
< praxeology>
Maybe... the people who would be duped into downloading/installing btc1... haven't even/don't/won't install Bitcoin Core in the first place. So that set of people is probably pretty small, like maybe 0 people?
< jimpo>
cfields: Is there a reason that the "send rejects" part of SendRejectsAndCheckBanned should be called at the end of ProcessMessages as introduced in https://github.com/bitcoin/bitcoin/pull/9720?
< praxeology>
I disagree with luke's suggested rename to "2X". Ideally we could get the whole bitcoin/altcoin community to change the name, but its too late now, should just stick w/ what everyone is familiar with
< jnewbery>
to be clear, I'm not disputing the quality of morcos's posts, and I personally agree with them, but I find the idea of 'Bitcoin Core thinks <x>' objectionable
< BlueMatt>
can also just quote tweet and say like "Some thoughts on 2x, from a major contributor to Bitcoin Core"
< luke-jr>
"Of the 25 Bitcoin Core developers who have stated a position on 2X, all of them are opposed."
< jnewbery>
Murch - I agree. Have misgivings about "Bitcoin Core" endorsing a personal opinion
< Murch>
gmaxwell: That needs a response from the actual Bitcoin Core twitter account to condemn it as false flag.
<@wumpus>
luke-jr: yes, something like https://planet.freedesktop.org/ would be nice, though on the other hand for bitcoin that would result in endless political discussions about who to include and who not
< bitcoin-git>
[bitcoin] jonasschnelli opened pull request #11081: Add length check for CExtKey deserialization (master...2017/08/fix_cextkey) https://github.com/bitcoin/bitcoin/pull/11081
< Murch>
BlueMatt: Yeah, Replay Protection might be a bit over the head for the general audience. It should be mentioned though that it is unrelated to and _not supported by Bitcoin Core_.
< BlueMatt>
yea, that seems reasonable, just "hey, this is unrelated to Bitcoin Core or Bitcoin, really, they are playing a very, very risky game and most folks dont condone this"
< gmaxwell>
praxeology: it's run by the bitcoin.org people. They're generally reasonable folks.
< Chris_St1>
maybe bitcoin.org people can throw up a warning about people promoting consensus imcompatible implementations
< praxeology>
How close is bitcoin.org w/ the core dev team? Who runs it?
< gmaxwell>
We can also talk to the bitcoin.org folks in general.
< BlueMatt>
ok, so objections to luke-jr's proposal to put something on bitcoincore.org that simply points out that s2x is unrelated to segwit, and a fork of bitcoin, not a "harmless upgrade"?
< gmaxwell>
E.g. seen it on reddit and hacker news; and our community links people to https://en.bitcoin.it/wiki/Segwit_support but then gets trolls responding that its "fake" and "censored by theymos"
< gmaxwell>
spreading misinformation about S2X describing it as a harmless "upgrade" to bitcoin, misstating that things like
< BlueMatt>
<gribble> https://github.com/bitcoin/bitcoin/issues/10267 | INew -includeconf argument for including external configuration files by kallewoof · Pull Request #10267 · bitcoin/bitcoin · GitHub
< bitcoin-git>
[bitcoin] practicalswift opened pull request #11071: Use static_assert(…, …) instead of assert(…) where appropriate (master...static_assert) https://github.com/bitcoin/bitcoin/pull/11071
< gribble>
https://github.com/bitcoin/bitcoin/issues/7522 | Bugfix: Only use git for build info if the repository is actually the right one by luke-jr · Pull Request #7522 · bitcoin/bitcoin · GitHub
< gribble>
https://github.com/bitcoin/bitcoin/issues/10923 | Use -Wthread-safety-analysis if available (+ -Werror=[…] if --enable-werror) by practicalswift · Pull Request #10923 · bitcoin/bitcoin · GitHub
< gribble>
https://github.com/bitcoin/bitcoin/issues/10923 | Use -Wthread-safety-analysis if available (+ -Werror=[…] if --enable-werror) by practicalswift · Pull Request #10923 · bitcoin/bitcoin · GitHub