1 2018-02-25 00:08:43	0|esotericnonsense|will have to fiddle when I have some more time. CWallet inherits from CValidationInterface, but it overrides TransactionRemovedFromMempool. if you kill it from CValidationInterface and remove the override along with a few other bits of code I think is dead you're there.
 2 2018-02-25 01:14:13	0|bitcoin-git|13bitcoin/06master 1490ba2df 15Jonas Schnelli: Fix missing cs_main lock for GuessVerificationProgress()
 3 2018-02-25 01:14:13	0|bitcoin-git|[13bitcoin] 15jonasschnelli pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/07090c533943...bf3353de9059
 4 2018-02-25 01:14:14	0|bitcoin-git|13bitcoin/06master 14bf3353d 15Jonas Schnelli: Merge #12287: Optimise lock behaviour for GuessVerificationProgress()...
 5 2018-02-25 01:15:06	0|bitcoin-git|[13bitcoin] 15jonasschnelli closed pull request #12287: Optimise lock behaviour for GuessVerificationProgress() (06master...062018/01/rescan_lock) 02https://github.com/bitcoin/bitcoin/pull/12287
 6 2018-02-25 01:19:43	0|jonasschnelli|wumpus: maybe confirm that your comment on https://github.com/bitcoin/bitcoin/pull/10387 has been fixed? thx
 7 2018-02-25 01:59:52	0|esotericnonsense|ah, it's finally clear now. connectTrace is the thing that actually uses it. just a few steps deep
 8 2018-02-25 02:41:50	0|ken2812221|Gitian build fails on Windows and Linux on master branch.
 9 2018-02-25 02:42:04	0|ken2812221|It says "No rule to make target `zlib/adler32.o', needed by `zlib/libz.a'.  Stop."
10 2018-02-25 02:42:41	0|ken2812221|But no problem when building MacOS
11 2018-02-25 02:44:27	0|ken2812221|Seems it fails after 2018-02-17, according to jonasschnelli's website.
12 2018-02-25 03:11:52	0|achow101|ken2812221: that's interesting. That's the same error showing up here: https://github.com/bitcoin/bitcoin/issues/12515 I think a dependency might have changed/updated which is now breaking that
13 2018-02-25 03:38:28	0|ken2812221|Just followed build-window.md, it failed on 32-bit and succeed on 64-bit
14 2018-02-25 03:46:33	0|achow101|The 64 bit build failed in a clean 14.04 vm for me. I'm trying a clean 17.10 vm next. It works in my normal 17.10 environment, but that could be because of some other things in my environment that makes it work
15 2018-02-25 03:47:00	0|achow101|ken2812221: which build of windows failed in gitian?
16 2018-02-25 03:47:08	0|achow101|32 or 64 bit?
17 2018-02-25 03:49:39	0|ken2812221|it fails when build 32-bit, so it won't built 64-bit
18 2018-02-25 04:13:40	0|ken2812221|ok, it seem like it  works on higher version since I test 64-bit on WSL and 32-bit on my Debian 8 vm
19 2018-02-25 04:14:58	0|ken2812221|64-bit also fails on my Debian vm
20 2018-02-25 04:53:32	0|achow101|you should be able to fix that problem by installing zlib1g-dev
21 2018-02-25 04:53:39	0|achow101|ken2812221: ^
22 2018-02-25 07:37:00	0|ken2812221|yeah, that works.
23 2018-02-25 09:39:04	0|fanquake|achow101 looks like the ccache zlib issue has been fixed upstream, should be included in 3.4.2
24 2018-02-25 10:45:11	0|bitcoin-git|[13bitcoin] 15practicalswift opened pull request #12534: tests: Specify skipped flake8 warnings (rather than enabled warnings) (06master...06lint-python-invert) 02https://github.com/bitcoin/bitcoin/pull/12534
25 2018-02-25 17:32:40	0|stevenroose|"For example, in Bitcoin Core 50KB within blocks are reserved for high priority transactions." Is this still the case?
26 2018-02-25 17:33:54	0|gmaxwell|Of course not.
27 2018-02-25 17:34:11	0|sipa|where do you read that?
28 2018-02-25 17:34:26	0|gmaxwell|miners turned that stuff off years ago, and the functionality was subsiquently removed (it massively slowed down block template creation)
29 2018-02-25 17:35:54	0|stevenroose|In a two-year-old paper
30 2018-02-25 17:35:56	0|stevenroose|https://github.com/sharathrao13/predicting-bitcoin-transaction-fees/blob/master/paper/smarter-fees.pdf
31 2018-02-25 17:36:12	0|stevenroose|Yeah makes sense to not do that
32 2018-02-25 17:36:54	0|gmaxwell|it should be a crime to end a url with .pdf but have it just be a webage.
33 2018-02-25 17:38:15	0|sipa|punishable by death by pwnage?
34 2018-02-25 17:38:30	0|stevenroose|Hahaha, agreed
35 2018-02-25 17:40:17	0|gmaxwell|maybe it's just me, but I continually wget urls like that then find myself irritated that I didn't get the file.
36 2018-02-25 17:46:09	0|meshcollider|gmaxwell: yeah it also messes up a lot of mobile browsers :(
37 2018-02-25 19:37:35	0|gmaxwell|cfields: re #12519  it's now been multiple _years_ since it was suggesed that we use MSG_MORE. We shouldn't keep delaying simple improvements forever.
38 2018-02-25 19:37:37	0|gribble|https://github.com/bitcoin/bitcoin/issues/12519 | Net: Pass MSG_MORE flag when sending non-final network messages by whitslack · Pull Request #12519 · bitcoin/bitcoin · GitHub
39 2018-02-25 19:42:44	0|gmaxwell|eklitzke: re #12495 is there a way to query if leveldb is going to do the mmap thing more directly rather than using 64/32 bitness as a proxy?
40 2018-02-25 19:42:45	0|gribble|https://github.com/bitcoin/bitcoin/issues/12495 | Increase LevelDB max_open_files on 64-bit POSIX systems by eklitzke · Pull Request #12495 · bitcoin/bitcoin · GitHub
41 2018-02-25 20:34:45	0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #12055: Prepare version scheme for upcoming release [take 2] (06master...06Mf1712-versionDropRedundantZero) 02https://github.com/bitcoin/bitcoin/pull/12055
42 2018-02-25 22:40:25	0|eklitzke|gmaxwell: not with the api they provide, but it would be simple enough to patch their code to expose that
43 2018-02-25 22:40:34	0|eklitzke|i was thinking about submitting a patch upstream to do that anyway
44 2018-02-25 23:45:46	0|gmaxwell|eklitzke: or just change it so the FD limit is actually a FD limit... and an open file limit is seperate.
45 2018-02-25 23:46:08	0|gmaxwell|though maybe our time would be better spent killing the FD limit in the daemon