Skip to content

Instantly share code, notes, and snippets.

@sorcerykid
Last active December 21, 2018 02:42
Show Gist options
  • Save sorcerykid/68056c844f927618bae1fb315b5bf9f7 to your computer and use it in GitHub Desktop.
Save sorcerykid/68056c844f927618bae1fb315b5bf9f7 to your computer and use it in GitHub Desktop.
I have been contributing code to Minetest for months, not just sitting on code.
@sorcerykid
Copy link
Author

sorcerykid commented Jul 16, 2018

It's okay if Shara complains about Minetest. nobody tells her to stop bitching. rubenwardy even says sure. Of course oddly Shara is also the reason the auth issue was reopened and put on high priority.

13:55 Shara I guess an explanation of how it's meant to be used in the readme was too much to expect as well.
2 more elements. Show/hide.
14:07 rubenwardy Minetest.conf
14:07 .example
14:07 :P
14:09 Shara rubenwardy: still nice to have things in the readme for the actual mod
14:09 johnnyjoy joined #minetest
14:10 Shara It's kind of silly that we don't do this
14:12 rubenwardy Sure

@sorcerykid
Copy link
Author

rubenwardy: well, socerykid does a nice job. perhaps try to catch as a core dev?
17:18 rubenwardy yeah, no
17:18 she's never helped review
17:18 or actually contributed
17:19 just thrown posted some random patches (which could cause data loss due to reduce save interval) on the forums
17:19 I'd welcome that to start now however :)
17:19 Sokomine perhaps it just takes a bit more talking with each other? most of socerykids work seems to be more of intrest for the engine and server owners than for individual players
17:19 that'd be nice, yes :-)
17:20 it's not as if core devs grow on trees .-)
17:20 plants a sapling...and...er...hopes
17:20 rubenwardy unfortunately not :(
17:20 we do need more core devs, and for that you need more contributors
17:21 the path usually goes:
17:21 Krock joined #minetest-hub
17:21 rubenwardy casual contributor -> contributor who also looks at other peoples PRs -> contributor who actively finds bugs and issues with other prs -> core devs

Apparently i posted multiple patches that can result in data loss, ignoring the fact that sfan5 contributed a PR that not only threatens data loss, but can actually take down a server. apparently these core devs don't understand the concept of a hotfix. my patch wasn't intended to be a "contribution", it was simply an interim fix for server operators to regain control of their server.

@sorcerykid
Copy link
Author

Thanks for your contribution. The mod works, but it's somewhat slow. You could use the Lua VoxelManipulator to speed up the node replacement a bit. If that's a bit complicated, then consider replacing set_node with swap_node, which also improves the performance a bit by skipping the Lua callbacks.

Apparently Krock even admits that "contributions" can include mods.
https://forum.minetest.net/viewtopic.php?p=326381#p326381

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment