Time  Nick             Message
09:24 IhrFussel        The docs say that this shouldn't work but it does
09:24 IhrFussel        on_metadata_inventory_take = function(pos, listname, index, stack, player) if minetest.is_protected(pos,player:get_player_name()) then return 0 end end,
09:25 IhrFussel        Docs say this callback doesn't allow any return values but I used this for years now and it cancels the taking when in foreign protection
09:33 IhrFussel        Wait I think I got it wrong... it doesn't prevent taking ... I got 1 node that had only this callback defined for years and nobody ever complained about it
12:19 Krock            !tell tenplus1 Hello tenplus1 (ninja'd, somehow)
12:19 MinetestBot      Krock: I'll pass that on when tenplus1 is around
13:13 Krock            uhm... request_http_api() seems to return nil regardless where I the mod exception
13:14 Krock            not even secure.enable_security=false help
13:14 Krock            +s
13:15 Krock            same for request_insecure_environment.. hm
13:30 Krock            > Script at mods/skinsdb/skin_updater.lua may not write to mods/skinsdb/meta/character_1.txt
13:30 Krock            wtf
13:59 Krock            okay. Minetest needs to be patched so that I can finish this project
14:50 Krock            Kinda satisfying to see two PRs referring to each other without that one of them was edited afterwards
14:53 rdococ           can a mod use voxelmanip to place and remove nodes while respecting area protections?
14:54 Krock            yes, call core.is_protected for each node
14:54 Krock            or protections_in_area or similar
14:54 Krock            minetest.is_area_protected(pos1, pos2, player_name, interval)`
14:54 rdococ           oh, cool
15:01 BuckarooBanzai58 Krock: how did you do that with the PR's? Did you count the numbers :)
15:02 Krock            BuckarooBanzai58: math.max(last_issue, last_pr) + 1
15:03 Krock            PREDICTION 100%
15:03 Krock            PREDITION 100%
15:05 BuckarooBanzai58 ;)
15:50 * Sokomine       is fighting with trying to apply a pr...
15:58 Krock            Sokomine: curl -Ls https://github.com/foo/bar/pull/1234.patch | git am -3 -
16:01 Sokomine         krock: yes, sadly that doesn't work. get the complaint "could not build fake ancestor", and it just aborts
16:02 Krock            Sokomine: broken git clone, or is that related to the build system?
16:02 Krock            [off[ BuckarooBanzai: besides me there's also atorian* (offline)
16:02 Krock            [fail[
16:03 Sokomine         it's a branch of my master, so it really ought to contain anything relevant. the patch has been sitting there for a while though...and i changed other things in the meantime
16:03 Sokomine         it's mostly adding intllib support for mg_villages. the user sadly deleted himself from git
16:04 BuckarooBanzai   Sokomine: won't a "git diff x y" suffice? (And then a git patch)
16:05 BuckarooBanzai   Or just "patch -p1"
16:06 Sokomine         hmm. i'll try with an older version...
16:36 Sokomine         or perhaps i ought to use gettext right from the start...just didn't take a look at that yet. is there a quick into somewhere?
16:42 DS-minetest      https://github.com/minetest/minetest/blob/7379aa74cf98c7e4c7aa5325ef1531d412a0abac/doc/lua_api.txt#L2514
16:44 Krock            FYI: that's just string-replace, not gettext
16:54 Sokomine         ah, well. if i'm at it anyway...what *is* the current way of doing the translations? client-side might be useful for servers (if that's really supported now?)
16:54 DS-minetest      it is afaik
16:55 Calinou          as of 5.0.0, client-side translation is supported
16:55 Calinou          translation files use a custom format (not .po files)
16:56 DS-minetest      and the format is specified here: https://github.com/minetest/minetest/blob/7379aa74cf98c7e4c7aa5325ef1531d412a0abac/doc/lua_api.txt#L2576
16:56 Krock            supported since 0.5.0-dev, but you might want to add a dummy string replace function to keep 0.4.x compat
16:56 Krock            as seen in inttlib. there's an example for a dummy S() function
16:57 Krock            that.. or depend on inttlib which will handle it as server-side translations for these old servers
18:55 Sokomine         french speakers present?
18:56 rubenwardy       Je parle pas français
18:59 BuckarooBanzai   Je *ne pas parle français :P
18:59 rubenwardy       the ne isn't needed
19:00 rubenwardy       it's usually left out in relaxed french
19:00 rubenwardy       ie:    Je sais pas pour qoui
19:01 rubenwardy       the pas is also after the verb/whatever
19:01 rubenwardy       my flatmates study french :/
19:08 BuckarooBanzai   :O
19:11 Sokomine         ah :-)
19:14 Krock            pourquoi tu cherches une personne qui parle français?
19:15 Krock            rubenwardy: it's not always "pas", also "plus", "rien" can follow after the verb in combination with "ne"
19:15 Sokomine         changed one text. and i wasn't sure how to change the french part correctly. hope the "avec" fits...Plot No. : @1, with @2 = Emplacement No. @1, avec @2
19:16 Krock            yes, that fits
19:16 Sokomine         thanks :-)
19:16 Krock            but usually you wouldn't put a comma there
19:17 Sokomine         yes, that's admittedly debatable. in this context, i think i find a comma to be more fitting
19:23 Sokomine         seems messages sent with chat_send_player get translated, while formspecs remain english
19:26 rubenwardy       are you using formspec_escape?
19:27 rubenwardy       that may be stripping out the translate tokens
19:27 Krock            @ does not require to be escaped
19:28 Krock            and labels are reverse-escaped client-side again for rendering so it should be fine again
19:49 Sokomine         just missed an intllib call in one file. thanks :-)