Times are UTC Toggle Colours
00:12:43 <Brot6> OpenGFX - Bug #1494 (New): Mirror does use other bundle name (Ammler) @ http://dev.openttdcoop.org/issues/1494 00:23:02 *** Yexo has quit IRC 00:23:21 *** Yexo has joined #openttdcoop.devzone 00:39:29 <Brot6> #openttdcoop Client Patch Pack - Revision 15:47c923e4cd60: Release 1.0.4, disabled ColorTownRatin... (Ammler) @ http://dev.openttdcoop.org/projects/clientpatches/repository/revisions/47c923e4cd60 00:39:29 <Brot6> #openttdcoop Client Patch Pack - Revision 16:81e0409b4412: Tagging for 1.0.4 (Ammler) @ http://dev.openttdcoop.org/projects/clientpatches/repository/revisions/81e0409b4412 00:47:17 <Brot6> clientpatches: update from 1.0.3 to 1.0.4 done (72 errors) - http://bundles.openttdcoop.org/clientpatches/releases/1.0.4 01:02:47 <Brot6> #openttdcoop NewGRF package - Revision 271:2bed588cd624: Cleanup: keep only 1 version for longer ... (Ammler) @ http://dev.openttdcoop.org/projects/grfpack/repository/revisions/2bed588cd624 02:06:49 *** V453000 has quit IRC 02:08:16 *** V453000 has joined #openttdcoop.devzone 06:00:22 <Brot6> NewGRF Meta Language - Feature Request #1492 (Closed): refer to own grfid easily (Hirundo) @ http://dev.openttdcoop.org/issues/1492 06:00:22 <Brot6> NewGRF Meta Language - Revision 775:05ecdf0d29a9: Feature #1492: make it easy to refer to your ow... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/05ecdf0d29a9 06:00:23 <Brot6> NewGRF Meta Language - Feature Request #1492 (Closed): refer to own grfid easily (yexo) @ http://dev.openttdcoop.org/issues/1492#change-3938 06:06:45 <Brot6> NewGRF Meta Language - Patch #1459 (Closed): Some documentation additions (yexo) @ http://dev.openttdcoop.org/issues/1459#change-3939 07:03:12 *** ODM has joined #openttdcoop.devzone 07:15:46 <Brot6> OpenGFX - Bug #1494 (Confirmed): Mirror does use other bundle name (Ammler) @ http://dev.openttdcoop.org/issues/1494 07:15:46 <Brot6> OpenGFX - Bug #1494 (Confirmed): Mirror does use other bundle name (planetmaker) @ http://dev.openttdcoop.org/issues/1494#change-3940 07:16:45 <planetmaker> Hm, I think brot is reporting too much. One of those would have been enough 07:16:55 <planetmaker> good morning also :-) 08:34:47 <Brot6> Swedish Rails - Revision 160:6c27ccb6cf53: Fix: [Makefile] Dependency check for source files, usi... (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/6c27ccb6cf53 09:03:56 <Brot6> Swedish Rails - Revision 161:597df02ed0e2: Change: [Makefile] Use a more unique filename for the ... (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/597df02ed0e2 09:19:23 <Brot6> Swedish Rails - Revision 162:582afebdb30a: Fix (r159): [Makefile] Wrong file extensions slipped in (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/582afebdb30a 09:37:19 <Hirundo> what's the status of the dutch train set? 09:43:14 <Rubidium> it's waiting for a leafs-on-rail railtype? 09:52:22 <planetmaker> but... leafs-on-railtype is possible already! ;-) 09:52:39 <planetmaker> Hm... which would be a nice idea actually :-) 09:52:56 <planetmaker> Look of tracks depending on the time of year. With some random component 09:53:26 <Rubidium> and lower TE when there's leafs on the rail :) 09:53:46 <planetmaker> hm... but that's something for the train, not for the tracks 09:56:55 <Ammler> make: *** No rule to make target `Makefile.dep', needed by `swedishrails.grf'. Stop. 09:57:26 <Ammler> Hirundo: djnekkind once worked on a version 2 09:57:34 <Ammler> somewhere on the devzone 10:03:47 <planetmaker> which rev, Ammler ? 10:05:19 <Hirundo> I ask, because I was pondering to use dutchset gfx for a nml vehicle example 10:06:37 <Ammler> planetmaker: tip 10:07:08 <Ammler> I made a "hg purge --all" 10:07:12 <planetmaker> hm 10:07:18 <Ammler> to be sure, it is a clean repo :-) 10:08:32 <planetmaker> tip is r162? 10:10:49 <Ammler> changeset: 162:582afebdb30a 10:11:37 <Ammler> same with a fresh clone 10:12:30 <Ammler> nml r775 10:16:33 <andythenorth> snow on tracks! 10:16:41 <andythenorth> and higher TE if a snowplow has passed by recently! 10:16:45 * andythenorth goes back to work 10:18:20 <planetmaker> andythenorth: snow on tracks is done 10:18:38 <planetmaker> you just don't use the 'proper' railtypes :-P 10:21:15 <andythenorth> and higher TE if snowplough? :P 10:21:21 <andythenorth> bad for performance... 10:21:27 <seberoth> hi 10:22:39 <planetmaker> andythenorth: that's for train newgrfs :-) 10:23:39 <Brot6> Swedish Rails - Revision 163:037284a6f5cb: Add: [Makefile] Depend again on the graphics files in ... (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/037284a6f5cb 10:25:04 <planetmaker> ah, thanks Ammler. 10:27:20 <planetmaker> try again 10:28:21 <Brot6> Swedish Rails - Revision 164:630bf067594e: Fix: Don't depend on files which may or may not be pre... (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/630bf067594e 10:31:35 <Ammler> cat: swedishrails.pnml: No such file or directory 10:31:54 <Ammler> changeset: 164:630bf067594e 10:32:38 <Ammler> after purge, I have again make: *** No rule to make target `Makefile.dep', needed by `swedishrails.grf'. Stop. 10:36:35 <planetmaker> hm 10:38:22 <planetmaker> gr. yes. another file 10:40:06 <planetmaker> better? 10:40:28 <Brot6> Swedish Rails - Revision 165:a32e644e7020: Fix: [Makefile] Don't depend on nfo-only files in NML ... (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/a32e644e7020 10:40:51 <planetmaker> thanks for testing :-) 10:41:24 <planetmaker> If so, I'll only have to transplant these dep changes somewhat to the other projects 10:41:34 <planetmaker> and implement a way to disable them for fb ;-) 10:47:27 <Ammler> fb? 10:47:31 <planetmaker> foobar 10:47:35 <Ammler> ah :-) 10:48:26 <Ammler> sorry my friend 10:48:35 <Ammler> still no success :-) 10:48:44 <Ammler> why does it work for you? 10:49:08 <Ammler> make: *** No rule to make target `normal.png', needed by `swedishrails.grf'. Stop. 10:50:26 <planetmaker> uh? Now I'm surprised... this worked for me :S 10:50:47 <planetmaker> hm... you worked with the the tar.gz or the hg repo? 10:52:15 <Ammler> hg pull -u 10:52:16 <planetmaker> hm. it works in both cases with a clean hg and tar 10:52:18 <Ammler> hg purge --all 10:52:21 <Ammler> make _V= 10:53:28 <Ammler> same with a fresh clone, so hg purge --all works 10:53:32 <planetmaker> hm. purge is a non-default extension 10:53:58 <Ammler> [extensions]\npurge= iirc 10:54:09 <planetmaker> you try with r165? 10:54:26 <Ammler> hgext.purge it is 10:54:52 <planetmaker> hm 10:54:55 <Ammler> yes 10:55:09 <Ammler> I made a fresh clone 10:55:11 <planetmaker> can you try something in the xterm? The regex is failing for you which I use, which works for me 10:56:53 <planetmaker> cat src/railtypes.pnml | grep -E -o "[a-zA-Z0-9\-_]*\.[pP][nNcC][gGxX]" 10:56:58 <planetmaker> what does it give? 10:57:09 <Brot6> swedishrails: update from r156 to r165 done - http://bundles.openttdcoop.org/swedishrails/nightlies/r165 10:57:16 <Ammler> a list of pngs 10:57:25 <planetmaker> and what do you need to get the complete filename, not the the truncated ones 10:57:40 <planetmaker> there's no normal.png 10:57:54 <planetmaker> there's depot_normal.png 10:57:58 <planetmaker> and other normal.pngs 10:58:03 <Ammler> http://pastebin.com/prcdCqCL 10:58:20 <planetmaker> that is very wrong :-( 10:58:31 <planetmaker> what if you escape the "_" in the regex? 10:58:41 <planetmaker> or what does help in your case to extract the names properly? 10:58:57 <planetmaker> it needs to not stop at "_" 10:58:58 <Ammler> just use -i for grep 10:59:10 <Ammler> and you don't have to bother about the case sensitivity 10:59:33 <planetmaker> that doesn't affect the "_" 10:59:40 <planetmaker> but good point 11:00:47 <Ammler> hmm, you don't define a starting point 11:01:02 <Ammler> for example the / 11:01:20 <planetmaker> I want the path ;-) 11:01:27 <planetmaker> actually. no 11:01:40 <planetmaker> but the full name 11:01:50 <planetmaker> which includes _-A-Za-z0-9 11:02:03 <planetmaker> and . 11:03:07 <Ammler> I moved the _ in front and now it works 11:03:22 <Ammler> [_a-z0-9\-]*\.[p][nc][gx] 11:04:07 <Ammler> excaping would work too 11:04:16 <Ammler> [a-z0-9\-\_]*\.[p][nc][gx] 11:05:06 <Ammler> are you sure, those are all allowed chars for filenames? 11:05:27 <Ammler> isn't space allowed? 11:05:47 <Ammler> or dot 11:05:59 <planetmaker> no spaces 11:06:03 <planetmaker> I don't support space filenames 11:06:25 <planetmaker> I might consider to support filenames with more than one dot. But currently not 11:06:56 <planetmaker> filenames with spaces are very evil with shell scripts 11:07:38 <Ammler> I know 11:07:41 <Ammler> :-) 11:08:49 <Ammler> will you commit the change? 11:09:31 <planetmaker> which? 11:09:41 <planetmaker> does the escaped _ work? 11:09:53 <planetmaker> oh... yes. missed that. 11:11:15 <planetmaker> done 11:11:25 <planetmaker> I escaped it 11:13:07 <Brot6> Swedish Rails - Revision 166:c3e86c59ff11: Fix: [Makefile] Also '_' needs an escape... (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/c3e86c59ff11 11:27:06 * planetmaker ponders to make a ser release. The parameters might be worth it 11:29:49 <Brot6> Swedish Rails - Feature #1495 (New): Sprites for menus (planetmaker) @ http://dev.openttdcoop.org/issues/1495 11:32:05 <Brot6> Swedish Rails - Feature #1496 (Assigned): Compatibility check (planetmaker) @ http://dev.openttdcoop.org/issues/1496 11:34:15 <Brot6> Swedish Rails - Bug #1497 (Confirmed): Alignment of tunnel sprites (planetmaker) @ http://dev.openttdcoop.org/issues/1497 11:55:33 <Brot6> Swedish Rails - Revision 167:222cf51491b0: Fix: Parameter description wasn't correct anymore (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/222cf51491b0 11:56:29 <Brot6> Swedish Rails - Revision 168:cb0805bb6d63: Change: Update changelog (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/cb0805bb6d63 11:58:08 *** michi_cc has quit IRC 11:58:08 *** OwenS has quit IRC 11:58:14 *** OwenS has joined #openttdcoop.devzone 11:59:09 *** michi_cc has joined #openttdcoop.devzone 12:00:20 <Ammler> planetmaker: ser works in ttdp now, btw. 12:00:42 <Ammler> your forums post lies 12:07:56 <planetmaker> yes. I'll fix that with the new release 12:08:23 <planetmaker> which cannot be off long, given the last commit ;) 12:11:57 <Ammler> I would like to close the ttdp ticket, since you asked me for the Makefile update, shall I commit that, I would also update the spec to build the version on the devzone too 12:13:18 <Brot6> OpenGFX - Revision 533:73086dd26e43: Fix #922 (r461): switch floppy sprites (Ammler) @ http://dev.openttdcoop.org/projects/opengfx/repository/revisions/73086dd26e43 12:13:53 <Ammler> http://dev.openttdcoop.org/attachments/916/bundle_ttdp.diff from #380 12:13:54 <Brot6> Ammler: http: #380 is http://dev.openttdcoop.org/issues/show/380 "OpenGFX - Feature #380: TTDPatch support - #openttdcoop Development Zone" 12:22:13 <planetmaker> I don't see why it needs a separate doc 12:22:47 <planetmaker> but ... might be better 12:26:54 <Ammler> sep doc? 12:27:08 <Ammler> ah, you mean the ttdpatch.txt? 12:27:38 <planetmaker> yes. but it's fine. 12:28:02 <Ammler> I think, we should split that, as the normal package isn't useable for it 12:28:11 <Ammler> well, we could add with preprocessing 12:30:59 <Ammler> planetmaker: can you ping haydn.openttdcoop.org, since it has the new card 12:31:20 <planetmaker> no, it's fine this way 12:31:45 <planetmaker> ping works 12:34:54 <planetmaker> no drop in 1500 pings 12:36:59 <Brot6> Swedish Rails - Revision 169:d7e7c6c806b8: Fix: One parameter was missing the the parameter GUI, ... (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/d7e7c6c806b8 12:39:53 <Brot6> swedishrails: update from 0.5.0 to 0.6.0 done - http://bundles.openttdcoop.org/swedishrails/releases/0.6.0 12:40:05 <Brot6> Swedish Rails - Revision 170:1d27693c9212: Added tag 0.6.0 for changeset d7e7c6c806b8 (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/1d27693c9212 12:40:05 <Brot6> Swedish Rails - Revision 171:2b7b3ebeeeda: Change: Update newgrf version to repository revision (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/2b7b3ebeeeda 12:40:05 <Brot6> Swedish Rails - Revision 172:2b7611dc62e2: Added tag 0.6.0 for changeset 2b7b3ebeeeda (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/2b7611dc62e2 12:40:07 <Brot6> Swedish Rails - Bug #1498 (New): DevZone compile failed (compiler) @ http://dev.openttdcoop.org/issues/1498 12:41:27 *** michi_cc has left #openttdcoop.devzone 12:41:29 *** michi_cc has joined #openttdcoop.devzone 12:46:10 <Ammler> hmm? 12:46:17 <Ammler> ah 12:51:36 <Ammler> 0.6.0 2010-09-15 swedishrails releases 2b7b3ebeeeda 12:51:44 <Ammler> but is it the right version? 12:52:01 <Ammler> have you explaination for the ticket? 12:54:09 <planetmaker> yes. I do somewhat... The version was broken. AND hg rollback didn't change back the .hgtags AND hg diff didn't show the change to hg tags 12:54:46 <Ammler> yes, you need to revert .hgtags 12:54:52 <planetmaker> yes, I forgot 12:55:00 <Ammler> hg rollback does just undo the commit 12:55:11 <planetmaker> with a tag it's somewhat stupid 12:55:17 <Ammler> :-) 12:55:39 <planetmaker> as a tag neds no commit 12:55:42 <Ammler> there is no nml in the log folder 12:55:50 <planetmaker> hm 12:56:14 <Ammler> maybe I didn't enable copy for releases :-/ 12:56:30 <planetmaker> dunno :-) 12:56:46 <planetmaker> Also NML projects need to set in future their version automatically. I did hack it manually this time 12:56:59 <planetmaker> also a missign feature ;-) 12:57:15 <Ammler> hmm, I use the custom_tags for it 12:57:38 <Ammler> ah 12:57:50 <Ammler> no, didn't use a14 yet 12:58:06 <Ammler> but why can't you set that with substitutes? 12:58:26 <Ammler> I mean, nml rarly can't assume, you are using hg 13:00:24 <planetmaker> the version is no string. So I'd need run that in the cpp phase, getting the value somewhat from custom_tags.txt 13:00:32 <planetmaker> It's just a thing which hasn't yet been done. 13:02:30 <Ammler> ah, I meant, it is a FR for nml 13:03:21 *** KenjiE20 has joined #openttdcoop.devzone 13:08:53 <planetmaker> FR? 13:09:11 <planetmaker> it's rather a FR for the makefile, I think 13:09:15 <planetmaker> though... maybe 13:09:51 <planetmaker> hm... :-( Menu and cursor sprites I can supply are base set dependent :S 13:11:16 <planetmaker> hm... so three cases there, too: no change, TTD and OpenGFX 13:14:07 <Ammler> 841 packets transmitted, 827 received, 1% packet loss, time 841060ms <-- how good is that? 13:14:23 <Ammler> rtt min/avg/max/mdev = 32.645/34.121/118.848/3.388 ms 13:19:03 <planetmaker> not good either, is it? 13:37:43 <Brot6> Swedish Rails - Swedish Rails 0.6.0 (planetmaker) @ http://dev.openttdcoop.org/news/46 16:20:30 <Brot6> nml: update from r774 to r775 done - http://bundles.openttdcoop.org/nml/nightlies/r775 16:21:10 <Brot6> swedishrails: update from r165 to r172 done - http://bundles.openttdcoop.org/swedishrails/nightlies/r172 16:21:17 <Brot6> Following repos didn't need a nightlies update: 2cctrainset (r613), 32bpp-extra (r39), airportsplus (r62), basecosts (r20), belarusiantowns (r7), comic-houses (r71), firs (r1364), fish (r390), frenchtowns (r4), grfcodec (r253), heqs (r372), metrotrackset (r56), newgrf_makefile (r190), nforenum (r502), nutracks (r115), ogfx-test (r529), ogfx-trees (r15), ogfxplus (r42), opengfx (r539), openmsx (r97), opensfx (r97), snowlinemod (r42), 16:21:18 <Brot6> swisstowns (r14), transrapidtrackset (r15), ttdviewer (r25), ttrs (r18), worldairlinersset (r663) 16:21:58 <Brot6> belarusiantowns: rebuild of r7 done (3 errors) (Diffsize: 22) (DiffDiffsize: 7) - http://bundles.openttdcoop.org/belarusiantowns/nightlies/r7/log 16:22:28 <Brot6> frenchtowns: rebuild of r4 done (4 errors) (Diffsize: 10) (DiffDiffsize: 7) - http://bundles.openttdcoop.org/frenchtowns/nightlies/r4/log 16:22:37 *** thgergo has joined #openttdcoop.devzone 16:23:05 <Brot6> ogfxplus: rebuild of r42 done (Diffsize: 7) (DiffDiffsize: 7) - http://bundles.openttdcoop.org/ogfxplus/nightlies/r42/log 16:23:32 <Brot6> swisstowns: rebuild of r14 done (4 errors) (Diffsize: 10) (DiffDiffsize: 7) - http://bundles.openttdcoop.org/swisstowns/nightlies/r14/log 16:23:39 <Brot6> Following repos rebuilds successful without any difference to earlier nightlies builds: airportsplus (Diffsize: 1) 16:32:37 <Ammler> nmlc: A GRF-block requires the 'name', 'desc', 'grfid', and 'version' properties to be set. <-- hehe 16:33:02 <Ammler> does that need to be a Error, could also be a warning? 16:33:18 <planetmaker> error is fine 16:33:46 <planetmaker> makes people aware of it. And in NML it's no big thing to just add it 16:34:16 <planetmaker> 10 characters if you want to be short ;-) 16:34:18 <Ammler> but it isn't required to work 16:34:27 <planetmaker> in NML it is :-P 16:34:32 <Ammler> :-) 16:34:55 <Ammler> then I might have a reason to test Alberths addons 16:34:56 <planetmaker> I think we all were for it as versioning the newgrf is a good thing 16:39:49 <Ammler> yes, warning would be fine, but exit is a bit too much, since it isn't required for the grf 16:40:18 <Ammler> or it should also error from openttd or nforenum/grfcodec 16:41:08 <planetmaker> I don't think it's too much 16:41:52 <Ammler> well, it's wrong 16:41:58 <planetmaker> the question is not what minimally is possible, but what helps in the overall context 16:41:58 <Ammler> or however you call it 16:42:16 <planetmaker> and the overall context is: versioned newgrfs are the way to go. So it may as well be forced 16:42:21 <planetmaker> it's not wrong 16:42:38 <planetmaker> it's forcing good behaviour on the authors 16:42:46 <Ammler> well, then openttd does it wrong 16:43:01 <planetmaker> I don't get you. What's wrong with it? 16:43:02 <Ammler> either missing version is a error or not 16:43:21 <planetmaker> missing version is a legacy thing 16:43:48 <Ammler> and why doesn't openttd error about it? 16:43:50 <planetmaker> like you may drive an old timer without seat belt, if it didn't have one. But a new car must have one 16:44:20 <planetmaker> it's no openttd error. But new newgrf should have it. So why allow new newgrfs without it? 16:45:01 <planetmaker> do you want to honestly tell me you want to support actively backward-oriented newgrf programming? 16:45:11 <Ammler> well, so nml grfs have it, nfo wont have, how does that help? 16:45:16 <planetmaker> Thus making things even harder for openttd to show the new versions only? 16:45:26 <planetmaker> Ammler: it helps in the long run 16:45:45 <planetmaker> Don't think 1 month only. Think 5 years 16:45:48 <planetmaker> or two 16:45:57 <Ammler> no, it should be a real error, speking openttd should also report it 16:45:59 <planetmaker> think about the dummy users 16:46:11 <Ammler> nml should not set its own "rules", imo 16:46:22 <planetmaker> they'll only be shown the newest newgrf. If the newgrf have no version, they'll have the option of all silly versions 16:46:41 <planetmaker> Of course having a missing version is no error nor will it be 16:46:48 <Ammler> why not? 16:47:59 <Ammler> again, error about it is fine, nml is just the wrong app to start with :-) 16:48:01 <planetmaker> what should the error be about and when should it be displayed? What sense would it serve? 16:48:18 <planetmaker> it would just disable all current newgrfs 16:48:20 <Ammler> what sense does nml serve with it? 16:48:36 <Ammler> no 14, no error 16:48:48 <planetmaker> producing higher-quality newgrf. Allowing Openttd to display only the newest to the default user 16:49:16 <planetmaker> reminding the author that he should version his newgrf 16:49:58 <planetmaker> I'm somewhat amazed that you either want to skip it in NML and on the other hand error out in OpenTTD on missing a14. Both makes not much sense 16:50:32 <Ammler> [18:47] <Ammler> again, error about it is fine, nml is just the wrong app to start with 16:50:50 <Ammler> it should be part of the grf spec 16:51:10 <planetmaker> yes. You remove all existing newgrfs from being valid. congratulations 16:51:23 <Ammler> [18:48] <Ammler> no 14, no error 16:51:32 <planetmaker> ? 16:51:51 <Ammler> if the grf doesn't have a14, it can't have a missing version :-) 16:52:06 <planetmaker> *sigh* 16:52:45 <planetmaker> Ammler: do you have the setting newgrf_show_old_versions activated? 16:52:51 <planetmaker> toggle it and test. 16:52:58 <planetmaker> see the gain of the setting 16:53:05 <planetmaker> then think again what you advocate 16:53:35 <Ammler> [18:47] <Ammler> again, error about it is fine, nml is just the wrong app to start with <-- you don't get hat? 16:53:38 <Ammler> that* 16:53:43 <planetmaker> no, I don't 16:53:52 <Ammler> I like to error about missing version 16:54:07 <Ammler> I just don't like it, that it only errors for nml 16:54:26 <planetmaker> I ask you again: how and where shall this error be? And what about existing newgrf? 16:54:40 <planetmaker> It is no newgrf error 16:54:47 <Ammler> I don't paste again :-P 16:55:06 <planetmaker> then please answer the question about where you want an error about a missing action14 16:55:13 <planetmaker> "not NML" is not constructive 16:55:32 <Ammler> grfspec - openttd 16:55:58 <planetmaker> that's also not an answer 16:56:18 <planetmaker> what is valid in grfspec7 cannot be made invalid in the same grfspec7 16:56:25 <planetmaker> it may be an option for grfspec8 16:56:53 <planetmaker> still, versions 0-7 won't have it. ever. as an error 16:57:04 <Ammler> so it possible to add a new action, but not to setup version in action as required? :-o 16:57:12 <planetmaker> of course 16:57:30 <planetmaker> it would make valid newgrfs invalid 16:57:41 <Ammler> why, they don't have a14 16:57:45 <planetmaker> you can only extend the specs, but not redefine a thing 16:58:34 <planetmaker> you need a new grfspec, if you want a mandatory grf version 16:58:41 <Ammler> well, nvm, we alreaey talked too much about a simple different oppinion :-) 16:58:49 <planetmaker> as in the current version it wasn't present in the beginning 17:00:56 *** frosch123 has joined #openttdcoop.devzone 17:51:04 <Brot6> Following repos didn't need a testing update: 32bpp-ez-patches (r20806), clientpatches (ERROR r20806), serverpatches (r20806) 17:59:13 <Ammler> no trunk commits? 17:59:58 <andythenorth> hi hi 18:01:18 <planetmaker> hi andythenorth 18:01:36 <Ammler> heya :-) 18:17:48 *** Alberth has joined #openttdcoop.devzone 18:29:53 <Brot6> Swedish Rails - Revision 173:b5035911b72b: Fix: [Makefile] Delete half finished grf, if NML fails... (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/b5035911b72b 18:30:22 <planetmaker> ^ Alberth should that not possibly be an NML task; I'm not sure. What is your opinion? 18:31:14 <Alberth> yep, either nothing, or everything would be my expectation 18:31:24 <planetmaker> :-) 18:31:40 <Alberth> comma not entirely at the right place :) 18:32:44 <planetmaker> in my commit message? It's a Germanism ;-) 18:32:53 <Brot6> NewGRF Meta Language - Bug #1499 (New): Delete half-finished grf upon failure (planetmaker) @ http://dev.openttdcoop.org/issues/1499 18:32:59 <Alberth> in my response :) 18:33:46 <planetmaker> :-) 18:40:45 <Alberth> hmm, reproducing that problem by 2 entries, where the 2nd fails, should be possible.. 18:41:39 <planetmaker> also, the error report is off by one (too little). where's a proper place to fix that? 18:41:49 <planetmaker> the line number reported I mean 18:42:00 <planetmaker> main.py? 18:42:15 <planetmaker> main.py:196? 18:42:17 <Alberth> parser.py or tokens.py 18:42:37 * Alberth looks what line 196 says 18:43:17 <planetmaker> I suspect it's an issue of 0-based vs. 1-based counting 18:44:16 <Alberth> for some errors (typically syntax errors), getting the line below the error reported is quite normal 18:45:27 <Alberth> I have quite good confidence that python reports the right line where it detected the error :) 18:45:31 <planetmaker> hm. It is complaining about a wrong identifier. Which is used in the line below 18:46:17 <planetmaker> nmlc: "src/railtypes.pnml", line 482: Unknown sprite set: gui_normal_opengfx_oldd <-- it's line 483 18:46:22 <Alberth> any way to reprocude the problem? 18:46:42 <planetmaker> rename an identifier like that on an existing project you have 18:46:57 <Alberth> that is not line 196 18:47:11 <planetmaker> :-) 18:47:22 <Alberth> 196 reports errors in the nml python code 18:47:54 <planetmaker> ok :-) That's why I'm asking 18:54:05 <planetmaker> hm... it's like poking in the dark... :S 18:55:26 <Alberth> start with printing the token stream, line numbers there should be correct 18:55:49 <planetmaker> hm... debugging option, I guess :-) 18:57:31 <planetmaker> hm... no. nmlc -d won't help me see line numbers of anything 18:58:07 <planetmaker> -s :-) 18:59:37 <Alberth> I think I created that, but no idea any more what it does :) 19:02:02 <planetmaker> http://pastebin.com/8zhW2nnF <-- that 19:02:12 <planetmaker> so... hm... adding a +1 there? 19:02:30 <planetmaker> doesn't sound fine either 19:05:03 <Alberth> ah, I remember now what -s does :) 19:05:15 <Alberth> no it does not sound like the right solution 19:07:33 <Alberth> http://www.dabeaz.com/ply/ply.html#ply_nn13 <-- explains how to get the tokens printed, although nml wrapped that a bit 19:07:36 <Webster> Title: PLY (Python Lex-Yacc) (at www.dabeaz.com) 19:08:28 <Alberth> you want to know where the first time the line becomes wrong 19:12:04 <andythenorth> someone tell me a colour for brickworks 19:12:14 <andythenorth> metal bits 19:12:35 <andythenorth> ach, company colour then 19:14:47 <Alberth> it would be good to have a function in tokens.py that outputs that token stream imho 19:16:16 <planetmaker> that sounds all a bit beyond my ability. Especially tonight 19:23:33 <Brot6> NewGRF Meta Language - Bug #1500 (New): off-by one with wrong sprite set (planetmaker) @ http://dev.openttdcoop.org/issues/1500 19:38:52 <Alberth> nmlc: Referencing unknown action2 id: ground__switch_overlay <-- I don't even get a line number :p 19:39:45 <planetmaker> that's bad :-) 19:44:36 <Alberth> and of course some swedish rails author added automagic deletion of the half-finished .grf, so I cannot reproduce that either :p 19:46:30 <Brot6> Swedish Rails - Revision 174:148b464723cb: Feature: Allow to select OpenGFX style GUI replacements (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/148b464723cb 19:46:47 <planetmaker> :-P 19:47:05 <planetmaker> try the revision before that ;-) 19:49:22 <Alberth> hmm, not trivial to report a position there 19:54:01 <Brot6> NewGRF Meta Language - Bug #1501 (New): Action2 does not report source file position (Alberth) @ http://dev.openttdcoop.org/issues/1501 19:54:27 <Alberth> planetmaker: this trick works nicely ^^ :) 19:54:53 <planetmaker> the wrong ID thing? Or 'outsourcing' via ticket? :-P 19:55:25 <Alberth> the 'outsourcing' :) 19:58:57 <planetmaker> :-P 20:00:34 <Alberth> http://pastebin.ca/1941577 <-- not having much luck, everything I try seems to fail tonight 20:01:15 <planetmaker> lol? 20:01:28 <planetmaker> That looks kinda worse than the last two nml bugs 20:01:47 <Alberth> ply does not do duck-typing correctly 20:02:04 <planetmaker> "duck-typing"? 20:02:27 <Hirundo> If it looks like a duck and it quacks like a duck, it probably is a duck 20:02:34 <Alberth> it looks like a duck, it quacks like a duck, therefore, it is a duck 20:03:12 <Alberth> it looks like a position, it is supposed to be a position, therefore it must be a position :) 20:03:28 <Hirundo> i.e. the type of an object doesn't really matter, as long as it has the required properties/methods 20:04:25 <ODM> it could be a small goose 20:07:21 <andythenorth> it could be a mongoose 20:07:26 <andythenorth> as long as it quacks 20:08:27 *** KenjiE20 has quit IRC 20:09:18 <ODM> what if it has a cold and you mistake his mating call for a quack? 20:12:27 <andythenorth> if he implements quack, treat him like a duck 20:12:53 <andythenorth> try 'fly' and see if he implements that :P 20:13:23 <planetmaker> ha, nice explanation :-) 20:15:21 <andythenorth> http://griddlenoise.blogspot.com/2005/12/zope-component-architecture-interfaces.html 20:15:22 <Webster> Title: Griddle Noise: The Zope Component Architecture - Interfaces, Adaptation, and Duck Typing (at griddlenoise.blogspot.com) 20:15:27 <ODM> yeah but hes not implementing, hes mistaking!:P 20:15:39 <andythenorth> might be Zope specific, not sure if adapters are standard across python :P 20:20:32 <Brot6> FIRS Industry Replacement Set - Revision 1365:8572123442a1: Change: improved graphics for Brick W... (andythenorth) @ http://dev.openttdcoop.org/projects/firs/repository/revisions/8572123442a1 20:25:35 <Hirundo> planetmaker: Have you already done something wrt #1421 ? 20:25:35 <Brot6> Hirundo: planetmaker: #1421 is http://dev.openttdcoop.org/issues/show/1421 "NewGRF Meta Language - Feature Request #1421: Example vehicle project - #openttdcoop Development Zone" 20:30:03 <Brot6> NewGRF Meta Language - Bug #1500: off-by one with wrong sprite set (Alberth) @ http://dev.openttdcoop.org/issues/1500#change-3941 20:46:56 <Alberth> good night 20:47:37 <planetmaker> g'night Alberth 20:47:46 <planetmaker> Hirundo: I started somewhat, but not too far 20:47:56 *** Alberth has left #openttdcoop.devzone 20:52:05 <Hirundo> I'll not dive into it, then 20:52:25 <Hirundo> "fix bugs before writing new code" 20:55:04 *** KenjiE20 has joined #openttdcoop.devzone 21:03:39 *** frosch123 has quit IRC 21:16:27 <planetmaker> Hirundo: I can send you what I have. Or add it to a ticket. I'm not sure I'll really have much time... 21:18:35 <planetmaker> I added it 21:20:09 <Brot6> NewGRF Meta Language - Feature Request #1421: Example vehicle project (planetmaker) @ http://dev.openttdcoop.org/issues/1421#change-3942 21:27:29 <Brot6> NewGRF Meta Language - Bug #1499: Delete half-finished grf upon failure (yexo) @ http://dev.openttdcoop.org/issues/1499#change-3943 21:31:53 <Brot6> Swedish Rails - Bug #1498 (Closed): DevZone compile failed (compiler) @ http://dev.openttdcoop.org/issues/1498 21:31:53 <Brot6> Swedish Rails - Bug #1498 (Closed): DevZone compile failed (planetmaker) @ http://dev.openttdcoop.org/issues/1498#change-3944 21:31:53 <Brot6> Swedish Rails - Revision 175:adb0f39fe750: Add #1495: Gui sprites in TTD style (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/adb0f39fe750 21:38:12 <Brot6> Swedish Rails - Revision 176:5c7a5bd7aaaa: Doc #1495: Amend readme for usage concerning new param... (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/5c7a5bd7aaaa 21:39:11 <Brot6> NewGRF Meta Language - Bug #1501 (Closed): Action2 does not report source file position (Alberth) @ http://dev.openttdcoop.org/issues/1501 21:39:11 <Brot6> NewGRF Meta Language - Revision 776:89533bcb8ac5: Fix #1501: print file/line information for unkn... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/89533bcb8ac5 21:39:11 <Brot6> NewGRF Meta Language - Bug #1501 (Closed): Action2 does not report source file position (yexo) @ http://dev.openttdcoop.org/issues/1501#change-3945 22:00:29 *** Westie has quit IRC 22:03:06 <Brot6> NewGRF Meta Language - Bug #1500 (Closed): off-by one with wrong sprite set (planetmaker) @ http://dev.openttdcoop.org/issues/1500 22:03:06 <Brot6> NewGRF Meta Language - Revision 777:f792a8ba27ec: Fix #1500: special #line directive lines didn't... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/f792a8ba27ec 22:03:06 <Brot6> NewGRF Meta Language - Bug #1500 (Closed): off-by one with wrong sprite set (yexo) @ http://dev.openttdcoop.org/issues/1500#change-3946 22:18:10 *** ODM has quit IRC 23:47:04 *** KenjiE20 has quit IRC