| Time |
Nick |
Message |
| 00:08 |
|
paramat joined #minetest-dev |
| 00:48 |
|
Tmanyo joined #minetest-dev |
| 01:18 |
|
Tmanyo joined #minetest-dev |
| 01:23 |
|
ssieb joined #minetest-dev |
| 01:27 |
paramat |
merging 7365 |
| 01:30 |
paramat |
i posted about merging that in -hub by mistake. will merge #7365 in 15mins if no objection |
| 01:30 |
ShadowBot |
https://github.com/minetest/minetest/issues/7365 -- Schematic decorations: Fix placement bug when centred and rotated by paramat |
| 01:51 |
paramat |
merging 7365 |
| 01:54 |
paramat |
done. no more floating logs |
| 01:55 |
lumberJ |
nice |
| 02:55 |
|
AndroBuilder joined #minetest-dev |
| 03:43 |
|
troller joined #minetest-dev |
| 04:12 |
|
Darcidride_ joined #minetest-dev |
| 04:25 |
|
Darcidride__ joined #minetest-dev |
| 04:26 |
|
Darcidride joined #minetest-dev |
| 04:59 |
|
AntumDeluge joined #minetest-dev |
| 05:09 |
nerzhul |
paramat, i think merging without objection at 3h30 AM doesn't mean anything as every other coredev sleep... |
| 05:13 |
|
longerstaff13 joined #minetest-dev |
| 06:22 |
|
Gael-de-Sailly joined #minetest-dev |
| 06:51 |
|
ensonic joined #minetest-dev |
| 08:51 |
|
Wuzzy joined #minetest-dev |
| 09:16 |
|
nyuszika7h joined #minetest-dev |
| 09:36 |
|
Beton joined #minetest-dev |
| 09:54 |
|
Fixer joined #minetest-dev |
| 10:31 |
|
Gael-de-Sailly joined #minetest-dev |
| 10:48 |
|
troller joined #minetest-dev |
| 10:59 |
|
ANAND joined #minetest-dev |
| 11:07 |
ANAND |
Greetings! Any specific reason why switch-case isn't used here? https://github.com/minetest/minetest/blob/master/src/game.cpp#L1859 |
| 11:44 |
sfan5 |
how would you use a switch-case there? |
| 12:12 |
|
longerstaff13 joined #minetest-dev |
| 12:15 |
ANAND |
since they're mutually exclusive, we could have `case KeyType::INVENTORY:` instead of 32 else ifs |
| 12:15 |
ANAND |
since the values are essentially enums |
| 12:16 |
ANAND |
no wait... |
| 12:18 |
ANAND |
the current API doesn't allow that |
| 12:33 |
|
lumberJ joined #minetest-dev |
| 12:39 |
|
twoelk joined #minetest-dev |
| 12:54 |
|
Taoki[mobile] joined #minetest-dev |
| 13:06 |
|
troller joined #minetest-dev |
| 13:51 |
|
Gael-de-Sailly joined #minetest-dev |
| 13:56 |
ANAND |
Would it make sense to replace wasKeyDown() with a method that returns an object of type KeyPress? |
| 13:56 |
ANAND |
Will that be more optimized? |
| 13:57 |
ANAND |
so if we're checking if the inventory key was pressed, we do this |
| 13:58 |
ANAND |
switch(getKeyDown()){ case KeyType::INVENTORY: {} } |
| 13:59 |
ANAND |
oops.. the method should ideally return an instance of KeyType, not KeyPress |
| 14:03 |
sfan5 |
you can press multiple keys you know... |
| 14:09 |
ANAND |
the return value is an object consisting of multiple bools right? |
| 14:11 |
|
antims joined #minetest-dev |
| 14:15 |
sfan5 |
wasKeyDown() returns bool, a single one |
| 14:18 |
ANAND |
ik... but nvm, sorry for the trouble :) |
| 14:23 |
ANAND |
I'd love to be able to modify client-side settings (like mouse-sensitivity) right from within the game - esp. when leaving the game can have certain disastrous effects... ;) |
| 14:23 |
ANAND |
Is any such functionality being planned for 0.5.0? |
| 15:00 |
|
Krock joined #minetest-dev |
| 15:09 |
|
grrr joined #minetest-dev |
| 15:28 |
|
proller__ joined #minetest-dev |
| 15:41 |
|
Cornelia joined #minetest-dev |
| 15:56 |
Krock |
Moving the discussion over here, nerzhul. Technically 0.4.17 is ready. If people miss features and bugfixes, they must switch to 0.5.x earlier then |
| 15:56 |
nerzhul |
then we are is a free now ? |
| 15:57 |
nerzhul |
or can we start the freeze on the discussion meeting on saturday |
| 15:57 |
nerzhul |
and release in 2 weeks if no blocker ? |
| 15:57 |
Krock |
I saw that GitLab fails due to project-unrelated errors |
| 15:58 |
Krock |
a 2 week freeze would mean that the release took over a year to happen 8), with one week instead, it would be only "near to a year" |
| 15:59 |
Krock |
but we should start the freeze asap. people have been waiting for it way too long |
| 15:59 |
nerzhul |
at least we have so many many commits added then go |
| 15:59 |
nerzhul |
sfan5, paramat are you okay ? rubenwardy ? |
| 15:59 |
nerzhul |
it seems we are in a freeze since months on 0.4.17 :p |
| 16:00 |
nerzhul |
we need a changelog too |
| 16:00 |
nerzhul |
at least we have the PR |
| 16:00 |
sfan5 |
not sure whether there's any more commits that need to be backported |
| 16:00 |
nerzhul |
i should update the 0.5.0 changelog too at a point |
| 16:00 |
nerzhul |
if we plan a 0.4.18 (i hope no) or 0.5.0 next month no need to backport :) |
| 16:00 |
nerzhul |
0.5.0 should be released at a point and cannot be perfect too |
| 16:00 |
Krock |
the question was about adding https://github.com/minetest/minetest/commit/4bb41a19dc74fa31cb021dc3b5622814d67fbd6f |
| 16:00 |
|
Cornelia joined #minetest-dev |
| 16:01 |
Krock |
well, the PR is the changelog .. basically |
| 16:01 |
nerzhul |
this is an interesting fix |
| 16:01 |
nerzhul |
the changelog is not user friendly :p |
| 16:01 |
nerzhul |
and not ordered |
| 16:01 |
nerzhul |
see http://dev.minetest.net/Changelog |
| 16:02 |
nerzhul |
don't forget we are a game, not a library for devs |
| 16:04 |
nerzhul |
:') |
| 17:10 |
|
Player-2 joined #minetest-dev |
| 17:30 |
|
SilverLuke joined #minetest-dev |
| 17:45 |
|
Lunatrius joined #minetest-dev |
| 17:50 |
|
entuland joined #minetest-dev |
| 18:02 |
|
Cornelia joined #minetest-dev |
| 18:40 |
|
paramat joined #minetest-dev |
| 18:41 |
Krock |
paramat, I agree. The logs here also show that nerzhul and me think that it's time for a freeze |
| 18:42 |
paramat |
core devs are more absent at the weekend so we should discuss now :) |
| 18:42 |
paramat |
and hi |
| 18:43 |
Krock |
already wanted to point out that you probably meant "active" but looking at the last meetings it appears that workdays are also quite well suited to talk :3 (may not be liked by everyone) |
| 18:43 |
paramat |
there are some bugfixes since the 13th but some are dependent on 0.5 changes, and mine are minor |
| 18:43 |
|
SilverLuke joined #minetest-dev |
| 18:43 |
paramat |
yes 'active' |
| 18:45 |
paramat |
i think freeze asap, today or tomorrow |
| 18:47 |
paramat |
i don't think fixed meetings are practical, instead meet when some people are available, like hmmmm used to do |
| 18:48 |
sfan5 |
are there any more commits that need to be backported? |
| 18:48 |
sfan5 |
paramat: nothing prevents you from dicussing topics when multiple coredevs happen to be online |
| 18:49 |
Krock |
So in the future there will be discussion overviews - likely one per month or similar? |
| 18:50 |
Krock |
I see that the saturday meetings aren't optimal as they are right now, so splitting and spreading the discussion points seem more appropriate |
| 18:50 |
nerzhul |
one per month can be better, and maybe open a discussion topic on the github discussion then for point in time questions ? |
| 18:51 |
nerzhul |
sfan5 are you okay for a freeze ? |
| 18:51 |
sfan5 |
generally yes |
| 18:51 |
sfan5 |
but first "are there any more commits that need to be backported?" should be answered |
| 18:51 |
nerzhul |
okay, we are 4 for it, i publish a communication about it on the forum. what is the expected release date ? 3 june ? |
| 18:52 |
sfan5 |
just don't mention a release date |
| 18:52 |
sfan5 |
but IMO a freeze period of 1 week is good eough |
| 18:52 |
sfan5 |
+n |
| 18:52 |
nerzhul |
okay, i give an estimation ? 1 or 2 weeks ? |
| 18:53 |
Krock |
until next weekend..? |
| 18:54 |
Krock |
as in: ~9 days |
| 18:54 |
nerzhul |
sounds reasonable |
| 18:56 |
nerzhul |
can a moderator stick https://forum.minetest.net/viewforum.php?f=18 ? |
| 18:58 |
sfan5 |
done |
| 18:59 |
nerzhul |
ty |
| 18:59 |
Krock |
/topic needs updating |
| 19:00 |
rubenwardy |
what a retarded topic |
| 19:00 |
rubenwardy |
the entire 0.4.17 release is bug fix only |
| 19:00 |
Krock |
rubenwardy, fix it. Improve it. Before the people see it. |
| 19:01 |
rubenwardy |
only way to fix is delete |
| 19:01 |
rubenwardy |
:P |
| 19:03 |
BillyS |
Yes, I suppose the topic is a bit dated, isn't it? |
| 19:04 |
paramat |
there will always be further bugfixes that didn't make it into 0.4.17 so i suggest not adding any more and freeze now while build is green? |
| 19:04 |
paramat |
however, nothing super important that i can see |
| 19:05 |
paramat |
but we will need a 0.4.18 after 0.5 |
| 19:12 |
|
Gael-de-Sailly joined #minetest-dev |
| 19:19 |
|
Beton_ joined #minetest-dev |
| 19:28 |
paramat |
so it looks like we are officially frozen? no more backporting |
| 19:31 |
paramat |
will merge #7371 in an hour or so |
| 19:31 |
ShadowBot |
https://github.com/minetest/minetest/issues/7371 -- Vein ore: Fix bug caused by changing perlinmap Y size by paramat |
| 19:33 |
Krock |
paramat, basically if (noise) delete noise; noise = new Noise(...); |
| 19:34 |
paramat |
ah ok |
| 19:34 |
paramat |
updated game#2128 to use paper in recipe, plus an optional ABM tweak |
| 19:34 |
ShadowBot |
https://github.com/minetest/minetest_game/issues/2128 -- TNT: Raise cost of TNT by adding a TNT stick crafting stage by paramat |
| 19:34 |
|
thatgraemeguy_ joined #minetest-dev |
| 19:34 |
|
thatgraemeguy_ joined #minetest-dev |
| 19:37 |
paramat |
ah yes, in most ore types the noise object is only created once, that's why. in vein ore the noise objects are recreated many times. ok will do that then |
| 19:38 |
paramat |
.. in this PR |
| 19:39 |
|
behalebabo joined #minetest-dev |
| 19:42 |
|
Wuzzy joined #minetest-dev |
| 19:52 |
paramat |
some OPs have been renamed to guest |
| 19:55 |
|
VanessaE joined #minetest-dev |
| 19:56 |
|
rubenwardy joined #minetest-dev |
| 19:57 |
paramat |
updated #7371, thanks for the advice |
| 19:57 |
ShadowBot |
https://github.com/minetest/minetest/issues/7371 -- Vein ore: Fix bug caused by changing perlinmap Y size by paramat |
| 19:59 |
Krock |
np. lgtm |
| 19:59 |
paramat |
will do after testing |
| 20:22 |
nerzhul |
i really thing TDD is really a good thing and it's possible in many cases in mapgen :p |
| 20:29 |
|
Calinou joined #minetest-dev |
| 20:30 |
|
entuland joined #minetest-dev |
| 20:33 |
Hijiri_ |
can mods in the mod store have version constraints on dependencies / engine? |
| 20:33 |
nerzhul |
oh github added the comment history when you edit it |
| 20:37 |
|
shivajiva joined #minetest-dev |
| 20:45 |
|
Guest54512 joined #minetest-dev |
| 20:46 |
|
`z joined #minetest-dev |
| 20:46 |
|
Gael-de-Sailly joined #minetest-dev |
| 21:04 |
|
Ruslan1 joined #minetest-dev |
| 21:05 |
paramat |
nerz removed 'if' lines, can i merge? |
| 21:06 |
nerzhul |
paramat: delete doesn't need pointer verification, it's a noop if pointer is null |
| 21:06 |
paramat |
ok cool |
| 21:09 |
paramat |
merging #7371 in 5 mins |
| 21:09 |
ShadowBot |
https://github.com/minetest/minetest/issues/7371 -- Vein ore: Fix bug caused by changing perlinmap Y size by paramat |
| 21:18 |
paramat |
merging |
| 21:20 |
paramat |
done |
| 21:22 |
paramat |
good back to 100 PRs |
| 21:30 |
|
entuland joined #minetest-dev |
| 21:55 |
|
Fixer joined #minetest-dev |
| 22:09 |
|
Fixer joined #minetest-dev |
| 22:27 |
|
Fixer joined #minetest-dev |
| 22:35 |
|
Fixer_ joined #minetest-dev |
| 23:02 |
|
AntumDeluge joined #minetest-dev |
| 23:07 |
|
paramat joined #minetest-dev |
| 23:54 |
|
kurennon joined #minetest-dev |
| 23:57 |
|
zistack joined #minetest-dev |
| 23:58 |
|
nickolas360 joined #minetest-dev |