00:05 < sipa> roasbeef, emzy: i've updated my seeder code to by default support masks 0x1,0x5,0x9,0xd,0x49,0x400,0x404,0x408,0x40c,0x448

00:05 < sipa> jonasschnelli: ^ also included NODE_NETWORK_LIMITED combinations

00:06 < sipa> provoostenator, jonasschnelli, luke-jr: you may want to update

00:07 < sipa> also another change is that it now sets fRelay=false, as it doesn't care about receiving tx invs

00:08 < sipa> i've only added NODE_COMPACT_FILTER in combination with NODE_WITNESS, and not combined with NODE_BLOOM (no software using BIP157 is non-segwit, or needs BIP37 filters, i think)

00:26 < roasbeef> sipa: nice! correct that afaik bip157 and segwit are more or less a pair

00:50 < luke-jr> sipa: what reason is there to update?

00:51 < luke-jr> I like to keep mine lagged when possible

00:54 < sipa> luke-jr: no strong reason

00:57 < sipa> just support for NODE_NETWORK_LIMITED and NODE_COMPACT_FILTER by default

00:58 < luke-jr> k

00:59 < sipa> (you can also configure those explicitly without code changes)

00:59 < luke-jr> right

01:00 < luke-jr> I should probably PR some fixes I did too

01:00 < luke-jr> the p2p version number it's using is old enough that some other node software rejects it

01:01 < luke-jr> http://luke.dashjr.org/programs/bitcoin/files/charts/services.html

01:01 < luke-jr> hmm, there's only 142 block-filters nodes

01:14 < roasbeef> it's a start ;)

06:35 < bitcoin-git> [bitcoin] practicalswift closed pull request #16320: ci: Add Travis check to make sure unit test coverage reports stay deterministic (master...test-deterministic-coverage-in-travis) https://github.com/bitcoin/bitcoin/pull/16320

06:36 < bitcoin-git> [bitcoin] practicalswift closed pull request #15191: validation: Add missing cs_{LastBlockFile,nBlockSequenceId} locks in PruneAndFlush() and UnloadBlockIndex(). Add missing locking annotations. (master...cs_LastBlockFile) https://github.com/bitcoin/bitcoin/pull/15191

06:36 < bitcoin-git> [bitcoin] practicalswift closed pull request #15192: Add missing cs_main locks in ThreadImport(...)/Shutdown(...)/gettxoutsetinfo(...)/InitScriptExecutionCache(). Add missing locking annotations. (master...validation-cs_main) https://github.com/bitcoin/bitcoin/pull/15192

07:55 < emzy> roasbeef, sipa: I have acrivated 0x1,0x5,0x9,0xd,0x49,0x400,0x404,0x408,0x40c,0x448 aka did the update.

12:56 < jonasschnelli> sipa: thanks. Changed the filter mask on my main-/testnet seeder

13:27 < provoostenator> sipa: thanks, done

13:28 < provoostenator> I added v0.18.1 to #12134

13:28 < gribble> https://github.com/bitcoin/bitcoin/issues/12134 | Build previous releases and run functional tests by Sjors · Pull Request #12134 · bitcoin/bitcoin · GitHub

16:04 < bitcoin-git> [bitcoin] sipsorcery opened pull request #16581: sipsorcery gitian key (master...add_gitian_key) https://github.com/bitcoin/bitcoin/pull/16581

16:16 < bitcoin-git> [bitcoin] MarcoFalke opened pull request #16582: test: Rework ci (Use travis only as fallback env) (master...1908-ciRework) https://github.com/bitcoin/bitcoin/pull/16582

16:41 < bitcoin-git> [bitcoin] gapeman opened pull request #16583: [QT] New filesave icon (master...master) https://github.com/bitcoin/bitcoin/pull/16583