Times are UTC Toggle Colours
01:34:07 *** Lakie has quit IRC 06:26:31 *** andythenorth has joined #openttdcoop.devzone 06:38:40 *** ODM has joined #openttdcoop.devzone 08:01:45 <Yexo> planetmaker: not sure if you remember the history, but dimme was the one who made the modular airports minigame 08:01:58 <Yexo> I don't think he's interested in the graphics, just in the statemachines 08:02:19 <planetmaker> yes, iirc he was that person 08:03:23 <planetmaker> I only try to get people play with what we have :-) If he doesn't catch it, maybe someone else 08:04:50 *** frosch123 has joined #openttdcoop.devzone 08:14:53 <planetmaker> ha... saving by layer name works :-) 08:15:49 <Terkhen> :) 08:17:00 <planetmaker> now only make it nicely usable ;-) 08:29:50 <andythenorth> planetmaker: that would be win 08:30:13 <planetmaker> definitely 08:30:26 <planetmaker> by number is a pain if a layer is added 08:30:39 <planetmaker> now it just needs a unique layer name 08:30:51 <planetmaker> s/a// 08:30:58 <planetmaker> s/name/names/ ;-) 08:55:04 <Yexo> frosch123: when skipping the groundsprite via the advanced sprite layout, will all 'extra' ground sprites also be skipped? 08:55:27 <Yexo> I mean all childsprites directly following the groundsprite before the first parent sprite 09:03:39 <Brot6> CHIPS Station Set - Revision 142:3e6027faf7a8: Feature: split menu to 'stations' and 'tiles' classes (andythenorth) @ http://dev.openttdcoop.org/projects/chips/repository/revisions/3e6027faf7a8 09:03:39 <Brot6> CHIPS Station Set - Feature #2690 (Closed): Decide whether to have one or two classes of station (andythenorth) @ http://dev.openttdcoop.org/issues/2690#change-6893 09:13:29 <Brot6> CHIPS Station Set - Feature #2689: Remove deprecated clay tiles (andythenorth) @ http://dev.openttdcoop.org/issues/2689#change-6894 09:13:30 <Brot6> CHIPS Station Set - Feature #2493: Parcels office (andythenorth) @ http://dev.openttdcoop.org/issues/2493#change-6895 09:16:06 *** andythenorth has left #openttdcoop.devzone 09:17:18 <frosch123> Yexo: no 09:17:37 <Yexo> ok, thanks 09:49:17 <Brot6> OpenGFX+ Trains - Revision 244:2b1394232927: -Feature: [Makefile] Allow to use xcf as direct sour... (planetmaker) @ http://dev.openttdcoop.org/projects/ogfx-trains/repository/revisions/2b1394232927 09:49:17 <Brot6> OpenGFX+ Trains - Bug #2654 (Closed): container colours upon train reversal (planetmaker) @ http://dev.openttdcoop.org/issues/2654#change-6900 09:57:28 *** andythenorth has joined #openttdcoop.devzone 10:22:28 *** JVassie has joined #openttdcoop.devzone 11:08:38 <Hirundo> Nice work on NML, Yexo 11:09:21 <Yexo> almost finished work on #2739 bit 0 11:09:21 <Brot6> Yexo: #2739 is http://dev.openttdcoop.org/issues/show/2739 "NewGRF Meta Language - Feature #2739: Advanced sprite layouts - #openttdcoop Development Zone" 11:11:03 <JVassie> oooh 11:11:30 <planetmaker> sometimes I really appreciate Brot's auto-quote of issues :-) 11:17:45 <Terkhen> :) 11:18:23 <Brot6> NewGRF Meta Language - Revision 1416:517aae5b906a: Feature #2739: hide_sprite property in tilelay... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/517aae5b906a 11:30:02 <Brot6> OpenGFX+ Trains - Bug #2734 (Rejected): DevZone compile failed (planetmaker) @ http://dev.openttdcoop.org/issues/2734#change-6901 11:55:27 <Ammler> IMO, we could release a 0.6.5-beta and upload to bananas with version restrictions 11:55:39 <Brot6> NewGRF Meta Language - Revision 1417:228652066b28: Fix: various bugs in hide_sprite implementation (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/228652066b28 11:55:45 <Ammler> or call it 0.6.5-nml :-) 11:56:04 <Yexo> Ammler: andy wanted to do more testing today before uploading it 11:56:08 <Ammler> (speaking about firs) 11:56:26 <Yexo> planetmaker: http://devs.openttd.org/~yexo/airport_fences.diff This adds the hide_sprite property to the airport fence defines 11:56:38 <Yexo> I couldn't find a good way to add those fence defines to every spritelayout 11:57:11 <Ammler> well, for andy it is not possible to upload a test version to bananas, which is wrong 11:58:01 <Ammler> I don't think, if you call the version beta, it does harm firs reputation ;-) 11:58:05 <Yexo> I'm not starting that discussion again, it's pointless. Uploading a new version with version restrictions would make the old version unavailable, which is bad 11:58:15 <Yexo> nobody cares about "beta", except developers 11:58:40 <Yexo> and there is nothing wrong with making reasonable sure a version works before uploading it 11:58:53 <Ammler> Yexo: I wasn't aware, bananas is broken, never mind then 11:59:09 <Yexo> https://secure.openttd.org/bugs/task/2966 12:01:06 <frosch123> [13:56] <Yexo> planetmaker: http://devs.openttd.org/~yexo/airport_fences.diff This adds the hide_sprite property to the airport fence defines <- does that mean, it works? :) 12:01:16 <Yexo> yes :) 12:02:17 <Yexo> frosch123: I was confused for a while about bit 0: "the "skip bounding box"-register uses value 1 to draw the sprite, and 0 to skip it, " 12:02:29 <Yexo> I had implemented it as 1, skips, 0 draws 12:02:48 <Ammler> well, the other alternative is to make a 2nd testing entry 12:02:48 <Yexo> that was also due to my naming "hide_sprite" 12:03:50 <frosch123> :) 12:05:28 <Ammler> (like LumberMill and LumberMill PS) 12:06:04 <planetmaker> I'll give it a shot, Yexo and try to enable it :-) Thanks a bunch 12:13:57 <Yexo> frosch123: did you plan on making nforenum support the advanced sprite layouts? if not I can work on that now 12:15:24 <Brot6> CHIPS Station Set - Revision 143:bd4c9ef58294: Change: replace Clay tiles with other tiles (andythenorth) @ http://dev.openttdcoop.org/projects/chips/repository/revisions/bd4c9ef58294 12:16:48 <frosch123> Yexo: http://devs.openttd.org/~frosch/diffs/wip_advsprlayout.diff 12:16:56 <frosch123> action 0 support is finished, but completely untested 12:17:09 <frosch123> action2 is basically unstarted 12:17:15 <frosch123> feel free to pick it up 12:17:21 <frosch123> i am doing wiki today :) 12:17:35 <Yexo> ok, I will try to finish action2 support 12:18:22 <frosch123> i guess best is to revert changes in src/act123.cpp :) 12:19:12 <frosch123> hmm, i guess i forgot to bump the action data format version 12:19:18 <Yexo> why is that? it looks like a good start 12:19:20 <frosch123> there are new syntax elements 12:19:38 <Hirundo> Yexo: It might make sense, to remove the difference between 'ttdsprite' and 'sprite' in spritelayouts 12:20:04 <Yexo> makes sense 12:20:05 <Hirundo> i.e. make 'ttdsprite' an alias of 'sprite', and emit a warning upon its use 12:20:19 <frosch123> Yexo: the whole offset issue is quite complicated there 12:21:19 <Brot6> FIRS Industry Replacement Set - Revision 2062:18c9ff60fb21: Fix: mistake with snow for Builders Yard (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/18c9ff60fb21 12:21:36 * andythenorth has to remember to be in right branch :P 12:23:14 <Yexo> Hirundo: currently we have "sprite: spriteset_name;", what about "sprite: spriteset_name(offset);" for the advanced layout? 12:23:24 <Yexo> where offset is any varaction2 expression 12:23:38 <Hirundo> There is already such thing implemented, for constants only 12:23:39 <Yexo> indicating the offset within the spriteset, ie bit 1 12:23:48 <Yexo> there is? good :) 12:23:50 <Yexo> that makes it easy 12:23:57 <Hirundo> Though I agree, it might be documented quite poorly 12:24:34 <Yexo> ah indeed, SpriteGroupRef can have params already 12:24:36 <Hirundo> There's also the possibility of adding labels to spritesets 12:27:04 <Hirundo> Are you gonna work on that now, or do you have any pending changes in action2layout? 12:27:15 <Yexo> I have no pending changes and I'm going to work on nforenum first 12:27:40 <Yexo> and before I do any more work on nml I'm going to document some of the new code 12:30:33 <Brot6> FIRS Industry Replacement Set - Revision 2063:d1ce3e29b605: Fix: Plastic Plant was using broken g... (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/d1ce3e29b605 12:30:33 <Brot6> FIRS Industry Replacement Set - Revision 2064:d0d1ccf81c27: Cleanup: remove unneeded pcx and temp... (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/d0d1ccf81c27 12:31:40 <Brot6> FIRS Industry Replacement Set - Bug #2747 (Closed): Plastic Plant snow sprites out of date (andythenorth) @ http://dev.openttdcoop.org/issues/2747#change-6902 12:31:40 <Brot6> FIRS Industry Replacement Set - Feature #1786 (Closed): Builders Yard needs snow graphics (andythenorth) @ http://dev.openttdcoop.org/issues/1786#change-6903 12:32:07 <planetmaker> hm... it would have helped, if I had pulled the CBF rename version straight away :-P 12:34:25 <Brot6> FIRS Industry Replacement Set - Revision 2065:038a50abad57: Cleanup: remove unneeded pcx files (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/038a50abad57 12:35:54 <planetmaker> works like a charm, Yexo :-) 12:36:09 <Yexo> great :) 12:36:30 <planetmaker> I guess I'll add fences to all airports and we could then release it as 0.2.0 or so 12:36:40 <planetmaker> re-written in NML and with one rotation for the small one 12:36:56 <Yexo> sure thing 12:37:07 <Yexo> only thing to watch out for is fences at the end of runways 12:37:18 <planetmaker> that's already taken care of. There'll never one 12:37:28 <planetmaker> +bw 12:37:31 <planetmaker> +be :-P 12:37:50 <planetmaker> though there are default airports where there is actually 12:37:57 <planetmaker> but I don't want to copy that detail 12:38:21 <Yexo> indeed, imo that's a bug of the default airports 12:44:28 <frosch123> all fences are ground/child sprites, right? 12:44:40 <planetmaker> yes 12:45:03 <frosch123> fine :) bounding boxes for them result in the classical foundation glitch 12:45:17 <planetmaker> actually... the fence is a building sprite. But... 12:45:54 <frosch123> so, you readd that bug :p 12:46:00 <planetmaker> hm? 12:46:52 <frosch123> r21272 | frosch | 2010-11-20 15:45:52 +0100 (Sa, 20 Nov 2010) | 1 line 12:46:53 <planetmaker> also with fences as building? 12:46:53 <frosch123> -Change [FS#119-ish]: Draw airport fences as groundsprites on runway/taxiway tiles. This makes the usual foundation-glitch a lot less likely to happen. 12:47:18 <planetmaker> well, it's easy to define them as child sprite 12:47:28 <frosch123> if the fence bounding box is above the foundation, it will force the foundation above the plaint in most cases 12:47:46 <frosch123> what do you mean with 'fences as buildings' ? 12:49:02 <planetmaker> currently I define them as building sprite 12:49:25 <planetmaker> within the sprite layout 12:49:28 <planetmaker> of a tile 12:50:33 <Yexo> planetmaker: yes, that is exactly what frosch was talking about 12:50:48 <Yexo> to make them childsprites make sure they're defined directly after the groundsprite 12:51:34 <planetmaker> hm, ok. also feasible, but currently they're not that, indeed 12:51:42 <planetmaker> should be easy to change :-) 12:53:34 <planetmaker> so, thanks for the hint, frosch123 :-) 12:53:41 <Brot6> FIRS Industry Replacement Set - Revision 2066:544788e66e75: Change: partial snow support for Lime... (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/544788e66e75 12:54:45 <planetmaker> andythenorth: I still think you should use the default ttd snowy ground tile and draw the other stuff on top of it. That'll make it blend in with both base sets 12:54:59 <andythenorth> that may be possible soon 12:55:07 <planetmaker> it's also possible now 12:55:20 <andythenorth> not without throwing away the current templating 12:55:42 <planetmaker> it needs slight modification to use always the ground sprite and an overlay 12:56:01 <andythenorth> if it's possible, I miss how :) 12:56:10 <andythenorth> it's one of the cases advanced was invented for 12:56:28 <andythenorth> even if it is solvable by some complex means, advanced layouts is about to solve it (hopefully) 12:59:39 <Yexo> frosch123: http://devs.openttd.org/~yexo/advanced_layouts.diff This works for the action2's I have so far 13:00:00 <Yexo> I didn't test any action0's, do you have a testcase for that already? 13:00:26 <frosch123> no :p 13:00:30 <Yexo> only update other than act123.cpp is updating the NDF_HEADER to a new version 13:00:49 <planetmaker> andythenorth: I'd simply add in each tile layout: use of the default ground sprite. use of a ground child sprite. And then optionally buildings 13:01:07 <andythenorth> how do you specify the ground child sprite? 13:01:11 <planetmaker> but... well... I just mean: don't bother to draw snowy ground. You'll have that for free 13:01:20 <planetmaker> I can only tell you in NML ;-) 13:01:22 <Brot6> FIRS Industry Replacement Set - advanced_action2_test_sheepfarm.patch (andythenorth) @ http://dev.openttdcoop.org/attachments/download/1632/advanced_action2_test_sheepfarm.patch 13:01:25 <andythenorth> hmm 13:01:27 <planetmaker> but that's what I just do with airports. 13:01:29 <andythenorth> that sounds workable and useful 13:01:39 <andythenorth> for certain cases 13:01:44 <andythenorth> like farm fields 13:01:46 <frosch123> Yexo: flat 0x08 has no data, thus no additional offset 13:01:47 <planetmaker> for all ;-) 13:01:58 <planetmaker> stuff which modifies the default ground is then just drawn as overlay 13:02:02 <andythenorth> I don't see how that works 13:02:13 <planetmaker> of course it doesn't make sense for concrete - unless you use default concrete 13:02:24 <andythenorth> exactly :P 13:02:43 <planetmaker> well... just define a tile layout. Add a ground sprite. Add a child sprite, add a building sprite. Done 13:02:49 <planetmaker> The child sprite is partially transparent 13:03:03 <planetmaker> and adds only what differs from the default ground tile 13:03:13 * andythenorth still doesn't understand :P 13:03:28 <planetmaker> uhm... you know what a tile layout is? 13:03:35 <planetmaker> you can add infinitely many sprites to a tile 13:03:39 <andythenorth> I don't use child sprites 13:03:42 <planetmaker> and arrange them there. 13:03:54 <planetmaker> Thus you could add a stack of wood as child sprite to the ground etc 13:03:56 <andythenorth> I just use sprites with their own bounding box afaik 13:03:57 <planetmaker> oh... you don't? 13:03:59 <planetmaker> You should! 13:04:06 <andythenorth> I don't see how it solves the case though 13:04:14 <andythenorth> a base set sprite number still has to be specific 13:04:16 <planetmaker> default ground: ttdsprite 13:04:16 <andythenorth> specified /s 13:04:20 <planetmaker> addon: yours 13:04:33 <planetmaker> yes. But the base set number of course is known 13:04:37 <andythenorth> ok I get it 13:04:42 <planetmaker> and 95% of your cases is flat 13:04:49 <andythenorth> it's almost what I'm doing now, but change a certain order 13:05:08 <planetmaker> i.e. don't provide ground yourself, just ground modifications 13:05:17 <andythenorth> it would work 13:05:33 <planetmaker> and add a lot compatibility wrt other graphics and also landscape newgrfs 13:06:00 <andythenorth> I'll consider it 13:06:17 <planetmaker> how I hate that word "consider" in this context 13:06:19 <planetmaker> ;-) 13:06:29 <andythenorth> we can't string concatenate in CPP can we ? 13:06:35 <planetmaker> we can 13:06:38 <andythenorth> we can? 13:06:41 <andythenorth> hmm 13:06:42 <planetmaker> blub ## blah 13:06:59 <planetmaker> at least that's how it works in macros 13:07:06 <andythenorth> it would mean reworking the templates for all industries 13:07:10 <planetmaker> #define MACRO(a, b) a ## b 13:07:24 <planetmaker> concatenates a and b 13:07:36 <andythenorth> we'd need (a,b,c) 13:07:37 <planetmaker> when you use MACRO(a, b) 13:08:00 <planetmaker> please elaborate a bit more 13:08:23 <andythenorth> current templating requires 13:08:27 <andythenorth> #define GRAPHICS fooindustry.png 13:08:29 <andythenorth> or 13:08:34 <andythenorth> #define GRAPHICS fooindustry_snow.png 13:08:56 <andythenorth> to make this method work we also need groundtiles.png 13:09:00 <planetmaker> you mean conditionally adding _snow? 13:09:02 <planetmaker> feasible 13:09:03 <andythenorth> and groundtiles_snow.png 13:09:25 <andythenorth> or I have to copy ground tile into every industry png, which I dislike 13:09:40 <andythenorth> already I am doing that for snow, but it's temporary I hope :P 13:10:11 <andythenorth> I don't know if templating will survive transition to nfo 13:10:12 <planetmaker> :-) well... I guess we can fix that when we have NML... 13:10:18 <andythenorth> nml / nfo /s 13:10:25 <planetmaker> the cpp templates for that can be copied from ogfx+airports 13:10:46 <planetmaker> even the conditional fence code now ;-) 13:10:54 * andythenorth wonders if it will be scriptable to migrate existing pnfo 13:10:58 <planetmaker> like draw fences on the outside automatically ;-) 13:11:00 <andythenorth> otherwise it is very painful 13:11:16 <planetmaker> that's hardly scriptable 13:11:29 <planetmaker> that's what the grf2nml does... and then it needs tidying 13:11:47 <planetmaker> but... you'll have help with that 13:11:52 <Yexo> * andythenorth wonders if it will be scriptable to migrate existing pnfo <- definitely not 13:12:01 * planetmaker thinks that 3 people will help andy there 13:12:13 <andythenorth> it's a big job you guys are taking on :P 13:12:26 <andythenorth> you are brave indeed :) 13:12:28 <Yexo> the sooner we can start the better :) 13:12:33 <frosch123> Yexo: diff looks fine 13:12:34 <planetmaker> ^ :-) 13:12:36 <andythenorth> I am quite happy to not be sole maintainer in future 13:12:49 <planetmaker> see. Only to stop that moaning it's worth it :-P 13:13:02 <Yexo> http://paste.openttdcoop.org/show/292/ <- frosch123: can you explain that warning? 13:13:06 <Yexo> I can't find it anywhere 13:15:04 <frosch123> if(!hasGround)IssueMessage(WARNING2,NO_GROUNDSPRITE,NONTRANS); 13:15:22 <frosch123> the example has \dx0 for the groundsprite 13:16:27 <Yexo> what is NONTRANS doing there instead of GROUND? 13:17:03 <frosch123> the layout only contains a single sprite, the groundsprite, and that one is 0 13:17:09 <frosch123> so nforenum sees no sprite at all 13:17:15 <Yexo> ah, ok 13:17:23 <frosch123> maybe just disable that test if there are flags? 13:18:09 <frosch123> Yexo: btw. your example is not valid 13:18:24 <frosch123> 40 does not use the syntax of the basic layout 13:18:35 <Yexo> ah, so it needs a zoffs 13:19:04 <frosch123> yes, 13:19:49 <Yexo> ok, it seems to work 13:22:48 <Yexo> frosch123: there is a bug in your action0 code though 13:22:55 <frosch123> :) 13:22:55 <Brot6> FIRS Industry Replacement Set - Revision 2067:f23201850384: Change: convert more things to png an... (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/f23201850384 13:23:05 <frosch123> what? 13:23:33 <Yexo> it spawns a lot of errors about invalid properties 13:23:40 <Yexo> on nfo lines it previously found correct 13:23:54 <Brot6> FIRS Industry Replacement Set - Revision 2068:7074bd71ab1d: Fix: missing files (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/7074bd71ab1d 13:23:56 <frosch123> ah, you mean it breaks something :( 13:25:26 <Brot6> FIRS Industry Replacement Set - Revision 2069:35650cf8b5f3: Cleanup: remove further unneeded pcx ... (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/35650cf8b5f3 13:31:18 <frosch123> Yexo: you can make the groundsprite test more strict, not all flags are valid there 13:31:31 * andythenorth should automate this pcx->png conversion :P 13:31:41 <andythenorth> but can't be bothered to handle edge cases 13:32:03 <Brot6> FIRS Industry Replacement Set - Revision 2070:9d90f5ffdc70: Cleanup: convert further pcx to png (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/9d90f5ffdc70 13:32:14 <frosch123> (i.e. xoffs and yoffs are not valid) 13:32:22 <Yexo> frosch123: bits 0, 1, and 2 are valid, 3 is valid but doesn't eat a byte, rest is invalid? 13:32:38 <frosch123> yes, FFF0 13:33:05 <Yexo> http://devs.openttd.org/~yexo/advanced_layouts.diff This works for all actions2's I've tested 13:33:17 <Yexo> I've left out the action0 code as I don't fully understand it and it breaks things 13:36:06 <frosch123> ground_flags&0x03 <- what is that for? 13:36:19 <frosch123> you mean 0x02 ? 13:36:27 <Yexo> no, I mean 0x03 13:36:35 <Yexo> it checks for ground != 0, so both sprite and palette 13:36:58 <frosch123> so 0x06? :p 13:37:19 <Yexo> yes 13:38:53 <frosch123> if(!CheckSpriteNum(building,off+1,act123::Instance().act1,feature,mismatch,hasGround)&& <- that +1 is too much now? 13:39:48 <frosch123> if(!building)IssueMessage(ERROR,NO_BUILDING_SPRITE,off+1); <- same 13:39:55 <Yexo> yep, indeed 13:40:08 <Yexo> if(!building)IssueMessage(ERROR,NO_BUILDING_SPRITE,off); <- and that needs a check for the flags too 13:40:25 <frosch123> + uint x=data.ExtractByte(curoff++),y=data.ExtractByte(curoff++); <- i guess excution order is not well defined there 13:40:53 <Yexo> also fixed 13:41:11 <Yexo> that however was original code 13:41:23 <frosch123> it had no ++ before 13:41:36 <Yexo> right 13:42:31 <frosch123> if(++i!=nument1)data.SetEol(off,2); <- no idea, whether that needs a - 1 now 13:42:59 <frosch123> looks fine :) 13:43:06 <frosch123> i should test action 0 better 13:49:33 <Yexo> fixed that and another SetEol line 13:49:45 <Yexo> shall I commit? or do you want to fix action0 and this in one go? 13:50:11 <frosch123> go ahead and commit :) 13:50:16 <Brot6> OpenGFX+ Airports - Revision 94:e9bbeee37fc7: Feature: Automatically draw fences around airport t... (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/e9bbeee37fc7 13:51:16 <planetmaker> and now building -> childsprite... that needs some serious offset fixes 13:51:42 <Brot6> GRFCodec - Revision 832:08511c778a07: Feature: support the advanced sprite layout in action2 (bas... (yexo) @ http://dev.openttdcoop.org/projects/grfcodec/repository/revisions/08511c778a07 13:52:51 <Yexo> Hirundo: to avoid double work: are you working on nml and if so, in which files? 13:53:27 <Hirundo> I'm working in action2layout currently 13:53:28 <frosch123> planetmaker: you can compute them 13:53:32 <Hirundo> I'll see if I can commit shortly 13:53:40 <Yexo> ok 13:54:11 <frosch123> planetmaker: xoffs = 2 * (dy - dx), yoffs = dx + dy - dz 13:55:35 <planetmaker> dx and dy of the ground sprite or the child sprite? 13:58:16 <Hirundo> Yexo: I'll commit now to avoid merging conflicts later, as I got to leave right now 13:58:33 <Yexo> fine by me 13:58:34 <Yexo> have fun :) 13:58:35 <Hirundo> Mind that the code is tested to the point that it passes regression 13:58:42 <Hirundo> i.e. pretty badly :) 14:00:14 <Hirundo> hmm... I missed a commit already and need to merge 14:00:23 <Hirundo> Just continue your work, I'll clean up the mess later 14:00:32 <Hirundo> goodbye for now 14:05:36 <frosch123> planetmaker: dx, dy,dz of the boudingbox 14:05:48 <frosch123> xoffs, yoffs of the equivalent childsprite offset 14:07:42 <planetmaker> but... ground tiles have no bounding box? 14:09:31 <planetmaker> i.e. I'd need to move the fences in the SE and SW left. By about 31 px. But the offset can only be positive... 14:12:55 <Yexo> try 15, 0, 0 for SW and 0, 15, 0 for SE 14:13:12 <frosch123> the groundsprite-childsprites offset from the same position as the groundsprite and as building sprites 14:13:22 <frosch123> so, no 31 etc needed 14:14:00 <frosch123> just put your current dx, dy, dz offsets into those formulas and get the new xoffs, yoffs 14:14:16 <Yexo> <Yexo> try 15, 0, 0 for SW and 0, 15, 0 for SE <- those values are from the openttd source code 14:14:39 <planetmaker> uhm... Yexo ? I only have xoffset and yoffset at my disposal 14:14:44 <planetmaker> or I need to use a building sprite 14:14:48 <Yexo> so leave out the last one 14:14:53 <planetmaker> But not using that was the whole issue, right? 14:15:00 <Yexo> yes 14:15:11 <planetmaker> and a positive x offset... is the wrong direction 14:15:20 <Yexo> so xoff=15, yoff=0 for SW, xoff=0, yoff=15 for SE 14:17:33 <planetmaker> http://devs.openttd.org/~planetmaker/patches/fence_offset.diff <-- see yourself. Applies to head 14:17:43 <frosch123> planetmaker: http://paste.openttdcoop.org/show/293/ 14:18:07 <frosch123> (i am not sure whether xoffset and yoffset is the right syntax in childsprite 14:18:14 <planetmaker> it is 14:18:28 <planetmaker> but that gives wrong results for me 14:19:21 <frosch123> you are not using the formula 14:19:40 <planetmaker> 15 is 0x0F 14:19:51 <frosch123> xoffset: 2 * (0x0F - 0x00);\ 14:19:53 <frosch123> yoffset: 0x0F + 0x00 + 0x00;\ 14:19:56 <frosch123> ^^ in my example 14:20:05 <frosch123> + xoffset: 0; \ 14:20:07 <frosch123> + yoffset: 15; \ 14:20:10 <frosch123> ^^ in your code 14:20:44 <Yexo> ah, I missed a macro 14:32:35 <planetmaker> well, I still don't get it 14:32:42 <planetmaker> or something's bugged 14:34:31 <planetmaker> hm... 14:37:00 <Yexo> planetmaker: xoffset:30, yoffset:15 works fine for the SE fences 14:39:45 <Yexo> frosch123: for the SE fence dx=15, dy=0 14:40:07 <Yexo> that means that the formula gives xoffset: 2*(0-15) which is -30 14:40:13 <Yexo> but negative values are not allowed 14:41:30 <Yexo> planetmaker: it seems to be a bug in nml 14:41:40 <Yexo> nfo specs has <xofs> and <yofs> as signed byte 14:42:01 <planetmaker> he 14:42:32 <planetmaker> and I've been playing with the SW fence ;-) 14:44:28 <Yexo> sorry for that, I'm not reading the specs properly 14:44:41 <Yexo> that was for sprites defining a new bounding box 14:44:53 <Yexo> for child sprites it just says "x offset from the top left corner of the previous sprite" 14:45:07 <Yexo> which makes sense to be unsigned, as it is in openttd 14:45:26 <Yexo> so currently it's not possible to fix this while using the default fence sprites 14:45:34 <Yexo> we'll have to include the fences in the grf 14:47:46 *** Lakie has joined #openttdcoop.devzone 14:49:04 <frosch123> ah, too bad. 14:49:28 <frosch123> the signed/unsigned thingie is a difference in default- and station spritelayouts to other spritelayouts 14:51:34 <frosch123> hmm, no, they are signed for action2 14:51:39 <frosch123> it should work 14:51:40 <Yexo> frosch123: on the station action0 page it's not defined whether they signed or unsigned 14:51:49 <Yexo> frosch123: but not for sprites sharing their bounding box 14:53:33 <frosch123> yeah, child_offset_is_unsigned is indeed true for action2 spritelayouts 14:54:04 <planetmaker> so... keep it as building sprite? 14:54:17 <Yexo> no, include the fence as spriteset in the grf 14:54:32 <planetmaker> ok, I'll try 15:46:40 <Brot6> OpenGFX+ Airports - Revision 95:062694f20612: Change: Define our own fences thus that glitches wi... (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/062694f20612 15:47:32 *** ODM has quit IRC 15:59:24 <Brot6> OpenGFX+ Airports - Revision 96:f5b2736da5b4: Change: Apply r95 also to the small airport (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/f5b2736da5b4 16:23:46 <Brot6> OpenGFX+ Airports - Revision 97:d32b0898e253: Chage: Add a bit more fence to the small airport (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/d32b0898e253 16:36:34 *** andythenorth is now known as Guest5108 16:36:34 *** andythenorth has joined #openttdcoop.devzone 16:39:45 <Brot6> OpenGFX+ Airports - Revision 98:89977e431db9: Doc: Add readme and changelog (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/89977e431db9 16:56:10 <Brot6> OpenGFX+ Airports - Revision 99:9648b1235b99: Added tag 0.2.0 for changeset 89977e431db9 (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/9648b1235b99 16:57:10 <Brot6> OpenGFX+ Airports - Bug #2752 (New): DevZone compile failed (compiler) @ http://dev.openttdcoop.org/issues/2752 17:01:33 <planetmaker> :-( Why, Ammler ? 17:01:52 <planetmaker> Is it (again) the last nightly of NML which triggers this? 17:02:48 <planetmaker> I don't see a real error in any of the logs... but maybe I'm blind 17:08:08 <Brot6> grfcodec: update from r831 to r832 done - http://bundles.openttdcoop.org/grfcodec/nightlies/r832 17:09:29 <Yexo> planetmaker: you've comitted the tag before the nightly run from nml 17:09:39 <Yexo> hence it uses the nml nightly from yesterday, which will indeed fail 17:10:07 <planetmaker> meh... but even then I don't see in the logs where it fails. That bothers me even more 17:10:17 <Yexo> that is indeed wrong 17:11:13 <Brot6> nml: update from r1414 to r1417 done - http://bundles.openttdcoop.org/nml/nightlies/r1417 17:11:19 <planetmaker> :-D 17:12:14 <Yexo> Ammler: I did some changes to the nml output recently, all errors / warnings now appear on stderr 17:12:18 <Yexo> previously some ended up on stdout 17:12:32 <Yexo> could that be the reason the error log doesn't contain the useful error messages? 17:14:02 <planetmaker> hm, possibly, yes. 17:18:03 <Brot6> airportsplus: update from r93 to r99 done - http://bundles.openttdcoop.org/airportsplus/nightlies/r99 17:18:23 <Brot6> chips: update from r141 to r143 done (1 errors) - http://bundles.openttdcoop.org/chips/nightlies/r143 17:19:30 <Brot6> firs: update from r2060 to r2070 done - http://bundles.openttdcoop.org/firs/nightlies/r2070 17:20:00 <Brot6> FIRS Industry Replacement Set - Revision 2071:30e5e2bfe6dd: Feature: snow for Lime Kiln (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/30e5e2bfe6dd 17:20:00 <Brot6> FIRS Industry Replacement Set - Feature #1794 (Closed): Lime Kiln needs snow graphics (andythenorth) @ http://dev.openttdcoop.org/issues/1794#change-6904 17:20:08 <Brot6> ogfx-trains: update from r242 to r244 done - http://bundles.openttdcoop.org/ogfx-trains/nightlies/r244 17:20:11 <Yexo> will the cf automatically rebuild airportsplus 0.2.0 or does that need a manual prod? 17:20:18 <Brot6> Following repos didn't need a nightlies update: 2cctrainset (r750), 32bpp-extra (r40), ai-admiralai (r75), ai-aroai (r39), ailib-common (r21), ailib-direction (r17), ailib-list (r32), ailib-string (r29), ailib-tile (r16), basecosts (r25), belarusiantowns (r8), bros (r52), comic-houses (r71), fish (r653), frenchtowns (r6), german-townnames (r34), grfcodec (r832), grfpack (r279), heqs (r605), indonesiantowns (r41), manindu (r7), 17:20:18 <Brot6> metrotrackset (r56), narvs (r37), newgrf_makefile (r294), nml (r1417), nutracks (r202), ogfx-industries (r119), ogfx-landscape (r69), ogfx-rv (r107), ogfx-trees (r50), opengfx (r678), openmsx (r97), opensfx (r97), smts (r19), snowlinemod (r49), spanishtowns (r10), sub-landscape (ERROR r66), sub-opengfx (ERROR r666), swedishrails (r202), swisstowns (r22), transrapidtrackset (r15), ttdviewer (r34), ttrs (r36), worldairlinersset (r672) 17:20:22 <planetmaker> I'm not sure. Let's see ;-) 17:24:36 <planetmaker> Yexo: AirportsPlus / climate dependent airports / OpenGFX+ Airports has no release thread yet? 17:24:55 <Yexo> not sure, quite possible 17:27:00 <planetmaker> oh... I didn't see zero.eight's very constructive and helpful postings... 17:27:12 <planetmaker> well :-) And he should be credited for the snowy versions of the graphics 17:27:20 <Yexo> yes, isn't he? 17:27:58 <Brot6> ogfx-industries: rebuild of r119 done (Diffsize: 32645) (DiffDiffsize: 32691) - http://bundles.openttdcoop.org/ogfx-industries/nightlies/r119/log 17:28:10 <planetmaker> no, he isn't. I wasn't aware of him doing that 17:28:18 <planetmaker> and there was no pre-existing readme in 0.1.0 17:28:40 * andythenorth climbs a FIRS ticket mountain slowly http://dev.openttdcoop.org/projects/firs/versions/145 17:28:41 <Brot6> ogfx-landscape: rebuild of r69 done (Diffsize: 1176) (DiffDiffsize: 1168) - http://bundles.openttdcoop.org/ogfx-landscape/nightlies/r69/log 17:29:03 <planetmaker> hm... I could add the release posting just to the dev thread... can be discussed much better IMHO 17:29:40 <planetmaker> could you change the first posting's title to OpenGFX+ Airports please, Yexo ? 17:29:46 <planetmaker> http://www.tt-forums.net/viewtopic.php?f=26&t=47475&hilit=airports+plus&start=20 17:29:47 <Webster> Title: Transport Tycoon Forums View topic - [OTTD] Climate dependant airports (at www.tt-forums.net) 17:30:00 <Yexo> sure 17:30:06 <Yexo> keep the [OTTD] or not? 17:30:25 <planetmaker> well... I don't mind either way 17:30:43 <Yexo> added a link to the forum topic here: http://dev.openttdcoop.org/projects/airportsplus 17:31:22 <Yexo> seems no rebuild will be done 17:31:33 <Yexo> I'd say credit zero.eight and just release 0.2.1 17:32:05 <planetmaker> 0.2.0a ;-) 17:32:28 <Yexo> I don't like messing that much with version numbers :) 17:32:35 <Brot6> FIRS Industry Replacement Set - Feature #2753 (New): Stockyard snow sprites (andythenorth) @ http://dev.openttdcoop.org/issues/2753 17:32:41 * planetmaker neither. Was rather meant as joke :-) 17:32:48 <Yexo> ;) 17:33:11 <Brot6> sub-landscape: compile of r66 still failed (#2616) - http://bundles.openttdcoop.org/sub-landscape/nightlies/ERROR/r66 17:33:31 <Brot6> FIRS Industry Replacement Set - Feature #2754 (New): Recycling Depot snow sprites (andythenorth) @ http://dev.openttdcoop.org/issues/2754 17:33:55 <andythenorth> it's like...one ticket closed, two tickets opened in FIRS land :P 17:34:42 <Brot6> sub-opengfx: compile of r666 still failed (#2586) - http://bundles.openttdcoop.org/sub-opengfx/nightlies/ERROR/r666 17:36:20 <Brot6> swedishrails: rebuild of r202 done (Diffsize: 3304) (DiffDiffsize: 3185) - http://bundles.openttdcoop.org/swedishrails/nightlies/r202/log 17:37:34 <Brot6> OpenGFX+ Airports - Revision 100:2a8040aa4ce1: Fix: Changelog and credits needed an update (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/2a8040aa4ce1 17:38:26 <Brot6> Following repos rebuilds successful without any difference to earlier nightlies builds: 2cctrainset (6 errors), 32bpp-extra (2 errors), basecosts, belarusiantowns (Diffsize: 30), bros (1 errors), comic-houses (3 errors) (Diffsize: 22), fish, frenchtowns, german-townnames, heqs, indonesiantowns (1 errors) (Diffsize: 1), manindu (Diffsize: 1), metrotrackset (Diffsize: 1), narvs (Diffsize: 3015), newgrf_makefile, nutracks (9 errors), 17:38:26 <Brot6> ogfx-rv (Diffsize: 4590), ogfx-trees, opengfx (Diffsize: 7), smts (Diffsize: 8), snowlinemod, spanishtowns (Diffsize: 1), swisstowns, transrapidtrackset (Diffsize: 12), ttrs (7 errors) (Diffsize: 1324), worldairlinersset 17:39:24 <Brot6> FIRS Industry Replacement Set - Revision 2072:64b16b77667d: Fix: windmill using old graphics in s... (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/64b16b77667d 17:40:29 <Brot6> FIRS Industry Replacement Set - Feature #2755 (New): Grain Mill snow sprites need finishing (andythenorth) @ http://dev.openttdcoop.org/issues/2755 17:42:03 <Brot6> repository /home/hg/bmss registered in Redmine with url /home/hg/bmss 17:42:03 <Brot6> repository /home/hg/bmss created 17:42:43 <planetmaker> bmss? 17:43:37 <Yexo> british modular station set 17:45:20 <Ammler> planetmaker: / Yexo, did the nml issue solve itself? 17:45:26 <Ammler> (didn't read everything back) 17:45:31 <Yexo> Ammler: yes, somewhat 17:45:33 <planetmaker> not really. 17:45:36 <Ammler> :-P 17:45:37 <Yexo> tag was pushed before the nml nightly 17:45:39 <planetmaker> :-D 17:45:45 <Yexo> so it used nml r1414, which failed 17:45:47 <planetmaker> Ammler: still: NML errors are not logged anymore 17:45:53 <Yexo> the nightly compiled fine using nml r1417 17:45:59 <Yexo> however the errors ^^ 17:46:00 <Ammler> planetmaker: nowhere? 17:46:07 <Ammler> there are different log files 17:46:13 <planetmaker> nowhere I see. Look at ogfx+airports 0.2.0 17:46:14 <Ammler> or just in the wrong 17:46:43 <Ammler> hmm, which is in the error dir 17:46:48 <Yexo> Ammler: see http://bundles.openttdcoop.org/airportsplus/releases/ERROR/0.2.0/ 17:47:00 <Ammler> planetmaker: retrigger building? 17:47:20 <Ammler> ./scheduler.sh -f -releases airportsplus 17:48:00 <planetmaker> hm... do you really think this is worth a new release, Yexo if we credit him in the forums? 17:48:20 <planetmaker> and of course right now in the nightlies? 17:48:24 <Yexo> planetmaker: yes, especially since the old version isn't published anywhere 17:48:27 <Brot6> airportsplus: compile of 0.2.0 still failed (#2752) - http://bundles.openttdcoop.org/airportsplus/releases/ERROR/0.2.0 17:48:48 <Ammler> wow, that was fast :-o 17:49:01 <Yexo> Server returned an error: HTTP Error 404: Not Found 17:49:05 <planetmaker> ^ 17:49:38 <Brot6> airportsplus: compile of 0.2.0 still failed (#2752) - http://bundles.openttdcoop.org/airportsplus/releases/ERROR/0.2.0 17:58:49 <Brot6> airportsplus: compile of 0.2.0 still failed (#2752) - http://bundles.openttdcoop.org/airportsplus/releases/ERROR/0.2.0 17:59:02 <Ammler> majonaise 18:01:17 <Brot6> airportsplus: compile of 0.2.0 still failed (#2752) - http://bundles.openttdcoop.org/airportsplus/releases/ERROR/0.2.0 18:02:18 <Brot6> airportsplus: [releases] openSUSE API not reachable, sleep an hour and try again... 18:02:49 <Ammler> added another test to the api 18:10:34 <planetmaker> hm... so it won't compile anything right now? 18:12:54 <Ammler> not with devzone v1 :-) 18:13:27 <Ammler> s/:-)/:-(/ 18:14:00 <planetmaker> meh 18:33:14 <Brot6> NewGRF Meta Language - Feature #2746: station support (yexo) @ http://dev.openttdcoop.org/issues/2746#change-6905 18:33:43 <Yexo> JVassie: I'm going to postpone starting work on bmss a bit, to see how far we can get with nml support for stations in the near future 18:34:27 <Brot6> FIRS Industry Replacement Set - Feature #2756 (New): Rework Lumber large shed (andythenorth) @ http://dev.openttdcoop.org/issues/2756 18:47:00 <planetmaker> hm... I should have added a version check... OpenGFX+ Airports of course fails with stable OpenTTD versions now :S 18:47:28 <Yexo> planetmaker: extra reason for 0.2.1 18:48:01 <Yexo> Hirundo: before continuing work on making ttdsprite obsolete, please read #2746 first. If we go for that solution we may not want to make ttdsprite obsolete 18:48:01 <Brot6> Yexo: Hirundo: #2746 is http://dev.openttdcoop.org/issues/show/2746 "NewGRF Meta Language - Feature #2746: station support - #openttdcoop Development Zone" 18:53:21 <Yexo> planetmaker: the intercontinental graphics are a step backwards from 0.1.0 18:53:51 <Yexo> see the tiles that don't belong to the airport but are inside it? those are fenced, leaving a grass triangle without fences 18:54:21 <Yexo> compare to http://www.tt-forums.net/download/file.php?id=125222&t=1 where the fencing is different 18:59:24 <planetmaker> Shall that be changed? Easy to do 18:59:41 <planetmaker> I left them out of the layout deliberately 19:00:03 <Yexo> that tile? that's ok, but imo it would look nicer if the tile next to it got a diagonal fence 19:00:16 <Yexo> so the empty tile + diagonal tile are 'together' 19:00:34 <planetmaker> that require a change to that tile. And to the autofencing an exception 19:00:45 <planetmaker> Rather I add the tile to the airport so that there's no fence 19:01:00 <Yexo> that's also better than current 19:01:19 <planetmaker> and the other two 'islands'? 19:02:13 *** ODM has joined #openttdcoop.devzone 19:03:37 <Brot6> airportsplus: update from 0.1.0 to 0.2.0 done - http://bundles.openttdcoop.org/airportsplus/releases/0.2.0 19:04:29 <Brot6> NewGRF Meta Language - Bug #2740 (Closed): Can't compile very big town names file (yexo) @ http://dev.openttdcoop.org/issues/2740#change-6906 19:16:52 <Brot6> OpenGFX+ Airports - Revision 101:e7e4b671aa86: Fix: Update the minimum required version and updat... (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/e7e4b671aa86 19:23:15 <Brot6> FIRS Industry Replacement Set - Revision 2073:c700e9b39f4d: Change: framework for snow sprites fo... (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/c700e9b39f4d 19:26:10 <Brot6> FIRS Industry Replacement Set - Feature #2757 (New): Forest greeble tile needs snow graphics (andythenorth) @ http://dev.openttdcoop.org/issues/2757 19:26:10 <Brot6> FIRS Industry Replacement Set - Revision 2074:8fec86bd5639: Change: convert Forest to png (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/8fec86bd5639 19:27:16 <Brot6> OpenGFX+ Airports - Revision 102:ce0de810a333: Change: don't fence the oblique grass tiles (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/ce0de810a333 19:28:09 <Yexo> planetmaker: doesn't compile due to missing semicolon 19:28:16 <Brot6> NewGRF Meta Language - Revision 1418:0112987af978: Add #2544: very basic man file (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/0112987af978 19:28:17 <planetmaker> I just noticed :S 19:29:21 <Yexo> Ammler: wrt #2544: The only issue left is the naming. Since even big packages like PIL doesn't do that automatically it's imo not something nml should fix 19:29:21 <Brot6> Yexo: Ammler: #2544 is http://dev.openttdcoop.org/issues/show/2544 "NewGRF Meta Language - Feature #2544: rpm - #openttdcoop Development Zone" 19:29:29 <Brot6> OpenGFX+ Airports - Revision 103:e5b56d4ef503: Fix (r102): Syntax error (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/e5b56d4ef503 19:29:33 <Yexo> if you want to get rid of that you'll have to fix it during packaging 19:31:32 <Brot6> FIRS Industry Replacement Set - Revision 2075:c8eaa9b927a2: Fix: Yet Another Missing File ® (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/c8eaa9b927a2 19:32:36 <planetmaker> http://devs.openttd.org/~planetmaker/patches/ogfx_airports_021.png <-- Yexo ? 19:32:55 <Yexo> testing it myself now too :) 19:33:00 <Yexo> looks better 19:33:15 <planetmaker> then I'll push the tag 19:33:24 <planetmaker> the rest can be fine-tuned later 19:33:25 <Yexo> wait a moment please 19:33:30 <planetmaker> ok 19:33:50 <andythenorth> one day we'll standardise a commit hook for missing files :P 19:34:05 <Yexo> build-on-push 19:34:23 <planetmaker> ? 19:34:35 <Yexo> for missing files, so you get noticed as soon as you push 19:34:39 <planetmaker> bb in a bit 19:35:06 <Yexo> planetmaker: is there no snowy version of the modern depot? 19:37:03 <Ammler> Yexo: "the name policy should be filtered (rpmlint)" 19:40:06 <Yexo> planetmaker: apart from the above all looks fine :) 19:40:18 <Yexo> and that can be fixed later if we don't have graphics now 20:12:25 <planetmaker> hm, you're right. Forgot about that. And yes, there's no snowy version of the (slightly) modified depot OpenGFX uses now. 20:12:45 <planetmaker> Its size is adjusted wrt the original one found in airports.png - which is what zero.eight made snowy 20:13:10 <Yexo> ok 20:13:27 <Yexo> so perhaps we should upload the new png to the forum so someone can make it snowy? 20:13:32 <Yexo> unless you want to do that yourself 20:13:57 <planetmaker> was not that quick as I hoped for. Postponed and forgot ;-) 20:14:04 <planetmaker> I'll amend my release posting 20:17:49 <Yexo> planetmaker: another request from the topic: make the grass under the antenna apron in the international (and perhaps also intercontinental) airport 20:17:50 <Yexo> see http://www.tt-forums.net/download/file.php?id=125771&mode=view 20:18:22 <planetmaker> apron means concrete, right? 20:18:28 <Yexo> yes, sorry 20:18:41 <planetmaker> (I always wondered, assumed, but never looked up) - thanks 20:18:50 <Yexo> especially now the fences are gone (international) it would look better 20:19:02 <Yexo> *intercontinental 20:19:12 <planetmaker> yeah, also for 0.2.2 or 0.3.0... whatever :-) 20:19:42 <Yexo> sure, should I make a ticket? 20:20:45 <planetmaker> would be good, I think :-) 20:22:08 <Brot6> OpenGFX+ Airports - Bug #2758 (New): ground under antenna on intercontinental should be apron ins... (yexo) @ http://dev.openttdcoop.org/issues/2758 20:22:09 <Brot6> OpenGFX+ Airports - Bug #2759 (New): add snowy version of 'modern' depot (yexo) @ http://dev.openttdcoop.org/issues/2759 20:22:09 <Brot6> OpenGFX+ Airports - Bug #2752 (Rejected): DevZone compile failed (yexo) @ http://dev.openttdcoop.org/issues/2752#change-6907 20:22:35 <planetmaker> @base 01 16 22518 20:22:42 <planetmaker> @base 10 16 22518 20:22:42 <Webster> planetmaker: 57F6 20:27:29 <planetmaker> @base 16 10 120057f6 20:27:29 <Webster> planetmaker: 302012406 20:30:07 <Brot6> OpenGFX+ Airports - Revision 104:ebb2cfbf4760: Added tag 0.2.1 for changeset e5b56d4ef503 (planetmaker) @ http://dev.openttdcoop.org/projects/airportsplus/repository/revisions/ebb2cfbf4760 20:31:04 <Brot6> airportsplus: update from 0.2.0 to 0.2.1 done - http://bundles.openttdcoop.org/airportsplus/releases/0.2.1 20:47:35 <Brot6> FIRS Industry Replacement Set - Revision 2076:2960e5eedeb2: Fix: conversion of basecosts was done... (yexo) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/2960e5eedeb2 21:42:24 <Ammler> hmm, nmlc isn't part of the source bundle anymore 21:51:21 *** andythenorth has quit IRC 21:57:31 *** JVassie has quit IRC 21:58:50 <Brot6> FIRS Industry Replacement Set - Revision 2077:f229ae5defaa: Fix: callbacks 0x38 and 0x3A always r... (yexo) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/f229ae5defaa 22:09:04 <Brot6> FIRS Industry Replacement Set - Revision 2078:c6244bcc8cd0: Fix: switch-blocks with nvar=0 (retur... (yexo) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/c6244bcc8cd0 22:23:36 *** ODM has quit IRC 22:25:43 <Brot6> NewGRF Meta Language - Revision 1419:b98645b020e4: Doc: action1.py (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/b98645b020e4 22:25:43 <Brot6> NewGRF Meta Language - Revision 1420:bf586d7db8c6: Doc: some basic functions in generic.py (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/bf586d7db8c6 22:25:43 <Brot6> NewGRF Meta Language - Feature #2544: rpm (Ammler) @ http://dev.openttdcoop.org/issues/2544#change-6908 22:32:33 *** frosch123 has quit IRC 22:37:02 <Brot6> NewGRF Meta Language - Feature #2544: rpm (yexo) @ http://dev.openttdcoop.org/issues/2544#change-6909 22:42:15 <Ammler> Yexo: I guess, simple workaround is to add the nmlc manually 22:42:28 <Ammler> as it is needed for test only 22:43:30 <Ammler> I guess, I have a fix :-) 22:43:43 <Ammler> cp /usr/src/packages/BUILDROOT/nml-0.1.1-0.x86_64/usr/bin/nmlc . 22:44:22 <Yexo> Ammler: in that case rather run it like: make -C regression/ NMLC="nmlc" 22:45:42 <Ammler> there is no nmlc installed 22:46:21 <Ammler> the build script has no root access 22:47:29 <Ammler> I could make an additional "testsuite rpm" 22:47:58 <Yexo> please don't and wait a bit until I can find a clean way to solve this 22:48:08 <Yexo> all that is just suggesting workarounds 22:48:28 <Ammler> well, the testsuite solution is the proper distro way :-) 22:48:30 <Yexo> all of which don't fix the fundamental problem that the source bundles are unusable 22:48:58 <Ammler> the source bundles aren't unusable 22:49:06 <Yexo> if they miss nmlc they are 22:49:07 <Ammler> just the regression test doesn't work 22:49:15 <Ammler> nmlc will be installed 22:50:42 <Ammler> I assume, if you use setup.py install without --root and as user root and then run regresion as you said, it works 22:51:02 <Ammler> the buildroot is just a special rpm way 22:53:22 <Ammler> on our cf, it works, because I use the whole hg repo 22:53:26 <Ammler> instead source bundle 22:54:42 <Yexo> source bundle indeed needs installation before you can run it 22:54:58 <Ammler> yep 22:56:32 <Ammler> which is (imo) fine 22:57:15 <Ammler> the question is can setup.py also build just a nmlc? 22:57:17 <Yexo> that means that the fix should indeed be somewhere in the spec file 22:57:58 <Ammler> I mean, can you you build nml to "run from source" 22:59:25 <Ammler> oh, did you change something so that nmlc doesn't need setuptools anymore for running? 23:00:14 <Ammler> ah no, still needed 23:00:20 <Yexo> Ammler: nmlc needs setuptools only if you install it via setuptools 23:00:30 <Yexo> if you just download the repo and run from there you don't need setuptools 23:00:56 <Ammler> yep, but it needs it also for runtime, that is very strange 23:01:17 <Ammler> uncommon, no other python tool I use had that requirement 23:01:37 <Yexo> again: only if you install it via setuptools 23:01:48 <Yexo> which is the same for all scripts build via setuptools 23:01:51 <Ammler> yep, which is the usual way for distro 23:02:30 <Ammler> hmm, doesn't matter 23:02:59 <Yexo> Ammler: I can work around that (=don't let setuptools generate nmlc but include the version from the repo), but then we're back at the same situation where we were with buildtools 23:03:01 <Ammler> just not logical 23:03:09 <Yexo> ie it doesn't work automatically for andy 23:03:43 <Ammler> yeah, that is preferable :-) 23:08:25 <Yexo> which of the two was preferable? 23:08:47 <Ammler> the andy way 23:09:17 <Ammler> the setuptools requirement is just something what makes me wondering, why 23:09:22 <Ammler> it does not really harm 23:47:32 <Brot6> NewGRF Meta Language - Revision 1421:fac192646f82: Fix r1416: switch-blocks with parent scope wer... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/fac192646f82 23:52:17 <Brot6> NewGRF Meta Language - Revision 1422:276b83ca3cb7: Fix: make sure the number of sets per action1 ... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/276b83ca3cb7