Time  Nick        Message
03:52 zat         Any ideas on how to be able to add a lua function to the list of decorations in register_decoration?
03:52 zat         So some decorations can be generated by Lua code.
03:52 zat         I tried to do it by keeping the lua_State* and a reference to the function in DecoSimple, then I called it with the coordinates of the decoration as argument.
03:52 zat         ...the result was crappy, partial trees and segfault after a few seconds.
03:52 zat         (Code available if someone wants to see)
03:56 * zat       pokes hmmmm by suggestion of VanessaE so blame her.
04:01 VanessaE    ha!
09:20 BlockMen    #1328
09:20 BlockMen    I would like to hear more comments on the question: new drawtype or change existing
09:20 BlockMen    on changing we have this so far
09:20 BlockMen    pro: existing torches are 3D aswell
09:20 BlockMen    contra: strange textures(like VanessaE's round torches) do not apply correct on blocky 3D torches
09:20 ShadowBot   https://github.com/minetest/minetest/issues/1328 -- Add 3D torches by BlockMen
09:21 VanessaE    my torches aren't round.
09:21 VanessaE    they're tapered.
09:21 sapier      wasn't there a mod doing exactly same?
09:21 VanessaE    like any other texture pack >32 pixels will be
09:22 VanessaE    you could easily have tapered, square torches, as in shaped like the Washington Monument
09:22 VanessaE    the problem is the model forces a shape on something that was entirely defined by the texture before, which means you must make it a new drawtype.
09:22 VanessaE    or you must use a new image filename.
09:23 VanessaE    (new drawtype is better)
09:23 BlockMen    sapier, e.g. wall mounted torches cant be done properly with nodeboxes
09:24 BlockMen    and there are at least 3 mods
09:24 BlockMen    (which shows ppl want 3D torches)
09:25 VanessaE    homedecor has one such mod
09:25 VanessaE    or rather, node
09:25 VanessaE    borrowed from 3dforniture
09:25 BlockMen    ok, at least 4 then
09:26 VanessaE    and then there's some mod out there, I forget who, that uses the same model but applies a wood texture to it
09:27 VanessaE    but anyway, my point is backwards compatibility not just for texture packs but for mods that use the existing drawtype for things besides torches.
09:27 VanessaE    (as the old panes mod used to do)
09:32 BlockMen    sapier, http://dev.minetest.net/Git_Guidelines -> "Commit messages must start with a capital letter..."
09:46 BlockMen    sapier, also #1329
09:46 ShadowBot   https://github.com/minetest/minetest/issues/1329 -- Pause- & Chatformspec invisible when HUD is hidden
09:46 BlockMen    is it reproducable under linux?
09:46 BlockMen    *too
09:47 PenguinDad  BlockMen: I'm able to reproduce it on my linux
09:47 VanessaE    yes
09:47 VanessaE    I can reproduce it in linux.
09:48 PenguinDad  Inventory is invisible too
09:48 VanessaE    yues
09:48 VanessaE    same here
09:49 PenguinDad  is the F10 console a formspec?
09:49 BlockMen    PenguinDad, ic, thx
09:49 BlockMen    and AFAIK its not
10:03 sapier      I wonder why I commit fixes from others ... next time I'll let others do this
10:04 sapier      is 1329 msvc only?
10:05 sapier      blockmen how do you hide hud?
10:05 sfan5       F1
10:05 sfan5       what else would you use?
10:05 BlockMen    sapier, look above
10:05 sapier      I never hide hud ;-)
10:05 BlockMen    and F1 yes
10:06 BlockMen    since its reproduceable on linux i doubt its msvc only
10:06 BlockMen    and furthermore it seems to be for all formspecs
10:06 PenguinDad  sapier: it hides everything also including chat, inventory, F10 console
10:06 BlockMen    or even more (since i guess console is not a formspec)
10:06 BlockMen    damn, ninja'd
10:07 sfan5       console was not a formspec last time I looked at the code
10:13 sapier      I think there's a quite simple fix let me check it first
10:15 sapier      fixed
10:16 sapier      I'm gonna merge #1313 too
10:16 ShadowBot   https://github.com/minetest/minetest/issues/1313 -- Remove settings not used any longer by sapier
10:18 sfan5       any comments to #1149?
10:19 ShadowBot   https://github.com/minetest/minetest/issues/1149 -- Add AppData file by davidgumberg
10:20 sapier      well I don't use in system install so I'm not affected by it but 1) squash 2) how much different "standards" do we want to suppoer?
10:21 sfan5       1) not an issue, can be done when merging 2) currently we only have .desktop
10:21 sfan5       >The GNOME Software Center currently shows a nag message that the upstream project doesn't ship the additional data. Additionally, we will penalize apps that do not ship the extra metadata by showing them lower in the search results.
10:21 sfan5       ^ situation for new fedora releases
10:21 sapier      I'd see files like that more being issue of a distribution package but I don't care enough to block it
10:22 sfan5       so you agree?
10:22 sapier      no I don't vote
10:23 sapier      btw we've got 2014
10:24 sapier      so if copyright is updated to 2014 and commits are merged you can do whatever you want ;)
11:06 sapier      #1331 is a fix for #111
11:06 ShadowBot   https://github.com/minetest/minetest/issues/1331 -- Hide pause menu prior drawing "shutting down..." message by sapier
11:06 ShadowBot   https://github.com/minetest/minetest/issues/111 -- Window close icon gives odd screen appearance on shutdown
11:30 sapier      I'm gonna merge the fix for #617 if there aren't objections
11:30 ShadowBot   https://github.com/minetest/minetest/issues/617 -- Minetest floating point exception
11:57 sapier      can someone please try #1200 travis always fails to complete without an error message
11:58 ShadowBot   https://github.com/minetest/minetest/issues/1200 -- Add support for exiting formspecs by doubleclicking outside by sapier
12:02 PenguinDad  sapier: it compiles without errors but if I doubleclick nothing happens
12:03 sapier      hmm that's not what it's expected to do
12:12 sapier      thanks PenguinDad I broke it on last rebase :-(
14:56 rubenwardy  Are world/players/*.txt files written for players that are offline?
15:10 CraigyDavi  rubenwardy, All player files are loaded regardless to wether they are online or not, if that's what you mean.
15:10 rubenwardy  ok
15:22 rubenwardy  In the linux code style it says to name pointers like Type *name;  Is that the same with functions?   Type *function();
15:23 jin_xi      the * is an operator
15:25 jin_xi      it declares pointer, so its not needed for general function calling unless you are defining a function which returns a pointer
15:29 sfan5       rubenwardy: function pointers are done like this: return_val (*func_name)(func_args);
15:29 sfan5       but you probably won't need function pointers
15:30 rubenwardy  I meant returning a variable as a pointer from a function
15:30 rubenwardy  like myclass *obj = someFunction();
15:30 sfan5       char *function();
15:31 rubenwardy  ok
15:31 sfan5       char *value = function();
16:09 sapier      I always wonder for what reason someone whats to add the * to the variable ... to me it's a modifyer to the type ... does anyone have an explanation for this quite insane rule?
16:10 sfan5       char* string, not_a_string;
16:10 sfan5       a bit confusing if you see it as type modifier
16:10 sapier      why? type is "Pointer of string"
16:11 sfan5       yeah
16:11 sfan5       int number1, number2;
16:11 sfan5       if you replace int with char* only the first var is a char*
16:11 sapier      exactly not "char *string1, *string2"
16:12 sapier      so why write "...func(char *string)"
16:13 sfan5       that is just reusing the style that is already used elsewhere
16:13 sapier      there's no style everywhere ;-)
16:17 sapier      wait  char* x,y truly creates two different types of variables ... hmm I'm programming for about 15 years now I wonder why I never happened to run into that issue
17:29 celeron55   i think that's one of the stupid parts of C syntax; the only logical way to write it is to put a space on both sides of *, but that then is too spacey
17:30 celeron55   an other stupid part is the mandatory parenthesis for if/for/whatever; it causes equally arbitrary formatting rules
17:31 celeron55   but... calling it stupid doesn't really solve anything so whatever 8)
19:38 VanessaE    build failure:  http://pastebin.ubuntu.com/7511747/
19:39 VanessaE    (irrlicht is as included in Xubuntu 14.04)
19:40 VanessaE    Minetest is freshly cloned just now.
19:54 VanessaE    sapier, you bwoke it!  http://pastebin.ubuntu.com/7511835/
19:55 PenguinDad  VanessaE: Hasufell actually broke it
19:56 * VanessaE  clobbers Hasufell then.
19:57 SoniEx2     suggestion: Java API + SecurityManager
19:57 * VanessaE  stops clobbering Hasufell and kills SoniEx2
21:36 VanessaE    sapier: check logs
21:39 sapier      that's crazy ... how are we supposed to fix link order if the order required for one distribution breaks another
21:39 ShadowNinja kaeza: Do you want your chatcommands.lua cleanup merged from the sorted /help PR?  I thought you wanted it as a seperate PR, but I don't see it.
21:40 VanessaE    sapier: dunno :)
21:40 ShadowNinja sapier: One got moved that didn't need to?
21:40 sapier      I moved it the way I assumed it to be logicaly correct ... and it did build for me
21:43 sapier      well I didn't create a new clone this time maybe that's been a mistake for moving around library dependencys
21:45 kaeza       ShadowNinja, I'm looking into cleaning up other files as part of the cleanup pull, but I think the sorted help can be merged
21:45 ShadowNinja `make clean && make distclean`
21:45 ShadowNinja kaeza: Already doen, I was just wondering about the second commit.  :-)
21:45 kaeza       alright :)
21:46 ShadowNinja sapier: ^ that should do it, although editing CMakeLists.txt should trigger a complete rebuild too.
21:46 sapier      "no rule to build distclean"
21:47 sapier      can't w just drop ubuntu support? that one isn't even a real linux anymore ;-)
21:47 * VanessaE  clobbers sapier
21:47 ShadowNinja sapier: I guess CMake doesn't add that target then, just clean should do.
21:48 sapier      just tried ... still builds guess I need to reclone
21:48 ShadowNinja sapier: Must be OS-dependent then.
21:48 sapier      that's not gonna make it better
21:48 proller__   git clean -fdx
21:48 VanessaE    which distro were you trying to correct for anyway?
21:49 sapier      ubuntu ;-)
21:49 VanessaE    ...
21:49 sapier      no wait ... gentoo
21:49 VanessaE    oh.
21:49 VanessaE    -funroll_loops ;)
21:51 ShadowNinja Latest git builds on my system (ArchLinux semi-recent x86_64).
21:51 sapier      bot end with u ... almost
21:55 sapier      ok still builds for me so I need to find out what's wrong for you vanessae
21:56 VanessaE    leave it to me to find the corner cases, as usual :D
21:57 sapier      I'm sorry but you seem to be quite qualified to do so ;-)
21:57 VanessaE    haha
22:02 sapier      the most simple way would be just revert it
22:03 VanessaE    but then that breaks whatever you were originally trying to fix
22:03 sapier      yes
22:03 sapier      but it's been broken before
22:04 VanessaE    oh.
22:04 VanessaE    revert that bastard then ;)
22:04 sapier      so it's just reverting to original state
22:04 sapier      but I'd prefere a real fix
22:04 VanessaE    (really, though, it sounds like this is a job for your ebuild to handle)
22:05 VanessaE    is a distro-dependent fix really the right thing to do though?
22:05 sapier      messing around in project build sys doesn't seem to be a issue for a ebuild
22:05 sapier      I guess we just haven't understood the real problem by now
22:06 sapier      ok I haven't understood
22:06 VanessaE    ?
22:06 sapier      so if anyone knows the correct answer ;-) I'm listening
22:08 sapier      according to this one http://irrlicht.sourceforge.net/forum/viewtopic.php?t=37091 it's correct now
22:11 sapier      vanessae can you try moving "CLIENT_PLATFORM_LIBS" next to X11_LIBRARIES?
22:12 VanessaE    I'm afraid to ;)
22:12 sapier      well it's already broken ;-)
22:12 VanessaE    sec\
22:13 ShadowNinja Irrlicht log: PNG warning: Application built with libpng-1.5.9 but running with 1.6.10 | Irrlicht log: LOAD PNG: Internal PNG create read struct failure
22:14 sapier      maybe you should fix your package set?
22:15 VanessaE    sapier: put CLIENT_PLATFORM_LIBS before, or after X11?
22:15 sapier      try before
22:15 VanessaE    ok
22:16 VanessaE    building....
22:17 VanessaE    sapier: no dice.  failed at link with the same error as before
22:23 VanessaE    https://sourceforge.net/p/dclib/discussion/442518/thread/7919aca0/
22:23 VanessaE    wtf is dlib?
22:25 sapier      I don't know but we already have X11 in there
22:25 VanessaE    (well maybe that's just another project with the same error anyway)
22:25 sapier      maybe my interpretation is wrong and link order is exactly reversed
22:26 VanessaE    how do I convince the linker to print the actual command line?
22:27 sapier      http://stackoverflow.com/questions/2670121/using-cmake-with-gnu-make-how-can-i-see-the-exact-commands
22:29 VanessaE    ah
22:30 ShadowNinja I ran make clean and rmed CMakeCache.txt, but CMake still finds the old version of libpng.
22:30 sapier      some arch specific issue?
22:30 VanessaE    sapier: I'll get you that command output in a sec.
22:33 VanessaE    sapier:  http://pastebin.ubuntu.com/7512505/
22:33 VanessaE    (just the end of the command with the important parts, obviously)
22:34 VanessaE    note to self:  use a pastebin that lets the text wrap when it's okay to let it wrap :P
22:38 sapier      your command line is missing -lSM -lICE -lX11 -lXext in the second block ... but I don't know why we have those blocks multiple times
22:38 sapier      https://gist.github.com/sapier/c995a26ec6ecc6c0c979
22:38 sapier      upper is yours lower is mine
22:42 sapier      maybe it's cmake version specific
22:42 sapier      I've got 2.8.9
23:01 ShadowNinja -- Found PNG: /usr/lib64/libpng.so (found version "1.6.10") I hope this means it's working now...
23:01 Megaf       Updating 660e636..f0a9e7d
23:01 Megaf       my minetest looks weird now
23:01 Megaf       fonts are really small
23:01 ShadowNinja Megaf: Define "weird".
23:02 ShadowNinja Megaf: font_size=x in minetest.conf?
23:02 Megaf       never changed that
23:02 Megaf       the whole UI is smaller
23:02 sapier      or another linkage issue ... I guess I should revert that one it's causing way more trouble then worth
23:02 sapier      stop whole ui?
23:03 sapier      how much versions did you jump?
23:03 Megaf       sapier; I changed the irrlicht too, wait a sec
23:04 ShadowNinja Nope, still broken.
23:04 ShadowNinja Must be Irrlicht.
23:09 Megaf       someborked borked the UI with Irrlich1.8.1
23:09 Megaf       http://www.zimagez.com/zimage/screenshot-240514-200759.php
23:10 Megaf       do you even test the code befor pushing it?
23:11 sapier      megaf you did update irrlicht not me ;-)
23:11 Megaf       sapier; well, both used to be exactely the same
23:11 Megaf       visually
23:12 Megaf       I always used 1.8.1
23:12 Megaf       now I was usin 1.7.3 because is debian defaults Im lazy
23:12 sapier      well now it isn't and I wont start another discussion about testing each of our 2000 possible combinations of os and libs prior merging a single commit ;-)
23:13 Megaf       http://www.zimagez.com/zimage/screenshot-240514-201128.php
23:13 sapier      I suggest trying to revert the commit in question localy if it's fixed by that action I'll reverte it as I don't see a sane way to fix it for all os
23:13 ShadowNinja Megaf: Are you getting libpng errors?
23:13 PilzAdam    sapier, your commit breaks it for me too
23:13 ShadowNinja Megaf: I am, and images don't show up.
23:13 sapier      ok then back to old variant
23:14 PilzAdam    yes
23:14 Megaf       ShadowNinja; yes, quite a lot
23:15 ShadowNinja Megaf: Hmmm, might be the link order, or libpng just got updated and irrlicht didn't.
23:15 Megaf       ShadowNinja; sapier http://paste.debian.net/101638/
23:15 Megaf       ShadowNinja; I'm on Debian, we don't get udates, libs are old here :P
23:15 PilzAdam    sapier, why exactly did you change it?
23:16 sapier      because we've got a issue about this order causing floating point exceptions on gentoo
23:16 Megaf       so its gentoo fault
23:16 Megaf       now you borked everything else
23:16 Megaf       everyone*
23:16 Megaf       and who uses gentoo now a days??
23:16 sapier      same way I could tell it's ubuntu's fault now
23:17 PilzAdam    sapier, and why is that?
23:17 sapier      but no need to discuss about it changing one way broken to another way broken isn't an option so back to "old broken"
23:18 sapier      ok link order is back to original now
23:19 Megaf       ^ VanessaE
23:20 Megaf       compiling
23:21 Megaf       yep
23:21 Megaf       back to normal
23:21 ShadowNinja sapier: It must be the order, it works before that commit.
23:22 sapier      I still wonder how this happens
23:22 sapier      things like that indicate there's still something broken but works accidentaly
23:22 Megaf       you could talk to folks on #irrlicht
23:23 sapier      maybe someone really understanding cmake should do that
23:23 VanessaE    sapier: ok, it builds/links fine now.