2017-12-12

< bitcoin-git> bitcoin/master d48ab83 Wladimir J. van der Laan: Merge #11831: Always return true if AppInitMain got to the end...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/c0902624b0ec...d48ab83f0053
< bitcoin-git> [bitcoin] laanwj closed pull request #10699: Make all script validation flags backward compatible (master...20170628_softflags) https://github.com/bitcoin/bitcoin/pull/10699
< bitcoin-git> bitcoin/master 2dd6f80 Pieter Wuille: Add a test that all flags are softforks
< bitcoin-git> bitcoin/master 01013f5 Pieter Wuille: Simplify tx validation tests
< bitcoin-git> bitcoin/master 2851b77 Pieter Wuille: Make all script verification flags softforks
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/0e722e8879a8...c0902624b0ec
< Provoostenator> sipa: your comment about why self.nodes[0].generate(1) was needed just solved another mystery for me: https://medium.com/provoost-on-crypto/debugging-bitcoin-core-functional-tests-cc0aa6e7fd3e
< gmaxwell> kallewoof: bitcoin core doesn't rebroadcast third party txn, but random bozos do, because they think they're helping in some cases, or because they want to pump up mempool stats. or god knows why

2017-12-11

< gribble> https://github.com/bitcoin/bitcoin/issues/10247 | cost too many fees? · Issue #10247 · bitcoin/bitcoin · GitHub
< adiabat> bitcoin-qt bump fee only gives you one option though; cli lets you set whatever you want
< gmaxwell> Provoostenator: the bumpfee in bitcoin core however lets you set whatever target you want AFAIR.
< gribble> https://github.com/bitcoin/bitcoin/issues/11605 | [Wallet] Enable RBF by default in QT by Sjors · Pull Request #11605 · bitcoin/bitcoin · GitHub
< BlueMatt> Provoostenator: well my concern is ux for average users...if I'm sending to someone I know understands bitcoin, fine, no issue, if its some guy who's receiving a withdraw from an atm/their exchange, they may be very confused
< BlueMatt> no need to relay it in bitcoin p2p net?
< gmaxwell> adiabat: there is no catch, if bitcoin core ever does replacement fully I assume it'll presign with locktimes.
< jonasschnelli> Not sure if we want a key leading to the Bitcoin Platinum project in our repository,... could be missued for advertising?
< jonasschnelli> The key links to btcplt.org (Bitcoin Platinum)
< BlueMatt> worth reaching out to people who use bitcoin core's wallet in reasonable volume to ask what they'd want from such an interface
< Randolf> alcipir: My suggestion is to learn C++ first, and then learn about programming Bitcoin/blockchain. The reason is that both have learning curves that are probably better-studied separately.
< Provoostenator> Or you could stalk your friends with C++ skills and manipulate them into becoming interested in Bitcoin, quitting their current job and helping out :-)
< gribble> https://github.com/bitcoin/bitcoin/issues/11383 | Basic Multiwallet GUI support by luke-jr · Pull Request #11383 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10740 | [wallet] dynamic loading/unloading of wallets by jnewbery · Pull Request #10740 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10794 | Add simple light-client mode (RPC only) by jonasschnelli · Pull Request #10794 · bitcoin/bitcoin · GitHub
< cluelessperson> I understand the frustration. It just keeps coming up for me because I keep getting morons that can't seem to understand how it works in #bitcoin
< cluelessperson> I have a serious question. What are thoughts on seperating the Bitcoin Core wallet and Bitcoin Core node?
< bitcoin-git> [bitcoin] promag closed pull request #11865: wallet: Improve ReacceptWalletTransactions performance (master...2017-12-improve-reaccept-wallet-transactions) https://github.com/bitcoin/bitcoin/pull/11865
< bitcoin-git> bitcoin/master 8ab6c0b Wladimir J. van der Laan: Merge #11839: don't attempt mempool entry for wallet transactions on startup if alr…...
< bitcoin-git> bitcoin/master 6697a70 Gregory Sanders: add test for unconfirmed balance between restarts
< bitcoin-git> bitcoin/master 6ba8f30 Gregory Sanders: don't attempt mempool entry for wallet transactions on startup if already in mempool
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/f60b4ad57912...8ab6c0b09e4e
< bitcoin-git> [bitcoin] promag opened pull request #11864: wallet: Make fund transaction atomic (master...2017-12-atomic-fundtransaction) https://github.com/bitcoin/bitcoin/pull/11864
< Provoostenator> AdilibA: I think this is more appropriate for e.g. #bitcoin-wizards or the bitcoin-dev mailinglist. Unless you have a proof-of-concept patch ready to go specifically for the Bitcoin Core client (which this channel is about).
< warren> The Bitcoin gitian instructions say to use only lxc but the way I've used it for the past few years is with qemu-kvm instead
< warren> Randolf: no, this is very gitian and bitcoin specific
< gribble> https://github.com/bitcoin/bitcoin/issues/11824 | Block ActivateBestChain to empty validationinterface queue by TheBlueMatt · Pull Request #11824 · bitcoin/bitcoin · GitHub

2017-12-10

< mryandao> maybe bitcoin-core should build in telemetry service and leave the generation of debug log for advanced users.
< Randolf> sipa: But this really isn't a Bitcoin issue -- it's an Operating System issue that has the same impact on other applications too.
< bitcoin-git> [bitcoin] irq13 opened pull request #11860: move banned peer log messages behind -debug=net (master...no-log-bans) https://github.com/bitcoin/bitcoin/pull/11860
< meshcollider> why is the boost::to_lower() inside a #ifdef WIN32 here: https://github.com/bitcoin/bitcoin/blob/master/src/util.cpp#L419

2017-12-09

< foobar_> does bitcoin rule or what
< cluelessperson> does bitcoin core support an "addrindex=1" ?
< bitcoin-git> [bitcoin] laanwj closed pull request #11740: Implement BIP159 NODE_NETWORK_LIMITED (pruned peers) *signaling only* (master...2017/11/NNL_signaling) https://github.com/bitcoin/bitcoin/pull/11740

2017-12-08

< cluelessperson> Question, does a prunine Bitcoin Core node keep a certain number of blocks?
< bitcoin-git> [bitcoin] sipa closed pull request #10149: gentlejack (master...0.12.1-bitcore) https://github.com/bitcoin/bitcoin/pull/10149
< bitcoin-git> [bitcoin] ryanofsky opened pull request #11854: Split up key and script metadata for better type safety (master...pr/scriptmet) https://github.com/bitcoin/bitcoin/pull/11854
< gribble> https://github.com/bitcoin/bitcoin/issues/11800 | Bitcoin is returning higher fees for 36 block window than 2 block window (on testnet) · Issue #11800 · bitcoin/bitcoin · GitHub
< adiabat> regarding the weird fee estimates mentioned yesterday, I have similar behavior on mainnet with bitcoin-qt right now
< bitcoin-git> [bitcoin] ryanofsky opened pull request #11851: scripted-diff: Rename wallet database classes (master...pr/wren) https://github.com/bitcoin/bitcoin/pull/11851
< valval> hi, anyone able to assists me in setting up of bitcoin core for modification?
< sipa> #bitcoin
< binarycat> I have a Bitcoin Core wallet.dat which I've had forever. Today, I made a fresh wallet.dat, created a receive address and went back to the original wallet.dat and transferred almost all of the Bitcoins to the new one, for the sake of being able to safely "claim" Bitcoin Gold/Cash/whatever without risking that the software just steals my Bitcoin instead of giving me other coins (and also to put the new wallet.dat in offline storage.... But the

2017-12-07

< aj> wumpus: curl -s -i https://api.github.com/repos/bitcoin/bitcoin/projects | grep ^X-RateLimit
< gribble> https://github.com/bitcoin/bitcoin/issues/5584 | Ability to examine, save locked-UTXO state · Issue #5584 · bitcoin/bitcoin · GitHub
< aj> wumpus: hmm, i might see if that works actually. i was thinking about having lightningbot update a repo of its meeting logs automatically (and use an Attendees file there to pick people to highlight), but got stuck on how to push... https://github.com/ajtowns/bitcoin-core-meetings
< wumpus> aj: yep non-protected branches would be fine; allow it to push to laanwj/bitcoin but not bitcoin/bitcoin master for example
< aj> wumpus: curl -s https://api.github.com/repos/bitcoin/bitcoin/pulls?"page=$a;per_page=100" | jshon -a -e number -u -p -e updated_at -u -p -e head -e sha -u -p -p -e base -e ref -u; done | paste -s -d ' \n'
< gribble> https://github.com/bitcoin/bitcoin/issues/10149 | gentlejack by gentlejack · Pull Request #10149 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10520 | 0.14 by gentlejack · Pull Request #10520 · bitcoin/bitcoin · GitHub
< gmaxwell> sipa: we do not want gmp as part of bitcoin's consensus critical paths, totally independently of using it as a dependency.
< BlueMatt> yea, ok, i meannnn, probably fine, but that is a real change in effective license of bitcoin core
< gribble> https://github.com/bitcoin/bitcoin/issues/10434 | [WIP] 3072-bit MuHash based hash_serialized by sipa · Pull Request #10434 · bitcoin/bitcoin · GitHub
< ryanofsky> Randolf maybe see https://github.com/bitcoin/bitcoin/pull/10244 which separates bitcoin gui code from wallet/node code
< Randolf> jonasschnelli: I've done a lot of Java development, and this is how JFC/Swing and the newer JavaFX handle things. The use of atomic variables and thread-safe queues becomes pretty important. Maybe the original GUI for Bitcoin was designed with more development convenience in mind just so that the
< gribble> https://github.com/bitcoin/bitcoin/issues/5896 | [Qt][PoC] introduce "core-pulse" by jonasschnelli · Pull Request #5896 · bitcoin/bitcoin · GitHub
< jonasschnelli> Bitcoin Core does a lot of things under the hood... and there is no way to see/control that
< gribble> https://github.com/bitcoin/bitcoin/issues/11826 | RFC: Activity feature · Issue #11826 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10267 | New -includeconf argument for including external configuration files by kallewoof · Pull Request #10267 · bitcoin/bitcoin · GitHub
< wumpus> currently if you define e.g. port or bind in bitcoin.conf you will get collisions when you run both testnet and mainnet on the same machine
< meshcollider> one idea I had was suggested in https://github.com/bitcoin/bitcoin/pull/10996#issuecomment-346189099, basically we default to using root-level bitcoin.conf and network specific network.conf if they exist, but if -conf is specified then we just use that and not the network specific one too
< gribble> https://github.com/bitcoin/bitcoin/issues/10267 | New -includeconf argument for including external configuration files by kallewoof · Pull Request #10267 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10996 | Add per-network config file network.conf by ajtowns · Pull Request #10996 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11826 | RFC: Activity feature · Issue #11826 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11800 | Bitcoin is returning higher fees for 36 block window than 2 block window (on testnet) · Issue #11800 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11363 | net: Split socket create/connect by theuni · Pull Request #11363 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11363 | net: Split socket create/connect by theuni · Pull Request #11363 · bitcoin/bitcoin · GitHub
< wumpus> #topic Bitcoin is returning higher fees for 36 block window than 2 block window (on testnet)
< gribble> https://github.com/bitcoin/bitcoin/issues/11800 | Bitcoin is returning higher fees for 36 block window than 2 block window (on testnet) · Issue #11800 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11824 | Block ActivateBestChain to empty validationinterface queue by TheBlueMatt · Pull Request #11824 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11824 | Block ActivateBestChain to empty validationinterface queue by TheBlueMatt · Pull Request #11824 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11824 | Block ActivateBestChain to empty validationinterface queue by TheBlueMatt · Pull Request #11824 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11839 | dont attempt mempool entry for wallet transactions on startup if alr… by instagibbs · Pull Request #11839 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10740 | [wallet] dynamic loading/unloading of wallets by jnewbery · Pull Request #10740 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11383 | Basic Multiwallet GUI support by luke-jr · Pull Request #11383 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11403 | SegWit wallet support by sipa · Pull Request #11403 · 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
< Guest82059> Mining of #bitcoin
< sipa> please, not here. go to #bitcoin
< sipa> #bitcoin
< belcher> wrong channel, try #bitcoin
< Guest82059> How can i do mining of bitcoin
< bitcoin-git> bitcoin/master 4ef4dfe Wladimir J. van der Laan: Merge #11847: Make boost::multi_index comparators const...
< bitcoin-git> [bitcoin] laanwj closed pull request #11847: Make boost::multi_index comparators const (master...2017-12-fix-const-comparator) https://github.com/bitcoin/bitcoin/pull/11847
< bitcoin-git> bitcoin/master 1ec0c0a Suhas Daftuar: Make boost::multi_index comparators const...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/80f9dad0b799...4ef4dfebbc07
< gribble> https://github.com/bitcoin/bitcoin/issues/11826 | RFC: Activity feature · Issue #11826 · bitcoin/bitcoin · GitHub
< darklegacy> can some one teach me how to mine bitcoin
< instagibbs> #bitcoin for chatter
< bitcoin-git> [bitcoin] laanwj closed pull request #11809: gui: Fix proxy setting options dialog crash (master...2017_12_gui_proxy_robustness) https://github.com/bitcoin/bitcoin/pull/11809
< bitcoin-git> bitcoin/master fa4c16d MarcoFalke: qa: Add getrawtransaction in_active_chain=False test
< bitcoin-git> bitcoin/master 3e50024 Wladimir J. van der Laan: Merge #11838: qa: Add getrawtransaction in_active_chain=False test...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7630a1fe9a4c...3e5002412002
< bitcoin-git> [bitcoin] sdaftuar opened pull request #11847: Make boost::multi_index comparators const (master...2017-12-fix-const-comparator) https://github.com/bitcoin/bitcoin/pull/11847
< wumpus> errors like /build/bitcoin-0.15.1/depends/arm-linux-androideabi/share/../lib/libboost_system-mt.a(error_code.o)(.ARM.extab+0xc): error: undefined reference to '__gxx_personality_v0' look scarier
< bitcoin-git> [bitcoin] wjcloud opened pull request #11845: Add gitian PGP key: wjcloud (master...p) https://github.com/bitcoin/bitcoin/pull/11845
< wumpus> (there's a similar consideration on whether to do flto on just bitcoin core or all the dependencies as well. Though that experiment was put on hold when it turned out flto somehow negatively affected performance, at least in a specific case)
< bitcoin-git> bitcoin/master 7630a1f Wladimir J. van der Laan: Merge #11829: Test datadir specified in conf file exists...
< bitcoin-git> bitcoin/master 529b866 MeshCollider: Test datadir in conf file exists
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/497d0e014cc7...7630a1fe9a4c
< wumpus> none of the dependencies that are not built into bitcoin's source tree are performance critical (maybe with the exception of boost, but that's mostly header only?)
< bitcoin-git> [bitcoin] laanwj closed pull request #11602: utils: removed deprecated check and function for OpenSSL compatiblity (master...old_openssl_names) https://github.com/bitcoin/bitcoin/pull/11602
< wumpus> agree, distro packages for bitcoin are a bad idea if they're not maintained
< gribble> https://github.com/bitcoin/bitcoin/issues/11388 | Building within termux fails with incomplete type CBlock · Issue #11388 · bitcoin/bitcoin · GitHub
< wumpus> FYI some people have also been building bitcoin core natively on android using termux (#11388), but that uses a debian derivative not NDK
< gribble> https://github.com/bitcoin/bitcoin/issues/11844 | Bitcoin core build on Android NDK · Issue #11844 · bitcoin/bitcoin · GitHub
< wumpus> so building bitcoin core on most recent ndk is an open issue?
< gribble> https://github.com/bitcoin/bitcoin/issues/11720 | iOS Deployment Target for RPC · Issue #11720 · bitcoin/bitcoin · GitHub
< wumpus> (it might in the future if someone adds support for the ARM crypto intrinsics to bitcoin)
< sipa> GAit: "--with-asm=arm --enable-experimental" are libsecp256k1 configure options; the " --enable-experimental-asm" is a bitcoin core configure option
< meshcollider> Usman_Mutawakil: the main() function for bitcoind is in bitcoind.cpp, for bitcoin-qt it is in qt/bitcoin.cpp and there is one in bitcoin-tx.cpp and bitcoin-cli.cpp for their respective binaries too

2017-12-06

< gribble> https://github.com/bitcoin/bitcoin/issues/3465 | [RFC] Post-0.9 network/protocol/main refactor · Issue #3465 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11829 | Test datadir specified in conf file exists by MeshCollider · Pull Request #11829 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11825 | When running bitcoind I keep getting - boost::filesystem::space: Operation not permitted · Issue #11825 · bitcoin/bitcoin · GitHub
< jonasschnelli> sipa: https://github.com/bitcoin/bitcoin/pull/11281/files#r154675029, does ReadBlockFromDisk requires cs_main? I don't think so.
< cluelessperson> is bitcoin core planning on implementing lightning built in?
< gribble> https://github.com/bitcoin/bitcoin/issues/11839 | dont attempt mempool entry for wallet transactions on startup if alr… by instagibbs · Pull Request #11839 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #11838: qa: Add getrawtransaction in_active_chain=False test (master...Mf1712-qaRpcRawTx) https://github.com/bitcoin/bitcoin/pull/11838
< Provoostenator> It should happen somewhere around here, is a row is inserted in the transaction table model: https://github.com/sipa/bitcoin/blob/e2e9ead25fe0ddb364e35f8eabb5a7f937d3973b/src/qt/walletview.cpp#L140
< gribble> https://github.com/bitcoin/bitcoin/issues/11403 | SegWit wallet support by sipa · Pull Request #11403 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #11741: Add -logdir option to control where debug.log lives (master...logdir) https://github.com/bitcoin/bitcoin/pull/11741
< hkjn0> hm, right. so I guess that L1289-1290 in bitcoin's configure.ac would be where to pass through the config flags to secp256k1: AC_CONFIG_SUBDIRS([src/secp256k1])
< wumpus> the only documentation of it is in ./configure. secp256's configure at that, not bitcoin's.
< Sentineo> I could not find a documentation of it, but if you check https://github.com/bitcoin-core/secp256k1/blob/master/configure.ac#L151 it is there
< gribble> https://github.com/bitcoin/bitcoin/issues/7759 | [WIP] rest: Stream entire utxo set by laanwj · Pull Request #7759 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] hkjn opened pull request #11836: Rename rpcuser.py to rpcauth.py (master...rename-rpcuser) https://github.com/bitcoin/bitcoin/pull/11836
< gribble> https://github.com/bitcoin/bitcoin/issues/11564 | [Qt] display fees in Sat / vByte instead of (μ/m)BTC/kB · Issue #11564 · bitcoin/bitcoin · GitHub
< wumpus> what version of bitcoin core?
< meshcollider> self-reminder i think we should discuss the config file situation at the meeting tomorrow, re https://github.com/bitcoin/bitcoin/pull/11829#issuecomment-349610380
< bitcoin-git> bitcoin/master 434526a Karl-Johan Alm: [test] Add tests for getrawtransaction with block hash.
< bitcoin-git> bitcoin/master b167951 Karl-Johan Alm: [rpc] Allow getrawtransaction to take optional blockhash to fetch transaction from a block directly.
< bitcoin-git> bitcoin/master a5f5a2c Karl-Johan Alm: [rpc] Fix fVerbose parsing (remove excess if cases).
< bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/a13e44385147...497d0e014cc7
< bitcoin-git> [bitcoin] laanwj closed pull request #11830: rpcuser.py: Use 'python' not 'python2' (master...rpcuser-py) https://github.com/bitcoin/bitcoin/pull/11830
< bitcoin-git> bitcoin/master a13e443 Wladimir J. van der Laan: Merge #11830: rpcuser.py: Use 'python' not 'python2'...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/5bea05bc1d17...a13e44385147
< bitcoin-git> bitcoin/master 6d2f277 Henrik Jonsson: rpcuser.py: Use 'python' not 'python2'
< jonasschnelli> Has anyone experimented with Jetson TX1 (NVIDIA) and Bitcoin Core?
< meshcollider> gurjeet: you should probably take this to #bitcoin not here
< gurjeet> I am looking bitcoin developer tatorial how i can setup bitcoin exchange
< jonasschnelli> BlueMatt: you wrote: "ReadBlockFromDisk requires cs_main" in https://github.com/bitcoin/bitcoin/pull/11281/files#r154681446
< gribble> https://github.com/bitcoin/bitcoin/issues/10275 | [rpc] Allow fetching tx directly from specified block in getrawtransaction by kallewoof · Pull Request #10275 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #11637: WIP: Remove dead service bits code (master...Mf1711-p2pDead) https://github.com/bitcoin/bitcoin/pull/11637

2017-12-05

< bitcoin-git> [bitcoin] sipa closed pull request #11834: [verify-commits] Fix gpg.sh's echoing for commits with '\n' (master...2017-12-verify-commits-fix) https://github.com/bitcoin/bitcoin/pull/11834
< bitcoin-git> bitcoin/master 5bea05b Pieter Wuille: Merge #11834: [verify-commits] Fix gpg.sh's echoing for commits with '\n'...
< bitcoin-git> bitcoin/master a38686c Matt Corallo: [verify-commits] Fix gpg.sh's echoing for commits with '\n'
< bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/91eeaa03354b...5bea05bc1d17
< spudowiar> sipa: Yeah, I'm using Emscripten and I have Bitcoin-Qt mostly working (except networking is disabled)
< spudowiar> Does --with-sysroot= work with Bitcoin Core?
< bitcoin-git> [bitcoin] practicalswift opened pull request #11835: Add Travis check for unused Python imports (master...lint-python) https://github.com/bitcoin/bitcoin/pull/11835
< BlueMatt> spudowiar: lots of folks seem like the "bitcoin has a p2p network, lets shove more features in it so that *everything* can use it!" approach, cause its easy, but it also doesnt scale....ultimately the bitcoin p2p network is full of garbage evil nodes anyway, its not magic...
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #11834: [verify-commits] Fix gpg.sh's echoing for commits with '\n' (master...2017-12-verify-commits-fix) https://github.com/bitcoin/bitcoin/pull/11834
< gribble> https://github.com/bitcoin/bitcoin/issues/11833 | [Net] WebSocket support · Issue #11833 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10773 | Shell script cleanups by practicalswift · Pull Request #10773 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #11831: Always return true if AppInitMain got to the end (master...2017-12-startup-exit-return-code-race) https://github.com/bitcoin/bitcoin/pull/11831
< bitcoin-git> [bitcoin] hkjn opened pull request #11830: rpcuser.py: Use 'python' not 'python2' (master...rpcuser-py) https://github.com/bitcoin/bitcoin/pull/11830
< gribble> https://github.com/bitcoin/bitcoin/issues/11829 | Test datadir specified in conf file exists by MeshCollider · Pull Request #11829 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] MeshCollider opened pull request #11829: Test datadir specified in conf file exists (master...201712_datadir_crash) https://github.com/bitcoin/bitcoin/pull/11829
< kallewoof> I guess I'll try linking directly and see how far that gets me. I suspect there will be a lot of issues where the bitcoin code expects a fully fledged bitcoin instance running.
< kallewoof> Hum. I could avoid a lot of reinventing wheels by simply requiring libbitcoin_[x].a in the indexer to link p2p stuff in. Potential minefield, but would be super easy to keep up to date with bitcoin core if it worked.
< kallewoof> And yeah, we agreed this should be an external thing that either uses hooks in bitcoin or implements the p2p protocol and talks to the network directly.
< gribble> https://github.com/bitcoin/bitcoin/issues/10275 | [rpc] Allow fetching tx directly from specified block in getrawtransaction by kallewoof · Pull Request #10275 · bitcoin/bitcoin · GitHub
< Randolf> esotericnonsense: For Bitcoin stuff, I'd create custom datatypes that save the data in binary format.
< kallewoof> I think having hooks in bitcoin core that let such a system exist separately is a good plan. It would only rely on API breakage, which bitcoin core is very careful about.
< kallewoof> I've been asked about address index on top of bitcoin core (to replace the unmaintained fork that exists somewhere else). What are people's opinions on this? I know some people prefer a minimal code base, but having an address index option seems like it would alleviate a lot of problems in a lot of places.
< bitcoin-git> [bitcoin] kallewoof closed pull request #11489: [wallet] sendtoaddress style argument (master...201709_segwitwallet2_sendtoaddress) https://github.com/bitcoin/bitcoin/pull/11489
< gribble> https://github.com/bitcoin/bitcoin/issues/11826 | RFC: Activity feature · Issue #11826 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11824 | Block ActivateBestChain to empty validationinterface queue by TheBlueMatt · Pull Request #11824 · bitcoin/bitcoin · GitHub
< BlueMatt> ls -lh ~/.bitcoin/debug.log
< BlueMatt> -rw------- 1 matt matt 6.5T Dec 4 19:45 /home/matt/.bitcoin/debug.log
< bitcoin-git> [bitcoin] TheBlueMatt opened pull request #11824: Block ActivateBestChain to empty validationinterface queue (master...2017-12-11822-debug) https://github.com/bitcoin/bitcoin/pull/11824

2017-12-04

< bitcoin-git> [bitcoin] practicalswift closed pull request #11799: wallet: Add compile-time checking of (non-)locking assumptions for BlockUntilSyncedToCurrentChain() [wip] (master...BlockUntilSyncedToCurrentChain-compile-time-warnings) https://github.com/bitcoin/bitcoin/pull/11799
< gribble> https://github.com/bitcoin/bitcoin/issues/11823 | Shouldnt this be nOut? · Issue #11823 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11799 | wallet: Add compile-time checking of (non-)locking assumptions for BlockUntilSyncedToCurrentChain() [wip] by practicalswift · Pull Request #11799 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9919 | UNIX sockets support for RPC by laanwj · Pull Request #9919 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/9979 | p2p: Bare minimum to support UNIX sockets by laanwj · Pull Request #9979 · bitcoin/bitcoin · GitHub
< GAit> bitcoin core rpc. Using a socket file rather than a port
< BlueMatt> achow101: can you try https://github.com/TheBlueMatt/bitcoin/tree/2017-12-11822-debug and see what debug log looks like?
< gribble> https://github.com/bitcoin/bitcoin/issues/11822 | Severe memory leak on current master · Issue #11822 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10773 | Shell script cleanups by practicalswift · Pull Request #10773 · bitcoin/bitcoin · GitHub
< BlueMatt> sipa: #11403 looks to have failed travis to to a bad iterator deref ( https://travis-ci.org/bitcoin/bitcoin/jobs/309801879#L2838 )
< gribble> https://github.com/bitcoin/bitcoin/issues/11403 | SegWit wallet support by sipa · Pull Request #11403 · bitcoin/bitcoin · GitHub
< notabot_> RK_: bitcoin.org is the best place to start.
< notabot_> mmhhtt: try #bitcoin channel or reddit
< mmhhtt> #bitcoin-core-dev
< RK_> I want to know about bitcoin
< mmhhtt> Hello, Is there anyone knows to how can build a exchange market for other country? Is there any request from bitcoin company to me for making exc market?
< gribble> https://github.com/bitcoin/bitcoin/issues/11822 | Severe memory leak on current master · Issue #11822 · bitcoin/bitcoin · GitHub
< achow101> Varunram: bitcoin.conf, not gui
< gribble> https://github.com/bitcoin/bitcoin/issues/11788 | Bitcoin-Qt (Settings) reports incorrect "Size of database cache" · Issue #11788 · bitcoin/bitcoin · GitHub
< Randolf> These servers don't have any GUI stuff installed. So I won't be able to try bitcoin-qt on them. But I could at least try bitcoin-qt on Windows 10 to confirm whether the same problem occurs cross-platform.
< achow101> also, i'm running bitcoin-qt right now, not bitcoind
< xRavenheart> Is anyone able to first provide me with a clear architecture diagram of bitcoin?
< xRavenheart> E.g. For instance Bitcoin Core has optimizations 1,2 and Electrum has optimizations 3,4
< sipa> xRavenheart: you're welcome here to ask questions about bitcoin's design and operation
< Randolf> Dizzle: Ah, yes. I suggested that asking here may be appropriate since it might involve core development. I also suggested that they ask in the #bitcoin and ##altcoins channel.
< Dizzle> Randolf: he's here, rejoined. I think luke-jr was just suggesting he ask elsewhere, e.g. #bitcoin
< sipa> 02:47:32 < Randolf> xRavenheart: What's your school project about? Bitcoin development?
< Randolf> xRavenheart: What's your school project about? Bitcoin development?

2017-12-03

< gribble> https://github.com/bitcoin/bitcoin/issues/11799 | wallet: Add compile-time checking of (non-)locking assumptions for BlockUntilSyncedToCurrentChain() [wip] by practicalswift · Pull Request #11799 · bitcoin/bitcoin · GitHub

2017-12-02

< gribble> https://github.com/bitcoin/bitcoin/issues/11045 | Offering my Bitcoin fuzzers · Issue #11045 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] jnewbery opened pull request #11817: [tests] Change bip68-112-113-p2p to use BitcoinTestFramework (master...refactor_bip68-112-113-p2p) https://github.com/bitcoin/bitcoin/pull/11817
< webuser323> Just wondering if this is relevant bitcoin-core: https://github.com/google/oss-fuzz/
< jimpo> luke-jr: That makes sense conceptually, but I don't understand make tools well enough to understand why. From what is looks like here, https://github.com/bitcoin/bitcoin/blob/master/src/Makefile.test.include#L105, the test is linked against the LIBBITCOIN_SERVER sources, which includes init.cpp.
< luke-jr> jimpo: likely they're never linked together.. test_bitcoin_main would be linked into test_bitcoin, and init.cpp into bitcoind/bitcoin-qt

2017-12-01

< Provoostenator> (feel free to answer in #bitcoin instead)
< Provoostenator> Varunram: I suppose I can try it for a day. Which begs the question how: https://bitcoin.stackexchange.com/questions/51169/how-can-i-solo-mine-using-bitcoin-core
< gribble> https://github.com/bitcoin/bitcoin/issues/8994 | Testchains: Introduce custom chain whose constructor... by jtimon · Pull Request #8994 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8994 | Testchains: Introduce custom chain whose constructor... by jtimon · Pull Request #8994 · bitcoin/bitcoin · GitHub
< Provoostenator> Tangentally related to testing the segwit branch. It looks like miners are no longer mining SegWit transactions on testnet. See my example in #bitcoin.
< bitcoin-git> bitcoin/master 00d25e9 Wladimir J. van der Laan: Merge #11804: [docs] Fixed outdated link with archive.is...
< bitcoin-git> bitcoin/master bf20a7d Tim Shimmin: [docs] Fixed outdated link with archive.is...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/0d7e0a3289ba...00d25e90db06
< bitcoin-git> bitcoin/master 0d7e0a3 Wladimir J. van der Laan: Merge #11337: Fix code constness in CBlockIndex::GetAncestor() overloads...
< bitcoin-git> bitcoin/master b4058ed Dan Raviv: Fix code constness in CBlockIndex::GetAncestor() overloads...
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/13e31dd6548d...0d7e0a3289ba
< wumpus> don't think Vision is still here but: https://github.com/bitcoin/bitcoin/pull/11809 should fix his issue
< gribble> https://github.com/bitcoin/bitcoin/issues/11403 | SegWit wallet support by sipa · Pull Request #11403 · bitcoin/bitcoin · GitHub

2017-11-30

< meshcollider> why would this validator return true for an empty string: https://github.com/bitcoin/bitcoin/blob/master/src/qt/optionsdialog.cpp#L337
< wumpus> the code here https://github.com/bitcoin/bitcoin/blob/master/src/qt/optionsmodel.cpp#L231 for ProxyIP and ProxyPORT needs to catch the condition where the list is empty, or not long enough
< wumpus> run bitcoin-qt with -resetguisettings
< wumpus> (of bitcoin core)
< meshcollider> Vision: in that case, the options will be in the registry, under HKEY_CURRENT_USER\Software\Bitcoin\Bitcoin-Qt iirc
< Vision> where are the proxy settings stored for bitcoin-qt? I cleared the proxy settings in the Options dialog, and now entering the settings dialog causes a crash. definitely a bug.
< bitcoin-git> bitcoin/master fbce66a MarcoFalke: Merge #10493: Use range-based for loops (C++11) when looping over map elements...
< bitcoin-git> bitcoin/master 680bc2c practicalswift: Use range-based for loops (C++11) when looping over map elements...
< bitcoin-git> [bitcoin] MarcoFalke pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9e38d357447e...fbce66a98267
< gmaxwell> and in the short term apply some mitigations in bitcoin, like increasing the FD count (ugh but I really don't like counting on FD magnitude sniffing)
< bitcoin-git> [bitcoin] luke-jr opened pull request #11803: Bugfix: RPC/Wallet: Include HD key metadata in dumpwallet (master...bugfix_dumpwallet_hdkeypath) https://github.com/bitcoin/bitcoin/pull/11803
< gribble> https://github.com/bitcoin/bitcoin/issues/11785 | Prevent file-descriptor exhaustion from RPC layer by vii · Pull Request #11785 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11227 | WIP: switch to libevent for node socket handling by theuni · Pull Request #11227 · bitcoin/bitcoin · GitHub
< Dizzle> cfields: this abstraction on libevent or bitcoin core?
< gmaxwell> I believe bluematt wrote a very small patch to change bitcoin to poll. We could take, that and wrap it in ifdefs so we still select on windows, and call that sub-issue done.
< wumpus> Dizzle: I don't think it's awfully useful in bitcoin-cli anyhow though it's useful for testing (though OTOH, the patch also changed the RPC testing framework to be able to use UNIX sockets)
< Dizzle> BTW, if anyone's curious how Electrum implements message signatures on native segwit addrs, we just iterate through the possible address types until there's match: https://github.com/spesmilo/electrum/blob/66cce115ef93c913d65ef5c7d781c8065f79bbaf/lib/bitcoin.py#L632
< wumpus> Dizzle: do you need it to be in bitcoin-cli for that?
< gribble> https://github.com/bitcoin/bitcoin/issues/10279 | Add a CChainState class to validation.cpp to take another step towards clarifying internal interfaces by TheBlueMatt · Pull Request #10279 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11785 | Prevent file-descriptor exhaustion from RPC layer by vii · Pull Request #11785 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10279 | Add a CChainState class to validation.cpp to take another step towards clarifying internal interfaces by TheBlueMatt · Pull Request #10279 · bitcoin/bitcoin · GitHub
< aj> do people think the +1, +0, -1, concept nak/ack thing from https://github.com/bitcoin/bitcoin/pull/11426#issuecomment-334091207 is good btw?
< gmaxwell> Provoostenator: that can be resolved with a better perspective; bitcoin development is a distributed process, your effort is your contribution not the fact that it was merged, etc.
< gribble> https://github.com/bitcoin/bitcoin/issues/8994 | Testchains: Introduce custom chain whose constructor... by jtimon · Pull Request #8994 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11699 | [travis-ci] Only run linters on Pull Requests by jnewbery · Pull Request #11699 · bitcoin/bitcoin · GitHub
< jonasschnelli> Two topic proposals: 1) review "nits" reduction, 2) bitcoin core code signing assoc.
< gribble> https://github.com/bitcoin/bitcoin/issues/11415 | [RPC] Disallow using addresses in createmultisig by achow101 · Pull Request #11415 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10579 | [RPC] Split signrawtransaction into wallet and non-wallet RPC command by achow101 · Pull Request #10579 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10583 | [RPC] Split part of validateaddress into getaddressinfo by achow101 · Pull Request #10583 · bitcoin/bitcoin · GitHub
< gmaxwell> same reason sighash flags can fail valid in bitcoin: If I pick an out of range sighash flag I could forge signatures for your pubkeys.
< gmaxwell> prior to having the explicit versions we couldn't do that, which is why we never even merged the patch in elements, much less brought it to bitcoin.
< wumpus> using bitcoin keys for anything else than signing transactions continues to make me uneasy
< sipa> the original bitcoin core message signing doesn't have a bip either
< karelb> sipa: ad sign message - in trezor we changed v constant - https://github.com/bitcoin/bitcoin/issues/10542#issuecomment-316032523 - no BIP, no time :(
< gribble> https://github.com/bitcoin/bitcoin/issues/10757 | RPC: Introduce getblockstats to plot things by jtimon · Pull Request #10757 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/8994 | Testchains: Introduce custom chain whose constructor... by jtimon · Pull Request #8994 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11403 | SegWit wallet support by sipa · Pull Request #11403 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11403 | SegWit wallet support by sipa · Pull Request #11403 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/10279 | Add a CChainState class to validation.cpp to take another step towards clarifying internal interfaces by TheBlueMatt · Pull Request #10279 · bitcoin/bitcoin · GitHub
< wumpus> 8 PRs have been tagged for https://github.com/bitcoin/bitcoin/projects/8
< 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
< saint_> any idea why can't bitcoin core download the blockchain on a remote NAS drive ?
< wumpus> saint_: ask in #bitcoin please, will answer there
< gribble> https://github.com/bitcoin/bitcoin/issues/11796 | [tests] Functional test naming convention by ajtowns · Pull Request #11796 · bitcoin/bitcoin · GitHub
< andytoshi> #bitcoin please, this is just about software development
< wordsToLiveBy> I'm reading the bitcoin white paper, and in section 11 Satoshi gives the calculations for how you can prove with probability that an attacker can't outpace the combined processing power of the rest of the nodes, but I am not quite getting the math being used.
< gribble> https://github.com/bitcoin/bitcoin/issues/11796 | [tests] Functional test naming convention by ajtowns · Pull Request #11796 · bitcoin/bitcoin · GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/11774 | [WIP] [tests] Rename functional tests by ajtowns · Pull Request #11774 · bitcoin/bitcoin · GitHub
< bitcoin-git> [bitcoin] laanwj closed pull request #11793: Docs: Bump OS X version to 10.13 (master...patch-1) https://github.com/bitcoin/bitcoin/pull/11793
< bitcoin-git> [bitcoin] laanwj closed pull request #9254: [depends] ZeroMQ 4.2.2 (master...zeromq-4-2-0) https://github.com/bitcoin/bitcoin/pull/9254
< bitcoin-git> [bitcoin] Varunram opened pull request #11793: Docs: Bump OS X version to 10.13 (master...patch-1) https://github.com/bitcoin/bitcoin/pull/11793

2017-11-29

< bitcoin-git> [bitcoin] aaron-hanson opened pull request #11792: Trivial: fix comments for ZeroMQ bitcoind args (master...trivial-zeromq-arguments-comments) https://github.com/bitcoin/bitcoin/pull/11792