Config
Log for #openttdcoop.devzone on 15th September 2010:
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

Powered by YARRSTE version: svn-trunk