2018-08-15

< Jmabsd> sipa: exactly, i'm rading the relay code and can't find it too. so i was thinking maybe that bitcoin.org article is bss*ahem*incorrect*ahem*obsolete.
< sipa> this discussion is more appropriate for #bitcoin or https://bitcoin.stackexchange.com
< Jmabsd> "Bitcoin Core 0.12.0 defaults to relaying and mining null data outputs with up to 83 bytes with any number of data pushes, provided the total byte limit is not exceeded. There must still only be a single null data output and it must still pay exactly 0 satoshis."
< Jmabsd> luke-jr: wrong url sorry, here! https://bitcoin.org/en/developer-guide#standard-transactions
< Jmabsd> there's a weird claim at https://en.bitcoin.it/wiki/Script#Constants that null outputs must have amount == 0 to be relayed. is it so?

2018-08-14

< jonasschnelli> The tor argument "collect now, decrypt later" may not be applicable 1:1 to bitcoin
< sipa> but you don't want existing addnode=IP lines in bitcoin.conf files suddenly fail
< bitcoin-git> [bitcoin] laanwj closed pull request #13960: Fix PSBT deserialization of 0-input transactions (master...fix-decodepsbt-no-in) https://github.com/bitcoin/bitcoin/pull/13960
< bitcoin-git> bitcoin/master 3e5424f Wladimir J. van der Laan: Merge #13960: Fix PSBT deserialization of 0-input transactions...
< bitcoin-git> bitcoin/master 43811e6 Andrew Chow: Fix PSBT deserialization of 0-input transactions...
< bitcoin-git> bitcoin/master bd19cc7 Andrew Chow: Serialize non-witness utxo as a non-witness tx but always deserialize as witness...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/dabfcb03071e...3e5424faf6ff
< gribble> https://github.com/bitcoin/bitcoin/issues/13901 | adduser: The user `bitcoin already exists. Exiting. · Issue #13901 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< gribble> https://github.com/bitcoin/bitcoin/issues/13826 | packaging: Auto-change datadir in ubuntu ppa · Issue #13826 · bitcoin/bitcoin · GitHub
< Gnappuraz> Hi, I was going through the bitcoin documentation but I can't find the rationale behind the fact of using the prevTx.scriptPubKey in the place of curTx.scriptSig when creating the signature
< wumpus> ken2812221_: problems with bitcoincore.org you can file at https://github.com/bitcoin-core/bitcoincore.org
< waiting2compile> hi, could someone walk me a bit through running Bitcoin's tests? I tried following the instructions in the README.md, though I suspect there's some extra setup that needs to be done that's missing, since running the command just gives make errors and the like (pardon me, I'm new to this and am trying to fix one of the good first bugs :))

2018-08-13

< nmnkgl> MarcoFalke: Could you point me to anything would prevent bitcoin core nodes from having crazy everlasting loops in case when there is a ring of whitelisted peers? In regular mode the loop will break at least because of using INVs. That may be very unlikely case though.
< MarcoFalke> This is bitcoin-core-dev
< gribble> https://github.com/bitcoin/bitcoin/issues/13951 | Hardcoded seeds update pre-0.17 branch by laanwj · Pull Request #13951 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13905 | docs: fixed bitcoin-cli -help output for help2man by hebasto · Pull Request #13905 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< MarcoFalke> wumpus: I think https://github.com/bitcoin/bitcoin/pull/13905 can go in to 0.17 as doc bug fix (tiny one)
< gribble> https://github.com/bitcoin/bitcoin/issues/13666 | Always create signatures with Low R values by achow101 · Pull Request #13666 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13938 | refactoring: Cleanup StartRest() by DesWurstes · Pull Request #13938 · bitcoin/bitcoin · GitHub
< kallewoof> I tried [ createpsbt "[]" "[{\"$(./bitcoin-cli getnewaddress)\":0.01}]" ], which worked fine. But decodepsbt errors for the result.
< gribble> https://github.com/bitcoin/bitcoin/issues/12391 | TODO for release notes 0.17.0 · Issue #12391 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/12624 | Release schedule for 0.17.0 · Issue #12624 · bitcoin/bitcoin · GitHub

2018-08-12

< gmaxwell> I totally agree that wallets shoudl be batching and whatnot, but consider: we don't even have a friendly way to do that... There is no dohicky in bitcoin core where you can queue a payment, have it draft it, but not send it, waiting for either more payments it can be bached with, timeout, or shutdown trigger.
< MarcoFalke> I was hoping we could do a primitive cache for now and later replace that with https://github.com/bitcoin/bitcoin/pull/13804 (tx pool layer)
< sipa> he left. also, https://bitcoin.stackexchange.com is your friend
< itaseski> i wasn't able to find anything bitcoin specific ...
< devmob> hi, I'd really like to know how bitcoin does gossip, like how the gossip protocol is implemented
< reald0ff1> well thanks for the answer. I think I will try to contact bitcoincore.org via email and bitcoin.org also via email to the website maintainer. Maybe some of them could provide me stats
< harding> reald0ff1: I don't know if anyone has that information for BitcoinCore.org, sorry. In addition, the binaries can also be downloaded from Bitcoin.org (maintained by a different team) or via a torrent (with optional magnet URI) that contains the binaries for all platforms.
< reald0ff1> Can someone please provide me download stats (or at least share in %) of bitcoin core, regarding the different platform versions (Win, Linux, OSX, etc) ?

2018-08-10

< gribble> https://github.com/bitcoin/bitcoin/issues/13930 | Better explain GetAncestor check for m_failed_blocks in AcceptBlockHeader by Sjors · Pull Request #13930 · bitcoin/bitcoin · GitHub
< ken2812221> I wonder why the linter failed? https://travis-ci.org/bitcoin/bitcoin/jobs/414449333
< sipa> this channel is mostly about developmemt of the bitcoin core software
< sipa> Jmabsd: i think your questions are probably better suited for bitcoin.stackexchange.com
< Jmabsd> are we in a place today where you can assume that every relevant participant in the Bitcoin ecosystem will be fine with Bitcoin transactions with version = 2?

2018-08-09

< sipa> 2018-08-09T18:55:00.301077Z receive version message: /bitcoinj:0.14.7/Bitcoin Wallet:6.28/: version 70001, blocks=535894, us=127.0.0.1:8333, peer=414
< sipa> and your ISP's cost, and the bitcoin exchange rate?
< gribble> https://github.com/bitcoin/bitcoin/issues/13808 | wallet: shuffle coins before grouping, where warranted by kallewoof · Pull Request #13808 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13922 | Lower default relay fees by ajtowns · Pull Request #13922 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13666 | Always create signatures with Low R values by achow101 · Pull Request #13666 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< gribble> https://github.com/bitcoin/bitcoin/issues/13922 | Lower default relay fees by ajtowns · Pull Request #13922 · bitcoin/bitcoin · GitHub
< meshcollider> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012 achow101 meshcollider jnewbery maaku fanquake promag provoostenator
< wumpus> pierre_rochard: I did some analysis of the bitcoin source using clang a long time ago, using the python clang bindings (clang.cindex)
< pierre_rochard> searching the repo turned up this recent comment from skeees https://github.com/bitcoin/bitcoin/pull/13815#issuecomment-409795628 which makes it sound like a material undertaking
< sipa> Jmabsd: https://bitcoin.stackexchange.com may be a better place to ask these things
< gribble> https://github.com/bitcoin/bitcoin/issues/13922 | Lower default relay fees by ajtowns · Pull Request #13922 · bitcoin/bitcoin · GitHub
< Jmabsd> aj: ah, so Bitcoin Core nodes will accept a nonstandard tx when it's in a block, but it will not accept them in gossiping, right?
< Jmabsd> Where in Bitcoin Core's code are the constraints on what scripts are allowed, implemented? E.g., gossiped transaction's lock scripts must be either of P2PK, P2PKH, multisig, nulldata or P2SH in order to be considered valid and be gossiped?
< Jmabsd> code question, https://github.com/bitcoin/bitcoin/blob/master/src/script/interpreter.cpp#L1162 - when generating the sign text for OP_CHECKSIG, the code to produce the signtext with respect to one transaction output, is just " ::Serialize(s, txTo.vout[nOutput]);".

2018-08-08

< gribble> https://github.com/bitcoin/bitcoin/issues/12254 | BIP 158: Compact Block Filters for Light Clients by jimpo · Pull Request #12254 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13442 | Convert the 1-way SSE4 SHA256 code from asm to intrinsics by sipa · Pull Request #13442 · bitcoin/bitcoin · GitHub
< gmaxwell> lordcow in #bitcoin reports a compile failure on freebsd: https://pastebin.com/wBN0YChc
< jonasschnelli> I'm implementing BIP151 message structures: https://github.com/bitcoin/bips/blob/master/bip-0151.mediawiki#encrypted-messages-structure

2018-08-07

< gribble> https://github.com/bitcoin/bitcoin/issues/10102 | [experimental] Multiprocess bitcoin by ryanofsky · Pull Request #10102 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< gribble> https://github.com/bitcoin/bitcoin/issues/10973 | Refactor: separate wallet from node by ryanofsky · Pull Request #10973 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< MarcoFalke> githubs send from:"notifications@github.com" to:"bitcoin@noreply.github.com" with cc:"comment@noreply.github.com" and cc you
< harding> MarcoFalke: to ping privately, the bot could just pull the email address of the committer from the most recent commit message and email them. Alternatively, the bot could open an issue on the repository they're making their PR from, e.g. github.com/harding/bitcoin/issues/new
< gribble> https://github.com/bitcoin/bitcoin/issues/13808 | wallet: shuffle coins before grouping, where warranted by kallewoof · Pull Request #13808 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< gribble> https://github.com/bitcoin/bitcoin/issues/13667 | wallet: Fix backupwallet for multiwallets by domob1812 · Pull Request #13667 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13788 | Fix --disable-asm for newer assembly checks/code by luke-jr · Pull Request #13788 · bitcoin/bitcoin · GitHub

2018-08-06

< gribble> https://github.com/bitcoin/bitcoin/issues/11640 | Make LOCK, LOCK2, TRY_LOCK work with CWaitableCriticalSection by ryanofsky · Pull Request #11640 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13890 | PGP: 18AE2F798E0D239755DA4FD24B79F986CBDF8736 BTC: bc1q9md8qvhuswr9wgj8re2wscsfcxel3wvmdmeg96 31orZSrFb8ES1f5g8ogyzZZZb4MzsGkreu · Issue #13890 · bitcoin/bitcoin · GitHub

2018-08-05

< viaj3ro> have an issue with my 0.16.2 node: https://github.com/bitcoin/bitcoin/issues/13885
< gmaxwell> jl2012: bitcoin core doesn't even serve SPV proofs for arbritary transactions today!
< gmaxwell> The bitcoin protocol doesn't generate SPV proofs for arbritary transactions, it certantly isn't consensus critical to do so! :)
< wumpus> depending on how technical the question is, #bitcoin might be a better place to ask if it's about high-level concepts and not implementation
< devmob> anyone here I can talk to about bitcoin's p2p layer ?

2018-08-04

< BlueMatt> is a rust-lightning client that uses bitcoin core's rpc

2018-08-03

< gribble> https://github.com/bitcoin/bitcoin/issues/13696 | Add aarch64 qt depends support for cross compiling bitcoin-qt by TheCharlatan · Pull Request #13696 · bitcoin/bitcoin · GitHub
< ossifrage> Oh, I was thinking of this: https://github.com/bitcoin/bitcoin/pull/12495

2018-08-02

< gmaxwell> https://github.com/bitcoin/bitcoin/pull/12495#issuecomment-367815005 < eklitzke suggests increasing leveldb's mmap limit here. ossifrage also apparently did that and reported it fixed his problems, though his specific problems are likely better fixed by switching to poll.
< gribble> https://github.com/bitcoin/bitcoin/issues/12495 | Increase LevelDB max_open_files by eklitzke · Pull Request #12495 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/12613 | [CI] Adding MSVC build to CI check with Appveyor · Issue #12613 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13789 | crypto/sha256: Use pragmas to enforce necessary intrinsics for GCC and Clang by luke-jr · Pull Request #13789 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/12624 | Release schedule for 0.17.0 · Issue #12624 · bitcoin/bitcoin · GitHub
< wumpus> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012 achow101 meshcollider jnewbery maaku fanquake promag provoostenator
< gribble> https://github.com/bitcoin/bitcoin/issues/13754 | Windows crashes for -wallet=你好 · Issue #13754 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13426 | [bugfix] Fix encoding issue for Windows by ken2812221 · Pull Request #13426 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13697 | Support output descriptors in scantxoutset by sipa · Pull Request #13697 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13844 | doc: correct the help output for -prune by hebasto · Pull Request #13844 · bitcoin/bitcoin · GitHub
< ossifrage> mmap() is a great way to generate a very large amount of write pressure, but it seems like most of the leveldb use in bitcoin has a very low change rate

2018-08-01

< sipa> the library implementing the elliptic curve things is in bhttps://github.com/bitcoin-core/secp256k1
< achow101> there's a library that implements all of that stuff: https://github.com/bitcoin-core/secp256k1
< sipa> actually, #bitcoin-commits already exists
< gribble> https://github.com/bitcoin/bitcoin/issues/13662 | Explain when reindex-chainstate can be used instead of reindex by Sjors · Pull Request #13662 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< gribble> https://github.com/bitcoin/bitcoin/issues/13676 | Explain that mempool memory is added to -dbcache during IBD by Sjors · Pull Request #13676 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< skeees> BlueMatt: sorry missed that one, updated in https://github.com/bitcoin/bitcoin/pull/13835
< ossifrage> Oh, in my case it was bitcoin-qt not bitcoind
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13247: Add tests to SingleThreadedSchedulerClient() and document the memory model (master...scheduler-tests) https://github.com/bitcoin/bitcoin/pull/13247
< bitcoin-git> bitcoin/master cbeaa91 Jesse Cohen: Update ValidationInterface() documentation to explicitly specify threading and memory model
< bitcoin-git> bitcoin/master b296b42 Jesse Cohen: Update documentation for SingleThreadedSchedulerClient() to specify the memory model
< bitcoin-git> bitcoin/master 9994d01 Jesse Cohen: Add Unit Test for SingleThreadedSchedulerClient...
< bitcoin-git> [bitcoin] MarcoFalke pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/0fb9c87815d1...e83d82a85c53
< sipa> bitcoind 11155 pw 40r REG 252,1 2173957 779300 /home/pw/.bitcoin/chainstate/864998.ldb
< sipa> bitcoind 11155 pw mem REG 252,1 2173885 783231 /home/pw/.bitcoin/chainstate/864555.ldb
< ossifrage> I have txindex and sadly my bitcoin data is on a btrfs filesystem (a mistake I won't make again)

2018-07-31

< bitcoin-git> [bitcoin] jnewbery opened pull request #13825: [wallet] [Do not merge until v0.18] Kill accounts (master...kill_accounts) https://github.com/bitcoin/bitcoin/pull/13825
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #13824: doc: Remove outdated net comment (master...Mf1808-docNetCritSec) https://github.com/bitcoin/bitcoin/pull/13824
< ossifrage> it looks like bitcoin-qt is using up 800 FDs for open files (chainstate, txindex, etc...)
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13822: bench: Make CoinSelection output groups pass eligibility filter (master...fix-out-groups-bench) https://github.com/bitcoin/bitcoin/pull/13822
< ossifrage> I didn't think bitcoin would be using select(), I thought it would be using epoll() and friends?
< bitcoin-git> bitcoin/master 0fb9c87 MarcoFalke: Merge #13822: bench: Make CoinSelection output groups pass eligibility filter...
< bitcoin-git> bitcoin/master 494634a Andrew Chow: bench: Make CoinSelection output groups pass eligibility filter...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7d3623794a10...0fb9c87815d1
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #13823: qa: quote path in authproxy for external multiwallets (master...Mf1808-qaAuthProxyQuotePath) https://github.com/bitcoin/bitcoin/pull/13823
< bitcoin-git> [bitcoin] laanwj closed pull request #13776: net: Add missing verification of IPv6 address in CNetAddr::GetIn6Addr(...) (master...call-IsIPv6-in-GetIn6Addr) https://github.com/bitcoin/bitcoin/pull/13776
< bitcoin-git> bitcoin/master 7d36237 Wladimir J. van der Laan: Merge #13776: net: Add missing verification of IPv6 address in CNetAddr::GetIn6Addr(...)...
< bitcoin-git> bitcoin/master 2fb0066 practicalswift: net: Add missing verification of IPv6 address in CNetAddr::GetIn6Addr(...)
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/230652cafc51...7d3623794a10
< bitcoin-git> [bitcoin] achow101 opened pull request #13822: bench: Make CoinSelection output groups pass eligibility filter (master...fix-out-groups-bench) https://github.com/bitcoin/bitcoin/pull/13822
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13821: qa: Re-enable test for unknown arg in conf file (master...Mf1808-qaConfWarn) https://github.com/bitcoin/bitcoin/pull/13821
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #13821: qa: Re-enable test for unknown arg in conf file (master...Mf1808-qaConfWarn) https://github.com/bitcoin/bitcoin/pull/13821
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13799: Ignore unknown config file options; warn instead of error (master...201807_warnunknown) https://github.com/bitcoin/bitcoin/pull/13799
< bitcoin-git> bitcoin/master 230652c MarcoFalke: Merge #13799: Ignore unknown config file options; warn instead of error...
< bitcoin-git> bitcoin/master 247d574 Pieter Wuille: Ignore unknown config file options for now
< bitcoin-git> bitcoin/master 04ce0d8 Pieter Wuille: Report when unknown config file options are ignored
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/77168f766f15...230652cafc51
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13809: contrib: Remove debian and rpm subfolder (master...Mf1808-debianContrib) https://github.com/bitcoin/bitcoin/pull/13809
< bitcoin-git> bitcoin/master 77168f7 MarcoFalke: Merge #13809: contrib: Remove debian and rpm subfolder...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/8ce55df70d31...77168f766f15
< bitcoin-git> bitcoin/master fa0e1e2 MarcoFalke: contrib: Remove debian and rpm subfolders
< bitcoin-git> [bitcoin] chipstar closed pull request #13820: Replace Bitcoin to XPChain (master...replace-botcoin-to-xpchain) https://github.com/bitcoin/bitcoin/pull/13820
< bitcoin-git> [bitcoin] chipstar opened pull request #13820: Replace Bitcoin to XPChain (master...replace-botcoin-to-xpchain) https://github.com/bitcoin/bitcoin/pull/13820
< bitcoin-git> [bitcoin] Sjors opened pull request #13818: More intuitive GUI settings behavior when -proxy is set (master...2018/07/gui-proxy) https://github.com/bitcoin/bitcoin/pull/13818
< bitcoin-git> [bitcoin] scravy opened pull request #13816: travis: build and run tests on os: osx (master...run-functional-tests-on-macos) https://github.com/bitcoin/bitcoin/pull/13816
< harding> various places in their home directory for different Bitcoin-related resources.
< harding> mryandao: ok. I guess I have a different backup philosophy than you do. I use cheap external HDs, and so I just grab everything indiscriminately. I think there probably is an advantage to fully separating stuff we really want users to backup (e..g bitcoin.conf and wallets) from stuff that's just nice to backup (e.g. old blocks), but I personally don't think that's enough of an advantage to make them have to go looking around
< mryandao> typically most linux distribution follow XDG specification for looking up config files, would this be preferable in Core? I know there was a previous closed issue which mentioned this, but we could keep the datadir cache separate in ~/.bitcoin as is
< sipa> bitcoin-cli needs to know what arguments to treat as string literals and which ones not
< luke-jr> kallewoof: for bitcoin-cli, which isn't tested by assert_raises_rpc_error
< bitcoin-git> [bitcoin] practicalswift opened pull request #13815: build: Add linter to make sure the return value of Parse[...](...) is checked (master...check-return-values-from-parse) https://github.com/bitcoin/bitcoin/pull/13815

2018-07-30

< gribble> https://github.com/bitcoin/bitcoin/issues/13791 | gui: Reject dialogs if key escape is pressed by promag · Pull Request #13791 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< gmaxwell> quitobro: you might want to also keep an eye on https://github.com/bitcoin/bitcoin/pull/12254
< quitobro> definitely; should we just keep our eyes on bitcoin-dev-mailing-list?
< wumpus> the only use of protobuf is the payment request code in bitcoin-qt and that hasn't seen serious changes since... forever
< gribble> https://github.com/bitcoin/bitcoin/issues/13804 | Stacked Transaction Pool Layer by MarcoFalke · Pull Request #13804 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< satwo> wumpus: thanks! I definitely need to familiarize myself with functional vs unit tests in bitcoin. I'll continue to come here to ask my noobish questions.
< fanquake> There was at one point a bitcoin test cases/writeups repo, but I don't think it ever got much traction
< gribble> https://github.com/bitcoin/bitcoin/issues/12879 | [scripted-diff] No extern function declarations by kallewoof · Pull Request #12879 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13803 | doc: add note to developer docs about warranted PRs by kallewoof · Pull Request #13803 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13802 | Dont use zero as null pointer constant by practicalswift · Pull Request #13802 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13795 | build: Add missing [[noreturn]] to handleRunawayException(...). Enable -Wsuggest-attribute=noreturn if available. by practicalswift · Pull Request #13795 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13770 | Use explicit captures in lambda expressions by practicalswift · Pull Request #13770 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13801 | wallet: -avoidreuse with destination filters by kallewoof · Pull Request #13801 · bitcoin/bitcoin · GitHub
< Fuzzbawls> Question: (please ping me directly with answer) Given the recent acquisition of GitHub by Microsoft, is a relocation of the bitcoin source code to an alternative platform (like GitLab) being considered at all? If so, has anyone been actively working towards porting the `.travis.yml` file over to an alternative CI provider? I've had mixed results with such a port; some features can be duplicated/represented, and some cannot (like the $TRAVIS_COMMIT
< gribble> https://github.com/bitcoin/bitcoin/issues/13756 | wallet: -avoidreuse feature for improved privacy by kallewoof · Pull Request #13756 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10065 | Support not reusing addresses · Issue #10065 · bitcoin/bitcoin · GitHub
< fanquake> Also locked the conversation on https://github.com/bitcoin/bitcoin/commit/d2186b3db61a9d4dc2d4a6211573790d9e34bf58#comments, as that's probably carried on long enough.

2018-07-29

< gmaxwell> I saw it, and so did some ordinary user in #bitcoin, and I'm pretty sure I didn't have anything cached.
< wumpus> they both show 0.16.2 don't they? https://bitcoin.org/en/download https://bitcoincore.org/en/download/

2018-07-28

< jonasschnelli> cfields: sigs ---> https://github.com/bitcoin-core/bitcoin-detached-sigs/pull/10 (thanks ken2812221)
< jonasschnelli> ken2812221: sry. :) now my fault. I need: e56b7d20bc7bfb8203f1651ddcf4fc18d8685acf942a8c7b90919f29e24db3a1 bitcoin-0.16.2-osx-unsigned.tar.gz
< jonasschnelli> You file: e56b7d20bc7bfb8203f1651ddcf4fc18d8685acf942a8c7b90919f29e24db3a1 bitcoin-osx-unsigned.tar.gz
< jonasschnelli> ken2812221: the file sha256sum doesn't match the available signatures: https://github.com/bitcoin-core/gitian.sigs/blob/master/0.16.2-osx-unsigned/laanwj/bitcoin-osx-0.16-build.assert
< gribble> https://github.com/bitcoin/bitcoin/issues/13782 | Fix osslsigncode compile issue in gitian-build by ken2812221 · Pull Request #13782 · bitcoin/bitcoin · GitHub
< windsok> +1 to making invalid options in bitcoin.conf a warning. I'm having trouble figuring out which option is causing the error

2018-07-27

< jhfrontz> I'm getting errors about missing symbols (like described here: https://github.com/bitcoin/bitcoin/issues/12511#issuecomment-386740137 ).
< wumpus> but a warning if in bitcoin.conf
< luke-jr> wumpus: actually, it might be annoying if users get errors for Knots configs in bitcoin.conf
< sipa> (by default it's enforced, unless you pass -ignoreunknownconfigoptions or so; and then it still only applies to bitcoin.conf and not to command line options)
< sipa> wumpus: or we need a way to specify bitcoin.conf options that only apply to a particular binary
< sipa> but we should probably make it ignore bitcoin-cli only options like this one, when present in bitcoin.conf
< sipa> right; you can't put it in bitcoin.conf anymore, because it's not a bitcoind option
< sipa> in bitcoin.conf it should work fine; that file should be read by both bitcoind and bitcoin-cli
< sipa> satwo: no, but it's a bitcoin-cli command line option; not a bitcoind one
< satwo> Has the bitcoind config param -rpcclienttimeout been removed? I'm getting errors trying to set it in both the command line and in bitcoin.conf.
< harding> brighton36: I'm not sure what you're talking about, but it feels a bit off topic for this channel. Would you mind joining #bitcoin-core-website to discuss further? (Or you can PM me if you're a registered user.)
< harding> brighton36: I help maintain the site. I don't relaly understand your question. There's a basic contributing policy at https://github.com/bitcoin-core/bitcoincore.org/blob/master/CONTRIBUTING.md but the basic process is the same for any open source project: people open PRs proposing changes, other people provide feedback, once the feedback is generally positive, the PR is merged. If the PR never gets to that stage, it's
< jonasschnelli> sipa: would adding 'h' as equivalent to ' in the descriptor xpub/xpriv range definition makes sense? >'< in conjunction with JSON and the GUI/bitcoin-cli/shell seems to be a hassle

2018-07-26

< gribble> https://github.com/bitcoin/bitcoin/issues/13311 | Dont edit Chainparams after initialization by jtimon · Pull Request #13311 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< gribble> https://github.com/bitcoin/bitcoin/issues/13426 | [bugfix] Fix encoding issue for Windows by ken2812221 · Pull Request #13426 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13426 | [bugfix] Fix encoding issue for Windows by ken2812221 · Pull Request #13426 · bitcoin/bitcoin · GitHub
< wumpus> jamesob: < wumpus> as for high priority for review, please review everything under the 0.17 milestone: https://github.com/bitcoin/bitcoin/milestone/33
< gribble> https://github.com/bitcoin/bitcoin/issues/13756 | wallet: -avoidreuse feature for improved privacy by kallewoof · Pull Request #13756 · bitcoin/bitcoin · GitHub
< wumpus> as for high priority for review, please review everything under the 0.17 milestone: https://github.com/bitcoin/bitcoin/milestone/33
< gribble> https://github.com/bitcoin/bitcoin/issues/12624 | Release schedule for 0.17.0 · Issue #12624 · bitcoin/bitcoin · GitHub
< BlueMatt> jonasschnelli: well I ask because there is debate about what to write in some docs in rust-bitcoin, and also what to call it on bitcoincore.org docs
< sipa> BlueMatt: scriptWitness is just in bitcoin core's source code though; is it called that way anywhere else?
< wumpus> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr btcdrak sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark mi
< satwo> Would modifying bitcoin-rpc to say something like "size (total):" or "total size:" be messy overkill? At the very least it would bring RPC and BIP 141 in harmony, potentially reducing some confusion
< satwo> Hi all. BIP-141 defines 4 ways to measure the size of a transaction: weight, virtual size, base size, and total size. Bitcoin-cli decoderawtransaction returns weight, vsize ("virtual size" - obvious), and size (“total size" - not obvious). I must not be the only one to have found it nontrivial to figure out how base size, total size in BIP141 and “size” in RPC are related. Even once one figures out that “BIP 141
< arubi> DDG first result for "scriptWitness" is the transaction.h file in the repo, and for "witnessScript bitcoin" the first result is is the core-dev docs site
< BlueMatt> Ugh, ok, poll time, what are peoples' thoughts on what to call the witness version of the redeemScript? https://github.com/bitcoin-core/bitcoincore.org/issues/581 and https://github.com/rust-bitcoin/rust-bitcoin/pull/109 for debate context
< ken2812221> bitcoin-git is dead?
< jonasschnelli> provoostenator: https://bitcoin.jonasschnelli.ch/build/715
< gribble> https://github.com/bitcoin/bitcoin/issues/13426 | [bugfix] Fix encoding issue for Windows by ken2812221 · Pull Request #13426 · bitcoin/bitcoin · GitHub

2018-07-25

< bitcoin-git> [bitcoin] fanquake closed pull request #12696: Fix possible data race when committing block files (master...fsync) https://github.com/bitcoin/bitcoin/pull/12696
< gribble> https://github.com/bitcoin/bitcoin/issues/13696 | Add aarch64 qt depends support for cross compiling bitcoin-qt by TheCharlatan · Pull Request #13696 · bitcoin/bitcoin · GitHub
< provoostenator> Mirroring everything in #bitcoin-core-dev-read-only is not ideal either I guess
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13714: contrib: Add lxc network setup for Bionic host (master...gitian-build-auto-install) https://github.com/bitcoin/bitcoin/pull/13714
< bitcoin-git> bitcoin/master 29b4ee6 MarcoFalke: Merge #13714: contrib: Add lxc network setup for Bionic host...
< bitcoin-git> bitcoin/master 60c3c23 Chun Kuan Lee: [contrib] Add automatic setup for Ubuntu Bionic
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/2d41af17282e...29b4ee64695f
< bitcoin-git> [bitcoin] laanwj closed pull request #13658: [moveonly] Extract RescanWallet to handle a simple rescan (master...rescan-from-time) https://github.com/bitcoin/bitcoin/pull/13658
< bitcoin-git> bitcoin/master 2d41af1 Wladimir J. van der Laan: Merge #13658: [moveonly] Extract RescanWallet to handle a simple rescan...
< bitcoin-git> bitcoin/master 3fe836b Ben Woosley: [moveonly] Extract RescanWallet to handle a simple rescan...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9f23c1659938...2d41af17282e
< bitcoin-git> [bitcoin] laanwj closed pull request #13721: Bugfixes for BIP 174 combining and deserialization (master...psbt-fixes) https://github.com/bitcoin/bitcoin/pull/13721
< bitcoin-git> bitcoin/master 9f23c16 Wladimir J. van der Laan: Merge #13721: Bugfixes for BIP 174 combining and deserialization...
< bitcoin-git> bitcoin/master fad231a Andrew Chow: Fix merging of global unknown data in PSBTs...
< bitcoin-git> bitcoin/master 41df035 Andrew Chow: Check that PSBT keys are the correct length...
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/94dd89e6fa37...9f23c1659938
< Lico> Hi guys I am sorry for the basic question, but I would like to debug the bitcoin and I would like to know what tools for debug do you use for Ubuntu ? Does anybody use and IDE for Ubuntu ? I tried to use netbeans, eclipse
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13706: doc: Minor improvements to release-process.md (master...release_process_doc) https://github.com/bitcoin/bitcoin/pull/13706
< bitcoin-git> bitcoin/master 94dd89e MarcoFalke: Merge #13706: doc: Minor improvements to release-process.md...
< bitcoin-git> bitcoin/master 1c22cc1 Mitchell Cash: doc: Update broken links to now point to gitian-build.py
< bitcoin-git> bitcoin/master 95464c7 Mitchell Cash: doc: Improve command to generate list of authors for release notes...
< bitcoin-git> [bitcoin] MarcoFalke pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/a140953907eb...94dd89e6fa37
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13753: scripted-diff: Remove trailing whitespaces (master...remove-trailing-whitespaces) https://github.com/bitcoin/bitcoin/pull/13753
< bitcoin-git> bitcoin/master a140953 MarcoFalke: Merge #13753: scripted-diff: Remove trailing whitespaces...
< bitcoin-git> bitcoin/master 12dd101 João Barbosa: scripted-diff: Remove trailing whitespaces...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c8836532024f...a140953907eb
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13750: depends: use MacOS friendly sed syntax in qt.mk (master...2018/07/depends-qt-macos) https://github.com/bitcoin/bitcoin/pull/13750
< bitcoin-git> bitcoin/master c883653 MarcoFalke: Merge #13750: depends: use MacOS friendly sed syntax in qt.mk...
< bitcoin-git> bitcoin/master 12c93cb Sjors Provoost: depends: use MacOS friendly sed syntax in qt.mk
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cf7f9ae34e66...c8836532024f
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13617: release: require macOS 10.10+ (master...macos-10-10) https://github.com/bitcoin/bitcoin/pull/13617
< bitcoin-git> bitcoin/master 6c6dbd8 fanquake: doc: mention that macOS 10.10 is now required
< bitcoin-git> bitcoin/master 84b0cfa fanquake: release: bump minimum required macOS to 10.10
< bitcoin-git> bitcoin/master 26b15df fanquake: depends: set OSX_MIN_VERSION to 10.10
< bitcoin-git> [bitcoin] MarcoFalke pushed 7 new commits to master: https://github.com/bitcoin/bitcoin/compare/1211b15bf6c0...cf7f9ae34e66
< bitcoin-git> [bitcoin] kallewoof opened pull request #13756: wallet: -avoidreuse feature for improved privacy (master...feature-avoidreuse) https://github.com/bitcoin/bitcoin/pull/13756
< bitcoin-git> [bitcoin] ken2812221 opened pull request #13755: [WIP] Full unicode support on Windows (master...bdb-unicode) https://github.com/bitcoin/bitcoin/pull/13755
< bitcoin-git> [bitcoin] kallewoof closed pull request #10386: [WIP] [wallet] Optional '-avoidreuse' flag which defaults to not reusing addresses in sends (master...feature-white-black-address) https://github.com/bitcoin/bitcoin/pull/10386
< bitbee> is roger ver still stealing bitcoin core code?

2018-07-24

< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13656: Remove the boost/algorithm/string/predicate.hpp dependency (master...patch/remove_boost_predicate_from_netbase) https://github.com/bitcoin/bitcoin/pull/13656
< bitcoin-git> bitcoin/master 1211b15 MarcoFalke: Merge #13656: Remove the boost/algorithm/string/predicate.hpp dependency...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7a9bca61fa55...1211b15bf6c0
< bitcoin-git> bitcoin/master e3245f2 251: Removes Boost predicate.hpp dependency...
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13650: [travis] Don't store debug info if --enable-debug is set (master...travis-debug-fix) https://github.com/bitcoin/bitcoin/pull/13650
< bitcoin-git> bitcoin/master 7a9bca6 MarcoFalke: Merge #13650: [travis] Don't store debug info if --enable-debug is set...
< bitcoin-git> bitcoin/master 14788fb Chun Kuan Lee: [travis] Don't store debug info if --enable-debug is set
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5f7575e26364...7a9bca61fa55
< bitcoin-git> [bitcoin] kallewoof reopened pull request #10386: [WIP] [wallet] Optional '-avoidreuse' flag which defaults to not reusing addresses in sends (master...feature-white-black-address) https://github.com/bitcoin/bitcoin/pull/10386
< bitcoin-git> [bitcoin] promag opened pull request #13753: scripted-diff: Remove trailing whitespaces (master...remove-trailing-whitespaces) https://github.com/bitcoin/bitcoin/pull/13753
< bitcoin-git> [bitcoin] laanwj closed pull request #12257: [wallet] Use destination groups instead of coins in coin select (master...feature-addrgrouped-coinselect) https://github.com/bitcoin/bitcoin/pull/12257
< bitcoin-git> bitcoin/master 65b3eda Karl-Johan Alm: wallet: Add input bytes to CInputCoin...
< bitcoin-git> [bitcoin] laanwj pushed 11 new commits to master: https://github.com/bitcoin/bitcoin/compare/7ebd8c638525...5f7575e26364
< bitcoin-git> bitcoin/master 173e18a Karl-Johan Alm: utils: Add insert() convenience templates
< bitcoin-git> bitcoin/master a443d7a Karl-Johan Alm: moveonly: CoinElegibilityFilter into coinselection.h
< gribble> https://github.com/bitcoin/bitcoin/issues/11881 | Remove Python2 support by jnewbery · Pull Request #11881 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< bitcoin-git> [bitcoin] 251Labs opened pull request #13751: Utils and libraries: Drops the boost/algorithm/string/split.hpp dependency (master...patch/remove_boost_split) https://github.com/bitcoin/bitcoin/pull/13751
< bitcoin-git> [bitcoin] Sjors opened pull request #13750: depends: use MacOS friendly sed syntax in qt.mk (master...2018/07/depends-qt-macos) https://github.com/bitcoin/bitcoin/pull/13750
< gribble> https://github.com/bitcoin/bitcoin/issues/13732 | Depends: Fix Qts rcc determinism by Fuzzbawls · Pull Request #13732 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] hmel closed pull request #13739: refactor: remove superfluous boost::thread_interrupted catch (master...dead-code-elimination) https://github.com/bitcoin/bitcoin/pull/13739
< bitcoin-git> [bitcoin] kallewoof closed pull request #13716: bitcoin-cli: -stdinwalletpassphrase and non-echo stdin passwords (master...stdinwalletpassphrase) https://github.com/bitcoin/bitcoin/pull/13716
< gribble> https://github.com/bitcoin/bitcoin/issues/13742 | doc: Adjust bitcoincore.org links by MarcoFalke · Pull Request #13742 · bitcoin/bitcoin · GitHub

2018-07-23

< gribble> https://github.com/bitcoin/bitcoin/issues/9502 | [Qt] Add option to pause/resume block downloads by jonasschnelli · Pull Request #9502 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9502 | [Qt] Add option to pause/resume block downloads by jonasschnelli · Pull Request #9502 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] jnewbery closed pull request #10160: Add updatepeer RPC (master...updatepeer) https://github.com/bitcoin/bitcoin/pull/10160
< gribble> https://github.com/bitcoin/bitcoin/issues/11637 | p2p: Remove dead service bits code by MarcoFalke · Pull Request #11637 · bitcoin/bitcoin · GitHubAsset 1Asset 1
< ossifrage> harding, in my case I think it is just that my node has been up for a while and the connection count is wandering up and with it the non-bitcoin nodes (recently lost my old ip address thanks to verizon-quality)
< gribble> https://github.com/bitcoin/bitcoin/issues/9502 | [Qt] Add option to pause/resume block downloads by jonasschnelli · Pull Request #9502 · bitcoin/bitcoin · GitHub
< bitcoin-git> bitcoin/master 7ebd8c6 Wladimir J. van der Laan: qt: Update translation source messages...
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/7ebd8c6385253bfa30175169a22f93474d8ddfaf
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #13747: tests: Skip P2PConnection's is_closing() check when not available (master...skip-isclosing) https://github.com/bitcoin/bitcoin/pull/13747
< bitcoin-git> bitcoin/master ab28b5b MarcoFalke: Merge #13747: tests: Skip P2PConnection's is_closing() check when not available...
< bitcoin-git> bitcoin/master 64b9f27 Daniel Kraft: Skip is_closing() check when not available....
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c0a47da72505...ab28b5b52702
< gribble> https://github.com/bitcoin/bitcoin/issues/12257 | [wallet] Use destination groups instead of coins in coin select by kallewoof · Pull Request #12257 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] domob1812 opened pull request #13747: tests: Skip P2PConnection's is_closing() check when not available (master...skip-isclosing) https://github.com/bitcoin/bitcoin/pull/13747
< gribble> https://github.com/bitcoin/bitcoin/issues/13478 | [RFC] gui: Minimum required Qt5 · Issue #13478 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/13529 | Use new Qt5 connect syntax by promag · Pull Request #13529 · bitcoin/bitcoin · GitHub