Config
Log for #openttdcoop.devzone on 26th June 2010:
Times are UTC Toggle Colours
06:59:09  <Brot6> #openttdcoop - Support #144: Tag will automatically create a release bundle (planetmaker) @ http://dev.openttdcoop.org/issues/144#change-2721
07:56:02  *** Webster has joined #openttdcoop.devzone
07:59:31  <Rubidium> attack of the typo!
07:59:44  <Rubidium> "parameter 1 wet to 3"
08:02:11  <planetmaker> damn
08:07:58  <Brot6> Swedish Rails - Revision 87:00b8f381ee31: Fix: Typo in the docs. Parameters really wet themselves (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/00b8f381ee31
08:30:32  <Brot6> OpenGFX - Feature #922: menu bar: save/load and statistics (Ammler) @ http://dev.openttdcoop.org/issues/922#change-2722
08:31:25  *** Alberth has joined #openttdcoop.devzone
08:34:36  *** Seberoth has joined #openttdcoop.devzone
08:36:02  <Ammler> planetmaker: also the 2nd parameter is a bit confusing, why is 0 year 1975?
08:36:15  <planetmaker> default
08:36:28  <planetmaker> so if one doesn't want to change it, use 0
08:36:30  <Ammler> you can set default to something else then 0
08:36:45  <planetmaker> yes. default usage is 1975
08:36:54  <planetmaker> but if the parameter is 0, 1975 is assumed
08:37:25  <Ammler> so if I like to use 1980, I need to set 5 as parameter?
08:37:32  <planetmaker> no
08:37:33  <planetmaker> 1980
08:37:36  <Ammler> I would just use the year
08:37:54  <planetmaker> but if you want to set parameter 4, you don't have to worry about parameter 3
08:39:19  <planetmaker> so... obviously the documentation is not clear :-)
08:39:29  <planetmaker> It just really is only the year which needs to be given
08:40:00  <planetmaker> the only exception is that "0" is interpreted as 1975
08:40:14  <planetmaker> or as 5000000
08:40:25  <planetmaker> depending upon OpenTTD version
08:42:15  <Ammler> I would change default to 1975
08:42:43  <Ammler> and just convert 0 to 1975 in the code but not documented, at least not that verbose
08:44:13  <planetmaker> hm, ok
08:48:56  <Ammler> http://pastebin.com/UYS5Daxj
08:52:20  <planetmaker> hm
08:52:32  <Ammler> I have a file called swedishrails.grf.nfo
08:53:41  <Ammler> oh, I need to update nml
08:53:57  <planetmaker> hm. but still... the source is not as it should be
08:54:01  <planetmaker> the source bundle
08:57:16  <Ammler> from nml or swr?
08:58:00  <planetmaker> I don't ship the lang dir
08:58:04  <planetmaker> as such building fails
08:58:57  <Ammler> oh :-)
08:59:53  <planetmaker> but otherwise everything builds fine
09:00:29  <Ammler> yes, it does, after rpm -Uhv http://bundles.openttdcoop.org/nml/nightlies/r499/nml-r499-suse1120.noarch.rpm
09:13:31  *** frosch123 has joined #openttdcoop.devzone
09:19:32  <planetmaker> Ammler: what would happen, if I re-tag 0.4.0, so that bundle_src gets built correctly?
09:19:38  <planetmaker> would the CF build it anew?
09:21:10  <Ammler> could you try :-)
09:21:18  <planetmaker> :-D
09:21:41  <Ammler> it does compare hashes to retrigger
09:22:42  <Ammler> but actually I didn't test that case
09:23:45  <planetmaker> ok, I will
09:28:31  <Brot6> Swedish Rails - Revision 88:1abea6861e27: Doc: Clarify documentation of depot introduction year p... (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/1abea6861e27
09:28:31  <Brot6> Swedish Rails - Revision 89:10cb3737dfe7: Fix: [Makefile] Also ship the language dir (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/10cb3737dfe7
09:30:49  <planetmaker> let's see
09:31:36  <Brot6> swedishrails: update from 0.4.0 to 0.4.0 done (2 errors) - http://bundles.openttdcoop.org/swedishrails/releases/0.4.0
09:32:12  <planetmaker> seems to work :-)
09:32:40  <planetmaker> nice
09:32:44  <Brot6> Swedish Rails - Revision 90:3482f4e160d9: Added tag 0.4.0 for changeset 10cb3737dfe7 (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/3482f4e160d9
09:33:38  <Ammler> the reason I compare hashes was for failed bundles
09:33:45  <planetmaker> :-)
09:33:50  <Ammler> nice, it works that way too :-)
09:34:40  <Ammler> but it should be said, that such update shouldn't be done after annoucement
09:36:10  <planetmaker> well, yes.
09:36:20  <planetmaker> The grf didn't change
09:36:35  <Ammler> ah ok
09:36:42  <Ammler> same md5sum there?
09:36:44  <planetmaker> yes
09:36:51  <planetmaker> otherwise I would not have gone for it
09:38:10  <planetmaker> it's really only a slight change in the makefile which allows to ship the lang dir, too when using bundle_src
09:38:29  <Brot6> Example NewGRF Project - Revision 102:e3bae3dbc985: Fix: [NML] Also ship the lang dir within sour... (planetmaker) @ http://dev.openttdcoop.org/projects/newgrf-makefile/repository/revisions/e3bae3dbc985
09:38:42  <Ammler> well, we know it works, we just shouldn't make big advertisement with it ;-)
09:38:49  <planetmaker> definitely not
09:39:04  <planetmaker> "kids don't do this at home" ;-)
10:12:41  *** KenjiE20 has joined #openttdcoop.devzone
10:40:42  <Brot6> NewGRF Meta Language - Feature #967: Random action2 (Hirundo) @ http://dev.openttdcoop.org/issues/967#change-2723
10:48:13  <Ammler> hmm, hook broken again
10:50:51  <planetmaker> the hook works for everyone?
11:11:28  <Ammler> it should :-(
11:12:50  <Ammler> this hook is installed: http://dev.openttdcoop.org/projects/home/repository/entry/mercurial/hooks/check_commit.py
11:22:27  <Alberth> ./devzone/hooks/repo_checks.ini   <--- shouldn't that be    .devzone/hooks/repo_checks.ini    ?
11:23:49  <Alberth> although that file may not exist atm
11:28:14  <Ammler> hm, true
11:28:50  <Ammler> yes, but I also included the global file
11:36:59  <Alberth> yes, those rules should be used
11:42:47  <Brot6> #openttdcoop - Revision 74:0e961d591fd3: [HG] Fix: wrong custom repo path (Ammler) @ http://dev.openttdcoop.org/projects/home/repository/revisions/0e961d591fd3
12:42:04  *** Seberoth has quit IRC
12:46:08  *** Seberoth has joined #openttdcoop.devzone
13:54:50  *** Seberoth has quit IRC
14:12:45  <Brot6> NewGRF Meta Language - Revision 503:53dfd9c7c424: Feature [#967]: Dependent or independent random... (Hirundo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/53dfd9c7c424
14:25:58  *** welshdragon has quit IRC
14:53:26  *** Seberoth has joined #openttdcoop.devzone
14:57:40  *** welshdragon has joined #openttdcoop.devzone
16:18:32  <Brot6> newgrf_makefile: compile of r102 failed - http://bundles.openttdcoop.org/newgrf_makefile/nightlies/ERROR/r102
16:19:12  <Brot6> nml: update from r499 to r503 done - http://bundles.openttdcoop.org/nml/nightlies/r503
16:19:45  <Brot6> swedishrails: update from r84 to r90 done - http://bundles.openttdcoop.org/swedishrails/nightlies/r90
16:19:47  <Brot6> Following repos didn't need a nightlies update: 2cctrainset (r557), 32bpp-extra (r36), airportsplus (r50), bros (r12), comic-houses (r70), firs (r1010), fish (r375), heqs (r320), nmts (r16), nutracks (r69), ogfxplus (r32), opengfx (r461), openmsx (r57), opensfx (r94), snowlinemod (r12), worldairlinersset (r643)
16:46:12  <Ammler> planetmaker: ogfxplus does also fail on ttdpatch, but only with an error message on the status screen
16:46:22  *** KenjiE20 has quit IRC
16:51:55  <Ammler> just tested NML regression grfs, only Action8 succeeded
16:52:47  <Ammler> hmm, might be because of the same ID?
16:54:12  *** KenjiE20 has joined #openttdcoop.devzone
17:09:43  *** Webster has joined #openttdcoop.devzone
17:22:41  <Hirundo> Ammler: regression works for me
17:23:08  <Ammler> Hirundo: tested the grfs with ttdpatch
17:23:23  <Ammler> but those might fail because of missing Action8
17:23:39  <Hirundo> vehicle (006) has action8 as well IIRC
17:25:01  <Ammler> the grfs should have unique IDs :-)
17:25:21  <Ammler> something like nmlX
17:33:59  *** DJNekkid has quit IRC
17:36:23  <Brot6> NewGRF Meta Language - Bug #1043 (New): NML fails on TTDPatch (Ammler) @ http://dev.openttdcoop.org/issues/1043
17:39:08  *** Seberoth has quit IRC
18:27:29  <Hirundo> Ammler: Does TTDP have any way to get more information about the error?
18:28:23  <Ammler> you mean more than that screen I posted?
18:28:34  <Hirundo> yes
18:29:39  <Ammler> I don't know, I just liked to test swr, but as that crashed ttdp
18:29:52  <Hirundo> what does 'code 4/7' mean?
18:29:57  *** Seberoth has joined #openttdcoop.devzone
18:30:19  <Ammler> so I only found one nml grf, which worked with patch :-)
18:33:33  <Ammler> Hirundo: I am/was ttdpatch player as I am openttd player, even less
18:33:47  <Ammler> not really experienced with any internals
18:34:42  <Ammler> but swedishrails is a good example, nobody plays ttdpatch anymore
18:35:13  <Ammler> since openttd supports around 99% of all features there :-)
18:36:44  <Ammler> I just liked to notify you about it, so you are aware, if someone complains about :-P
19:42:50  <frosch123> http://wiki.ttdpatch.net/tiki-index.php?page=SpriteErrors <- the error means, sprite #16 refers to an invalid vehicle/text/town part ID
20:11:29  <Brot6> NewGRF Meta Language - Bug #1043: NML fails on TTDPatch (frosch) @ http://dev.openttdcoop.org/issues/1043#change-2724
20:13:46  <Ammler> frosch123: and the other grfs fail because those don't have Action8?
20:14:26  <Ammler> well, still have no idea, why swr crashes patch
20:15:54  <frosch123> no idea about the others :) i only looked at 006_vehicle. but without action8 it will very likely not work
20:20:54  <Alberth> those are unit tests for commands, and not intended for actually loading into a game
20:22:28  <Alberth> for example 002_sounds, defines incorrect data as 'sound', and gives no option to actually play the sound in the game. Ie pretty useless in-game.
20:23:38  <Alberth> perhaps 006_vehicle is somewhat better, I don't know
20:25:05  <frosch123> swedish rails uses undefined action2 ids in the action3 default cases
20:26:12  <frosch123> if railtypes get callbacks that will likely cause trouble, maybe even OTTD crashes :)
20:26:58  <Ammler> well, it shouldn't use railtypes for ttdpatch
20:27:21  <Ammler> planetmaker: made some effort to make it ttdpatch compatible, that is why I tested it there
20:27:52  <Rubidium> that colon shouldn't be there... now it means something completely different
20:28:50  * frosch123 does not know whether the missing default case is that a bug of swedishrails, or whether nml should rather enforce a "callback failed"-result for the default case
20:38:10  <frosch123> Ammler: but it only skips the action3 for ttdp, neither action1 nor action2, ... so who knows :)
20:39:25  <Ammler> hmm, seems like it is simple hazard (or however), that all nml projects failed
20:39:53  <Ammler> ogfx+ might also fail because of high id using
20:41:15  <Alberth> good night all
20:41:22  <frosch123> night albert
20:41:43  *** Alberth has left #openttdcoop.devzone
21:28:12  *** ODM has quit IRC
21:51:12  <planetmaker> [23:43]	<frosch123>	[22:25:05] swedish rails uses undefined action2 ids in the action3 default cases <-- hm, it does?
21:51:27  <planetmaker> afaik all my switch statements have valid targets, also as default...
21:51:33  <planetmaker> oh, and good evening everyone
21:51:43  <Rubidium> in that case: blame shinkansen!
21:52:03  <frosch123> planetmaker: not the switch varact2, but the action3
21:52:45  <frosch123> graphics { <- that lists only TRACKOVERLAY ... FENCES, but no default
21:54:25  <planetmaker> uh? they all do, don't they?
21:54:37  <planetmaker> oh... the graphics
21:54:46  <planetmaker> hm...
22:02:48  *** dan123 has joined #openttdcoop.devzone
22:03:56  <dan123> hello all, I've made some fixes to 2cctrainset (wagon weights and 1gen flatbed availability):
22:03:57  <dan123> http://www.tt-forums.net/viewtopic.php?f=26&t=43838&p=886071#p886071
22:03:59  <Webster> Title: Transport Tycoon Forums • View topic - [OTTD]2cc TrainSetThe Trains of the world in2cc.Now:2.0Beta2 (at www.tt-forums.net)
22:04:41  <dan123> can somebody try it in free time? :)
22:15:00  <Ammler> dan123: hello :-)
22:15:07  <Ammler> don't you use the hg repo?
22:15:22  <dan123> I did download source from there
22:15:54  <dan123> but I don't know how to do anything else with this system :)
22:16:12  <Ammler> how did you download the source?
22:16:17  <Ammler> with hg clone?
22:16:31  <dan123> yes
22:16:51  <Ammler> just it would be easier to review a patch
22:17:03  <Ammler> hg diff > mypatch.diff
22:17:15  <dan123> I'll try it
22:17:35  <Ammler> instead of the zip with the modified files
22:19:55  <dan123> Ammler: I've updated my post :)
22:20:03  <dan123> thanks for help :)
22:20:27  <Ammler> well, the quy which can comment you best is currently on vacation
22:20:51  <dan123> yea, I've read whole topic
22:22:03  <Ammler> you added a new file?
22:22:55  <dan123> I've replaced zip with diff
22:23:08  <dan123> should I also add zip back?
22:23:20  <dan123> I'm kinda new in pathing business :)
22:23:35  <dan123> in fact that's my first patch I've ever made
22:24:47  <Ammler> no
22:25:00  <Ammler> it is just because you had 3 files in the zip, afaik
22:25:17  <Ammler> but the diff shows only changes in 2
22:25:41  <frosch123> count again :)
22:25:59  <Ammler> oh true
22:26:15  <dan123> don't scare me
22:26:16  <dan123> :D
22:26:24  <Ammler> stupid me using Kompare
22:27:44  <Ammler> dan123: maybe you can also register at the devzone so we can add you as contributor
22:28:04  <Ammler> but that is up to djn
22:29:17  <dan123> I'll register, I do like openttd and like to mess around with code so maybe I'll contribute some more
22:30:01  <dan123> mail there has to be public?
22:30:15  <Ammler> no, you can hide that in the settings
22:31:12  <Ammler> the email is needed for notifications
22:31:20  <Ammler> you can also customize that :-)
22:42:58  *** andythenorth has left #openttdcoop.devzone
22:55:06  <planetmaker> dan123: what exactly did you change in a0.txt? It's so very long lines and the change(s) hard to find... only the weight?
22:55:08  *** frosch123 has quit IRC
22:55:20  <dan123> planetmaker: yes
22:55:51  <planetmaker> hm... DJN somehow copy & pastes these lines from his magic excel file
22:55:59  <planetmaker> (which is also part of the repo)
22:55:59  <dan123> i figured a0.txt is just a file with all stats in there
22:56:04  <planetmaker> yes, it is
22:56:17  <planetmaker> it's short for action0 - which is the action which defines the stats
22:56:55  <dan123> hmm, I should also make those changes in this excel file then
22:58:13  <planetmaker> -//Flatbed-short1 <-- you remove that comment at the start of the file short1_flatbed
22:59:10  <planetmaker> dan123: if you can make the changes to the excel file, too, that'd be great
22:59:27  <planetmaker> otherwise I fear that it'll get overwritten with the next update
22:59:52  <planetmaker> mind that not every stat is just typed in there but some are 'just' calculated.
23:00:24  <dan123> in flatbed file i just changed all "LONG" into "SHORT"
23:00:27  <planetmaker> But if you can provide an update to that file, too... I see no reason to not commit it
23:00:50  <planetmaker> dan123: yes, and removed the comment and replaced it by a blank line :-)
23:02:25  <dan123> I figured that it shouldn't be there, since there's no such line in files associated with other wagons
23:02:45  <planetmaker> hm, that might be
23:06:15  <dan123> excel file doesn't need updates, the right weights are right there, I'll adjust mine to those and make another version of my patch
23:07:47  <planetmaker> ah, good
23:12:49  <planetmaker> dan123: I'll look at it possibly tomorrow. I'm looking forward to it. But now bedtime
23:12:53  <planetmaker> Good night everyone
23:13:06  <dan123> goodnight
23:14:35  <PeterT> gn8 sir
23:39:24  *** Seberoth has quit IRC
23:46:43  *** KenjiE20 has quit IRC

Powered by YARRSTE version: svn-trunk