< fanquake> Would someone like to review the backporting in #17881
< gribble> https://github.com/bitcoin/bitcoin/issues/17881 | [0.18] scripts: fix check-symbols & check-security argument passing by fanquake . Pull Request #17881 . bitcoin/bitcoin . GitHub
< kallewoof> luke-jr: I ran into that issue last night! (not auto-gen block413567.raw.h) I was scratching my head for awhile over that one...
< bitcoin-git> [bitcoin] sipsorcery opened pull request #17899: msvc: Ignore msvc linker warning and update to msvc build instructions. (master...mavc_linkerwarns) https://github.com/bitcoin/bitcoin/pull/17899
< promag> #17792 is ready i think
< gribble> https://github.com/bitcoin/bitcoin/issues/17792 | 0.19: Backports by promag . Pull Request #17792 . bitcoin/bitcoin . GitHub
< promag> also, if possible, #17878 could be in HP chasing ack
< gribble> https://github.com/bitcoin/bitcoin/issues/17878 | wip: zmq: Support -zmqpubwallettx by promag . Pull Request #17878 . bitcoin/bitcoin . GitHub
< bitcoin-git> [bitcoin] hebasto closed pull request #17898: [WIP] qt: Make Information tab accessible via keyboard (master...20200108-accessibility-info) https://github.com/bitcoin/bitcoin/pull/17898
< bitcoin-git> [bitcoin] hebasto reopened pull request #14810: qt: Enable tabbing through labels (master...20181124-tab-through-labels) https://github.com/bitcoin/bitcoin/pull/14810
< bitcoin-git> [bitcoin] theStack opened pull request #17900: ci: Combine 32-bit build with CentOS 7 build (master...20200108-ci-add-32-bit-centos-build) https://github.com/bitcoin/bitcoin/pull/17900
< promag> anyone can explain why importmulti calls EnsureWalletIsUnlocked?
< promag> ah ok nm
< luke-jr> kallewoof: another one is src/qt/moc_paymentserver.cpp - been deleting it for years now :x
< jnewbery> #16688 ready for merge I think
< gribble> https://github.com/bitcoin/bitcoin/issues/16688 | log: Add validation interface logging by jkczyz . Pull Request #16688 . bitcoin/bitcoin . GitHub
< bitcoin-git> [bitcoin] emilengler opened pull request #17901: Add feature to get specific help (master...2020-01-get-specific-help) https://github.com/bitcoin/bitcoin/pull/17901
< hebasto> MarcoFalke: mind looking into #17893 ?
< gribble> https://github.com/bitcoin/bitcoin/issues/17893 | qa: Fix double-negative arg test by hebasto . Pull Request #17893 . bitcoin/bitcoin . GitHub
< wumpus> #startmeeting
< lightningbot> Meeting started Thu Jan 9 19:00:21 2020 UTC. The chair is wumpus. Information about MeetBot at http://wiki.debian.org/MeetBot.
< lightningbot> Useful Commands: #action #agreed #help #info #idea #link #topic.
< kanzure> hi
< wumpus> #bitcoin-core-dev Meeting: wumpus sipa gmaxwell jonasschnelli morcos luke-jr sdaftuar jtimon cfields petertodd kanzure bluematt instagibbs phantomcircuit codeshark michagogo marcofalke paveljanik NicolasDorier jl2012 achow101 meshcollider jnewbery maaku fanquake promag provoostenator aj Chris_Stewart_5 dongcarl gwillen jamesob ken281221 ryanofsky gleb moneyball kvaciral ariard digi_james amiti fjahr
< wumpus> jeremyrubin lightlike emilengler jonatack hebasto
< hebasto> hi
< achow101> hi
< jamesob> hi
< jb55> ih
< wumpus> no pre-proposed topics for today in https://gist.github.com/moneyball/071d608fdae217c2a6d7c35955881d8a
< wumpus> any last minute proposals?
< instagibbs> hi
< emilengler> hi
< aj> hi
< wumpus> #topic High priority for review
< promag> hi
< wumpus> https://github.com/bitcoin/bitcoin/projects/8 9 blockers, 5 chasing concept for now
< ariard> hi
< wumpus> anything to add/remove or ready for merge?
< promag> can #17878 be in hp?
< gribble> https://github.com/bitcoin/bitcoin/issues/17878 | wip: zmq: Support -zmqpubwallettx by promag . Pull Request #17878 . bitcoin/bitcoin . GitHub
< promag> "Chasing Concept ACK"
< wumpus> I don't think a wip PR should be in high priority or review
< wumpus> ohh yes that's okay
< promag> ok, ty
< nehan_> hi
< jb55> looks like #13339 is finally ready though?
< gribble> https://github.com/bitcoin/bitcoin/issues/13339 | wallet: Replace %w by wallet name in -walletnotify script by promag . Pull Request #13339 . bitcoin/bitcoin . GitHub
< promag> going to address latest ryanofsky review
< jb55> testing it locally now
< wumpus> that's good to know
< jonatack> hi
< promag> prolly a release note saying that win is not supported
< wumpus> that definitely took a long and difficult review cycle for on first glance, easy functionality
< wumpus> #topic 0.19.1 release
< emilengler> 3 PRs need backports rn
< wumpus> I think it'd be a good idea to do a 0.19.1 release soon, there have been quite a few bug fixes
< promag> yeah, could be in 0.19.1
< jb55> that would be great
< hebasto> nice
< wumpus> emilengler: yes, there are still a few things tagged for backport to 0.19 that could make it in
< achow101> Can #17156 be added for backport 10 0.19.1?
< promag> I have 2 bug fixes #16923 #17457 but not sure they will get enough review
< gribble> https://github.com/bitcoin/bitcoin/issues/17156 | psbt: check that various indexes and amounts are within bounds by achow101 . Pull Request #17156 . bitcoin/bitcoin . GitHub
< gribble> https://github.com/bitcoin/bitcoin/issues/16923 | wallet: Fix duplicates fileid exception on start by promag . Pull Request #16923 . bitcoin/bitcoin . GitHub
< wumpus> especialy the macosx disk usage
< gribble> https://github.com/bitcoin/bitcoin/issues/17457 | gui: Fix manual coin control with multiple wallets loaded by promag . Pull Request #17457 . bitcoin/bitcoin . GitHub
< wumpus> well, my idea is to do the 0.19.1 release soon-ish, so things that require long review cycle and aren't super urgent could wait for 0.19.2 I think
< wumpus> would be nice though
< promag> when then?
< wumpus> when the remaining tagged PRs are merged
< wumpus> and backported ofc
< wumpus> dunno about #15706 though
< gribble> https://github.com/bitcoin/bitcoin/issues/15706 | build: Check QT library version by lucayepa . Pull Request #15706 . bitcoin/bitcoin . GitHub
< promag> then please tag #13339
< gribble> https://github.com/bitcoin/bitcoin/issues/13339 | wallet: Replace %w by wallet name in -walletnotify script by promag . Pull Request #13339 . bitcoin/bitcoin . GitHub
< wumpus> that's not a bugfix
< promag> ah!
< hebasto> #15706 needs more eyes
< promag> so 0.20 I guess
< gribble> https://github.com/bitcoin/bitcoin/issues/15706 | build: Check QT library version by lucayepa . Pull Request #15706 . bitcoin/bitcoin . GitHub
< wumpus> promag: yes I'd hope so
< wumpus> if it wouldn't make 0.20 either it'd be absurd
< wumpus> hebasto: it's hard to get review for m4 build system stuff, could hope cfields_ takes a look at it :)
< wumpus> any other topics?
< wumpus> #endmeeting
< lightningbot> Meeting ended Thu Jan 9 19:15:16 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
< sipa> hi
< instagibbs> sipa, bye
< wumpus> hi
< jnewbery> wumpus: I missed the meeting but #16688 is RFM
< gribble> https://github.com/bitcoin/bitcoin/issues/16688 | log: Add validation interface logging by jkczyz . Pull Request #16688 . bitcoin/bitcoin . GitHub
< wumpus> jnewbery: thanks
< bitcoin-git> [bitcoin] laanwj pushed 5 commits to master: https://github.com/bitcoin/bitcoin/compare/6196e9300181...e7f84503571c
< bitcoin-git> bitcoin/master 428ac70 Jeffrey Czyz: Add VALIDATION to BCLog::LogFlags
< bitcoin-git> bitcoin/master 72f3227 Jeffrey Czyz: Format CValidationState properly in all cases
< bitcoin-git> bitcoin/master 6edebac Jeffrey Czyz: Refactor FormatStateMessage for clarity
< bitcoin-git> [bitcoin] laanwj merged pull request #16688: log: Add validation interface logging (master...2019-08-validation-logging) https://github.com/bitcoin/bitcoin/pull/16688
< bitcoin-git> [bitcoin] bblarney opened pull request #17902: fixed typo in comment, iff -> if (master...blarney-typo) https://github.com/bitcoin/bitcoin/pull/17902
< aj> is secp256k1 tests failing on travis normal? https://travis-ci.org/bitcoin/bitcoin/jobs/634722879
< fanquake> aj: no, not that I've ever seen
< promag> "Waiting for author" can be removed from 17843