Meetings

Upcoming Meetings

2019-06-X

Organisation discussion

• Is 5.0.2 needed for the newlines in setting name bug? (https://github.com/minetest/minetest/pull/8590)
• What is needed to have good mobs in Minetest / Minetest Game? I suggest roping in mob devs like tenplus1 and stujones1

PR discussion/reviews

MTG discussion/reviews

Past Meetings

July-August 2018

Organisation discussion

• Will there be a 0.4.18 release?
  • http://irc.minetest.net/minetest-dev/2018-07-14#i_5358759 - sounds like a definitive "no"?
• Specify a rough 5.0.0 release date (month, year)
  • Freeze for multiple weeks for testing & fixing
• Forums: Clarify which rules (amongst license) apply for WIP modpacks/mods/games
  • https://forum.minetest.net/viewtopic.php?t=20630

PR discussion/reviews

• Colorize command parameters and privilege names (https://github.com/minetest/minetest/pull/7019)
  • Improves readability - does it need `string.format` for a possibly prettier code?
• Protocol cleanups https://github.com/minetest/minetest/pull/7348
  • Review needed (compat break)

June 2018

Organisation discussion

• Will there be a 0.4.18 release?
• Bugfixing the bugfix release
  • https://github.com/minetest/minetest/commit/014a1a08 fix for https://github.com/minetest/minetest/issues/6092
  • https://github.com/minetest/minetest/commit/a1598e1b (c_internal.cpp only) fix for https://github.com/minetest/minetest/issues/7419
• Proposal: change master version scheme to: 5.0.0

PR discussion/reviews

• Colorize command parameters and privilege names (https://github.com/minetest/minetest/pull/7019)
  • Improves readability - does it need `string.format` for a possibly prettier code?
• Lua parameter reading https://github.com/minetest/minetest/pull/7410
  • Good design concept? Comments?
• Protocol cleanups https://github.com/minetest/minetest/pull/7348
  • Review needed (compat break)
• CSM keypress - is this advantage acceptable? https://github.com/minetest/minetest/pull/7008
  • Needs flavors to be added: http://irc.minetest.net/minetest-dev/2018-06-11#i_5329651

2018-05-19

Proposed time: 18:00 UTC

Organisation discussion

• 0.4.17 - backport missing fixes and set date for freeze
  • ~Android-specific fix for https://pastebin.com/j4mjniaq must be added too (during freeze?)~
  • Needs buildbot fixes (404 packages)
  • Backport https://github.com/minetest/minetest/commit/4bb41a19d (http://irc.minetest.net/minetest-dev/2018-05-19#i_5310266)
• 0.4.17 - set release date

PR discussion/reviews

• Colorize command parameters and privilege names (https://github.com/minetest/minetest/pull/7019)
  • Improves readability - does it need `string.format` for a possibly prettier code?
• Fix missing ignore textures (https://github.com/minetest/minetest/pull/7326)
  • Needs more dev opinions & reviewing

2018-05-05

It's now almost a month since the last meeting - it time for the next one. Proposed time: 18:00 UTC

Organisation discussion

• When do we have time to release 0.4.17?
  • Still unclear freeze and release dates. 1 MTE needs backporting first
• Whether to leave the issue number link in a commit message when merging on Github
  • MTE: Keep the numbers if they're inserted automatically (high PR count, helpful for the web interface blame or fast jumps in commits)
  • MTG: The numbers may be removed if wanted (small amount of PRs)
  • http://irc.minetest.net/minetest-dev/2018-05-05#i_5298887

PR discussion/reviews

• Colorize command parameters and privilege names (https://github.com/minetest/minetest/pull/7019)
  • Improves readability - does it need `string.format` for a possibly prettier code?
• Abort when trying to set a not registered node (https://github.com/minetest/minetest/pull/7011)
  • Needs more dev opinions & reviewing

2018-04-07

Organisation discussion

• Questions related to 0.4.17, 0.4.18 or future development?
• 0.4.17 release should be moved to 21/22 April for backports (http://irc.minetest.net/minetest-dev/2018-04-07#i_5272834)
  • Feature/backport freeze after 14 April to have enough time for testing

PR discussion/review

• Android joystick (https://github.com/minetest/minetest/pull/7126)
• Formspecs: Add a <use_color_alpha> parameter to the box[] element (https://github.com/minetest/minetest/pull/7116)
  • Causes errors in-game for older clients but doesn't affect playability - merge or not?
• Colorize command parameters and privilege names (https://github.com/minetest/minetest/pull/7019)
  • Improves readability - does it need `string.format` for a possibly prettier code?

MTG discussion/reviews

• Tools: rebalance (https://github.com/minetest/minetest_game/issues/1681)
  • Needs either a levelling system and/or special abilities for different materials to add variety
  • paramat suggests removing bronze, which is incorrectly a stronger tool than steel (http://irc.minetest.net/minetest-dev/2018-04-07#i_5272959)
  • Future mese concept: http://irc.minetest.net/minetest-dev/2018-04-07#i_5273013

2018-03-31

PR discussion/reviews

• Introduce clang-tidy to the CI (https://github.com/minetest/minetest/pull/6295)
• Android joystick (https://github.com/minetest/minetest/pull/7126)
  • Minor change requests, discussion about joystick placement

0.4.17 release schedule

• Definitive release date
  • Announced. 15 April 2018, see https://forum.minetest.net/viewtopic.php?f=18&t=19905
• Any new, important bugfixes to backport?
  • Added the `core.rotate_node` bugfix, otherwise probably not. It's pretty much stable.
• Volunteer for cherry-picking the ToDo backport commits (https://github.com/minetest/minetest/pull/6746)
  • `<sfan5> probably me` (http://irc.minetest.net/minetest-dev/2018-03-31#i_5264276)

2018-03-10

Organisation discussion

• New Doxygen rule
  • Log: http://irc.minetest.net/minetest-dev/2018-03-10#i_5249578
  • Opposed: paramat, sfan5, SmallJoker, Shara
  • Comments in the functions directly are more welcome
• Group issues in projects?
  • Log: http://irc.minetest.net/minetest-dev/2018-03-10#i_5249600
  • Purpose is unclear and questionable due missing information, needs to be discussed later again

PR discussion/reviews

• Remove texture pack caching (https://github.com/minetest/minetest/pull/6660)
• Don’t create CSM script env (https://github.com/minetest/minetest/pull/6951)
  • Log: http://irc.minetest.net/minetest-dev/2018-03-10#i_5249625
  • Needs improvement, questionable solution due request of server-provided CSM (https://github.com/minetest/minetest/issues/5393)
• “Bugfix” for 0.4-backported rotate_node calls (https://github.com/minetest/minetest/pull/6900)
  • SmallJoker will test again and possibly merge after success
• Android C++11 build fix (https://github.com/minetest/minetest/pull/6796)
  • Log: http://irc.minetest.net/minetest-dev/2018-03-10#i_5249684
  • sfan5 and nerzhul will try to test it soon. It does not break the regular setup, so a merge is to expect soon
  • Confirmed working by Wayward_One (http://irc.minetest.net/minetest-dev/2018-03-10#i_5249720)
• Autogenerated settings header (https://github.com/minetest/minetest/pull/6728)
  • Log: http://irc.minetest.net/minetest-dev/2018-03-10#i_5249703 (stub)
  • Needs attention

0.4.17 release schedule

• Definitive release date
  • absent nerzhul: “release 0.4.17 ASAP” (http://irc.minetest.net/minetest-dev/2018-03-10#i_5249422)
• Are there any new, important bugfixes to backport?
• Volunteer for cherry-picking the 6 TODO (+ ^ from above) commits (https://github.com/minetest/minetest/pull/6746)