< CubicEarths> When starting Core for the first time, there is an option about where to put the data directory. It seems like that would be a good time to ask about a few other options, such as pruning and dbcache size, and create a .conf file with those settings.
< CubicEarths> The settings can be adjusted through the GUI after the program starts, but it needs to be restarted from them take effect...
< luke-jr> CubicEarths: Knots already does the pruning bit, FWIW
< luke-jr> would be nice to get it merged, but review is slow..
< luke-jr> (the pending PR step is #11082)
< gribble> https://github.com/bitcoin/bitcoin/issues/11082 | Add new bitcoin_rw.conf file that is used for settings modified by this software itself by luke-jr · Pull Request #11082 · bitcoin/bitcoin · GitHub
< CubicEarths> luke-jr: yeah. I was mainly thinking about the settings that a) are relevant to initial syncing and b) need restarting to activate
< Randolf> It was good to see so many Pull Requests merged and closed today.
< bitcoin-git> [bitcoin] jerryldh opened pull request #12609: 陈独秀同志,你请讲! (master...master) https://github.com/bitcoin/bitcoin/pull/12609
< * Randolf> laughs
< bitcoin-git> [bitcoin] jerryldh closed pull request #12609: 陈独秀同志,你请讲! (master...master) https://github.com/bitcoin/bitcoin/pull/12609
< Randolf> The changed file in that Chinese Pull Request seemed to be about creating a fork of Bitcoin and then changing just one line of code. Closing it without merging was the correct decision.
< ossifrage> I'm trying to track down a QT UI bug. The 0.16.0 release is fine, but what I build from the head as the problem. Where can I find the exact config options used to make the release binaries?
< ken2812221> ossifrage: ./configure --help
< ossifrage> ken2812221, I know how to use configure, but I don't know what is used to build the release builds, the ldd output for the release binary is fairly minimal, but my build is huge (60 vs 18)
< ken2812221> you can take a look at contrib/gitian-descriptors/gitian-*.yml ,this is the script for release build
< ossifrage> ken2812221, thanks, building with the config options from gitian (doubt it is going to help, most likely a local qt problem)
< bitcoin-git> [bitcoin] jonasschnelli opened pull request #12610: Multiwallet for the GUI (master...2018/03/multiwallet) https://github.com/bitcoin/bitcoin/pull/12610
< jonasschnelli> ossifrage: see my answer on the issue...
< jonasschnelli> ossifrage: the release builds use the dependency compile environment (check the depends/ folder)
< jonasschnelli> "relesse builds" == "official binaries"
< jonasschnelli> Most stuff is linked static there... Qt5.7 is used and linked static (you won't see it in ldd)
< ossifrage> jonasschnelli, I'm using qt5.9.4 (fedora27)
< jonasschnelli> ossifrage: and you have the issue when compiling against qt5.9.4?
< jonasschnelli> please doublecheck with ldd
< ossifrage> Yeah the debug peer list column resize thing is with 5.9.4 (Can't verify with ldd, because I did a 'make clean', building the 0.16.0 tag)
< ossifrage> Yup libQt5Core.so -> libQt5Core.so.5 -> libQt5Core.so.5.9.4
< jonasschnelli> ossifrage: your having the issue with Qt5.9.4 only? But not with the static Qt5.7 from our depends build... strange
< jonasschnelli> txindex: is there a reason why we do not allow -txindex with -prune?
< jonasschnelli> Example: you only interested to index stuff > 1.1.2017 (I know hard with the current MB based prune target)
< ossifrage> The problem is not specific to the head, it also happens with the 0.16.0 tag (using qt5.9.4). I haven't gotten the depends to build, need to try again later
< jonasschnelli> ossifrage: its probably very likely that you will your column issue with older tags
< ossifrage> jonasschnelli, yup
< bitcoin-git> [bitcoin] practicalswift opened pull request #12611: consensus: Use safer casting in IsValidSignatureEncoding(...) (master...safer-casting-in-IsValidSignatureEncoding) https://github.com/bitcoin/bitcoin/pull/12611
< jonasschnelli> coredev.tech is currently down due to server hardware issue, sry for the bad timing
< jonasschnelli> broken CPU fan
< jonasschnelli> which was just fixed... so nm, it's running again
< bitcoin-git> [bitcoin] laanwj pushed 1 new commit to master: https://github.com/bitcoin/bitcoin/commit/7deba93bdc76616011a9f493cbc203d60084416f
< bitcoin-git> bitcoin/master 7deba93 Wladimir J. van der Laan: test: Update trust git root...
< wumpus> jonasschnelli: thanks for fixing!
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #12606: [depends] Revert "ccache 3.4.1" (master...Mf1803-dependsCCache) https://github.com/bitcoin/bitcoin/pull/12606
< bitcoin-git> [bitcoin] ken2812221 opened pull request #12615: doc: SIGNER can contains space inside now. (master...patch-1) https://github.com/bitcoin/bitcoin/pull/12615
< bitcoin-git> [bitcoin] promag opened pull request #12616: Set modal overlay hide button as default (master...2018-03-modaloverlay) https://github.com/bitcoin/bitcoin/pull/12616
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7deba93bdc76...9903537750ac
< bitcoin-git> bitcoin/master 741f017 Evan Klitzke: Add DynamicMemoryUsage() to LevelDB...
< bitcoin-git> bitcoin/master 9903537 Wladimir J. van der Laan: Merge #12604: Add DynamicMemoryUsage() to CDBWrapper to estimate LevelDB memory use...
< bitcoin-git> [bitcoin] laanwj closed pull request #12604: Add DynamicMemoryUsage() to CDBWrapper to estimate LevelDB memory use (master...leveldb_memory) https://github.com/bitcoin/bitcoin/pull/12604
< bitcoin-git> [bitcoin] NicolasDorier closed pull request #11653: [RPC] Add utility getsignaturehash (master...getsignaturehash) https://github.com/bitcoin/bitcoin/pull/11653
< bitcoin-git> [bitcoin] NicolasDorier closed pull request #10980: [Refactor] Decouple CKeyStore from CWatchOnlyStore (master...decouplewatchonly) https://github.com/bitcoin/bitcoin/pull/10980
< bitcoin-git> [bitcoin] NicolasDorier closed pull request #9728: Can create External HD wallet with -externalhd (master...watchonlyhd) https://github.com/bitcoin/bitcoin/pull/9728
< bitcoin-git> [bitcoin] NicolasDorier closed pull request #9991: listreceivedbyaddress Filter Address (master...listreceivedbyaddress-filtered) https://github.com/bitcoin/bitcoin/pull/9991
< bitcoin-git> [bitcoin] NicolasDorier closed pull request #10202: [Wallet] FundRawTransaction can fund a transaction with preset inputs found in the CoinView (master...fundrawtransaction) https://github.com/bitcoin/bitcoin/pull/10202
< bitcoin-git> [bitcoin] NicolasDorier closed pull request #10216: [Wallet] FundRawTransaction can accept pre-set inputs whose parent is not yet broadcasted (master...fundrawtransaction2) https://github.com/bitcoin/bitcoin/pull/10216
< bitcoin-git> [bitcoin] NicolasDorier closed pull request #11666: [RPC] Add utility signinput (master...signinput) https://github.com/bitcoin/bitcoin/pull/11666
< aj> wumpus, *: what do people think about changing the merge script to add "Acked-By: ..." credits for reviewers in the merge commit, or similar?
< xyxxy> have there been any notable articles or papers published pertaining to a security code review of bitcoin core?
< bitcoin-git> [bitcoin] laanwj opened pull request #12617: gui: Show messages as text not html (master...2018_03_gui_textbox) https://github.com/bitcoin/bitcoin/pull/12617
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9903537750ac...1f9349129d57
< bitcoin-git> bitcoin/master e2b2e48 Ken Lee: doc: SIGNER can contains space inside now....
< bitcoin-git> bitcoin/master 1f93491 Wladimir J. van der Laan: Merge #12615: doc: allow for SIGNER containing spaces...
< bitcoin-git> [bitcoin] laanwj closed pull request #12615: doc: allow for SIGNER containing spaces (master...patch-1) https://github.com/bitcoin/bitcoin/pull/12615
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/1f9349129d57...cff95a63250e
< bitcoin-git> bitcoin/master cfdd895 João Barbosa: qt: Set modal overlay hide button as default
< bitcoin-git> bitcoin/master cff95a6 Wladimir J. van der Laan: Merge #12616: Set modal overlay hide button as default...
< sipa> xyxxy: look at the eclipse attacks on bitcoin's peer to peer network paper
< bitcoin-git> [bitcoin] laanwj closed pull request #12616: Set modal overlay hide button as default (master...2018-03-modaloverlay) https://github.com/bitcoin/bitcoin/pull/12616
< sipa> for example
< wumpus> aj: yeah, I've seen that in Linux I think?
< wumpus> aj: not sure how to go about that, though; automatically scanning all replies for "ACK" seems vulnerable to false positives.
< wumpus> aj: would be easier if everyone used the github 'approve' button
< wumpus> but giving reviewers more credit is a nice idea in itself
< aj> wumpus: scanning replies for ACK <commitid> should be pretty safe from false positives
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/cff95a63250e...d74b01db0c13
< bitcoin-git> bitcoin/master 6058766 Michał Zabielski: Remove deprecated PyZMQ call from Python ZMQ example...
< bitcoin-git> bitcoin/master d74b01d Wladimir J. van der Laan: Merge #12588: [Utils] Remove deprecated PyZMQ call from Python ZMQ example...
< bitcoin-git> [bitcoin] laanwj closed pull request #12588: [Utils] Remove deprecated PyZMQ call from Python ZMQ example (master...zmq_example_fix) https://github.com/bitcoin/bitcoin/pull/12588
< bitcoin-git> [bitcoin] ajtowns closed pull request #10996: Add per-network config file network.conf (master...netconf) https://github.com/bitcoin/bitcoin/pull/10996
< bitcoin-git> [bitcoin] JeremyRubin closed pull request #9938: Lock-Free CheckQueue (master...PR-lockfree-checkqueue) https://github.com/bitcoin/bitcoin/pull/9938
< bitcoin-git> [bitcoin] instagibbs closed pull request #11545: [wallet] Support CMV signing of 17~20 key template (master...20of20) https://github.com/bitcoin/bitcoin/pull/11545
< bitcoin-git> [bitcoin] NicolasDorier reopened pull request #9991: listreceivedbyaddress Filter Address (master...listreceivedbyaddress-filtered) https://github.com/bitcoin/bitcoin/pull/9991
< promag> fanquake: please add "Needs release notes" label to #11120
< gribble> https://github.com/bitcoin/bitcoin/issues/11120 | [RPC] verifytxoutproof returns object including blockhash by instagibbs · Pull Request #11120 · bitcoin/bitcoin · GitHub
< wumpus> promag: the PR is still open, it's preferable if the author adds to the release notes themselves
< wumpus> 'needs release notes' tag is really for PRs which that was forgotten
< bitcoin-git> [bitcoin] ajtowns closed pull request #10994: Add option to avoid warning on certain network upgrades (master...ignore-vbits) https://github.com/bitcoin/bitcoin/pull/10994
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/d74b01db0c13...bc679829e2d3
< bitcoin-git> bitcoin/master 937bf43 fanquake: Use std::thread::hardware_concurrency, instead of Boost, to determine available cores
< bitcoin-git> bitcoin/master bc67982 Wladimir J. van der Laan: Merge #10271: Use std::thread::hardware_concurrency, instead of Boost, to determine available cores...
< bitcoin-git> [bitcoin] laanwj closed pull request #10271: Use std::thread::hardware_concurrency, instead of Boost, to determine available cores (master...replace-boost-getnumcores) https://github.com/bitcoin/bitcoin/pull/10271
< bitcoin-git> [bitcoin] eklitzke opened pull request #12618: Set SCHED_BATCH priority on the loadblk thread. (master...sched_batch) https://github.com/bitcoin/bitcoin/pull/12618
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/bc679829e2d3...765a3ebb0268
< bitcoin-git> bitcoin/master 8f79226 fanquake: depends: zeromq 4.2.3
< bitcoin-git> bitcoin/master 13a399a Cory Fields: depends: patch pthread_set_name_np out of zeromq
< bitcoin-git> bitcoin/master 765a3eb Wladimir J. van der Laan: Merge #11986: [depends] zeromq 4.2.3...
< bitcoin-git> [bitcoin] laanwj closed pull request #11986: [depends] zeromq 4.2.3 (master...zeromq-4-2-3) https://github.com/bitcoin/bitcoin/pull/11986
< CubicEarths> Here is the bandwidth chart from my recent sync - https://pasteboard.co/HaGmNtt.png
< CubicEarths> I was connecting to a single node over my lan
< aj> CubicEarths: kinda weird looking? cpu or disk bound in the middle?
< CubicEarths> The graph stops around Jan 2018. I restarted and it is currently processing about 2 blocks / second again
< aj> oh, right
< CubicEarths> I was asleep when it was doing that - but 6gb dbcache, 1tb empty ssd, 2015 i5-processor (laptop)
< CubicEarths> And that first drop off - that was about 12 hours into the sync
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/765a3ebb0268...f13d756cdd94
< bitcoin-git> bitcoin/master cfaac2a murrayn: Add build support for 'gprof' profiling.
< bitcoin-git> bitcoin/master f13d756 Wladimir J. van der Laan: Merge #12373: Build: Add build support for profiling....
< bitcoin-git> [bitcoin] laanwj closed pull request #12373: Build: Add build support for profiling. (master...profiling) https://github.com/bitcoin/bitcoin/pull/12373
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/f13d756cdd94...20e3b9a485fd
< bitcoin-git> bitcoin/master 6fbc098 Wladimir J. van der Laan: gui: Show messages as text not html...
< bitcoin-git> bitcoin/master 20e3b9a Wladimir J. van der Laan: Merge #12617: gui: Show messages as text not html...
< bitcoin-git> [bitcoin] laanwj closed pull request #12617: gui: Show messages as text not html (master...2018_03_gui_textbox) https://github.com/bitcoin/bitcoin/pull/12617
< bitcoin-git> [bitcoin] theuni closed pull request #9566: threading: use std::chrono for timestamps (master...nuke-boost-chrono2) https://github.com/bitcoin/bitcoin/pull/9566
< ossifrage> CubicEarths, trimmed empty or just no files in the file system?
< bitcoin-git> [bitcoin] kallewoof opened pull request #12619: Give hint about gitian not able to download (master...patch-2) https://github.com/bitcoin/bitcoin/pull/12619
< CubicEarths> The SSD? It just has Ubuntu on it - and I just remembered - a 200GB partition. But it has about 700GB of free space, so there shouldn't be issues related to the drive full or fragmented
< bitcoin-git> [bitcoin] promag opened pull request #12620: Remove TransactionTableModel::TxIDRole (master...2018-03-remove-txidrole) https://github.com/bitcoin/bitcoin/pull/12620
< ossifrage> CubicEarths, the SSD itself can be fragmented or out of erased blocks, but I doubt it could be so bad to show up on your graph
< ossifrage> CubicEarths, assuming your SSD supports trim you can run fstrim before running tests like that to reduce the impact of the SSD stalling
< CubicEarths> interesting. The ssd is new, btw
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/20e3b9a485fd...a36834f10b80
< bitcoin-git> bitcoin/master e68172e Ethan Heilman: Add test-before-evict discipline to addrman...
< bitcoin-git> bitcoin/master a36834f Wladimir J. van der Laan: Merge #9037: net: Add test-before-evict discipline to addrman...
< bitcoin-git> [bitcoin] promag opened pull request #12621: Avoid querying unnecessary model data when filtering transactions (master...2018-03-transaction-filter-avoid-querying) https://github.com/bitcoin/bitcoin/pull/12621
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/a36834f10b80...85424d79eded
< bitcoin-git> bitcoin/master b7cd08b James O'Beirne: Add documentation to PeerLogicValidation interface and related functions
< bitcoin-git> bitcoin/master 85424d7 Wladimir J. van der Laan: Merge #12603: [docs] PeerLogicValidation interface...
< bitcoin-git> [bitcoin] laanwj closed pull request #12603: [docs] PeerLogicValidation interface (master...jamesob/2018-03-peerlogicvalidation-doc) https://github.com/bitcoin/bitcoin/pull/12603
< bitcoin-git> [bitcoin] laanwj opened pull request #12622: net: Correct addrman logging (master...2018_03_addrman_log_fixup) https://github.com/bitcoin/bitcoin/pull/12622
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/85424d79eded...b7c8812ad0c2
< bitcoin-git> bitcoin/master 22b4aae Karl-Johan Alm: [arith_uint256] Avoid unnecessary this-copy using prefix operator
< bitcoin-git> bitcoin/master b7c8812 Wladimir J. van der Laan: Merge #12564: [arith_uint256] Avoid unnecessary this-copy using prefix operator...
< bitcoin-git> [bitcoin] laanwj closed pull request #12564: [arith_uint256] Avoid unnecessary this-copy using prefix operator (master...optim-arith_uint256-operator-dec) https://github.com/bitcoin/bitcoin/pull/12564
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/b7c8812ad0c2...12ac2f0d7129
< bitcoin-git> bitcoin/master c5be37e Dan Bolser: doc: Mention configure without wallet in FreeBSD instructions...
< bitcoin-git> bitcoin/master 12ac2f0 Wladimir J. van der Laan: Merge #12581: doc: Mention configure without wallet in FreeBSD instructions...
< bitcoin-git> [bitcoin] laanwj closed pull request #12581: doc: Mention configure without wallet in FreeBSD instructions (0.16...patch-3) https://github.com/bitcoin/bitcoin/pull/12581
< bitcoin-git> [bitcoin] laanwj pushed 3 new commits to master: https://github.com/bitcoin/bitcoin/compare/12ac2f0d7129...cd5e4381d46d
< bitcoin-git> bitcoin/master fc44cb1 Conor Scott: [RPC] Add list of child transactions to verbose output of getrawmempool
< bitcoin-git> bitcoin/master 1dfb4e7 Conor Scott: [Tests] Check output of parent/child tx list from getrawmempool, getmempooldescendants, getmempoolancestors, and REST interface
< bitcoin-git> bitcoin/master cd5e438 Wladimir J. van der Laan: Merge #12479: RPC: Add child transactions to getrawmempool verbose output...
< bitcoin-git> [bitcoin] laanwj closed pull request #12479: RPC: Add child transactions to getrawmempool verbose output (master...getrawmempool_descendent_pointers) https://github.com/bitcoin/bitcoin/pull/12479
< bitcoin-git> [bitcoin] laanwj closed pull request #12622: net: Correct addrman logging (master...2018_03_addrman_log_fixup) https://github.com/bitcoin/bitcoin/pull/12622
< bitcoin-git> [bitcoin] laanwj closed pull request #12519: Net: Pass MSG_MORE flag when sending non-final network messages (master...MSG_MORE) https://github.com/bitcoin/bitcoin/pull/12519
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/e1d6e2af6d89...9fb3898c3fb0
< bitcoin-git> bitcoin/master f506c0a Sjors Provoost: [qt] send: Clear All also resets coin control options
< bitcoin-git> bitcoin/master 9fb3898 Wladimir J. van der Laan: Merge #12432: [qt] send: Clear All also resets coin control options...
< bitcoin-git> [bitcoin] laanwj closed pull request #12432: [qt] send: Clear All also resets coin control options (master...2018/02/qt-send-clear-all-coin-control) https://github.com/bitcoin/bitcoin/pull/12432
< wumpus> I've added meshcollider to the team that's able to open/close issues and add/remove labels to issues, so that he can help there
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/9fb3898c3fb0...14475e2dcdc8
< bitcoin-git> bitcoin/master 40b17f5 Sjors Provoost: [scripts] lint-whitespace: use perl instead of grep -P...
< bitcoin-git> bitcoin/master 14475e2 Wladimir J. van der Laan: Merge #12097: [scripts] lint-whitespace: use perl instead of grep -P...
< bitcoin-git> [bitcoin] laanwj closed pull request #12097: [scripts] lint-whitespace: use perl instead of grep -P (master...lint-whitespace-no-grep-P) https://github.com/bitcoin/bitcoin/pull/12097
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/14475e2dcdc8...17c44b208f1c
< bitcoin-git> bitcoin/master 8dbf740 Sjors Provoost: [scripts] lint-whitespace: check last N commits or unstaged changes...
< bitcoin-git> bitcoin/master 17c44b2 Wladimir J. van der Laan: Merge #12098: [scripts] lint-whitespace: add param to check last N commits...
< bitcoin-git> [bitcoin] laanwj closed pull request #12098: [scripts] lint-whitespace: add param to check last N commits (master...lint-whitespace-n-commits) https://github.com/bitcoin/bitcoin/pull/12098
< bitcoin-git> [bitcoin] laanwj pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/17c44b208f1c...29088b14ee4b
< bitcoin-git> bitcoin/master 5f8cc0d Richard Kiss: Add a test for large tx output scripts with segwit input.
< bitcoin-git> bitcoin/master 29088b1 Wladimir J. van der Laan: Merge #12600: Add a test for large tx output scripts with segwit input....
< bitcoin-git> [bitcoin] laanwj closed pull request #12600: Add a test for large tx output scripts with segwit input. (master...feature/segwit_big_txout_script) https://github.com/bitcoin/bitcoin/pull/12600
< bitcoin-git> [bitcoin] laanwj pushed 5 new commits to master: https://github.com/bitcoin/bitcoin/compare/29088b14ee4b...b225010a808d
< bitcoin-git> bitcoin/master 32e69fa Pieter Wuille: Replace CBitcoinSecret with {Encode,Decode}Secret
< bitcoin-git> bitcoin/master ebfe217 Pieter Wuille: Stop using CBase58Data for ext keys
< bitcoin-git> bitcoin/master 119b0f8 Pieter Wuille: Split key_io (address/key encodings) off from base58
< bitcoin-git> [bitcoin] laanwj closed pull request #11372: Address encoding cleanup (master...201709_addr_cleanup) https://github.com/bitcoin/bitcoin/pull/11372
< bitcoin-git> [bitcoin] fanquake opened pull request #12625: depends: biplist 1.0.3 (master...biplist-1-0-3) https://github.com/bitcoin/bitcoin/pull/12625
< bitcoin-git> [bitcoin] EthanHeilman opened pull request #12626: Limit the number of IPs addrman learns from each DNS seeder (master...master) https://github.com/bitcoin/bitcoin/pull/12626
< bitcoin-git> [bitcoin] MarcoFalke opened pull request #12627: qa: Fix some tests to work on native windows (master...Mf1803-qaWindowsTests) https://github.com/bitcoin/bitcoin/pull/12627