| Time |
Nick |
Message |
| 00:23 |
|
octacian joined #minetest-dev |
| 00:49 |
|
hfranqui joined #minetest-dev |
| 00:51 |
hfranqui |
Hi, the other days I asked how to merge PR locally, and I wasn't able to thank you all paramat, nore, Fixer and nerzhul who helped me |
| 00:51 |
hfranqui |
Thanks |
| 01:17 |
|
ssieb joined #minetest-dev |
| 02:21 |
|
Tmanyo joined #minetest-dev |
| 02:24 |
|
RKINGLEGEND joined #minetest-dev |
| 02:24 |
|
RKINGLEGEND_ joined #minetest-dev |
| 02:24 |
|
RKINGLEGEND__ joined #minetest-dev |
| 03:49 |
|
Puka joined #minetest-dev |
| 04:06 |
|
YuGiOhJCJ joined #minetest-dev |
| 04:25 |
Zeno` |
#5781 |
| 04:25 |
ShadowBot |
https://github.com/minetest/minetest/issues/5781 -- Fix mem leak in mesh cache by Zeno- |
| 06:01 |
|
nerzhul joined #minetest-dev |
| 06:12 |
|
Hunterz joined #minetest-dev |
| 06:13 |
nerzhul |
hello, merging #5772 in ~5 mins |
| 06:13 |
ShadowBot |
https://github.com/minetest/minetest/issues/5772 -- Various code cleanup & little performance improvement on HTTP download by nerzhul |
| 06:15 |
nerzhul |
nice Zeno` |
| 06:15 |
nerzhul |
i missed it |
| 06:15 |
|
Fritigern joined #minetest-dev |
| 06:15 |
nerzhul |
merging #5781 too |
| 06:15 |
ShadowBot |
https://github.com/minetest/minetest/issues/5781 -- Fix mem leak in mesh cache by Zeno- |
| 06:27 |
|
troller joined #minetest-dev |
| 06:36 |
|
Fritigern joined #minetest-dev |
| 07:01 |
|
troller joined #minetest-dev |
| 07:36 |
|
proller joined #minetest-dev |
| 07:52 |
|
DS-minetest joined #minetest-dev |
| 08:12 |
|
Krock joined #minetest-dev |
| 08:12 |
nerzhul |
just pushing a trivial fix for https://github.com/minetest/minetest/commit/a4a377ecad6f1732cc54e7ac329fdef9949f9bf7 (x = cur +1 instead of x = cur) it works as intended |
| 08:16 |
Krock |
another trivial fix: https://hastebin.com/azupabewag.patch |
| 08:17 |
Krock |
fixes https://github.com/minetest/minetest/commit/e25a38e#commitcomment-22215004 |
| 08:17 |
nerzhul |
Krock, i was working on it :p |
| 08:17 |
Krock |
ninja'd :P |
| 08:17 |
Krock |
pushing in 10 mins |
| 08:18 |
nerzhul |
okay :p i prefer having if not minimap then |
| 08:18 |
nerzhul |
return |
| 08:18 |
nerzhul |
end |
| 08:18 |
nerzhul |
in the preview_minimap function |
| 08:18 |
Krock |
that's also a possibility |
| 08:24 |
nerzhul |
it permit to prevent error if calling multiple times this function :) |
| 08:26 |
Krock |
k.. whatever, changed it |
| 08:26 |
Krock |
print, return |
| 08:28 |
nerzhul |
nice |
| 08:28 |
nerzhul |
can you review tiny #5775 it's a wrong test done in our legacy code crashing server on wide chars |
| 08:28 |
ShadowBot |
https://github.com/minetest/minetest/issues/5775 -- [needs testing] Potential fix for chat/csm related crash by red-001 |
| 08:29 |
Krock |
k, so 5746 and 5775 testing together. let's see if that works |
| 08:32 |
nerzhul |
today it's time to merge and test the last remaining features. feature freeze will start just after our meeting (there is a meeting, right ?) |
| 08:33 |
Krock |
I hope there's one planned. if not, we'll need luck to get the devs online. |
| 08:36 |
Krock |
I'm afraid paramat is right and the sn*ak pull won't make it into this release |
| 08:36 |
nerzhul |
yeah, 3 or 4 dev is sufficient to make PR merged before feature freeze |
| 08:36 |
nerzhul |
what do you mean ? |
| 08:37 |
Krock |
talking about PR #5533, where it seems as nobody wants to give a real feedback |
| 08:37 |
ShadowBot |
https://github.com/minetest/minetest/issues/5533 -- Sneak: Strip down version by SmallJoker |
| 08:38 |
Krock |
nobody with the exception of paramat himself who keeps testing it :) |
| 08:40 |
Krock |
red-001, how can I reproduce the bug for #5775? |
| 08:40 |
ShadowBot |
https://github.com/minetest/minetest/issues/5775 -- [needs testing] Potential fix for chat/csm related crash by red-001 |
| 08:40 |
nerzhul |
it's very very difficult, but the fix is lgtm as we don't compare the good char types |
| 08:41 |
Krock |
ah, one of these crashes that happen once in a few days of playing. the fix looks good |
| 08:42 |
nerzhul |
yeah isspace & iswspace are standard functions i'm pretty confident it helps |
| 08:43 |
nerzhul |
could be nice to have a unittest on this function but it's not easy |
| 08:43 |
nerzhul |
(note: i also think we should change our unittest framework to use standard unittest framework cppunit which is great, but we will see this later |
| 08:44 |
nerzhul |
hmmm lol |
| 08:44 |
nerzhul |
if (word_end < m_line.size() && isspace(word_end)) |
| 08:44 |
nerzhul |
i think we have a problem on this function |
| 08:44 |
nerzhul |
isspace(int) |
| 08:44 |
nerzhul |
word_end is a position... |
| 08:44 |
nerzhul |
not the char itself |
| 08:45 |
nerzhul |
there is same problem in many places |
| 08:46 |
nerzhul |
i'm looking for cpp doc |
| 08:48 |
nerzhul |
okay it's really a char type mismatch |
| 08:48 |
nerzhul |
merging this fix, i need it to provide the next patch |
| 08:50 |
Zeno` |
hmm |
| 08:51 |
Zeno` |
there is really that problem in places? |
| 08:51 |
nerzhul |
what do you mean Zeno` ? |
| 08:51 |
Zeno` |
<nerzhul> if (word_end < m_line.size() && isspace(word_end)) |
| 08:52 |
Zeno` |
checking that the end "index" is a space or not?! |
| 08:52 |
nerzhul |
/ Last nick completion start (index into m_line) |
| 08:52 |
nerzhul |
it's the variable initing word_end |
| 08:52 |
Zeno` |
link to code? |
| 08:52 |
nerzhul |
isspace(index) doesn't mean anything, it's an index |
| 08:52 |
Zeno` |
correct |
| 08:53 |
nerzhul |
chat.cpp L541 |
| 08:53 |
Krock |
eeks |
| 08:53 |
nerzhul |
you will find this problem, u32 word_end = prefix_end; and u32 prefix_end = m_nick_completion_end; & previous comment :p |
| 08:53 |
Krock |
that's horrible |
| 08:53 |
Zeno` |
yikes |
| 08:53 |
Zeno` |
how did that happen?! |
| 08:54 |
nerzhul |
it's kahrl :p |
| 08:54 |
Zeno` |
nah, I'm sure kahrl could not have made that elementary mistake |
| 08:55 |
nerzhul |
sorry Zeno` => https://lut.im/HsENk2sfZL/t9zQGHPtxp5woajh.png |
| 08:56 |
Krock |
https://github.com/minetest/minetest/commit/967f2546diff-845c1df7811b52248eace7e5b209e757R534 |
| 08:56 |
Krock |
s/diff/#diff/ |
| 08:57 |
nerzhul |
very very old file :p |
| 08:57 |
nerzhul |
minetest was a baby |
| 08:57 |
Krock |
yeah |
| 08:57 |
nerzhul |
i don't know if a static analysis can find this, i hope yes |
| 08:59 |
Zeno` |
I don't it would because technically there is nothing wrong with that call to isspace (apart from it being nonsense) |
| 09:00 |
nerzhul |
isspace looks for 1 byte, is wspace look for 2 bytes, it's different when you use unicode chars |
| 09:05 |
|
lisac joined #minetest-dev |
| 09:07 |
nerzhul |
Zeno`, Krock #5783 |
| 09:08 |
ShadowBot |
https://github.com/minetest/minetest/issues/5783 -- chat.cpp fix wchar_t isspace -> iswspace & wrong isspace on an index by nerzhul |
| 09:08 |
nerzhul |
tested on autocompletion on MFF it works great |
| 09:11 |
Krock |
any other check to be fixed? |
| 09:14 |
red-001 |
unicode in setting files crashes minetest |
| 09:16 |
nerzhul |
no i searched over all files |
| 09:27 |
Zeno` |
#4810 |
| 09:27 |
ShadowBot |
https://github.com/minetest/minetest/issues/4810 -- Unittests crash on Release build |
| 09:27 |
Zeno` |
sofar's suggested fix does absolutely nothing, so there is no point merging it |
| 09:28 |
nerzhul |
yeah |
| 09:28 |
Zeno` |
see my new comment (where the exception really is happening) |
| 09:29 |
Zeno` |
it could be the compiler using more than 32 bits (for some kind of optimisation reason) |
| 09:32 |
Zeno` |
I really don't want to read the academic papers regarding that algorithm used so maybe the fix can be found another way |
| 09:32 |
nerzhul |
PCg random ? |
| 09:32 |
nerzhul |
when switching to C++11 we can remove this part to use std::mt15539 |
| 09:32 |
nerzhul |
:p |
| 09:34 |
Zeno` |
well hmmmm seemed to think pcg was better than mt |
| 09:34 |
Zeno` |
mt is easily implemented even now (I did it years ago) so no need to even wait for c++11 |
| 09:37 |
Zeno` |
personally I dunno why we don't just use MT |
| 09:37 |
Zeno` |
I mean it's a game |
| 09:37 |
Zeno` |
surely the distribution it generates is good enough for a game heh |
| 09:38 |
Zeno` |
plus it's understood a lot better |
| 09:41 |
nerzhul |
lol irony powa |
| 09:43 |
red-001 |
could someone review #5757? |
| 09:43 |
ShadowBot |
https://github.com/minetest/minetest/issues/5757 -- Improve password change menu by red-001 |
| 09:44 |
red-001 |
it's mostly a bug fix |
| 09:45 |
Krock |
I'm not really a fan of having passwords in variables |
| 09:45 |
|
paramat joined #minetest-dev |
| 09:45 |
Krock |
hello paramat |
| 09:45 |
nerzhul |
not me, but this object is detroyed after formspec close |
| 09:45 |
Krock |
s/not/neither/ ? |
| 09:46 |
red-001 |
Krock, no way around it |
| 09:47 |
red-001 |
resizing in Irrlich or at least the way minetest uses it requires recreating the whole interface from scratch |
| 09:47 |
Krock |
grabbing patch |
| 09:49 |
nerzhul |
red-001, can you explain us #5779 |
| 09:49 |
ShadowBot |
https://github.com/minetest/minetest/issues/5779 -- Fix CSM crash by red-001 |
| 09:49 |
Krock |
^ |
| 09:50 |
red-001 |
the client has no objectref |
| 09:50 |
nerzhul |
yeah, it's just a typo .? because server side we said it's ref whereas it's an id, exact ? |
| 09:50 |
red-001 |
so trying to return one crashes minetest |
| 09:51 |
paramat |
i have a suggestion: since freeze is planned for tomorrow (21st) and since the weekend is when more devs are present and active, it makes good sense to freeze at the end of tomorrow to get more stuff done first. rubenwardy sofar Zeno` nore and all |
| 09:52 |
red-001 |
nerzhul, object refs are stored in a lua table on the server-side and the client doesn't have access to it |
| 09:52 |
nerzhul |
we can delay it to tomorrow, but we really should use this evening meeting to merge the last features |
| 09:53 |
nerzhul |
red-001, okay, it's just the name ref on server side which is not really consistent with the stored id, i find that strange (not especially your code) |
| 09:53 |
nerzhul |
i approved #5757 |
| 09:53 |
ShadowBot |
https://github.com/minetest/minetest/issues/5757 -- Improve password change menu by red-001 |
| 09:56 |
Zeno` |
oh, don't freeze today |
| 09:56 |
Zeno` |
there is still much to be done I think |
| 09:56 |
nerzhul |
Zeno`, it was planed since months ago and we can keep features for a next release :p |
| 09:56 |
nerzhul |
but okay for delayed to tomorrow |
| 09:57 |
Zeno` |
I'm sure a day is not going to make a huge difference |
| 09:57 |
Zeno` |
(in fact it probably won't make any difference at all LOL) |
| 09:57 |
nerzhul |
sufficient, released is planed in june, in 3 or 4 weeks (look at forum post) |
| 09:58 |
Krock |
if we move the feature freeze phase to next week, there's a higher risk of having bugs in the stable version |
| 09:58 |
Zeno` |
well at least the major bugs are fixed I guess |
| 09:58 |
Krock |
but 4 weeks? that's more than enough |
| 09:58 |
nerzhul |
feature freeze is for bugs, yes |
| 09:58 |
Zeno` |
(or getting close to being fixed) |
| 10:09 |
nerzhul |
merging #5757 #5746 & #5777 in 10 mins |
| 10:09 |
ShadowBot |
https://github.com/minetest/minetest/issues/5757 -- Improve password change menu by red-001 |
| 10:09 |
ShadowBot |
https://github.com/minetest/minetest/issues/5746 -- Cleanup in content_mapblock by numberZero |
| 10:09 |
ShadowBot |
https://github.com/minetest/minetest/issues/5777 -- Fix shift key producing space in console by davisonio |
| 10:10 |
nerzhul |
and maybe #5783 is Zeno` or Krock looks at :D |
| 10:10 |
ShadowBot |
https://github.com/minetest/minetest/issues/5783 -- chat.cpp fix wchar_t isspace -> iswspace & wrong isspace on an index by nerzhul |
| 10:13 |
|
Topic for #minetest-dev is now Minetest core development and maintenance. Feature freeze for 0.4.16: 22st May 23:59 UTC. Chit-chat goes to #minetest. http://irc.minetest.net/minetest-dev/ http://dev.minetest.net/ |
| 10:13 |
nerzhul |
Zeno`, i was wrong, 2 weeks of feature freeze |
| 10:14 |
|
Topic for #minetest-dev is now Minetest core development and maintenance. Feature freeze for 0.4.16: 21st May 23:59 UTC. Chit-chat goes to #minetest. http://irc.minetest.net/minetest-dev/ http://dev.minetest.net/ |
| 10:14 |
nerzhul |
and i delayed the FF from 0:00 to 23:59 paramat |
| 10:14 |
paramat |
nice |
| 10:16 |
|
Zeno` joined #minetest-dev |
| 10:30 |
nerzhul |
the 3 pr are merged |
| 10:34 |
nerzhul |
okay second patchset to merge in 10 mins: #4994 & #5783 (ty for review Zeno` |
| 10:34 |
ShadowBot |
https://github.com/minetest/minetest/issues/4994 -- Clear Inventory Chatcommand (/clearinv) by octacian |
| 10:34 |
ShadowBot |
https://github.com/minetest/minetest/issues/5783 -- chat.cpp fix wchar_t isspace -> iswspace & wrong isspace on an index by nerzhul |
| 10:34 |
nerzhul |
(after that less than 150 PR) |
| 10:35 |
paramat |
wow |
| 10:40 |
paramat |
#5766 is ready for merge, retested by 2 people |
| 10:40 |
ShadowBot |
https://github.com/minetest/minetest/issues/5766 -- Particle spawner: Do not spawn particles distant from player by paramat |
| 10:44 |
|
cx384 joined #minetest-dev |
| 10:56 |
nerzhul |
merging done |
| 10:59 |
|
juhdanad joined #minetest-dev |
| 11:16 |
|
DS-minetest joined #minetest-dev |
| 11:43 |
|
Fixer joined #minetest-dev |
| 11:43 |
|
Zeno` joined #minetest-dev |
| 11:55 |
Krock |
Edge case of feature/bugfix: #5784 |
| 11:55 |
ShadowBot |
https://github.com/minetest/minetest/issues/5784 -- [CSM] Correct the log destination of print() by SmallJoker |
| 12:08 |
|
Raven262 joined #minetest-dev |
| 12:11 |
|
Raven262 joined #minetest-dev |
| 12:18 |
|
Raven262 joined #minetest-dev |
| 12:24 |
Krock |
merging game#1739 in 5 mins |
| 12:24 |
ShadowBot |
https://github.com/minetest/minetest_game/issues/1739 -- shorter and better abm's by theFox6 |
| 12:25 |
paramat |
ok |
| 12:29 |
|
garywhite joined #minetest-dev |
| 13:20 |
|
Gundul joined #minetest-dev |
| 13:27 |
|
Gundul joined #minetest-dev |
| 13:29 |
|
nerzhul joined #minetest-dev |
| 13:30 |
|
Thomas-S joined #minetest-dev |
| 13:37 |
|
Thomas-S_ joined #minetest-dev |
| 13:38 |
|
paramat joined #minetest-dev |
| 13:39 |
nerzhul |
we really should do a pass on issues and close the unneeded issues |
| 13:39 |
nerzhul |
840 issues is impossible to manage, i think there are duplicates issues |
| 13:41 |
nerzhul |
Krock, is #664 fixed ? |
| 13:41 |
ShadowBot |
https://github.com/minetest/minetest/issues/664 -- Jumping on a node higher than you will make you slowly fall until you land |
| 13:43 |
Krock |
too bad the video is gone |
| 13:43 |
nerzhul |
Calinou, ^ |
| 13:43 |
red-001 |
#2989 |
| 13:43 |
ShadowBot |
https://github.com/minetest/minetest/issues/2989 -- Can't serialize data of type userdata |
| 13:43 |
red-001 |
^ doesn't seem like an issue |
| 13:44 |
Krock |
^ can't fix |
| 13:44 |
nerzhul |
we cant fix 2989 it's just bad implementation |
| 13:44 |
nerzhul |
#1106 is fixed no ? |
| 13:45 |
ShadowBot |
https://github.com/minetest/minetest/issues/1106 -- Codepages support (utf-8) for all platforms |
| 13:45 |
red-001 |
it doesn't even sound like something that can be fixed |
| 13:45 |
nerzhul |
paramat, #1404 is fixed no ? |
| 13:45 |
ShadowBot |
https://github.com/minetest/minetest/issues/1404 -- water opacity bug with irrlicht 1.8.x |
| 13:46 |
paramat |
no idea |
| 13:46 |
nerzhul |
i don't encounter this problem since a long time |
| 13:46 |
nerzhul |
Krock, #1460 ? |
| 13:46 |
ShadowBot |
https://github.com/minetest/minetest/issues/1460 -- Sound repeats on ladders |
| 13:47 |
Krock |
still present |
| 13:47 |
paramat |
but minor, happens in several situations |
| 13:47 |
nerzhul |
okay |
| 13:48 |
nerzhul |
#1508 and #1959 are obscure i think we should close them |
| 13:48 |
ShadowBot |
https://github.com/minetest/minetest/issues/1508 -- ERROR[main]: WARNING: updateListener: invalid value |
| 13:48 |
ShadowBot |
https://github.com/minetest/minetest/issues/1959 -- Public serverlist causes Minetest to freeze on client tab |
| 13:49 |
paramat |
ah 1404 may be fixed now that i have removed the duplicate water surface shader, only water top tiles were sent to that |
| 13:49 |
nerzhul |
yeah i think it's this thing |
| 13:49 |
nerzhul |
Krock, #2231 fixed ? |
| 13:49 |
ShadowBot |
https://github.com/minetest/minetest/issues/2231 -- Unit test fails |
| 13:50 |
Krock |
can refresh with a new error |
| 13:52 |
nerzhul |
i think #2283 can be closed too, we massively improve shader performance, i always play with shaders enabled and i don't notice any performance drop due to them |
| 13:52 |
ShadowBot |
https://github.com/minetest/minetest/issues/2283 -- Bad render performance based on shader data |
| 13:53 |
Krock |
well, the test fail ratio got better - at least that :D |
| 13:53 |
Krock |
1 / 13 vs 6 / 87 |
| 13:53 |
nerzhul |
paramat, #2399 does the flowing commits helps ? |
| 13:53 |
ShadowBot |
https://github.com/minetest/minetest/issues/2399 -- Lava doesnt flow correct |
| 13:53 |
nerzhul |
lol |
| 13:54 |
nerzhul |
Krock, #2659 |
| 13:54 |
ShadowBot |
https://github.com/minetest/minetest/issues/2659 -- Deleting blocks - unable deactivate deleted objects |
| 13:54 |
paramat |
664 still happens, but only in rare cases, keep open i think but low priority |
| 13:55 |
nerzhul |
#3389 should be easily fixed, why we can't do it now ? |
| 13:55 |
ShadowBot |
https://github.com/minetest/minetest/issues/3389 -- Unknown items don't dropping on minetest 0.4.13-dev (18.11.2015) |
| 13:56 |
Krock |
is that even a bug? unknown stuff shouldn't lay around |
| 13:58 |
nerzhul |
i don't know but you should have the possibility to remove them from your inventory no ? |
| 13:58 |
paramat |
i doubt 2399 is fixed, i think it's inevitable behaviour caused by 2 liquids fighting to enter a node |
| 14:00 |
paramat |
Krock see my comment in 1508 the actual issue here may be teleport to 'inf' co-ordinate |
| 14:03 |
Krock |
or a mod that somehow managed to get in a calculation a inf value |
| 14:03 |
paramat |
i agree to closing 2283 |
| 14:03 |
red-001 |
#3389 is a really annoying bug |
| 14:03 |
ShadowBot |
https://github.com/minetest/minetest/issues/3389 -- Unknown items don't dropping on minetest 0.4.13-dev (18.11.2015) |
| 14:03 |
paramat |
closed 2283 |
| 14:03 |
Krock |
checking against being valid positions would help there |
| 14:04 |
nerzhul |
pushing a typo fix for #4293 (in README.md, case sensitive PostgreSQL_INCLUDE_DIR variable |
| 14:04 |
ShadowBot |
https://github.com/minetest/minetest/issues/4293 -- POSTGRE_SQL_INCLUDE_DIR option doesn't work. |
| 14:10 |
paramat |
i agree with closing 1508 and commented |
| 14:24 |
nerzhul |
i don't know what to do with this amount of issues :p |
| 14:24 |
nerzhul |
many seems lgtm |
| 14:24 |
nerzhul |
we should do many fixed, especially on recent reports in the feature freeze |
| 14:35 |
nerzhul |
merging #5779 & #5784 in ~10 mins |
| 14:36 |
ShadowBot |
https://github.com/minetest/minetest/issues/5779 -- Fix CSM crash by red-001 |
| 14:36 |
ShadowBot |
https://github.com/minetest/minetest/issues/5784 -- [CSM] Correct the log destination of print() by SmallJoker |
| 14:38 |
paramat |
#5766 ? |
| 14:38 |
ShadowBot |
https://github.com/minetest/minetest/issues/5766 -- Particle spawner: Do not spawn particles distant from player by paramat |
| 14:39 |
Krock |
seems to be tested, thus ok. |
| 14:40 |
|
Gundul joined #minetest-dev |
| 14:40 |
nerzhul |
the particle object is just shit :p |
| 14:40 |
nerzhul |
look at this constructor.. |
| 14:40 |
nerzhul |
braces are all wrong paramat fix it please |
| 14:40 |
Krock |
that's copypasta |
| 14:40 |
nerzhul |
yeah but our rules said fix :) |
| 14:41 |
nerzhul |
and it's good :p |
| 14:41 |
Krock |
uh.. where? |
| 14:43 |
paramat |
yes there are code style issues nearby, but i would like to fix those in a separate PR |
| 14:43 |
paramat |
the whole file has bad code style, i will fix this separately |
| 14:44 |
Krock |
the freeminer devs will cry |
| 14:44 |
nerzhul |
i can link a PR where some said, forks are not our problem, and i agree :p |
| 14:44 |
paramat |
the diff makes it appear i have added bad code style, but those added lines are mostly pre-existing lines |
| 14:45 |
paramat |
the diff is quite confusing actually |
| 14:45 |
nerzhul |
merging the 3 PR , but at a point we really should fix all those code style use clang-format define which rules to use with clang-format and conform to it and give end user the guidelines |
| 14:45 |
nerzhul |
merging the 3 PR then |
| 14:46 |
paramat |
i have only added a few lines, but indented a lot of existing code |
| 14:47 |
|
kaeza joined #minetest-dev |
| 14:47 |
paramat |
wow thanks :] |
| 14:47 |
nerzhul |
but on next PR ensure it's consisten |
| 14:47 |
nerzhul |
t* |
| 14:49 |
paramat |
who approved 5766? just so that i can add the label |
| 14:50 |
nerzhul |
#5766 |
| 14:50 |
ShadowBot |
https://github.com/minetest/minetest/issues/5766 -- Particle spawner: Do not spawn particles distant from player by paramat |
| 14:50 |
nerzhul |
me |
| 14:50 |
paramat |
ok |
| 14:50 |
paramat |
thanks |
| 14:50 |
nerzhul |
no better soluce without breaking things |
| 14:50 |
nerzhul |
the next release will need good C++ competences to review PR |
| 14:50 |
nerzhul |
i started a project about the C++11 conversion |
| 14:51 |
nerzhul |
https://github.com/minetest/minetest/projects |
| 14:51 |
red-001 |
clangformat doesn't always generate the cleanest looking code |
| 14:51 |
paramat |
have we agreed to move to C++11? |
| 14:51 |
nerzhul |
yes |
| 14:52 |
paramat |
ok i missed that, thought it was just your suggestion at the moment |
| 14:52 |
nerzhul |
no, i suggest it at... first or second meeting and it was accepted, C++11 is supported since Windows Vista, and GCC 4.9/Clang 3.5 (we are now with GCC 7.1 and Clang 3.9/4.0 |
| 14:53 |
nerzhul |
Debian 8 i just good, the only problematic thing is android i should test the NDK build with C++11 when flags will be set on project |
| 14:53 |
nerzhul |
i will essential work on this maintenance after release |
| 14:54 |
nerzhul |
on your C++ code parts, the randomgen will be converted to C++11 random generators which are better than our algorithm (STL quality :p) |
| 14:54 |
Calinou |
Krock: nerzhul: I need to check if this still occurs (Jumping on a node higher than you will make you slowly fall until you land) |
| 14:55 |
nerzhul |
but we have 2 weeks for finish this C++03 build |
| 14:55 |
Krock |
would be glad if you could refresh this issue :) |
| 14:56 |
nerzhul |
i think i will finish the chat protocol rewrite, and as it's a breakage version i will really make it good heh (compat is a pain, especially on lua api for this fix) |
| 14:56 |
nerzhul |
also i will work on long time request mod channels (sofar <-) |
| 14:59 |
red-001 |
wasn't est31 working on utf-8 for chat or something? |
| 15:00 |
Calinou |
Krock, nerzhul: tested on latest Minetest Git commit (60baf81), still occurs |
| 15:00 |
nerzhul |
i can't remember, but i will ensure it uses wchat_t |
| 15:00 |
Calinou |
it's easy to test (works in both singleplayer and multiplayer, and any subgame) |
| 15:00 |
nerzhul |
wchar_t* |
| 15:00 |
nerzhul |
Calinou, ty |
| 15:00 |
Calinou |
jump on a node that's 1 node higher than you |
| 15:01 |
Calinou |
you will slowly sink into the ground (but it's a bit faster than before now), still noticeable |
| 15:01 |
Calinou |
well, not "into" but "onto" |
| 15:01 |
Calinou |
you'll never be stuck in the node |
| 15:01 |
Calinou |
but it still feels strange, no other game engine I know does this |
| 15:01 |
Calinou |
(including older versions of Minetest, and that other voxel game) |
| 15:02 |
Calinou |
nerzhul: should I comment on the issue to indicate this? |
| 15:02 |
red-001 |
I wonder is this linking to the bug that lets you glitch into a node |
| 15:03 |
nerzhul |
Calinou, can be nice to have an update yes |
| 15:03 |
Calinou |
red-001: I don't think so |
| 15:03 |
Calinou |
I remember being able to fix this bug at the cost of another issue |
| 15:03 |
Calinou |
(related to step height or something?) |
| 15:04 |
Krock |
looks like it doesn't set the Y position to the surface and only resets the speed when almost touching the ground |
| 15:04 |
Calinou |
this doesn't seem to happen when sneak-jumping onto a node, too |
| 15:06 |
Calinou |
this occurs regardless of your horizontal speed, too (even if you're using fast mode, it can occur) |
| 15:07 |
|
Gundul joined #minetest-dev |
| 15:10 |
|
Gundul joined #minetest-dev |
| 15:53 |
paramat |
#5785 |
| 15:53 |
ShadowBot |
https://github.com/minetest/minetest/issues/5785 -- Fix instant digging by HybridDog |
| 15:54 |
paramat |
new attempt to fix #5728 |
| 15:54 |
ShadowBot |
https://github.com/minetest/minetest/issues/5728 -- Block instadig after tool switch |
| 15:59 |
VanessaE |
so am I to assume with the closing of #4459 that minetest will never get a dynamic view range again? |
| 15:59 |
ShadowBot |
https://github.com/minetest/minetest/issues/4459 -- Dynamic view range depending on frame rate |
| 16:03 |
nerzhul |
seems no atm |
| 16:03 |
VanessaE |
that sucks |
| 16:03 |
Krock |
"never" is quite a strong word. I'd say this won't be much a thing for the next few months until there are more players who want this |
| 16:05 |
VanessaE |
Krock: every time someone calls for auto view range, it gets closed or shouted down |
| 16:05 |
VanessaE |
the old algorithm WORKED |
| 16:05 |
VanessaE |
even if it was buggy |
| 16:05 |
VanessaE |
but nooooooo let's just remove it instead of improving it |
| 16:05 |
Krock |
buggy behaviour isn't really a desired feature |
| 16:06 |
VanessaE |
Krock: simple logic: leave the buggy feature in, as long as it doesn't break. those that don't like it can turn it off. those who can deal with the bugs, turn it on. but if you remove it, then NO one can turn it on. |
| 16:07 |
|
torgdor joined #minetest-dev |
| 16:11 |
red-001 |
#5554 |
| 16:11 |
ShadowBot |
https://github.com/minetest/minetest/issues/5554 -- [CSM] Allow enabling and disabling mods. by red-001 |
| 16:13 |
paramat |
it didn't work, and so was removed. we tried tuning it but it still had severe problems which many complained about, we have few complaints now about manual control |
| 16:16 |
VanessaE |
it DID work |
| 16:16 |
VanessaE |
it was just buggy |
| 16:17 |
VanessaE |
again, those who didn't like it could just disable it anyway |
| 16:17 |
VanessaE |
but now it's gone entirely, so even those of us who could tolerate the bugs have NO OPTION now |
| 16:17 |
VanessaE |
come on manm |
| 16:17 |
VanessaE |
-m |
| 16:17 |
VanessaE |
and fixed range DOES NOT WORK in a production environment |
| 16:18 |
paramat |
it's not fixed |
| 16:20 |
VanessaE |
um, yes it is. |
| 16:20 |
paramat |
maybe someone will come up with a much better implementation |
| 16:20 |
VanessaE |
it's "fixed" if the engine doesn't automatically adjust it |
| 16:20 |
VanessaE |
having to sit there on the +/- keys as you play kills immersion |
| 16:21 |
paramat |
manual control means not fixed |
| 16:21 |
VanessaE |
manual control |
| 16:21 |
* VanessaE |
grumbles both at the enter key and at paramat |
| 16:22 |
VanessaE |
manual control is still "fixed", the way we would use the term in American English |
| 16:22 |
nerzhul |
just wait |
| 16:22 |
VanessaE |
like a "fixed width font" can still be scaled up or down with manual controls. |
| 16:23 |
VanessaE |
nerzhul: bad advise.... |
| 16:23 |
VanessaE |
advice* |
| 16:23 |
paramat |
in the UK we drive fixed gear cars with only one gear :] |
| 16:23 |
VanessaE |
not the same thing :P |
| 16:26 |
Fixer |
i wonder why in creative pickaxes are so slow? |
| 16:27 |
paramat |
erm i seem to remember some reason, just use your hand |
| 16:40 |
|
garywhite joined #minetest-dev |
| 16:46 |
|
halt_ joined #minetest-dev |
| 16:47 |
|
Grandolf joined #minetest-dev |
| 17:12 |
kilbith_ |
> Dynamic view range |
| 17:12 |
kilbith_ |
no, no, no and NO. |
| 17:13 |
VanessaE |
then. turn. it. off. |
| 17:13 |
|
paramat joined #minetest-dev |
| 17:13 |
VanessaE |
(if it existed that is) |
| 17:13 |
|
YuGiOhJCJ joined #minetest-dev |
| 17:14 |
VanessaE |
"oh, this feature doesn't work for me, let's just remove it from the engine entirely." |
| 17:14 |
kilbith_ |
s/for me// |
| 17:14 |
kilbith_ |
fuck this shit really |
| 17:15 |
|
juhdanad joined #minetest-dev |
| 17:18 |
VanessaE |
again, it worked for me |
| 17:18 |
VanessaE |
buggy or not, it DID work |
| 17:18 |
kilbith_ |
re-add it in your fork |
| 17:18 |
VanessaE |
I ain't gonna fork for something stupid like that |
| 17:21 |
VanessaE |
it needs to be made a hard, fast, unbreakable rule in this project: if a feature exists and works for some people, and doesn't break anything (read: crash, memory corruption, or the like), then it shall not, under any circumstances be removed from the engine/game unless something is immediately ready to replace it. |
| 17:24 |
kilbith |
there was basically only you and your mate cheapie that wanted to keep this shit |
| 17:24 |
paramat |
'for some people'.. . and inflexible rules are bad |
| 17:26 |
VanessaE |
then come up with a set of reasonable circumstances where it's okay to remove a feature. |
| 17:26 |
paramat |
that's also an inflexible rule |
| 17:30 |
VanessaE |
there have to be rules. |
| 17:30 |
VanessaE |
flexibility is fine but there needs to be a line we do not cross. |
| 17:31 |
Fixer |
https://github.com/minetest/minetest/pull/5777#issuecomment-302883704 |
| 17:31 |
Fixer |
new bug? |
| 17:31 |
ShadowNinja |
nerzhul, rubenwardy, sfan5, sofar, Krock, nore, paramat: Meeting in 30 minutes. |
| 17:31 |
sfan5 |
welp, i'll not be here |
| 17:33 |
nerzhul |
ShadowNinja, nice, will be there |
| 17:33 |
VanessaE |
I'll be here, if it matters. |
| 17:33 |
paramat |
flexible lines are best |
| 17:34 |
paramat |
today has been one big meeting really, no need to wait 30 mins |
| 17:34 |
nerzhul |
paramat, i go to lunch |
| 17:34 |
nerzhul |
it's lunch time for me :p |
| 17:34 |
kilbith |
*to dinner |
| 17:36 |
Fixer |
confirm unicode input bug |
| 17:38 |
Fixer |
some nonlatin symbols work, some not - broken since today, will check exact commit |
| 17:39 |
Fixer |
possibly broken by this https://github.com/minetest/minetest/pull/5777 |
| 17:39 |
kilbith |
why would need to type unicode symbols anyway? |
| 17:40 |
Fixer |
i mean nonlatin symbols, for example cyrillic/etc |
| 17:40 |
Fixer |
people use it |
| 17:41 |
VanessaE |
kilbith: because some useful chars are only unicode. X11_compose-o-o --> ° .... X11_compose-t-m --> ™ .... X11_compose-o-r --> ® |
| 17:42 |
VanessaE |
and so on |
| 17:42 |
VanessaE |
(degrees, "TM", and (R), respectively) |
| 17:43 |
red-001 |
kilbith, people that don't use english? |
| 17:43 |
VanessaE |
^^^ those are routinely used by English speakers also |
| 17:43 |
kilbith |
or this one: âš¡âš¡ |
| 17:43 |
kilbith |
very useful |
| 17:44 |
red-001 |
I have no idea what you are arguing for |
| 17:44 |
ShadowNinja |
nerzhul: #5782: Isn't CSM disabled by default since it's still experimental? |
| 17:44 |
ShadowBot |
https://github.com/minetest/minetest/issues/5782 -- [CSM] Disable preview mod before release |
| 17:47 |
|
torgdor joined #minetest-dev |
| 17:53 |
nerzhul |
Fixer, seems possible, testing in a few minutes |
| 18:01 |
Krock |
yes, CSM is off by default |
| 18:02 |
rubenwardy |
I feel like that release really needs abuse protection for minetest.get_node on the client side |
| 18:02 |
rubenwardy |
it's a blocker to me |
| 18:02 |
rubenwardy |
well |
| 18:03 |
rubenwardy |
meh, it makes it easier to ore detect, but doesn't make more cheating possible |
| 18:03 |
nerzhul |
CSM is off, but if user enable it prview mod will just disrupt his gameplay |
| 18:04 |
ShadowNinja |
Alright, the dev meeting starts now. |
| 18:04 |
nerzhul |
nice |
| 18:04 |
nerzhul |
it's the last meeting before feature freeze |
| 18:04 |
ShadowNinja |
Krock: Anything on inventory manipulation callbacks? |
| 18:04 |
Krock |
as soon we have a digging function in CSM, it's no longer required to build a cheater version of Minetest and simply run the oredetect mod :3 |
| 18:05 |
nerzhul |
we will never add this Krock |
| 18:05 |
ShadowNinja |
sofar: You around? Anything on list_predict? |
| 18:05 |
nerzhul |
ShadowNinja, we really should focus on new features, old business is not important on this meeting |
| 18:05 |
ShadowNinja |
web#78: Lets choose someone to handle this. |
| 18:05 |
ShadowBot |
https://github.com/minetest/minetest.github.io/issues/78 -- Add better screenshots, and more screenshots |
| 18:06 |
Krock |
ShadowNinja, haven't looked into it anymore. I'd be glad if there's someone around who could overtake this one. |
| 18:06 |
rubenwardy |
I vote Wuzzy :D |
| 18:06 |
ShadowNinja |
nerzhul: Patience, we'll get to all the blockers too :-) |
| 18:06 |
Calinou |
hmm |
| 18:06 |
Calinou |
beware, really |
| 18:06 |
Calinou |
screenshots should show a vanilla or vanilla-ish game, buildings are fine but please no "I have 500 nodes in my subgame" crap |
| 18:07 |
Calinou |
they probably should not have HUD too |
| 18:07 |
Calinou |
except for one maybe, but even then I'm not sure, our HUD is not something to be proud of today |
| 18:07 |
rubenwardy |
agreed. Any screenshots should have consistent contents |
| 18:07 |
ShadowNinja |
Calinou: I think it's fine to have a few that show what's possible. Minetest is more than minetest_game. |
| 18:07 |
Calinou |
having occasional mods is OK, as I said |
| 18:07 |
rubenwardy |
I suggest different subgames, such as Pixture and Rpgtest |
| 18:07 |
Calinou |
that might just confuse people though |
| 18:07 |
VanessaE |
make some screenshots from one of my servers, there's lots of nice stuff to see there. |
| 18:07 |
rubenwardy |
they tend to look consistent, and show different game play |
| 18:08 |
Calinou |
the former has entirely different textures |
| 18:08 |
ShadowNinja |
Calinou: Also, 500 -> more like 10000 in some games :-P |
| 18:08 |
rubenwardy |
maybe point the different subgame stuff on the customise page though |
| 18:08 |
rubenwardy |
*put |
| 18:08 |
Calinou |
I don't know of any game whose official screenshots feature different base assets |
| 18:09 |
rubenwardy |
i think it's fine for the customise section |
| 18:09 |
ShadowNinja |
Calinou: Well, would you want to choose some screenshots and add them? Something a bit more interesting than the cave that we've got now. |
| 18:09 |
rubenwardy |
plus, we're supposed to be an engine primarily |
| 18:10 |
paramat |
last time we discussed this it was strongly suggested to show off mods and other subgames, and i agree |
| 18:10 |
nerzhul |
agree variety of screens are important |
| 18:10 |
rubenwardy |
maybe we should have Godot style images on the homepage? Ie: showing lots of different games that are possible |
| 18:10 |
Krock |
a few multiplayer screenshots wouldn't be too bad either |
| 18:11 |
nerzhul |
Fixer, for the problem i see same problem as we fixed this morning... |
| 18:11 |
nerzhul |
isprint vs iswprint !! |
| 18:11 |
nerzhul |
i push directly the fix to master, the char is a wchar_t |
| 18:11 |
Calinou |
rubenwardy: we're both an engine and game, really |
| 18:11 |
nerzhul |
(just as code consistency), tell me if it's okay for you, i tryed é€@ĸ and it works for me |
| 18:12 |
Calinou |
don't fall into "we're an engine!!1!1" syndrome that has been plaguing projects like ioquake3 :( |
| 18:12 |
paramat |
but maybe sections for screenshots: vanilla MTG, mods, subgames |
| 18:12 |
* VanessaE |
hands ShadowNinja a gavel |
| 18:12 |
Calinou |
multiplayer screenshots make sense |
| 18:12 |
ShadowNinja |
Calinou: Can you pick a good mix of stock and moded screenshots for the site and install them? |
| 18:12 |
Calinou |
ShadowNinja: sure, but I'll probably retake the screenshots myself |
| 18:12 |
Calinou |
to make sure they all use the same settings |
| 18:13 |
|
destt_ joined #minetest-dev |
| 18:13 |
Calinou |
anti-aliasing/anisotropic filtering are very important to turn on, and the view distance should preferably be the same on all |
| 18:13 |
paramat |
nah |
| 18:13 |
Krock |
would be nice to have but variety comes from different players |
| 18:13 |
ShadowNinja |
Calinou: Alright, make sure you show off what minetest can do. Include things like large buildings, heavy games (dreambuilder) and action (TNT going off?). |
| 18:13 |
Calinou |
but design by committee sucks :) |
| 18:14 |
Calinou |
(c55 himself said that a lot back in the day) |
| 18:14 |
Krock |
indeed, it's less consistent |
| 18:14 |
Calinou |
ShadowNinja: yeah I'm fine with that; large mods are not a problem as long as it looks good |
| 18:14 |
Calinou |
a lot of mods have terrible textures, let's face it |
| 18:14 |
ShadowNinja |
Calinou: Exactly, that's why I want to assign one person :-) |
| 18:14 |
Calinou |
we could even have an official trailer, but that's for later |
| 18:14 |
paramat |
we don't want everything looking the same, and you then restrict yourself to what you can currently access, there is a huge variety of pre-existing screenshots |
| 18:14 |
Calinou |
Minecraft had awesome trailers back in the day |
| 18:14 |
nerzhul |
Fixer, https://github.com/minetest/minetest/commit/a2bb776ea81715a7b9eae4b2ea8aca032d42fb5e |
| 18:15 |
Krock |
ShadowNinja, dreambuilder is no longer a heavy subgame. it's now a very heavy modpack :P |
| 18:15 |
ShadowNinja |
Next: #5767 |
| 18:15 |
ShadowBot |
https://github.com/minetest/minetest/issues/5767 -- Don't add damage flash while punch texture modifier is active by stujones11 |
| 18:15 |
Calinou |
this is the post-1.0 one from Minecraft, https://www.youtube.com/watch?v=MmB9b5njVbA |
| 18:15 |
Calinou |
there was a Beta 1.0-1.8 one but I can't find it, it's made by the same person |
| 18:15 |
nerzhul |
i didn't get time to test it but the algorithm seems fine |
| 18:15 |
ShadowNinja |
Blocker, needs some review and merge. |
| 18:16 |
Krock |
didn't test but the code looks good |
| 18:16 |
nerzhul |
should be merged before release, we have two weeks for testing it |
| 18:16 |
Calinou |
https://www.youtube.com/watch?v=m_yqOoUMHPg |
| 18:16 |
Calinou |
that's the old one, found it! |
| 18:16 |
Fixer |
nerzhul: i'm checking that shift commit first, since someone mentioned it |
| 18:16 |
rubenwardy |
Mods suck |
| 18:16 |
Calinou |
obviously we're not able to reach that production quality |
| 18:17 |
Calinou |
but we should eventually try something |
| 18:17 |
rubenwardy |
the real power of Minetest is the subgames - being able to make your own consistent game |
| 18:17 |
Calinou |
we have cinematic camera, maybe we need a few more camera tools |
| 18:17 |
Calinou |
not all mods suck, plenty of them make sense |
| 18:17 |
Calinou |
the more a mod follows the UNIX philosophy, the better it is, usually |
| 18:17 |
rubenwardy |
do only one thing? |
| 18:17 |
Fixer |
beta 1.7.3 like gameplay is what you need :trollface: |
| 18:17 |
Calinou |
yeah, mostly |
| 18:17 |
Calinou |
"Write programs that do one thing and do it well." |
| 18:18 |
rubenwardy |
yeah agreed |
| 18:18 |
* rubenwardy |
looks at default |
| 18:18 |
nerzhul |
ShadowNinja, i suggest to delay 5767 to next meeting if not merged, letting us time to test it |
| 18:18 |
ShadowNinja |
For #5782, CSM is disabled so I don't think this is really a blocker. |
| 18:18 |
ShadowBot |
https://github.com/minetest/minetest/issues/5782 -- [CSM] Disable preview mod before release |
| 18:18 |
Calinou |
anyway I'll try putting some screenshots up soon |
| 18:18 |
ShadowNinja |
nerzhul: Yep, I'll wait for review on that one. |
| 18:18 |
Calinou |
I guess I'll hop on VanessaE's servers, try some mods/subgames locally |
| 18:18 |
Calinou |
may the block loading be with me! |
| 18:19 |
Calinou |
(I have to walk around quite a bit to make sure all blocks in view range are generated/loaded…) |
| 18:19 |
ShadowNinja |
Calinou: Yeah, uh, don't include the FPS counter :-P |
| 18:19 |
nerzhul |
5782 is a blocker as if a user enable CSM (accidentaly or not) preview mod will just do many unneeded things on gameplay |
| 18:19 |
Calinou |
I have a GTX 1080, I can get > 60 FPS literally anywhere |
| 18:20 |
Calinou |
all screenshots should be 1920×1080 I guess |
| 18:20 |
Calinou |
no need for more, and we can make smaller thumbnails |
| 18:20 |
nerzhul |
we should have this to permit CSM adoption and let modders start the CSM without problems (also it's basicly a refactor of the mod loading interface to make it more generic) |
| 18:20 |
paramat |
and a screenshot from each mapgen to show the variety |
| 18:21 |
ShadowNinja |
nerzhul: Well, if the user enables an experimental feature they can expect some things to change. |
| 18:21 |
nerzhul |
yeah but preview mod is not just a change, it can make the gameplay... unusable :p |
| 18:21 |
nerzhul |
preview contain all CSM calls , or nearly all |
| 18:22 |
nerzhul |
ShadowNinja, for disabling it, it's not the issue itself to merge |
| 18:22 |
ShadowNinja |
nerzhul: Well, still experimental so IMO it doesn't matter. |
| 18:22 |
nerzhul |
it's #5554 |
| 18:22 |
ShadowBot |
https://github.com/minetest/minetest/issues/5554 -- [CSM] Allow enabling and disabling mods. by red-001 |
| 18:22 |
nerzhul |
this PR merge solve the issue :) |
| 18:24 |
nerzhul |
without that you cannot disable any CSM mod (except if you remove it) |
| 18:24 |
nerzhul |
but anyway i'm okay for removing blocker if 5554 is merged |
| 18:25 |
ShadowNinja |
nerzhul: Alright, well I think we can merge 5554. I looked over it and it looks pretty reasonable. Any other comments on merging? |
| 18:25 |
nerzhul |
just merge :) |
| 18:25 |
nerzhul |
just factorize mod config code permitting to expose it to CSM engine :) |
| 18:26 |
Fixer |
nerzhul: Fix shift key producing space in console (#5777) introduced that bug (https://github.com/minetest/minetest/commit/605599b6f150b89ba6539c4d088231b326adcb48) |
| 18:26 |
ShadowBot |
https://github.com/minetest/minetest/issues/5777 -- Fix shift key producing space in console by davisonio |
| 18:26 |
nerzhul |
Krock, rubenwardy ? |
| 18:26 |
rubenwardy |
#5554 ? |
| 18:26 |
Krock |
5554, +1 for the concept |
| 18:26 |
ShadowBot |
https://github.com/minetest/minetest/issues/5554 -- [CSM] Allow enabling and disabling mods. by red-001 |
| 18:27 |
nerzhul |
y |
| 18:27 |
paramat |
uh but there is no description, how are we meant to judge this? |
| 18:27 |
rubenwardy |
I think that it's ultimately pointless, as the point of CSM should be to add client-side prediction, and make GUIs better |
| 18:27 |
rubenwardy |
not for users to change their own gameplay client-side |
| 18:28 |
Krock |
so just as extentions? |
| 18:28 |
nerzhul |
paramat, i did the descriptiong: move the Server mod configuration handling code to common interface permitting CSM to have the same interface :) |
| 18:28 |
paramat |
5782 is better |
| 18:28 |
Fixer |
nerzhul: oh, you already pushed a fix? |
| 18:28 |
nerzhul |
rubenwardy, both are important |
| 18:28 |
nerzhul |
Fixer, i pushed a fix on this PR |
| 18:28 |
nerzhul |
you should test it |
| 18:28 |
Fixer |
nerzhul: will check |
| 18:28 |
kilbith |
CSM is not usable for closed-source stuff, unfortunately |
| 18:29 |
nerzhul |
then let's go for 5554, can we look at next PR ? |
| 18:29 |
rubenwardy |
well, running GPL javascript in a web browser doesn't make the browser GPL, unless it's bundled with it |
| 18:29 |
rubenwardy |
ianal though |
| 18:30 |
paramat |
hmm so csm mods appear and are enabled just like all other mods? |
| 18:30 |
nerzhul |
Fixer, i close #5667 both paramat PR has been merged |
| 18:30 |
rubenwardy |
paramat, there's no GUI |
| 18:30 |
ShadowBot |
https://github.com/minetest/minetest/issues/5667 -- Particles rendered for everyone on server irregardless of distance/visibility |
| 18:30 |
paramat |
no description in a PR is unnacceptable |
| 18:30 |
rubenwardy |
just a config file |
| 18:30 |
ShadowNinja |
rubenwardy: We can't do that yet though, because the current snadbox is quite weak and we can't execute arbitrary remote code without a strong sandbox. |
| 18:30 |
nerzhul |
paramat, there is no formspec interface atm, it's just configuration, and they are not confused with other mods |
| 18:30 |
Fixer |
nerzhul: right, i will also close another issue if i confirm it is fixed |
| 18:31 |
Fixer |
lof fps bug* |
| 18:31 |
Fixer |
low* |
| 18:31 |
rubenwardy |
ShadowNinja, I'm not saying that we should start that now, I just worry that once we have this feature it'll never be removed |
| 18:32 |
paramat |
hmm ok, but i am very unimpressed by lack of description |
| 18:32 |
nerzhul |
both feature are important, prediction on formspec (example a craft guide seems good client side, and is not affected by lag), but for that in fact you need client GUI customization because your formspec is a client customization :D |
| 18:32 |
rubenwardy |
I think the best way for this is for clients to download client mods at the server's request from somewhere else than the server, ie: a mod store. For debugging, a secure setting is needed to allow unsecured mods |
| 18:33 |
nerzhul |
rubenwardy, yeah, it's not the purpose of this meeting, we can discuss about this after :) |
| 18:33 |
nerzhul |
this feature will not be added for tomorrow :p |
| 18:33 |
ShadowNinja |
nerzhul: I think you can go ahead and merge that PR after testing it now. |
| 18:34 |
paramat |
no description was what RBA did, irritating |
| 18:34 |
nerzhul |
ShadowNinja, nice, will re-verify if after meeting and merge |
| 18:34 |
nerzhul |
paramat, RBA did more complex PR :p |
| 18:34 |
ShadowNinja |
Next: https://github.com/minetest/minetest/issues/5728 Block instadig after tool switch |
| 18:35 |
nerzhul |
next issue is #5728, but there is no real PR associated |
| 18:35 |
ShadowBot |
https://github.com/minetest/minetest/issues/5728 -- Block instadig after tool switch |
| 18:35 |
nerzhul |
i tested one and it was worst |
| 18:35 |
rubenwardy |
I think you can merge #5554, as it's needed currently for testing - but you should accept that it can be thrown away at a later date when needed. Or at least the server should take priority |
| 18:35 |
paramat |
there is a new PR for that |
| 18:35 |
ShadowBot |
https://github.com/minetest/minetest/issues/5554 -- [CSM] Allow enabling and disabling mods. by red-001 |
| 18:35 |
paramat |
#5785 |
| 18:35 |
ShadowBot |
https://github.com/minetest/minetest/issues/5785 -- Fix instant digging by HybridDog |
| 18:35 |
nerzhul |
rubenwardy, priority can be done, but client should customize his game himself without server if needed, we will have an algo to determine this |
| 18:36 |
nerzhul |
okay this code move seems more reasonable |
| 18:36 |
nerzhul |
i'm testing it now |
| 18:36 |
VanessaE |
I need to remind you all of #5640 also... |
| 18:36 |
ShadowBot |
https://github.com/minetest/minetest/issues/5640 -- Automatic item and node colorization by juhdanad |
| 18:38 |
paramat |
yeah i notified reviewers of previous colour PRs, seems none are present |
| 18:38 |
nerzhul |
seems to fix the instadig bug |
| 18:38 |
|
torgdor joined #minetest-dev |
| 18:38 |
nerzhul |
cannot reproduce it |
| 18:38 |
nerzhul |
5785 is fine for me |
| 18:38 |
paramat |
maybe we can get 5640 reviewed by end of tomorrow |
| 18:39 |
nerzhul |
if others are okay we can merge it |
| 18:42 |
ShadowNinja |
How about #5627? I think it looks pretty good visually now. |
| 18:42 |
ShadowBot |
https://github.com/minetest/minetest/issues/5627 -- Menu: Merge singleplayer and server tabs by octacian |
| 18:42 |
nerzhul |
ShadowNinja, merge |
| 18:42 |
nerzhul |
Krock, are you okay with 5785 ? |
| 18:42 |
nerzhul |
Fixer has tested it and me too |
| 18:42 |
paramat |
well, 2 have tested 5785 and is simple, i'm sure we can merge this within a day |
| 18:42 |
ShadowNinja |
The file should probably be renamed too by whoever merges though. |
| 18:43 |
nerzhul |
ShadowNinja, which file ? |
| 18:43 |
nerzhul |
merging 5785 |
| 18:43 |
ShadowNinja |
(and the tab name at the bottom) |
| 18:43 |
|
tenplus1 joined #minetest-dev |
| 18:43 |
Krock |
nerzhul, if you're sure that it's fixed, all-ok. |
| 18:43 |
ShadowNinja |
nerzhul: tab_singleplayer -> tab_play |
| 18:43 |
tenplus1 |
hi peeps |
| 18:43 |
tenplus1 |
imma leave this here for devs: https://github.com/minetest/minetest_game/pull/1543 |
| 18:43 |
nerzhul |
tenplus1, it's core meeting not game :) |
| 18:43 |
VanessaE |
ShadowNinja: imho that's a bad name |
| 18:44 |
VanessaE |
because what is "client" then, if not "playing" the game also? |
| 18:44 |
tenplus1 |
ooh, meeting time :) |
| 18:44 |
ShadowNinja |
VanessaE: Maybe not an ideal name, but I don't want to bikeshed too much :-P |
| 18:45 |
tenplus1 |
? |
| 18:45 |
VanessaE |
maybe not, but the user has to understand it too |
| 18:45 |
VanessaE |
maybe "Local Game" for the "play" tab, and "Go Online" for the "client" tab? |
| 18:45 |
Fixer |
nerzhul: unicode bug fixed |
| 18:46 |
nerzhul |
Fixer, nice, if there is an issue, just close it :) |
| 18:46 |
nerzhul |
VanessaE, i proposed that but not very accepted :p |
| 18:46 |
nerzhul |
i proposed (online) |
| 18:47 |
nerzhul |
also Local Game permits to enable server mode with public list |
| 18:47 |
VanessaE |
that's the idea. |
| 18:47 |
VanessaE |
"Local Game" more easily encompasses both singleplayer and hosting, without implying that "client" ("whatever that is!") isn't also a "play" option. |
| 18:48 |
paramat |
client is not necessarily online |
| 18:48 |
VanessaE |
paramat: usually it is, I'd say. |
| 18:48 |
paramat |
and 'play' becomes hosting a server |
| 18:49 |
paramat |
.. which is not a local game |
| 18:49 |
VanessaE |
99% of the users who download this are going want to do two things: play a singleplayer game, or go on some public server. |
| 18:49 |
rubenwardy |
as it's the day before the freeze, we can talk about MTG features that are wanted desperately |
| 18:49 |
ShadowNinja |
I think local game and play online are good. |
| 18:49 |
rubenwardy |
(and already have PRs for) |
| 18:49 |
rubenwardy |
game#1543 |
| 18:49 |
ShadowBot |
https://github.com/minetest/minetest_game/issues/1543 -- Add 'creative' privilege for survival servers by tenplus1 |
| 18:49 |
|
torgdor joined #minetest-dev |
| 18:49 |
paramat |
many play LAN which is not really 'online' |
| 18:50 |
VanessaE |
paramat: from the standpoint of the computer running the client, it is. |
| 18:50 |
tenplus1 |
why not simply "singleplayer" and "multiplayer" ? |
| 18:50 |
ShadowNinja |
tenplus1: Because local games can be multiplayer. |
| 18:50 |
rubenwardy |
heh, game#1543 is non-urgent |
| 18:50 |
ShadowBot |
https://github.com/minetest/minetest_game/issues/1543 -- Add 'creative' privilege for survival servers by tenplus1 |
| 18:50 |
paramat |
yeah, technically |
| 18:50 |
tenplus1 |
not urgent, but a damn good feature for servers |
| 18:50 |
paramat |
game 1543 is not critical |
| 18:51 |
tenplus1 |
you realise how many owners will grant trusted players 'creative' to help out |
| 18:51 |
Krock |
well, there's "give" |
| 18:51 |
ShadowNinja |
Can we agree to merge with the tabs renamed "Local Game" and "Play Online"? |
| 18:51 |
tenplus1 |
I wouldnt give 'give'... too paranoid they give themselves hidden/special nodes... creative is much easier to regulate |
| 18:52 |
tenplus1 |
sounds good shadow |
| 18:52 |
paramat |
i think you're rushing 5627, we have 24 hrs |
| 18:52 |
nerzhul |
ShadowNinja, seems good yeah |
| 18:52 |
Krock |
+1 |
| 18:53 |
VanessaE |
+1 |
| 18:53 |
ShadowNinja |
paramat: There's plenty of time for bugfixes after the freeze if there's an issue. |
| 18:53 |
paramat |
and i disagree with 'local game' and 'play online' |
| 18:53 |
Krock |
tenplus1: the creative priv looks good |
| 18:53 |
nerzhul |
paramat, yeah but having a consensus permit to ensure merging :p |
| 18:53 |
tenplus1 |
thanks krock, and even sofar likes it... |
| 18:53 |
ShadowNinja |
paramat: Why? You would prefer? Is it that big of a deal? |
| 18:54 |
Krock |
paramat, how would you name them? Keep as-is? |
| 18:54 |
VanessaE |
ShadowNinja: one thing though, Jordach had a whole revamped menu he wanted to get in before 0.4.16. might be worth checking with him first. |
| 18:54 |
paramat |
so 'local game' contains internet multiplayer? =/ |
| 18:54 |
VanessaE |
"local game" refers to who hosts the server. :) |
| 18:54 |
ShadowNinja |
paramat: Yep, but hosted locally. |
| 18:54 |
rubenwardy |
no rewrite's getting into 0.4.16 at this point |
| 18:55 |
paramat |
we have 24 hrs, don't rush |
| 18:55 |
paramat |
but ok, it's borderline reasonable |
| 18:55 |
VanessaE |
paramat: or delay the freeze if needed |
| 18:55 |
ShadowNinja |
VanessaE: Eh, it's too late to merge it at this point, and this wouldn't conflict since he would just be replacing all of this. |
| 18:55 |
VanessaE |
ShadowNinja: ok, no worries. |
| 18:56 |
Fixer |
nerzhul: interestingly, i can't type one symbol i have... U+2116 â„– (Numero Sign) that was working before |
| 18:56 |
ShadowNinja |
paramat, rubenwardy: Does it seem mergable with "Local Game"/"Play Online"? |
| 18:57 |
paramat |
it's just a little weird you changing 5627 without the author knowing |
| 18:58 |
Fixer |
Local/Internet |
| 18:58 |
Fixer |
simple as that |
| 18:58 |
VanessaE |
Fixer: I can't type unicode chars at all in MT, either with X11-compose or shift-ctrl-u<digits> |
| 18:58 |
paramat |
discuss it with the author and we can merge it tomorrow |
| 18:59 |
paramat |
a problem with 'meetings' is the tendancy to rush things, it happened before |
| 18:59 |
ShadowNinja |
paramat: Well, the author isn't the one who has to approve it... I'll take that as an approval. |
| 18:59 |
kaeza |
I can't type accents into the console either |
| 19:00 |
paramat |
ok -1 |
| 19:00 |
Fixer |
kaeza: even before? |
| 19:00 |
kaeza |
things like áéÃóú show as ´a´e´i etc |
| 19:00 |
kaeza |
Fixer, has been this way forever |
| 19:00 |
Fixer |
ah |
| 19:00 |
kaeza |
well, since MT ditched the chat "dialog" |
| 19:00 |
paramat |
please wait and discuss with the author |
| 19:00 |
VanessaE |
X11_compose-t-m should be " â„¢ " but is just "tm" in minetest., |
| 19:01 |
paramat |
he'll probably be around tonight |
| 19:01 |
paramat |
i'll link him to this discussion |
| 19:02 |
kaeza |
those are probably because of dead keys; things like "ñ" (which is a separate key on spanish keyboards) work |
| 19:03 |
ShadowNinja |
paramat: See my comment. |
| 19:03 |
paramat |
5627 isn't even essential for release or a blocker, lets merge it tomorrow |
| 19:03 |
ShadowNinja |
Is it accurate? |
| 19:04 |
nerzhul |
yeah i agree we can delay before feature freeze. Next ? |
| 19:04 |
paramat |
sort of, i'm neutral |
| 19:04 |
paramat |
thanks |
| 19:04 |
tenplus1 |
will this be an all format release ? (win, linux, android) ? |
| 19:06 |
nerzhul |
?? we always release on all supports |
| 19:06 |
nerzhul |
except mac osx |
| 19:07 |
tenplus1 |
I mean at the same time... last few releases had android lagging behind |
| 19:07 |
ShadowNinja |
Anything else that's important we cover before the freeze? |
| 19:08 |
nerzhul |
#4206 closing |
| 19:08 |
ShadowBot |
https://github.com/minetest/minetest/issues/4206 -- Implement an LMDB backend by ERIIX |
| 19:08 |
VanessaE |
the only thing I had in mind was that "simple metals" idea (see #minetest)( |
| 19:08 |
VanessaE |
but it's too late now |
| 19:08 |
nerzhul |
#5589 PR before feature freeze can be nice |
| 19:08 |
ShadowBot |
https://github.com/minetest/minetest/issues/5589 -- Accommodate duplicate and empty world names automatically by metarmask |
| 19:09 |
tenplus1 |
stu's #5767 would be nice but only noticable when using player textures |
| 19:09 |
ShadowBot |
https://github.com/minetest/minetest/issues/5767 -- Don't add damage flash while punch texture modifier is active by stujones11 |
| 19:09 |
Krock |
from my side, the strip down sneak PR but that's too time consuming to process that one. |
| 19:09 |
Krock |
tenplus1, we already had this here |
| 19:09 |
tenplus1 |
w00t |
| 19:10 |
Krock |
it's going to be tested and merged before release |
| 19:10 |
ShadowNinja |
It's basically the same as #4962 though, and that PR is more comprehensive. |
| 19:10 |
ShadowBot |
https://github.com/minetest/minetest/issues/4962 -- Save the name of the world in world.mt and remove invalid characters from the directory name by red-001 |
| 19:10 |
paramat |
no it's not the same |
| 19:12 |
nerzhul |
Fixer, i want to tell a thing: encoding problems on each platform is just a pain ! :p |
| 19:12 |
tenplus1 |
encoding? |
| 19:13 |
nerzhul |
keyboard issues :p |
| 19:13 |
tenplus1 |
ahh |
| 19:15 |
nerzhul |
Fixer, for your character problem it just depend on your locale |
| 19:15 |
nerzhul |
http://en.cppreference.com/w/cpp/string/wide/iswprint |
| 19:15 |
nerzhul |
if your locale is wrong you cannot print some chars |
| 19:16 |
Fixer |
nerzhul: funnily it worked on yesterday build |
| 19:16 |
Fixer |
before that changes |
| 19:16 |
nerzhul |
but there was another problem... the char on mac osx |
| 19:17 |
nerzhul |
with the shift key |
| 19:17 |
Fixer |
yes |
| 19:20 |
nerzhul |
Fixer, can you test one thing for me |
| 19:20 |
Fixer |
yeah |
| 19:20 |
nerzhul |
replace iswprint condition with !iswcntrl |
| 19:21 |
nerzhul |
i think it's a better test :p |
| 19:21 |
nerzhul |
and yeah i can print € in chat now |
| 19:21 |
nerzhul |
seems to be the real fix |
| 19:22 |
Fixer |
will test |
| 19:22 |
garywhite |
Oh, 0.4.16 freeze is only about 28.5 hours away |
| 19:23 |
tenplus1 |
hi gary |
| 19:23 |
garywhite |
hi ten |
| 19:25 |
nerzhul |
who does some MacOSX builds there ? |
| 19:26 |
tenplus1 |
doesnt it still have to be jailbroken to run ? |
| 19:28 |
nerzhul |
lol |
| 19:28 |
Calinou |
no, not for macOS |
| 19:28 |
Calinou |
that'd be iOS |
| 19:28 |
Calinou |
I could do a macOS build but I'd need instructions :P |
| 19:28 |
tenplus1 |
ahh |
| 19:30 |
garywhite |
isn't macOS kind of like Linux? (I know they're both UNIX-based) |
| 19:31 |
nerzhul |
no |
| 19:31 |
nerzhul |
Linux is not a UNIX based, it just has unix philosophy |
| 19:31 |
Fixer |
linux is not uni... |
| 19:31 |
nerzhul |
BSD is UNIX based, and MacOSX is built upon FreeBSD |
| 19:35 |
|
DS-minetest joined #minetest-dev |
| 19:36 |
|
GreenDimond joined #minetest-dev |
| 19:37 |
GreenDimond |
I'm just gonna leave this here in case any core devs (or anyone for that matter) is interested. https://forum.minetest.net/viewtopic.php?p=271095#p271095 Dunno if Exeter reported it already. Screenshots of it are further down posted by vita. |
| 19:37 |
GreenDimond |
I am rather concerned. |
| 19:38 |
GreenDimond |
If this is a Minetest bug not caused by mods, it better be fixed quick :o |
| 19:38 |
tenplus1 |
oof, what mods he got running ??? |
| 19:38 |
paramat |
looking |
| 19:39 |
GreenDimond |
yeah check servers.minetest.net for mod list |
| 19:39 |
* VanessaE |
's server is #2 on the list today :D |
| 19:39 |
ShadowBot |
https://github.com/minetest/minetest/issues/2 -- Burned wood |
| 19:39 |
VanessaE |
(was #1 earlier) |
| 19:39 |
ShadowBot |
https://github.com/minetest/minetest/issues/1 -- GlowStone code by anonymousAwesome |
| 19:39 |
paramat |
hehe |
| 19:39 |
GreenDimond |
lol |
| 19:39 |
VanessaE |
shit, wrong channel |
| 19:40 |
GreenDimond |
well derp nvm the server is off so you cant access it :/ |
| 19:40 |
GreenDimond |
lemme see if the test server is up and Ill give you the list |
| 19:40 |
GreenDimond |
well, an older list. |
| 19:40 |
paramat |
mgv7 with floatlands enabled? |
| 19:40 |
GreenDimond |
nvm he has that one off too |
| 19:40 |
VanessaE |
paramat: with THAT much water? no chance. |
| 19:40 |
GreenDimond |
mg7 yes |
| 19:40 |
GreenDimond |
dunno about floatlands (i think no) |
| 19:41 |
paramat |
floatlands are disabled by default in mgv7 |
| 19:41 |
paramat |
and should not create unsupported water |
| 19:41 |
GreenDimond |
I sealed up a similar source yesterday that we thought was a WE hacker |
| 19:41 |
GreenDimond |
it wasnt growing tho |
| 19:43 |
Fixer |
nerzhul: yep, â„– works with your suggested function |
| 19:43 |
nerzhul |
i hope this works for mac osx too, waiting for input to merge |
| 19:44 |
shivajiva |
^ +1 |
| 19:45 |
GreenDimond |
off the top of my head he has default mods, vending machines, stairs+, a few custom mods, 3d_armor, colored wood, protector redo i think, bakedclay, homedecor... erm... |
| 19:46 |
tenplus1 |
nothing that would generate water in mid air ? |
| 19:46 |
GreenDimond |
dont think so |
| 19:46 |
GreenDimond |
I could be entirely wrong tho |
| 19:46 |
GreenDimond |
I dont have the mod list in front of me :/ |
| 19:47 |
GreenDimond |
he has mobs_redo aswell |
| 19:47 |
GreenDimond |
oh and WE ofc |
| 19:48 |
shivajiva |
possible injection via the exploit? |
| 19:48 |
tenplus1 |
ya think anyone would have WE priv apart from admin ? |
| 19:48 |
VanessaE |
you don't need privs to use the exploit |
| 19:48 |
VanessaE |
(or didn't, anyway) |
| 19:48 |
VanessaE |
check all of your configuration files, GreenDimond |
| 19:48 |
VanessaE |
make sure none of them have code inserted |
| 19:48 |
VanessaE |
some mods use dofile() to load configs |
| 19:49 |
VanessaE |
and when you do that, you can also put code into the config |
| 19:49 |
tenplus1 |
is /lua able to set water_level to a new height ? maybe causing this issue |
| 19:49 |
VanessaE |
also refresh all of your mods, the exploit can be used to rewrite a mod also |
| 19:50 |
VanessaE |
wait, why are we talking about this here? |
| 19:50 |
VanessaE |
--> #minetest |
| 19:50 |
tenplus1 |
lol |
| 19:51 |
GreenDimond |
Not my server |
| 19:52 |
GreenDimond |
I posted it here because it could be a possible mapgen bug |
| 19:52 |
GreenDimond |
he uses latest |
| 19:52 |
GreenDimond |
and he updated it recently |
| 19:52 |
|
bigfoot547 joined #minetest-dev |
| 19:55 |
|
tenplus1 left #minetest-dev |
| 20:04 |
Fixer |
did I just evade fall damage again while _continiously_ falling from 2000 nodes high at huge speed? |
| 20:05 |
Fixer |
damage enabled |
| 20:05 |
|
Taoki joined #minetest-dev |
| 20:05 |
Fixer |
fuck you and fuck me, it really does |
| 20:06 |
Fixer |
3 seconds cooldown? hmmm |
| 20:06 |
Fixer |
hmhmhmhmhm |
| 20:11 |
Fixer |
ok, I'm creating ANOTHER fall evasion bug, this is literally fucked up |
| 20:17 |
Fixer |
survived fall at 270 nodes/sec |
| 20:29 |
nerzhul |
merging trivial #5787 which really solve the mac osx char, confirmed by original PR owner |
| 20:29 |
ShadowBot |
https://github.com/minetest/minetest/issues/5787 -- Real control fix by nerzhul |
| 20:29 |
Fixer |
https://imgur.com/a/GuvtT ??? :D ??? :D ??? :D |
| 20:30 |
Fixer |
i'm speechless, i've seen everything now |
| 20:31 |
Fixer |
nerzhul: ty |
| 20:32 |
paramat |
doesn't look like you hit ignore on the way down, but can you reproduce it? |
| 20:33 |
Fixer |
paramat: yes, every time |
| 20:33 |
Fixer |
paramat: if you hit ignore - you will die at landing |
| 20:33 |
Fixer |
=) ! |
| 20:33 |
Fixer |
paramat: the key is preload the area by falling at least once and then go back to that place, wait for cooldown, or 10 sec to be sure, jump off... |
| 20:34 |
Fixer |
easily reproducable |
| 20:34 |
paramat |
yeah |
| 20:35 |
Fixer |
don't ask me, why I did fall 2000 nodes down in the first place, I wanted to check for water places, and decided to fall through... I did not die... |
| 20:46 |
Fixer |
https://github.com/minetest/minetest/issues/5788 |
| 20:47 |
Fixer |
btw, i've noticed another bug |
| 20:48 |
Fixer |
something with chat dropdown... sometimes it is not dropped down to usual... depth |
| 20:54 |
red-001 |
paramat, added a description to #5554 |
| 20:54 |
ShadowBot |
https://github.com/minetest/minetest/issues/5554 -- [CSM] Allow enabling and disabling mods. by red-001 |
| 20:54 |
paramat |
thanks |
| 21:27 |
|
destt_ joined #minetest-dev |
| 21:58 |
red-hunter-002 |
why was #5554 merged in such a strange way? |
| 21:58 |
ShadowBot |
https://github.com/minetest/minetest/issues/5554 -- [CSM] Allow enabling and disabling mods. by red-001 |
| 22:00 |
red-hunter-002 |
see #90808a4 |
| 22:03 |
red-hunter-002 |
https://github.com/minetest/minetest/commit/90808a4f34dca379e872074bdfd472faf1c48cf3 |
| 22:03 |
red-hunter-002 |
!title |
| 22:03 |
ShadowBot |
Real control fix (#5787) · minetest/minetest 90808a4 · GitHub |
| 22:03 |
red-hunter-002 |
@90808a4 |
| 23:03 |
|
Fritigern joined #minetest-dev |
| 23:04 |
nerzhul |
5554 is not merged |
| 23:22 |
|
Taoki joined #minetest-dev |
| 23:48 |
|
DI3HARD139 joined #minetest-dev |
| 23:57 |
|
paramat joined #minetest-dev |
