Time |
Nick |
Message |
01:11 |
|
SEx2 joined #minetest-dev |
01:55 |
|
smoke_fumus joined #minetest-dev |
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:53 |
|
cheapie joined #minetest-dev |
03:56 |
* zat |
pokes hmmmm by suggestion of VanessaE so blame her. |
04:01 |
VanessaE |
ha! |
04:04 |
|
grrk-bzzt joined #minetest-dev |
04:58 |
|
cheapie joined #minetest-dev |
05:05 |
|
cheapie joined #minetest-dev |
05:06 |
|
cheapie joined #minetest-dev |
05:08 |
|
cheapie joined #minetest-dev |
05:09 |
|
cheapie joined #minetest-dev |
05:12 |
|
cheapie joined #minetest-dev |
05:28 |
|
catninja joined #minetest-dev |
05:47 |
|
Hunterz joined #minetest-dev |
06:09 |
|
crazyR_ joined #minetest-dev |
06:19 |
|
kaeza joined #minetest-dev |
06:28 |
|
Bhima joined #minetest-dev |
07:27 |
|
darkrose joined #minetest-dev |
07:28 |
|
Calinou joined #minetest-dev |
07:30 |
|
tomreyn joined #minetest-dev |
07:38 |
|
Anchakor joined #minetest-dev |
07:39 |
|
kaeza joined #minetest-dev |
07:50 |
|
Fresh_me_ joined #minetest-dev |
08:05 |
|
CraigyDavi joined #minetest-dev |
08:15 |
|
PenguinDad joined #minetest-dev |
08:30 |
|
jin_xi joined #minetest-dev |
08:43 |
|
rsiska joined #minetest-dev |
08:53 |
|
john3213 joined #minetest-dev |
08:58 |
|
john3213 left #minetest-dev |
09:11 |
|
sapier joined #minetest-dev |
09:14 |
|
Jordach joined #minetest-dev |
09:20 |
|
BlockMen joined #minetest-dev |
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 |
|
PilzAdam joined #minetest-dev |
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 |
09:53 |
|
ImQ009 joined #minetest-dev |
09:53 |
|
rsiska joined #minetest-dev |
10:00 |
|
Calinou joined #minetest-dev |
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 |
|
Bhima__ joined #minetest-dev |
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 |
|
Bhima joined #minetest-dev |
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 ;) |
10:30 |
|
Bhima joined #minetest-dev |
10:40 |
|
vifino joined #minetest-dev |
10:46 |
|
kaeza joined #minetest-dev |
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:12 |
|
nore joined #minetest-dev |
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:40 |
|
BlockMen left #minetest-dev |
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 :-( |
13:45 |
|
Hunterz joined #minetest-dev |
13:59 |
|
hmmmm joined #minetest-dev |
14:04 |
|
Zeitgeist_ joined #minetest-dev |
14:15 |
|
Megaf joined #minetest-dev |
14:16 |
|
init joined #minetest-dev |
14:20 |
|
grrk-bzzt joined #minetest-dev |
14:38 |
|
rubenwardy joined #minetest-dev |
14:56 |
rubenwardy |
Are world/players/*.txt files written for players that are offline? |
14:59 |
|
pro joined #minetest-dev |
15:09 |
|
Calinou joined #minetest-dev |
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:12 |
|
Exio4 joined #minetest-dev |
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:04 |
|
Calinou joined #minetest-dev |
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 |
16:37 |
|
sapier joined #minetest-dev |
16:44 |
|
zat joined #minetest-dev |
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) |
18:06 |
|
grrk-bzzt joined #minetest-dev |
18:15 |
|
vifino joined #minetest-dev |
18:19 |
|
diemartin joined #minetest-dev |
18:31 |
|
kaeza joined #minetest-dev |
18:31 |
|
kaeza joined #minetest-dev |
18:32 |
|
ImQ009 joined #minetest-dev |
18:43 |
|
Hunterz joined #minetest-dev |
18:45 |
|
Hunterz joined #minetest-dev |
18:51 |
|
pro joined #minetest-dev |
19:30 |
|
Taoki joined #minetest-dev |
19:38 |
VanessaE |
build failure: http://pastebin.ubuntu.com/7511747/ |
19:39 |
|
Zeitgeist_ joined #minetest-dev |
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 |
20:08 |
|
kaeza joined #minetest-dev |
20:09 |
|
diemartin joined #minetest-dev |
20:27 |
|
troller joined #minetest-dev |
20:27 |
|
tomreyn joined #minetest-dev |
20:31 |
|
proller__ joined #minetest-dev |
20:39 |
|
tomreyn joined #minetest-dev |
21:00 |
|
smoke_fumus|2 joined #minetest-dev |
21:29 |
|
sapier joined #minetest-dev |
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:44 |
|
deltib joined #minetest-dev |
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:18 |
|
stormchaser3000 joined #minetest-dev |
22:19 |
|
kahrl joined #minetest-dev |
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 |
22:56 |
|
iqualfragile joined #minetest-dev |
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 |
|
ImQ009 joined #minetest-dev |
23:02 |
Megaf |
the whole UI is smaller |
23:02 |
|
Taoki joined #minetest-dev |
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. |
23:38 |
|
grrk-bzzt joined #minetest-dev |