Meetings

= Upcoming Meetings =

2020-02-01
Most important first.

Organisation Discussion
 * Next steps in developer recruitment?

PR discussion/reviews
 * Android XXL PR https://github.com/minetest/minetest/pull/9066
 * What need to be changed?
 * What about dependency management? -> repo needed?
 * GUIFormspecMenu: Add animated_image[] element https://github.com/minetest/minetest/pull/9258
 * Texture format - use ".NN.png" or vertically stacked?
 * Images in tabs https://github.com/minetest/minetest/pull/8621
 * Concept discussion only

MTG discussion/reviews
 * None yet

= Past Meetings =

2020-01-11
Arranged by: Krock

Most important first.

Organisation Discussion
 * Looking for new Core Developer members
 * What's missing for 5.1.1?

PR discussion/reviews
 * Formspec text cursor feature https://github.com/minetest/minetest/pull/8665
 * Scroll container https://github.com/minetest/minetest/pull/9101
 * Images in tabs https://github.com/minetest/minetest/pull/8621

MTG discussion/reviews
 * Craftguide from pauloue https://github.com/minetest/minetest_game/pull/2396

2019-08-10
Arranged by: Krock

PR discussion/reviews


 * Custom particle generators for particle spawners https://github.com/minetest/minetest/pull/6688
 * Should this be closed?
 * Fix debug.txt growing bigger and bigger https://github.com/minetest/minetest/pull/8382
 * What to change? Or reject & close?
 * Punchwear https://github.com/minetest/minetest/pull/6804
 * Who wants to adopt it? The feature was requested many times.
 * Add support for set_formspec_prepend in main menu https://github.com/minetest/minetest/pull/8611
 * Hide chat when console is open https://github.com/minetest/minetest/pull/8656

2019-06-29
Arranged by: rubenwardy

Logs: http://irc.minetest.net/minetest-dev/2019-06-29#i_5561978

Organisation discussion


 * Is 5.0.2 needed for the newlines in setting name bug? (https://github.com/minetest/minetest/pull/8590)
 * Yet no clear decision
 * What is needed to have good mobs in Minetest (- Game)?
 * I suggest roping in mob devs like tenplus1 and stujones1
 * No blood included. Belongs to another mod
 * Combine pathfinder issues into a single issue
 * https://github.com/minetest/minetest/issues/8642
 * Possible close issues/PRs: https://github.com/minetest/minetest/pulls?q=is%3Apr+is%3Aopen+sort%3Aupdated-desc+label%3A%22Possible+close%22
 * Does 1 disapproval block a PR?
 * SmallJoker seems to think it does

Proposed order of merging formspec PRs

The idea is to merge the most important PR first (top of list) to not drown in conflicts: https://github.com/minetest/minetest/projects/6


 * style[] https://github.com/minetest/minetest/pull/8383
 * scroll_container[] https://github.com/minetest/minetest/pull/8591
 * list spacing https://github.com/minetest/minetest/pull/7971 (perhaps integrate into style[]?)
 * Unarelith's refactors

July-August 2018
Arranged by: Krock

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
Arranged by: Krock

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
Arranged by: Krock

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
Arranged by: Krock

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
Arranged by: Krock

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  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
Arranged by: Krock

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)
 * ` probably me` (http://irc.minetest.net/minetest-dev/2018-03-31#i_5264276)

2018-03-10
Arranged by: Krock

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)

2017-05-20
Minetest developer meeting 2017-05-20

Last chance to merge/improve the remaining PR before feature freeze:

Old business: * Additional hooks for inventory manipulations callbacks (#4035) * Krock was working on it (#5647), but had an issue with a mysterious crash bug. Progress? * Timed move (#1489) * Merge/close? * nerzhul: close for me  * nore: Merge * Krock: Merge * Split block position in SQLite3 database (and use R*Tree) (#1845) * Rebased, performance tests to do soon(tm). * nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date) * Add list_predict formspec element (#1988) * sofar worked on a rebase a while ago. Close or merge a modern version? * New screenshots for website (web#78) https://github.com/minetest/minetest.github.io/issues/78 * ShadowNinja: "good" screenshots are fairly subjective and it doesn't look like we're actually getting anywhere near a decision in the issue thread, therefore I propose assigning someone to choose and install new screenshots. * > Calinou will handle it

New business: * First, release blockers: * https://github.com/minetest/minetest/pull/5767 Don't add damage flash while punch texture modifier is active #5767 * > Bugfix, punted to next meeting. * https://github.com/minetest/minetest/issues/5782 [CSM] Disable preview mod before release #5782 * CSM disabled by default anyways since it's experimental * > Fixed by #5554 * https://github.com/minetest/minetest/issues/5728 Block instadig after tool switch #5728 * > Fixed by #5785 * nerzhul: * https://github.com/minetest/minetest/pull/5732 (CSM nodedefs/itemdefs read) * maybe close https://github.com/minetest/minetest/pull/5654 * Single/Multiplayer tab merge https://github.com/minetest/minetest/pull/5627 (user UI experience) * > Merge with minor changes * Create world better experience: https://github.com/minetest/minetest/pull/5589 * Permit to enable/disable mods on CSM https://github.com/minetest/minetest/pull/5554 (very important) * ShadowNina: CSM is experimental, so I don't think any CSM issues should be blockers * nerzhul: here the problem if a user enable client side modding his experience will be affected by this preview mod, dedicated to modders)  * > nerzhul will merge * https://github.com/minetest/minetest/pull/5589 (start world after creation) * LMDB backend (https://github.com/minetest/minetest/pull/4206)   * nerzhul: Close? We already have 5 backends to maintain and we already cover all the usecases (client usage, performance usage, reliability usage)

2017-05-13
Minetest developer meeting 2017-05-13

Old business: * Additional hooks for inventory manipulations callbacks (#4035) * Krock was working on it (#5647), but had an issue with a mysterious crash bug. * > No progress * Timed move (#1489) * Merge/close? * > nerzhul: close for me  * > nore: Merge * > Krock: Merge * Split block position in SQLite3 database (and use R*Tree) (#1845) * Rebased, performance tests to do soon(tm). * nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date) * Add list_predict formspec element (#1988) * sofar worked on a rebase a while ago. Close or merge a modern version?

New business: * New screenshots for website (web#78) https://github.com/minetest/minetest.github.io/issues/78 * ShadowNinja: "good" screenshots are fairly subjective and it doesn't look like we're actually getting anywhere near a decision in the issue thread, therefore I propose assigning someone to choose and install new screenshots. * Fix 'alpha' property for liquid nodes (#5494) * > paramat testing. * CSM: Fix undocumented API calls (#5756)

nerzhul: please look at recent PR for this meeting. Feature Freeze is in 8 days Here is a list to look at: * https://github.com/minetest/minetest/pull/5746 (cleanup content_mapblock) * https://github.com/minetest/minetest/pull/5732 (CSM nodedefs/itemdefs read) * maybe close https://github.com/minetest/minetest/pull/5654 * Single/Multiplayer tab merge https://github.com/minetest/minetest/pull/5627 (user UI experience) * Create world better experience: https://github.com/minetest/minetest/pull/5589 * Permit to enable/disable mods on CSM https://github.com/minetest/minetest/pull/5554 (very important)

2017-05-06
Minetest developer meeting 2017-05-06

Old business: "Add configurable key bindings" (#1439) Rebased by nore, needs tweaks and merge. Additional hooks for inventory manipulations callbacks (#4035) Krock is working on it, progress? Timed move (#1489) Merge/close? nerzhul: close for me   Use a settings object for the main settings (#1949) Rebased, should be merged soon. Split block position in SQLite3 database (and use R*Tree) (#1845) Rebased, performance tests to do soon(tm). nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date) Add list_predict formspec element (#1988) sofar worked on a rebase a while ago. Close or merge a modern version? Use object property "stepheight" for player stepheight too. (#2056)       Agreed to merge, but nobody yet assigned. Make players respect makes_footstep_sound in the object properties (#2852) Agreed to merge, nobody assigned.

New business: Add support for multiple listen addresses (#2604) Big patch, but can't really be split up much more since everything's interdependent. Make the player collisionbox settable (#2738) Clean up numeric.h and split FaceFositionCache from it (#3256) Up-to-date and looks more-or-less ready to merge, just needs formal approval. :       https://github.com/minetest/minetest/pull/5361 Background color on textarea/field/pwdfield)        https://github.com/minetest/minetest/pull/5355: vertical bar in text areas        Possible close: https://github.com/minetest/minetest/pull/5281 and https://github.com/minetest/minetest/pull/5279        Documentation update: https://github.com/minetest/minetest/pull/4968 close or not close ?

For future meeting: If finished: https://github.com/minetest/minetest/pull/5544 (on_item_use CSM)

2017-04-29
Arranged by: paramat

ShadowNinja is unavailable today, paramat will be hosting in his stead.

Old business: (Things mentioned at previous meetings that haven't yet been resolved)


 * "Add configurable key bindings" (#1439)
 * Rebased by nore, needs tweaks and merge.
 * Additional hooks for inventory manipulations callbacks (#4035)
 * Krock is working on it, progress?
 * Minetest subgame meeting
 * paramat held one at 1900Z
 * Timed move (#1489)
 * Merge/close?
 * Use a settings object for the main settings (#1949)
 * Rebased, should be merged soon.
 * Split block position in SQLite3 database (and use R*Tree) (#1845)
 * Rebased, performance tests to do soon(tm).
 * nerzhul: performance is a little bit slower, but it's ridiculous compared to mapblock deserialization part (no bench to show it atm, but i did it 2 years ago on my fork, and SQLite was improved since this date)
 * Add list_predict formspec element (#1988)
 * sofar worked on a rebase a while ago. Close or merge a modern version?

New business: (mainly oldest or least recently updated PRs that haven't been handled yet)


 * Added formspec element 'KeyEventBox' for charbased input by ninnghazad (#1737)
 * Closed: better done with client-side mods.
 * Use object property "stepheight" for player stepheight too. (#2056)
 * Agreed to merge, but nobody assigned.
 * Add mapgen settings to create world dialog by srifqi (#2561)
 * Merge, but needs changes.

2017-04-22
Minetest developer meeting 2017-04-22

Old business: "getTime refactoring" (#1188) not yet merged (nore) "Delayed shutdown" merged by nerzhul. "Disable Android LevelDB" rebased by nore and merged by nerzhul. "Add configurable key bindings" (#1439) rebased by nore, needs tweaks and merge. Additional hooks for inventory manipulations callbacks (#4035) Needs rebase and cleanup, volunteers? Rename .txt to .md (#5451)?

New business: (PRs obtained by sorting by age on GitHub) Schedule Minetest subgame meeting? Coverity: https://scan.coverity.com/projects/12441 Need to triage issues, many legitimate issues to fix. Timed move (#1489) Added formspec element 'KeyEventBox' for charbased input (#1737) Add ability to escape commands with a back-slash (#1768)  I may want to tweak this so that it doesn't strip the slash when you type "\foo" (or maybe not?    It should at least work for the client-side dot-commands too) Split block position in SQLite3 database (and use R*Tree) (#1845) Use a settings object for the main settings (#1949)  I also unified Initialize and InitiallizeAsync, which was necessary to implement this. I think the unification is worth merging even without the settings object change. This conflicts with just about the entire ScriptAPI, so I don't want to rebase it too often :-)   Add listpredict to the formspec (#1988)    Add non-global mod namespaces (#2039)