Minetest logo

IRC log for #minetest-dev, 2014-03-03

| Channels | #minetest-dev index | Today | | Google Search | Plaintext

All times shown according to UTC.

Time Nick Message
00:29 ShadowNinja "Minetest engine core main application" -- This sounds redundant.  Like saying "Minetest core core core application".  It's obviously an application too, so just "Minetest engine" should do.
00:34 VanessaE yes
00:35 VanessaE I refer to the engine as "core" locally but "engine" is just as good
01:46 werwerwer joined #minetest-dev
02:57 BrandonReese joined #minetest-dev
03:00 ImQ009 joined #minetest-dev
03:14 khonkhortisan joined #minetest-dev
03:46 khonkhortisan joined #minetest-dev
04:07 Miner_48er joined #minetest-dev
04:18 proller joined #minetest-dev
05:18 VanessaE joined #minetest-dev
05:40 grrk-bzzt joined #minetest-dev
05:41 RealBadAngel joined #minetest-dev
06:07 nore joined #minetest-dev
06:21 darkrose joined #minetest-dev
06:30 salamanderrake joined #minetest-dev
06:43 damiel joined #minetest-dev
07:06 sfan5 umm... why was #973 merged?
07:06 ShadowBot https://github.com/minetest/minetest/issues/973 -- improved win32 file version information by sapier
07:19 RealBadAngel whats wrong with it?
07:20 sfan5 <ShadowNinja> "Minetest engine core main application" -- This sounds redundant.  Like saying "Minetest core core core application".  It's obviously an application too, so just "Minetest engine" should do.
07:20 sfan5 I would like to have that fixed
07:21 RealBadAngel ah that. agree
07:21 RealBadAngel that sounds strange even for me ;)
07:36 rsiska joined #minetest-dev
07:38 kahrl why are some of the values in the version block null terminated and some not?
07:38 sfan5 who even agreed to merge that?
07:38 kahrl I don't see anything here
07:50 RealBadAngel so just reverse it and fix
07:51 RealBadAngel have you guys tried new shaders?
08:30 RealBadAngel new effect: http://i.imgur.com/9ivjFGe.jpg http://i.imgur.com/zgkBrpb.jpg
08:31 RealBadAngel how do you like it?
08:38 Calinou joined #minetest-dev
08:58 sfan5 RealBadAngel: looks nice
09:07 darkrose joined #minetest-dev
09:07 darkrose joined #minetest-dev
09:37 OldCoder joined #minetest-dev
10:30 ImQ009 joined #minetest-dev
10:30 proller joined #minetest-dev
10:46 RealBadAngel joined #minetest-dev
11:03 1JTAAH6BG joined #minetest-dev
11:04 OldCoder joined #minetest-dev
11:20 PilzAdam joined #minetest-dev
11:28 ImQ009 joined #minetest-dev
11:47 Naked joined #minetest-dev
12:26 tomreyn joined #minetest-dev
12:44 tomreyn joined #minetest-dev
13:30 proller joined #minetest-dev
13:39 ImQ009 joined #minetest-dev
13:46 ShadowNinja joined #minetest-dev
13:48 hmmmm joined #minetest-dev
13:50 proller joined #minetest-dev
13:57 salamanderrake joined #minetest-dev
14:05 proller joined #minetest-dev
14:09 Megaf joined #minetest-dev
14:23 deltib joined #minetest-dev
14:58 iqualfragile joined #minetest-dev
15:03 PenguinDad joined #minetest-dev
15:06 Megaf folks, how do I compile minetest for OpenGL ES?
15:10 xyz -DENABLE_GLES=1
15:12 zat joined #minetest-dev
15:16 Megaf hm
15:18 Anchakor_ joined #minetest-dev
15:39 sfan5 we don't have a documentation for cmake flags, do we?
15:48 PenguinDad sfan5, the only documentation I know for them is -LH
15:48 sfan5 so no
15:48 sfan5 I guess we shall have some at dev.minetest.net
16:03 Jordach joined #minetest-dev
16:10 Calinou joined #minetest-dev
16:34 EvergreenTree joined #minetest-dev
16:34 EvergreenTree joined #minetest-dev
16:35 sapier joined #minetest-dev
16:39 rsiska joined #minetest-dev
16:39 sapier #973 was sitting around for half a year now I mentioned it multiple times, it's qute strange that complains always pop up AFTER something was merged. Especially as those complains most time are for formal issues only.
16:39 ShadowBot https://github.com/minetest/minetest/issues/973 -- improved win32 file version information by sapier
16:40 EvergreenTree joined #minetest-dev
16:40 sapier Yet the file version information is there now, everyone who wants to improve it is welcome to do so.
16:41 EvergreenTree joined #minetest-dev
16:42 sapier I'm gonna merge #1164 at about 21MET so if someone wants to check it BEFORE merge you're welcome.
16:42 ShadowBot https://github.com/minetest/minetest/issues/1164 -- Replace builtin pause and message menus by formspec based ones by sapier
16:43 xyz wtf is MET?
16:44 sapier middle european timezone? ;-P
16:44 sapier GMT+1
16:44 sapier IRC
16:44 xyz ah, so same as CET but you call it MET
16:44 xyz and there's a compliant already in the issue comments
16:46 sapier well complains form shadow about coding style are on my ignore list
16:46 xyz why?
16:46 sapier he'll always find something right after I fixed the last thing he complained
16:47 xyz ugh
16:47 sapier and that file doesn't comply to any coding style anyway
16:49 sapier so unless someone fixes minetests whole coding style or allows usage for tools on existing files I'm just ignore minor issues like that
16:49 xyz I don't think this should be done at once
16:50 xyz but I've been told I have no right to complain
16:50 xyz so I won't
16:50 sapier doing it at once is the only way to do
16:50 sapier you wont be able to fix it step by step
16:50 Naked joined #minetest-dev
16:50 sapier especialy as we don't even HAVE a defined coding style
16:51 sapier we have something with some exceptions somewhere
16:51 sapier but I'm starting to waste time on usless discussions again
16:51 xyz I wonder why "and no, Boost will not even be considered, so forget it"
16:52 sapier because boost is crap
16:52 sfan5 I do not want to install 126MB of headers to compile something
16:53 sapier it breaks compatibility on each single version they release, no distribution has the version bundled you need right now and and and
16:53 sfan5 sapier: astyle -z2 -W3 -k3 -p -w -Y -S -K -N -t -m1 is pretty near to Minetest's coding style
16:53 sapier and of course each application requires a different version of boost
16:53 sapier yes but I've been forbidden to use it on any file I checkin so this is useless
16:53 xyz hmm, dunno
16:54 xyz anyway, what I wanted to complain about
16:54 sapier and "near to" wont stop shadow from complaining about those things not beeing exactly how he did understand our unprecise specification
16:55 sapier complains without reason and suggestion are ignored too ;-)
16:57 sapier yxz CET is american name for MET
16:57 sfan5 can we just ue UTC for important things?
16:57 sapier MET-1 ;-)
16:58 sapier btw I don't consider a simple cleanup to be something important
16:58 xyz using struct for non-pod should be discouraged
16:58 sapier xyz can you give an example?
16:59 xyz an example of what?
16:59 sapier well what you did is a generall saying I assume you have some context to it ?
17:00 sapier as far as I know whole minetest code uses structs for non pod types so unless you wanna change it everywhere I see no use in a rule like that except of bugging developers doing things like they are already done
17:00 sapier +for
17:02 xyz ugh
17:02 xyz why is it always all or nothing
17:02 sapier what's use of rules for non critical things?
17:03 xyz you can apply new rules to new code so quality of code will gradually increase
17:03 sapier because it is already crazy to get a commit in shape for merge right now ... not because of bugs but because of unspecified questionable rules
17:03 xyz yet you apply your all-or-nothing argument and in the end quality of new code is even worse
17:04 sapier and the real bugs aren't even found on review
17:04 sapier if there would be found only half the amount of bugs  formal issues are found quality would rise drasticaly
17:05 sapier Yes because it's crap to fix some locations and have exactly same bug in other locations
17:05 sapier and usually once I fix something I get bugged for not fixing other things I haven't even touched ... which is extremely annoying
17:06 xyz it's always simple things
17:06 sapier if at least one of the really critical bugs would've been found prior merge I'd be way less annoyed about that formal things
17:06 xyz i.e. why have void gotText(std::string message) if it's not used anywhere and only produces an error and makes it harder to write new code?
17:07 sapier because that function is part of interface
17:07 xyz that nothing now implements, correct?
17:07 sapier and no I it's not my interface
17:08 sapier no the function is called
17:08 xyz where?
17:09 sapier iirc some exit function in guiformspec calls it
17:09 sapier and usually people write their own functions if they're not remembered about the existing ones
17:10 sapier but that's a formal thing again, did anyone compile and use it yet?
17:12 xyz I certainly didn't
17:14 xyz see, you're not interested in my suggestions so I too am not interested in compiling this code
17:14 sapier I am interested as long as it's not add a tab there move that function 2 lines above and things like that
17:15 sapier the getText you mention is just specified by interface if it's used where it shouldn't be used imho it's worth an error
17:18 xyz left #minetest-dev
17:19 rubenwardy joined #minetest-dev
17:22 sfan5 I'm not completly done but here is documentation for cmake switches: http://dev.minetest.net/CMake_Options
17:24 sapier good work, guess we can add this to readme too?
17:24 VanessaE sfan5: cURL enables the use of remote media fetching via http also, no>
17:24 VanessaE ?
17:25 sapier sfan5 did you check your luajit detection fixes to really stop usage of old luajit versions?
17:25 VanessaE this looks good (wish I'd had this a couple of times in the past, in fact)
17:25 proller sapier, did you play in MT, or only write code?
17:28 VanessaE fwiw, the luajit version check works on ubuntu :P  and forced me to pull and install that from git
17:29 sapier sfan5 because for my tests your changes just break the check completely you could've just reverted that commit too
17:31 sapier well actually it'd be better if you did remove it instead of making it non functional
17:35 sapier VanessaE prior or after sfan5's changes?
17:36 sapier because that cmake submodule just checks if the supplied code compiles which is always true if you do it the way he did change it
17:39 sfan5 sapier: the check works for me
17:39 sfan5 VanessaE: yes it does, add that
17:40 VanessaE sapier: well I'm at commit 9a3b7715 + your 1143 patch, so whatever that does :P
17:41 sfan5 VanessaE: i'll add it for you (b/c I will complete it now)
17:41 sapier sfan5? did you read my comments above?
17:41 VanessaE sfan5: done.
17:42 sfan5 ...
17:42 sfan5 sapier: I'm pretty sure I have not broken the check
17:43 Zeitgeist_ joined #minetest-dev
17:44 sapier ok I'm gonna make c fresh pull and only add my current luajit version to broken versions again ... just to make sure no old relics are in my dir
17:48 sapier ok my fault.  The check only fails for existing clones ... well I guess everyone already having a cloned repo will know about the broken luajit versions
17:51 PenguinDad sapier, just tried #1164 and I think the menus are too large
17:51 ShadowBot https://github.com/minetest/minetest/issues/1164 -- Replace builtin pause and message menus by formspec based ones by sapier
17:52 sapier too large? thought they're exactly same size as before? can you provide a screenshot?
17:54 vifino joined #minetest-dev
17:54 vifino If someone who codes on MineTest wanna help me to get a macbuild working, say it!
17:54 vifino I am working on that
17:54 vifino I got cmake working
17:55 vifino ( AKA no errors )
17:55 vifino the make -j2 failes
17:55 sapier just post a link to the error message
17:55 vifino kay
17:55 VanessaE PenguinDad: screenshots to compare?
17:55 sfan5 ok
17:55 sfan5 I completed the cmake options page on the wiki
17:56 sfan5 someone please look if I missed something
17:57 sapier good work
17:57 sapier DISABLE_LUAJITcould be interesting too
17:58 vifino http://pastebin.com/W2QHJW7S
17:58 VanessaE VERSION_EXTRA - Text too append   s/too/to/
17:58 vifino there it is
17:58 VanessaE (fixed)
17:58 VanessaE this seems good to me.
17:58 vifino sem_timedwait doesnt exists on mac
17:58 vifino like some bsds
17:58 sapier that's a problem
17:59 sapier is there something similar?
17:59 vifino sem_wait and sem_trywait exists
17:59 VanessaE sapier: wait, I thought there already was such a flag?
17:59 sapier yes it's missing in doc only
18:00 sapier vifiny you can build a timedwait replacement by using trywait ... it's not best style but will work
18:00 sapier get current time, do a trywait, loop until time is passed or your trywait succeeds
18:00 vifino sapier How?
18:01 vifino please help me
18:01 vifino There is my current reposiroty: https://github.com/vifino/minetest-osx
18:02 sapier give me a minute
18:02 vifino sapier take your time
18:04 Jordach joined #minetest-dev
18:05 sfan5 sapier: added DISABLE_LUAJIT
18:06 vifino brb
18:07 Jordach joined #minetest-dev
18:07 sapier really good work ... I didn't even know about the EGL flags before
18:08 vifino back
18:08 sfan5 the GLES flag was added to prevent it automatically using gles libs that seem to be present on archlinux and screwing up irrlicht
18:13 sapier https://gist.github.com/sapier/9331103 untested directly written to gist
18:14 vifino sapier what do i do with that?
18:14 sapier that code isn't best to do but may work as replaxement for timed wait
18:14 sapier vifino do you have any experience in coding?
18:15 vifino In several languages
18:15 vifino but not C
18:15 sapier ok then there's at least some chance to get it ported
18:15 vifino ...
18:15 * sfan5 just categorized the Library specific CMake flags
18:16 sapier sfan5 can you merge the old version to readme?
18:16 vifino sapier join #vifino to see my latest project in action
18:16 sfan5 sure
18:17 sfan5 sapier: http://dev.minetest.net/index.php?title=CMake_Options&amp;oldid=1806 that one?
18:17 sapier great, it'd be a pitty if we'd have to do the job twice
18:17 sapier yes
18:18 vifino sapier what do i do with that piece of code
18:18 vifino Do i paste it in place of the line or what?
18:18 sapier it's a replacement for sem_timedwait
18:19 sapier but as I said it's untested it may or may not work
18:19 vifino sapier yes, but WHERE do i put that piece of code
18:19 sapier exactly at that location where sem_timedwait is used
18:19 sapier src/jthread/pthread/jsemaphore.cpp
18:20 sapier wait a second
18:20 vifino waittime
18:21 sapier it's better this way
18:21 vifino sapier or just put it in my github
18:21 vifino or else i would probably fail
18:22 sapier ok I updated the whole function for you
18:22 vifino mkay
18:25 sfan5 ''Document CMake options in README'' incoming in 10 minutes
18:26 sapier does anyone know HOW the code style for what shadow did complain in #1164 should be?
18:26 ShadowBot https://github.com/minetest/minetest/issues/1164 -- Replace builtin pause and message menus by formspec based ones by sapier
18:28 sapier well that's annoying ... someone complains ... you fix it and then someone else complains
18:32 PenguinDad sapier, VanessaE: http://imgur.com/a3ef3lb,bvejYtu
18:33 VanessaE wow
18:34 sapier what's wrong about it?
18:34 VanessaE it's... um...huge
18:34 VanessaE and honestly it's not well-formatted but that can be fixed
18:35 sapier well the formating is exactly as original one ;-) ... I guess I know what's wrong ... formspecs scale to window size ... I did only check with default size
18:35 sapier not a big deal to fix
18:53 EvergreenTree joined #minetest-dev
18:59 sapier PenguinDad: now it should look exactly like before even on larger/smaller resolutions
18:59 sapier ok on larger only :-)
19:00 sapier oops ... forget about it :-)
19:00 VanessaE bbl
19:01 sapier broke mainmenu by adding a feature in a generic way :-)
19:01 salamanderrake joined #minetest-dev
19:03 sapier but now it's fixed
19:05 sapier but I'm gonna give that fix another day of testing maybe there's not only PenguinDad who does real testing
19:11 Zeitgeist_ joined #minetest-dev
19:23 Garmine left #minetest-dev
19:23 Anchakor_ joined #minetest-dev
19:43 Garmine joined #minetest-dev
19:53 NakedFury joined #minetest-dev
20:22 BrandonReese_ joined #minetest-dev
20:47 proller joined #minetest-dev
21:13 khonkhortisan joined #minetest-dev
21:37 grrk-bzzt joined #minetest-dev
21:38 VanessaE ShadowBot: tell sapier nope, that last build (bugfix for 4.1.1) didn't work either. same crash as the others.
21:38 ShadowBot VanessaE: O.K.
21:39 vifino joined #minetest-dev
21:45 sapier joined #minetest-dev
23:06 Zeitgeist_ joined #minetest-dev
23:38 smoke_fumus joined #minetest-dev

| Channels | #minetest-dev index | Today | | Google Search | Plaintext