< GitHub106>
[bitcoin] jrmithdobbs closed pull request #7243: This community NEEDS to adopt a code of conduct. Recent events make t… (master...code_of_conduct) https://github.com/bitcoin/bitcoin/pull/7243
< jcorgan>
blade: this is not the correct channel for this, please go to #bitcoin
< blade>
im useig the bitcoin core wallet
2015-12-21
< GitHub71>
[bitcoin] jrmithdobbs opened pull request #7243: This community NEEDS to adopt a code of conduct. Recent events make t… (master...code_of_conduct) https://github.com/bitcoin/bitcoin/pull/7243
< GitHub85>
[bitcoin] jtimon closed pull request #6625: BLOCKING: Consensus: Move blocksize and related parameters to consensusparams ...without removing consensus/consensus.h [#6526 alternative] (master...consensus-blocksize-0.12.99) https://github.com/bitcoin/bitcoin/pull/6625
< GitHub159>
[bitcoin] jtimon opened pull request #7238: Blocksize: Some small preparations for a blocksize hardfork (master...6526-6625-remainings-0.13.99) https://github.com/bitcoin/bitcoin/pull/7238
2015-12-20
< GitHub4>
[bitcoin] dgenr8 opened pull request #7236: Use createrawtx locktime parm in txn_clone (master...use_rpc_locktime_clone) https://github.com/bitcoin/bitcoin/pull/7236
< maaku>
that's effectively useless given how bitcoin development works...
< jonasschnelli>
Yes. We kept bitcoin-qt because otherwise new versions would not overwrite old ones.
< jonasschnelli>
cfields: Luke-Jr's name unify PR works really good. We might want to fix the osx application name ("Bitcoin-Qt").
< GitHub173>
[bitcoin] sdaftuar opened pull request #7225: Eliminate unnecessary call to CheckBlock (master...eliminate-extra-checkblock) https://github.com/bitcoin/bitcoin/pull/7225
< cfields>
Luke-Jr: what's missing in the bitcoin build itself? the python libs?
< wumpus>
<jgarzik> benjyz1, #bitcoin-dev please
< wumpus>
off topic things not related to bitcoin development, If it fits into the above category, you're a legit contributor and somehow got 'censored' feel free to send a complaint to me
< jonasschnelli>
Luke-Jr: your TestyCOIN-Crazy projects still uses "Bitcoin-Qt.app" as filename (as you mentioned). But I guess the whole dmg process should also work if we would use a different filename? right?
< Luke-Jr>
oh, it works because we don't actually use "Bitcoin Core" in filenames
< Luke-Jr>
looking at the osx descriptor, it seems to assume bitcoin-* prefix in the filename
< GitHub197>
bitcoin/0.12 f3ad812 Wladimir J. van der Laan: test: don't override BITCOIND and BITCOINCLI if they're set...
< GitHub68>
[bitcoin] sdaftuar opened pull request #7222: [WIP] RPC: Indicate which transactions are signaling opt-in RBF (master...add-optin-info) https://github.com/bitcoin/bitcoin/pull/7222
< GitHub4>
[bitcoin] luke-jr closed pull request #7220: RBF: Allow replacements to pay for minRelayFee(replaced)+minRelayFee(replacement) rather than actualFee(replaced)+minRelayFee(replacement) (master...rbf_feecomp) https://github.com/bitcoin/bitcoin/pull/7220
< helo>
someone familiar with p2pool code may want to take a look at KipIngram's messages in #bitcoin
< jonasschnelli>
Luke-Jr: hmm... IPv6 seems to be set up coorect. dig AAAA bitcoin.jonasschnelli.ch gives me the right answer, .. i can ping6 from another machine (in the same lan although).
< GitHub143>
[bitcoin] luke-jr opened pull request #7220: RBF: Allow replacements to pay for minRelayFee(replaced)+minRelayFee(replacement) rather than actualFee(replaced)+minRelayFee(replacement) (master...rbf_feecomp) https://github.com/bitcoin/bitcoin/pull/7220
< BlueMatt>
(pool servers are largely bit proprietary blobs that get essentially no external review, whereas bitcoin core is....not that)
2015-12-15
< GitHub85>
[bitcoin] sdaftuar opened pull request #7217: Mark blocks with too many sigops as failed (master...fix-sigops-rejection) https://github.com/bitcoin/bitcoin/pull/7217
< sipa>
#bitcoin-dev is more for development of the protocol
< sipa>
This is specifically about the Bitcoin Core software, and its implementation details
< desantis>
honest question: How does this channel differ from bitcoin-dev?
< GitHub92>
[bitcoin] MarcoFalke opened pull request #7214: qt5: Use the fixed font the system recommends (master...MarcoFalke-2015-qt5monospace) https://github.com/bitcoin/bitcoin/pull/7214
< sdaftuar>
hm. does bitcoin core use the container-based environment?
< sdaftuar>
is there anything special that needs to be done to make travis work well on personal repos? i enabled travis on my fork of the project so i could test branches before opening PRs, but they fail a lot for strange reasons (even when the same branch succeeds as part of a bitcoin core PR)
< GitHub156>
[bitcoin] EthanHeilman opened pull request #7212: Adds unittests for CAddrMan and CAddrinfo, removes source of non-determinism. (master...unittest) https://github.com/bitcoin/bitcoin/pull/7212
< GitHub158>
[bitcoin] laanwj opened pull request #7209: test: don't override BITCOIND and BITCOINCLI if they're set (master...2015_12_rpctests_setenviron) https://github.com/bitcoin/bitcoin/pull/7209
< GitHub12>
[bitcoin] laanwj opened pull request #7208: Make max tip age an option instead of chainparam (master...2015_12_maxtipage) https://github.com/bitcoin/bitcoin/pull/7208
< GitHub92>
[bitcoin] laanwj closed pull request #7200: Checks for null data transaction before issuing error to debug.log (master...null-tx-debug) https://github.com/bitcoin/bitcoin/pull/7200
< GitHub149>
bitcoin/master d812daf accraze: fix logic for error log
< GitHub149>
bitcoin/master c611acc accraze: wallet: check if tx scriptPubKey is unspendable
< GitHub149>
bitcoin/master b6915b8 accraze: checks for null data transaction before debug.log...
< GitHub60>
[bitcoin] laanwj closed pull request #7068: [RPC-Tests] add simple way to run rpc test over QT clients (master...2015/11/rpc_tests_qt) https://github.com/bitcoin/bitcoin/pull/7068
< GitHub91>
bitcoin/master 5f3c670 Wladimir J. van der Laan: Merge pull request #7068...
< GitHub91>
bitcoin/master 979698c Jonas Schnelli: [RPC-Tests] add option to run rpc test over QT clients
< GitHub104>
[bitcoin] laanwj closed pull request #7185: Note that reviewers should mention the id of the commits they reviewed. (master...2015-12-07-contributingackcommit) https://github.com/bitcoin/bitcoin/pull/7185
< GitHub69>
bitcoin/master ea0f5a2 Wladimir J. van der Laan: Merge pull request #7185...
< GitHub69>
bitcoin/master e1030dd Patrick Strateman: Note that reviewers should mention the commit hash of the commits they reviewed.
< Luke-Jr>
An error occurred during a connection to bitcoin.jonasschnelli.ch. SSL received a record that exceeded the maximum permissible length. (Error code: ssl_error_rx_record_too_long)
< GitHub16>
[bitcoin] jonasschnelli closed pull request #7206: Add "NODE_BLOOM" to guiutil so that peers don't get UNKNOWN[4] (master...2015_12_bloom_gui) https://github.com/bitcoin/bitcoin/pull/7206
< GitHub4>
bitcoin/master b7c704a Jonas Schnelli: Merge pull request #7206...
< GitHub4>
bitcoin/master daf6466 Matt Corallo: Add "NODE_BLOOM" to guiutil so that peers don't get UNKNOWN[4]
< GitHub39>
[bitcoin] TheBlueMatt opened pull request #7206: Add "NODE_BLOOM" to guiutil so that peers don't get UNKNOWN[4] (master...2015_12_bloom_gui) https://github.com/bitcoin/bitcoin/pull/7206
< btcdrak>
Luke-Jr: Yes, we must actively say that. Bitcoin Core devs (as I have said repeatedly in private) are playing into the hands of disruption by being too politically correct and "nice". We need to be prepared to call a spade a spade.
< btcdrak>
Luke-Jr: "Bitcoin Core" as a phrase is probably trademarkable, but I really dont see the benefit.
< midnightmagic>
what is bitcoin's licence anyway.. mit/bsd?
< CodeShark>
if phantomcircuit is specifically referring to actions taken by someone on "behalf" of Bitcoin Core, I think the policy needs to be specifically directed at that...and we need to come out with one voice saying that someone doesn't represent Bitcoin Core
< btcdrak>
gavinandresen: you dont contribute to Bitcoin Core just to creating chaos. While others might be more polite, I'm going right ahead and saying this, please go away, you are not wanted here.
< GitHub115>
[bitcoin] pstratem closed pull request #7104: [Mining] Build empty block on when chainTip changes. (master...2015-11-26-gbt-latency) https://github.com/bitcoin/bitcoin/pull/7104
< GitHub101>
[bitcoin] pstratem closed pull request #7150: Print size of pcoinsTip cache in AcceptToMemoryPool (master...2015-12-1-printcachesize) https://github.com/bitcoin/bitcoin/pull/7150
< GitHub178>
[bitcoin] pstratem closed pull request #7203: [WIP][Mining]Release cs_main in CreateNewBlock while selecting transactions from mempool. (master...2015-12-12-createnewblocklocks) https://github.com/bitcoin/bitcoin/pull/7203
< gmaxwell>
gavinandresen: Hi. I'm concerned that I keep hearing reports of you taking actions on behalf of Bitcoin Core while not actually communicating with anyone involved in the project, or being involved yourself. I wasn't sure of how much of this was rumor or miscommunication, but this appears to be you directly confirming it:
< GitHub0>
[bitcoin] pstratem opened pull request #7203: Release cs_main in CreateNewBlock while selecting transactions from mempool. (master...2015-12-12-createnewblocklocks) https://github.com/bitcoin/bitcoin/pull/7203
< gmaxwell>
Basically the system currently encourages you to play block arrival time chicken. Pay massively low fees, get mined fast. hurray! ... next time, mined slowly "Boo bitcoin is broken!"
< morcos>
the idea being that before the block has decent size that from a long term perspective you need to be adding at least some txs or bitcoin is useless, but once it gets of a decent size, then you need to worry more about what the profit maximizing strategy is?
< gmaxwell>
Thats what Bitcoin did in 2012 and before, and I think the actual implementation wasn't a good path. It's weird to have such a huge amount of statefulness regarding when transactions come in.
< GitHub130>
[bitcoin] accraze opened pull request #7200: Checks for null data transaction before issuing error to debug.log (master...null-tx-debug) https://github.com/bitcoin/bitcoin/pull/7200
< gmaxwell>
So what bitcoin did pre-2012 was more reasonable; in the sense that it provided gradual back pressure, but it depended on future state and it was strangely order dependant. (e.g. first txn into a block could go in with low fees, then higher paying things were excluded later).
2015-12-11
< gmaxwell>
I've been giving some though to bringing back the tiered fee behavior bitcoin had untul mid 2012. The earlier logic had a rate table where transactions were added in arrival order, with the required fee gradually going up as the non-fee-sorted candidate block became bigger. In 2012 it was changed to the sorting based selection on the basis that this was the rational behavior to miners, and the '
< gmaxwell>
the whole coinselector in bitcoin core appears to have been written with a strong assumption that an address isn't used more than once. :(
< wumpus>
at last I prefer not including unnecessary commits, I admit other people may have different opinions, but that's the annoying thing about having one wallet implementation bound to bitcoin core
< Luke-Jr>
do you know if src/qt/res/bitcoin-qt-res.rc can use PRODUCT_NAME?
< Luke-Jr>
jonasschnelli: re isolating "Bitcoin Core" to fewer places.. there's 8 instances in the .ui files; I could move those to the equivalent .cpp, but it would look not-so-nice in Designer; thoughts?
< GitHub35>
[bitcoin] luke-jr opened pull request #7192: Unify product name to as few places as possible without major changes (master...single_prodname) https://github.com/bitcoin/bitcoin/pull/7192
< Luke-Jr>
let's rename to Nucleul Bitcoin in all languages. it sounds nicer.
< jonasschnelli_>
nucleul sounded after a reasonable Bitcoin wallet name...:-)
< wumpus>
jonasschnelli_: hehe in Dutch it's called "Bitcoin Kern"
< jonasschnelli_>
Maybe my lol was inappropriate (I thought he is using a different wallet (Nucleul Bitcoin versiunea v0.11.0 (32-bit)), but looks like core. Could be that the user really face an issue/bug.
< GitHub51>
[bitcoin] laanwj closed pull request #7154: [Qt] add InMempool() info to transaction details (master...2015/12/qt_conflicts) https://github.com/bitcoin/bitcoin/pull/7154
< GitHub113>
bitcoin/master 00b4b8d Wladimir J. van der Laan: Merge pull request #7154...
< dcousens>
those bastards, lol, yeah no worries, jsut wanted to quickly compile something and realised I had to put include "config/bitcoin-config.h" in a bunch of files just to stop it exploding
< GitHub148>
[bitcoin] pstratem opened pull request #7185: Note that reviewers should mention the id of the commits they reviewed. (master...2015-12-07-contributingackcommit) https://github.com/bitcoin/bitcoin/pull/7185
< GitHub173>
[bitcoin] Xekyo opened pull request #7183: Removed obsolete lines of code, broke continuation of loop. (master...Fix-#7182) https://github.com/bitcoin/bitcoin/pull/7183
< GitHub25>
[bitcoin] laanwj opened pull request #7181: net: Add and document network messages in protocol.h (master...2015_12_network_messages) https://github.com/bitcoin/bitcoin/pull/7181
< GitHub38>
[bitcoin] laanwj opened pull request #7180: net: Account for `sendheaders` `verack` messages (master...2015_12_log_sendheaders_verack) https://github.com/bitcoin/bitcoin/pull/7180
< GitHub92>
[bitcoin] laanwj opened pull request #7179: net: Fix sent reject messages for blocks and transactions (master...2015_12_fix_reject_message) https://github.com/bitcoin/bitcoin/pull/7179
< GitHub85>
bitcoin/master dc0305d Wladimir J. van der Laan: Merge pull request #6589...
< dcousens>
what do y'all get for "bitcoin-cli getrawtransaction 4a5e1e4baab89f3a32518a88c31bc87f618f76673e2cc77ab2127b7afdeda33b"
2015-12-06
< GitHub123>
[bitcoin] MarcoFalke opened pull request #7177: [qa] Change default block priority size to 0 (master...MarcoFalke-2015-rpcNoPrio) https://github.com/bitcoin/bitcoin/pull/7177
2015-12-05
< GitHub32>
[bitcoin] gmaxwell closed pull request #7174: Don't do mempool lookups for "mempool" command without a filter (master...2015-12-braindead-mempool) https://github.com/bitcoin/bitcoin/pull/7174
< gmaxwell>
AFAIK bitcoin core is the only wallet thats ever supported priority on the send side, and it's never even optimized for using priority wisely.
< paveljanik>
gmaxwell, using this argument, this calls for a lot of NEW users to bitcoin to call for "restart" of the network...
< phantomcircuit>
Luke-Jr, either fees become a significant portion of the total block reward and miners acting in their own self interest ignore priority, or we have massively misjudged the economic rational for mining and bitcoin will fail
< Luke-Jr>
believe it or not, most miners *do* care to some degree about Bitcoin's success
< GitHub43>
[bitcoin] TheBlueMatt opened pull request #7174: Don't do mempool lookups for "mempool" command without a filter (master...2015-12-braindead-mempool) https://github.com/bitcoin/bitcoin/pull/7174
< GitHub164>
bitcoin/master 7922592 Wladimir J. van der Laan: Merge pull request #7166...
< GitHub147>
[bitcoin] laanwj closed pull request #7166: Disconnect on mempool requests from peers when over the upload limit. (master...mempool_p2p_when_overlimit) https://github.com/bitcoin/bitcoin/pull/7166
< GitHub164>
bitcoin/master 6aadc75 Gregory Maxwell: Disconnect on mempool requests from peers when over the upload limit....
< GitHub68>
[bitcoin] jameshilliard opened pull request #7167: Move TestBlockValidity into a background thread (master...TBVBackground) https://github.com/bitcoin/bitcoin/pull/7167
2015-12-03
< GitHub29>
[bitcoin] gmaxwell opened pull request #7166: Disconnect on mempool requests from peers when over the upload limit. (master...mempool_p2p_when_overlimit) https://github.com/bitcoin/bitcoin/pull/7166
< jtimon>
the code is in jtimon/libconsensus-f2 (pre-bitcoin-jt) anyway
< jtimon>
and I don't close it because you have objections to it, just because I'm not going to rebase it as is if it's incompatible with building only once, which I also want for bitcoin jt
< GitHub142>
[bitcoin] laanwj opened pull request #7165: WIP: build: Enable C++11 in build, require C++11 compiler (master...2015_12_c++11) https://github.com/bitcoin/bitcoin/pull/7165
< GitHub156>
[bitcoin] ptschip opened pull request #7164: Do not download transactions during initial blockchain sync (master...tx_getdata) https://github.com/bitcoin/bitcoin/pull/7164