Time |
Nick |
Message |
00:25 |
|
Tmanyo joined #minetest-dev |
00:47 |
paramat |
thanks for 6708 |
00:47 |
rubenwardy |
:D |
00:47 |
rubenwardy |
have you seen the bug in the comment? |
00:47 |
rubenwardy |
it's a bug in master too |
00:48 |
rubenwardy |
we only have 47 PRs older than this year |
00:48 |
rubenwardy |
*only* |
00:51 |
rubenwardy |
closed another one, duplicate of #6708 |
00:51 |
ShadowBot |
https://github.com/minetest/minetest/issues/6708 -- Add file and line to NodeDef field deprecation warning by rubenwardy |
00:53 |
rubenwardy |
is Rogier-5 active anywhere? |
00:56 |
paramat |
erm i don't understand the comment in 6708 |
00:57 |
paramat |
line 5 where? |
00:57 |
rubenwardy |
in the code snippet |
00:57 |
rubenwardy |
the comments indicate it |
00:57 |
rubenwardy |
it gives the caller of the function that calls the deprecated function |
00:57 |
rubenwardy |
not the line of the deprecated function |
00:58 |
rubenwardy |
note that the NodeDef field warning points to the correct line |
00:59 |
paramat |
oh i see |
01:00 |
|
thePalindrome joined #minetest-dev |
01:01 |
paramat |
however 'minetest.get_mapgen_params()' is deprecated, if that's relevant |
01:02 |
rubenwardy |
that's the point lol :D |
01:02 |
rubenwardy |
the PR is about deprecation messages |
01:02 |
paramat |
"we only have 47 PRs older than this year" rising contributions and/or falling dev time |
01:03 |
rubenwardy |
well, it means we've been fairly successful at closing old PR |
01:04 |
paramat |
oh i was looking at 3957 which is about 'warn if field exists' |
01:04 |
rubenwardy |
lol |
01:05 |
paramat |
so i can't see how 6708 replaces it |
01:05 |
paramat |
never mind, i don't understand |
01:06 |
rubenwardy |
it's the suggestion given in the PR |
01:06 |
rubenwardy |
3957 adds a backtrace to warn fields |
01:06 |
rubenwardy |
6708 adds a file and line number to warn fields |
01:06 |
rubenwardy |
warn fields is a function called when something in register_node is deprecated |
01:06 |
paramat |
ahh |
01:09 |
rubenwardy |
#6153 would be a shame to miss |
01:09 |
ShadowBot |
https://github.com/minetest/minetest/issues/6153 -- Windows: Cpack wix installer by adrido |
02:02 |
|
EvergreenTree joined #minetest-dev |
02:14 |
rubenwardy |
paramat, https://github.com/minetest/minetest/pull/6670#issuecomment-347729315 |
02:14 |
rubenwardy |
you'll like that |
02:14 |
rubenwardy |
I voted against the first chat send PR, and it was rejected #5452 |
02:14 |
rubenwardy |
The PR was then remade when I was not around and merged: #5747 |
02:14 |
ShadowBot |
https://github.com/minetest/minetest/issues/5452 -- [CSM] Add send_chat_message() function by ronsoros |
02:14 |
ShadowBot |
https://github.com/minetest/minetest/issues/5747 -- [CSM] Add send_chat_message and run_server_chatcommand API functions by paly2 |
02:17 |
paramat |
looking |
02:19 |
paramat |
hmm |
02:20 |
paramat |
5452 didn't really have any support |
02:21 |
rubenwardy |
none of them did |
02:21 |
paramat |
considering the strength of feeling i think we have to take that into account and merge 6670 |
02:21 |
rubenwardy |
paramat, what do you think about merging #6709 with "flavor" |
02:21 |
ShadowBot |
https://github.com/minetest/minetest/issues/6709 -- Spelling fixes for the client Lua API document. by BluebirdGreycoat |
02:21 |
rubenwardy |
given the API is still flavor |
02:21 |
rubenwardy |
*flavour |
02:23 |
rubenwardy |
if this was a PR to add chat messages, and the thumbs was the other way around (3 up, 9 down) then it wouldn't be merged |
02:23 |
paramat |
it's ok to merge, spelling mismatch between docs and code is not critical, and flavor will very likely be used in csm code soon |
02:23 |
rubenwardy |
kk, merging now |
02:24 |
rubenwardy |
done |
02:25 |
|
torgdor joined #minetest-dev |
02:26 |
rubenwardy |
lol paramat, approving after merge :D |
02:28 |
paramat |
i'll add a comment to 6670 |
02:29 |
rubenwardy |
I wish http://dev.minetest.net/Client_scripting_plans was stuck to |
02:36 |
|
ThomasMonroe joined #minetest-dev |
03:09 |
|
CalebDavis joined #minetest-dev |
04:50 |
|
paramat joined #minetest-dev |
05:58 |
|
Fritigern joined #minetest-dev |
06:58 |
|
Puka joined #minetest-dev |
07:00 |
nore |
IMHO, concerning #6662, we should give the choice to the user between downloading and installing mods manually (with a hash check to ensure the same mod runs on the server), or automatically from the server while the mod is signed by its author, giving the possibility to the user to trust some mod authors (and needing to individually accept mods that are not by these authors or are not signed) |
07:00 |
ShadowBot |
https://github.com/minetest/minetest/issues/6662 -- CSM causes security issues |
07:01 |
nore |
The rationale behind all that is that whatever sandbox we do, there will be a way, at some point, to escape it -- sandboxes *are* buggy |
07:03 |
nore |
On the other hand, downloading mods separately by the client already implies trust in the given code - and malicious code can be far more easily detected and removed |
07:03 |
nore |
(As with server-side mods) |
07:05 |
|
Puka_ joined #minetest-dev |
07:05 |
|
ashtrayoz joined #minetest-dev |
07:06 |
ashtrayoz |
Might be OT, but: how do I get a voice on #minetest-hub? |
07:10 |
|
nerzhul joined #minetest-dev |
07:16 |
jas_ |
ashtrayoz: ask Shara |
07:17 |
jas_ |
(any op can give it to you, but shara can put you on the list.) |
07:18 |
ashtrayoz |
jas_: thanks |
07:21 |
|
geospeck joined #minetest-dev |
07:22 |
ashtrayoz |
jas_: BTW (and mostly on topic), if your terminal needs engine support to preserve metadata, #6604 has been update with suggestions from sfan5 and paramat. It even has one thumbs-up :-) |
07:22 |
ShadowBot |
https://github.com/minetest/minetest/issues/6604 -- Preserve node metadata as item metadata. by ashtrayoz |
07:33 |
jas_ |
i keep meaning to test this, i will do so now (well, after my coffee is done making) |
07:44 |
|
ssieb joined #minetest-dev |
07:46 |
|
ashtrayoz joined #minetest-dev |
07:50 |
|
Puka joined #minetest-dev |
07:59 |
|
ashtrayoz left #minetest-dev |
08:42 |
|
Darcidride joined #minetest-dev |
08:43 |
|
Hunterz joined #minetest-dev |
08:45 |
|
nerzhul joined #minetest-dev |
09:07 |
|
compunerd joined #minetest-dev |
09:43 |
|
behalebabo joined #minetest-dev |
09:50 |
|
Hunterz joined #minetest-dev |
10:26 |
|
Icedream joined #minetest-dev |
11:19 |
|
Fixer joined #minetest-dev |
11:30 |
|
twoelk joined #minetest-dev |
12:26 |
|
Icedream joined #minetest-dev |
12:38 |
|
geospeck joined #minetest-dev |
12:40 |
|
ThomasMonroe joined #minetest-dev |
12:49 |
|
Jordach joined #minetest-dev |
13:40 |
|
EvergreenTree joined #minetest-dev |
13:44 |
|
lisac joined #minetest-dev |
14:07 |
|
CalebDavis joined #minetest-dev |
14:17 |
jas_ |
https://github.com/minetest/minetest/issues/6714 |
14:17 |
jas_ |
oops. #6714 |
14:17 |
ShadowBot |
https://github.com/minetest/minetest/issues/6714 -- minetest.sound_play Offset |
14:17 |
jas_ |
(that's a feature request, hope it's ok.) |
14:18 |
|
kilbith joined #minetest-dev |
14:19 |
jas_ |
#6715 *steps away from github* |
14:19 |
ShadowBot |
https://github.com/minetest/minetest/issues/6715 -- OpenAL device gets recreated each time the main menu is loaded. |
14:20 |
kilbith |
https://www.youtube.com/watch?v=meWr1G7aO7E |
14:21 |
kilbith |
!title |
14:21 |
ShadowBot |
kilbith: Error: That URL appears to have no HTML title within the first 4KB. |
14:22 |
|
ensonic joined #minetest-dev |
14:23 |
|
antims joined #minetest-dev |
14:26 |
jas_ |
n1 kilbith, now if only one could literally scroll inventory lists! ha |
14:26 |
|
jas_ left #minetest-dev |
14:27 |
kilbith |
soon (tm) |
14:27 |
|
nerzhul joined #minetest-dev |
14:37 |
nerzhul |
is this me or the CSM debate is ridiculous because solution is in tree since months, but we just don't have release MT ? |
15:13 |
|
Hunterz joined #minetest-dev |
16:45 |
|
YuGiOhJCJ joined #minetest-dev |
16:45 |
|
YuGiOhJCJ joined #minetest-dev |
17:12 |
|
YuGiOhJCJ joined #minetest-dev |
17:57 |
|
Darcidride_ joined #minetest-dev |
18:01 |
|
AntumDeluge joined #minetest-dev |
18:05 |
|
Krock joined #minetest-dev |
18:12 |
|
AntumDeluge joined #minetest-dev |
18:18 |
|
nerzhul joined #minetest-dev |
18:59 |
|
Jordach joined #minetest-dev |
19:44 |
|
CalebDavis joined #minetest-dev |
19:54 |
|
nrzkt joined #minetest-dev |
19:56 |
|
geospeck joined #minetest-dev |
20:18 |
|
CalebDavis joined #minetest-dev |
20:36 |
|
geospeck joined #minetest-dev |
21:07 |
|
ensonic joined #minetest-dev |
21:11 |
|
troller joined #minetest-dev |
21:12 |
|
ThomasMonroe joined #minetest-dev |
21:28 |
|
Puka joined #minetest-dev |
21:40 |
|
troller joined #minetest-dev |
21:51 |
|
CBugDCoder joined #minetest-dev |
22:08 |
|
Sokomine joined #minetest-dev |
22:30 |
|
troller joined #minetest-dev |
22:34 |
Hijiri |
anyone know how to fix this build failure https://travis-ci.org/minetest/minetest/builds/307565206?utm_source=github_status&utm_medium=notification |
22:35 |
Hijiri |
also is there a current planned API for remote CSM? By that, I mean specific functions, architecture etc. |
22:35 |
Hijiri |
I was under the impression that it would be file-based like current server and local CSM, but BBGC suggested a simpler API where the server just sends code strings to execute on the client |
22:36 |
Hijiri |
Or maybe it was the server registers code strings to be sent to the client |
22:37 |
Hijiri |
That would make more sense, since you could isolate script environments by mod (though you would want to provide some other way of interoperation) |
22:37 |
|
ThomasMonroe joined #minetest-dev |
22:41 |
|
troller joined #minetest-dev |
22:41 |
Hijiri |
From what I can tell particles.cpp isn't built before it's needed by the linker |
22:55 |
|
troller joined #minetest-dev |
23:01 |
|
troller joined #minetest-dev |
23:06 |
|
troller joined #minetest-dev |
23:11 |
|
troller joined #minetest-dev |
23:18 |
|
troller joined #minetest-dev |
23:46 |
|
troller joined #minetest-dev |
23:51 |
|
troller joined #minetest-dev |