Time Nick Message 01:25 M13 celeror55 04:58 hmmmm hmm 04:58 hmmmm where would a function like isNodeLiquid() belong? 05:00 hmmmm obviously in a class that exists the entire lifetime of the execution 05:02 hmmmm ah, CNodeDefManager 05:04 hmmmm oh this is great - you already did this 05:05 hmmmm ahh, this is very handy. i was sort of mislead with the whole "caching" thing going on in the original mapgen code. 06:38 celeron55 it was some wrapping for legacy code, mostly 18:26 PilzAdam this commit changes the protocol so you cant connect to old servers: https://github.com/celeron55/minetest/commit/3cab24fbcfbbeb12cdd9f085167969e1a934bd59 18:27 PilzAdam thexyz, this was your patch, is it possible to add this withou the protocol change? 18:28 celeron55 oh dog 18:28 celeron55 have you added something to ContentFeatures without adding it as the last one in a SerializationError catch? 18:29 celeron55 baaaaaaaaad 18:29 VanessaE [11-08 12:36] 12:35:45: ERROR[main]: A serialization error occurred: 18:29 VanessaE [11-08 12:36] 12:35:45: ERROR[main]: unsupported NodeBox version 18:29 VanessaE [11-08 12:36] 12:35:45: ERROR[main]: The server is probably running a different version of Minetest.c 18:29 * celeron55 fixes this crap 18:29 VanessaE celeron55: does that answer your question? :-) 18:34 celeron55 you are VERY lucky for doing this to something that isn't saved on disk 18:34 celeron55 otherwise we'd have some serious shit going on 18:36 celeron55 VanessaE: tell example server 18:36 VanessaE recrab.suret.net:30401 18:36 PilzAdam redrab.suret.net:30401 18:36 Calinou redcrab.suret.net:30401 18:36 Calinou there 18:36 VanessaE lol 18:36 VanessaE anyone else wanna chime in? 18:37 PilzAdam Calinou's is right 18:37 celeron55 and actually, i think i merged that 18:38 Calinou btw, I get can't resolve address when putting that address:port directly 18:38 celeron55 wtf 18:38 celeron55 that is a month old 18:38 Calinou I have to remove the port part then specify the port by hand in the port field 18:38 celeron55 how hasn't anyone noticed 18:38 PilzAdam it was merged on sunday 18:39 celeron55 ah, github screws up the timeline for merges 18:39 celeron55 so it was PilzAdam 18:40 PilzAdam well, xyz and darkrose told me that this works; and i tested it myself (local) 18:40 celeron55 thexyz, PilzAdam: this comment in there exists for a reason: // If you add anything here, insert it primarily inside the try-catch block to not need to increase the version. 18:43 celeron55 pushed 18:46 VanessaE thanks c55. 18:48 VanessaE my fork is up-to-date now as well. 18:54 VanessaE http://digitalaudioconcepts.com/vanessa/hobbies/minetest/builds/minetestc55_20121108-filtering-1_amd64.deb 18:54 VanessaE there's the binary build for whoever needs that. 18:56 VanessaE confirmed working on redcrab:30401.