IRC meeting summary for 2016-07-28

Overview


Notes / short topics

  • Jtimon would like to see the removal of ISM, talked about in last weeks meeting, merged quickly as it is important for other libconsensus refactors.
  • NicolasDorier asks to review/test PR #8422 (Cache hashes), which needs to be merged and backported to 0.13 before segwit release.

Main topics

  • 0.13.0
  • boost threads/sync replacement for master

0.13.0

background

The Bitcoin Core team is working towards the 0.13.0 release (full schedule) and RC2 is available since 2016-07-31 (3 days after the meeting).

meeting comments

PR #8408, which fixes a bug in compactblocks, is the only thing remaining that’s tagged for 0.13.

Jtimon made PR #8412, which he thinks should be included in 0.13. Everyone agrees.

Luke-jr reiterates the release notes have a bad policy of encouraging changing blockmaxsize to blockmaxweight for which he has a pull request including some codechanges. Wumpus notes not everyone agrees on what is ‘bad policy’. Luke-jr argues if that’s the case the release notes should not recommend anything. Gmaxwell thinks it’s foolish we’re still releasing with default settings that don’t reflect near ubiquitous network usage as in practice nearly every miner will set blockmaxsize and blockmaxweight to the maximum allowed value. Default has been 750k, however there are no 750k blocks to be seen. (at this point in the discussion luke-jr has to catch his plane)

Wumpus thinks a positive consequence of these settings is that it forces miners to not use the default settings. Gmaxwell also notes changing the value of blockmaxweight is more complicated as it needs to be 4 times the desired blockmaxsize, explaining how to set this to the maximum value in the release notes might be seen as a recommendation though. Eliel_ suggests the option of making the mining part refuse to function without the user manually setting the required configuration values, therefor avoiding setting default values, which is something luke-jr argued for years.

meeting conclusion

  • review PR #8408 (Prevent fingerprinting, disk-DoS with compact blocks)

boost threads/sync replacement for master

background

Bitcoin Core is working towards removing the dependency on the boost library. Cfields has a pull request ready to move away from boost threads.

meeting comments

Cfields asks if he should do the replacements one chunk at the time, or all in one go. Wumpus states it makes the most sense to do it all at once, to make it just a one-time pain.

There’s a prerequisite for #8023, which he’ll do a pull request for after the meeting.

Cfields is also still working on the network refactor, for which review/ACKs of PR #8128 and #8085 would be helpful.

meeting conclusion

Review PR #8128 and #8085 (“Net: Turn net structures into dumb storage classes” & “p2p: Begin encapsulation”)

Comic relief

lightningbot      Meeting ended Thu Jul 28 20:00:26 2016 UTC.
jonasschnelli     sipa: time for your Pokemon walk. :P

Participants

IRC nick Name/Nym
sipa Pieter Wuille
gmaxwell Gregory Maxwell
wumpus Wladimir van der Laan
btcdrak BtcDrak
kanzure Bryan Bishop
cfields cory Fields
sdaftuar Suhas Daftuar
jonasschnelli Jonas Schnelli
achow101 Andrew Chow
luke-jr Luke Dashjr
jtimon Jorge Timón
morcos Alex Morcos
instagibbs Gregory Sanders
NicolasDorier Nicolas Dorier
Eliel_ Eliel_

Disclaimer

This summary was compiled without input from any of the participants in the discussion, so any errors are the fault of the summary author and not the discussion participants.


Show your support

Github