Times are UTC Toggle Colours
00:38:24 *** OwenS has quit IRC 06:38:45 *** ODM has joined #openttdcoop.devzone 07:48:50 *** Yexo has joined #openttdcoop.devzone 07:53:05 <Webster> Latest update from devactivity: NFO Meta Language - Revision 192: Change: make -o autodetect the output type (nfo/grf) from the f... <http://dev.openttdcoop.org/projects/nml/repository/revisions/192> || NFO Meta Language - Patch #958 (Closed): Add -o option <http://dev.openttdcoop.org/issues/958#change-2549> 08:40:05 *** Seberoth has joined #openttdcoop.devzone 08:43:46 *** Alberth has joined #openttdcoop.devzone 08:51:07 <Alberth> Hirundo: with revision be698ff9e914 it is now allowed to have an empty expression before the first comma 08:51:51 <Alberth> moin all 08:58:07 <Webster> Latest update from devactivity: NFO Meta Language - Revision 193: Change: Use a set of commutative_operators. <http://dev.openttdcoop.org/projects/nml/repository/revisions/193> 09:00:19 <planetmaker> moin Alberth 09:00:28 <planetmaker> seems that the "moin" seems to spread :-) 09:02:05 <Alberth> form a committee against the spread of the informal 'moin' at IRC channels. Make a web site, ask the government for some funding, start an advertising campaign, go for president. 09:02:17 <Alberth> Hmm, that last bit may be a bit overkill :) 09:03:24 <Alberth> hg revision numbers are quite ugly 09:05:38 <Yexo> just use the r* number from the server, those shouldn't change 09:10:51 <Ammler> http.//new.dev.openttdcoop.org will support both 09:10:55 <Ammler> :* 09:11:02 <Ammler> and yeah, moin 09:12:02 <Ammler> http://new.dev.openttdcoop.org/projects/nml/repository/revisions/be698ff9e914 09:14:18 <planetmaker> Alberth: I certainly won't form a committee. I'd even go as far as to claim to have started it ;-) 09:16:40 <Alberth> Forming a counter-committee against reduction of the use of moin at irc channels is also allowed 09:16:51 <planetmaker> :-P 09:17:22 <Alberth> but you first have to wait until someone has formed the organization you are against :) 09:17:50 <Rubidium> itteh bitteh complaining committeh? 09:18:35 <Alberth> more like adding enough negations so nobody understands what your point of view is 09:22:54 *** frosch123 has joined #openttdcoop.devzone 09:25:13 <Alberth> good morning 09:25:16 <Yexo> the anti-counter-committee that is not against reduction of the use of moin :p 09:27:37 <Alberth> hmm, lexer line count is wrong 09:27:44 <Yexo> yes 09:27:50 <Yexo> I suspect it's partly due to comments 09:28:45 <Alberth> it is 09:29:01 <Alberth> let me find an anti-dote :) 09:29:19 <Yexo> make t_ignore_COMMENT a function and increase the linecount like in t_newline 09:31:09 <Alberth> I was thinking it might be an example in the ply docs, but it isn't 09:32:09 <Yexo> the examples didn't support multi-line comments 09:35:12 <Alberth> fixed 09:44:44 <Webster> Latest update from devactivity: NFO Meta Language - Revision 194: Fix: Also count new lines in comment. <http://dev.openttdcoop.org/projects/nml/repository/revisions/194> 10:01:51 *** OwenS has joined #openttdcoop.devzone 10:40:08 *** KenjiE20 has joined #openttdcoop.devzone 10:52:55 *** welshdragon has left #openttdcoop.devzone 11:25:51 <Hirundo> Alberth: bug fixed 11:36:35 <Webster> Latest update from devactivity: NFO Meta Language - Revision 195: Fix: The first parameter in a parameter list could be empty. <http://dev.openttdcoop.org/projects/nml/repository/revisions/195> 12:16:35 *** Doorslammer has joined #openttdcoop.devzone 12:52:39 *** Seberoth has quit IRC 13:26:30 *** Yexo has quit IRC 13:31:44 *** Yexo has joined #openttdcoop.devzone 13:43:49 *** OwenS is now known as Guest1307 13:46:04 *** Guest1307 has quit IRC 13:46:10 *** OwenS has joined #openttdcoop.devzone 13:58:54 <Alberth> \o/ http://paste.openttd.org/225837 14:04:19 <Brot6> test: abort: push creates new remote heads! 14:17:42 <Alberth> Yexo: a review would be useful imho: http://devs.openttd.org/~alberth/diffs/add_actionf.patch 14:17:57 <Alberth> or should I add it to a tracker? 14:28:11 <Alberth> http://devs.openttd.org/~alberth/diffs/actionF.py missed the new file 14:28:17 <Rubidium> yes! (oh, how such a useless answer) 14:31:27 <Alberth> not all, I learned that at least one of both options is the right action. 14:31:42 <Yexo> Alberth: instead of the "dutch: ...; any: ..." just use string(STR_NAMES) so it can be translated via the language files 14:32:16 <Alberth> nope, I need all the translations, not just one 14:33:10 <Alberth> at least, that's what I learned from the discussion yesterday with RB and PM 14:33:25 <planetmaker> Alberth: but the string also returns all languages 14:33:28 <Yexo> that is no problem, all translations can be accessed that way 14:33:46 <Yexo> town_names(0, 99) } <- what does that line do exactly? 14:33:57 <Alberth> ok, have to find out how that works then 14:34:37 <Alberth> it inserts a town name generated from town_name { definition : 0 ; .... } with 99/100 probablity 14:35:32 <Yexo> action4.py also needs all translations 14:35:55 <planetmaker> the problem is that it's not an action4 stringID 14:36:04 <Yexo> that doesn't matter at all 14:36:20 <Yexo> Alberth: is the id in any way relevant? 14:37:00 <Webster> Latest update from devactivity: NFO Meta Language - Revision 196: Change: Actions now also use new style classes. <http://dev.openttdcoop.org/projects/nml/repository/revisions/196> 14:37:03 <Alberth> the value itself not really 14:37:35 <Alberth> but you need a way to refer to an earlier town_names 14:37:38 <Yexo> if not, can they be autogenerated? (for example by using "town_names(first_part) { .... ...} town_names(final) { { text("capital", 1), town_names(first_part, 99) } }" ? 14:37:43 <Yexo> ie use ids instead of numbers? 14:38:17 <Alberth> how do you collect those ids? 14:38:33 <Alberth> is there a callback or so for it? 14:38:40 <Yexo> nope 14:39:00 <Yexo> just increase a counter and assign that id 14:39:13 <Alberth> ok 14:39:13 <Yexo> if the ids can be reused it's slightly more tricky 14:39:58 <Yexo> I'm off for now, will be back later tonight 14:40:01 <Alberth> the spec is not clear about that, but I doubt you will ever reach 0x7f entries 14:40:14 <Yexo> true, keep it simple then 14:40:14 <Alberth> ok, thanks for the review. 14:40:15 <Alberth> bye 14:40:18 *** Yexo has quit IRC 14:52:02 <Webster> Latest update from devactivity: NFO Meta Language - Revision 197: Change: Use a tuple of classes instead of several instance(). <http://dev.openttdcoop.org/projects/nml/repository/revisions/197> 15:17:09 <Brot6> airportsplus: update from r48 to r49 done (0 errors) - http://bundles.openttdcoop.org/airportsplus/nightlies/ 15:17:16 <Brot6> nutracks: update from r60 to r61 done (1 errors) - http://bundles.openttdcoop.org/nutracks/nightlies/ 15:17:21 <Brot6> snowlinemod: update from r10 to r11 done (0 errors) - http://bundles.openttdcoop.org/snowlinemod/nightlies/ 15:17:21 <Brot6> Following repos didn't need a update: bros (r10), comic-houses (r69), nmts (r15) 15:22:34 <Ammler> hello :-) 15:22:46 <Ammler> planetmaker: you do change your projects self... 15:23:34 <planetmaker> hm? 15:23:41 *** frosch has joined #openttdcoop.devzone 15:23:41 <Ammler> to the new compiler... 15:29:23 *** frosch123 has quit IRC 15:32:52 <planetmaker> I guess I didn't :-) And I'd appreciate, if you would do that as you know that :-) 15:33:53 <Ammler> well, it would be nice to see, if someone else is able to... 15:34:26 <planetmaker> I think I added it to ... one project 15:34:37 <planetmaker> newgrf_makefile should have it 15:34:42 <planetmaker> if it doesn't work there, I failed 15:35:06 <Ammler> you never pushed the enable 15:35:18 <planetmaker> hu? 15:35:29 <Ammler> and it uses different names for identifier and repo 15:35:40 <Ammler> (but that shouldn't fail) 15:36:54 <planetmaker> isn't that it: http://paste.openttd.org/225838 15:37:19 <planetmaker> otherwise I'm missing something 15:37:21 <planetmaker> as said 15:37:36 <Ammler> yes, that is quite old 15:37:43 <planetmaker> what? Like 3 days 15:37:45 <Ammler> there is no make anymore 15:38:06 <Ammler> you also don't need to define files 15:38:08 <planetmaker> sorry, then I have no clue what to add 15:38:15 <Ammler> if you use default grf build 15:38:17 <planetmaker> give me a diff and I can do that 15:38:31 <Ammler> check the existing projects 15:38:52 <planetmaker> guess where I got that diff from... 15:38:54 <Ammler> touch .devzone/build/nightlies/enable works for your grfs 15:39:15 <planetmaker> "existing repos" is obviously a too broad term 15:39:28 <Ammler> :-) 15:39:42 <Ammler> http://dev.openttdcoop.org/projects/home/wiki/DevZone 15:40:28 <Ammler> http://dev.openttdcoop.org/projects/fish/repository/revisions/371 15:41:29 <planetmaker> excuse me? That's EXACTLY the same diff 15:41:43 <planetmaker> no. exactly the same TAB :-P 15:41:46 <planetmaker> I look at 15:42:32 <planetmaker> hm. Redmine should show git style patches 15:45:53 <Ammler> example? 15:46:52 <planetmaker> the last link you give 15:46:56 <Ammler> http://dev.openttdcoop.org/projects/opensfx/repository/show/.devzone/build <-- special config for non-grfs 15:46:57 <planetmaker> empty files are not displayed 15:47:11 <Ammler> enable is there 15:48:35 <planetmaker> hm... .devzone/build/files/license.txt ? 15:49:26 <planetmaker> and which repos now have it? 15:49:45 <Ammler> why should you want do add such a file? 15:49:53 <Ammler> where do you get that idea from? 15:50:43 <planetmaker> as entry in .devzone/build/files 15:50:46 <Ammler> the only files are enable, *.spec, requires and files 15:51:34 <Ammler> well, such things are easy changeable... 15:52:10 <Webster> Latest update from devactivity: OpenSFX - Revision 94: DevZone: add custom files list <http://dev.openttdcoop.org/projects/opensfx/repository/revisions/94> || OpenSFX - Revision 93: DevZone: add buildrequires mercurial and p7zip <http://dev.openttdcoop.org/projects/opensfx/repository/revisions/93> 15:54:33 <planetmaker> well yes, but better right in the first place :-) 15:55:10 <planetmaker> hg diff --git works actually quite well :-) 15:55:10 <Ammler> well, I don't think, we need it there... 15:55:29 <planetmaker> so... which do have it? 15:55:32 <Ammler> but I also don't care, if you like to add it 15:55:36 <planetmaker> which don't? I have no overview 15:55:40 <Ammler> but we should do it locally 15:55:48 <Ammler> globally* 15:56:05 <planetmaker> I thought that file list gives what gets shipped somehow? or what does it represent? 15:57:07 <planetmaker> given the different project types I nearly start to end up at a ./configure ;-) 15:57:38 <Ammler> yes 15:57:57 <Ammler> well, grfs with your makefile need a enable only 15:58:08 <Ammler> everyhting else will be done by the global file 15:59:03 <Ammler> http://dev.openttdcoop.org/projects/home/repository/show/compiler/.default/nightlies 15:59:41 <planetmaker> meh. But patching doesn't work :-( 16:00:57 <Ammler> what is that hard with "touch .devzone/build/nightlies/enable"? 16:01:02 <planetmaker> hm, idea for you: make new Makefile target / a patch which just needs applying so that the CF works 16:01:19 <Ammler> that is already possible 16:01:24 <planetmaker> Ammler: that I can't use patch -p1 < new_cf.diff 16:01:57 <Ammler> patch ... is as hard as touch ... 16:01:59 <Ammler> :-P 16:02:02 <planetmaker> honestly, I fear I'll mess up 16:02:09 <planetmaker> not removing unnecessary stuff 16:02:13 <planetmaker> or removing too much 16:02:25 <Ammler> currently there is nothing in the repo 16:02:30 <Ammler> so you can't remove something 16:02:35 <Ammler> you can only add 16:02:38 <planetmaker> ! .devzone/build/nightlies/files 16:02:39 <planetmaker> ! .devzone/build/nightlies/make 16:02:41 <planetmaker> ? .devzone/backup.push <-- doesn't look alright to me for example 16:02:51 <Ammler> and maybe move backup.push from . to .devzone/ 16:03:05 <planetmaker> after patch -p1 -R < old_cf.diff && patch -p1 < ../new_cf.diff 16:03:23 <planetmaker> yes, moving that to .devzone makes definitely sense 16:03:25 <Ammler> there is no old cf conig in the repo 16:03:34 <Ammler> config* 16:03:40 <planetmaker> wish that were true :-) 16:03:50 <planetmaker> you see the diff :-) 16:04:01 <Ammler> the old CF used a Makefile.nightly on server side workcopy 16:04:03 <planetmaker> s/diff/status/ 16:04:18 <planetmaker> I don't know 16:04:24 <Ammler> :-) 16:04:42 <Ammler> the only DevZone file in the repo was backup.push 16:04:53 <Ammler> but that isn't used for the compiler 16:05:11 <planetmaker> yes. and Makefile.nightly and... .devzone/build/nighlies/make 16:05:23 <Ammler> Makefile.nightly was never part of the repo 16:06:03 <Ammler> that make file was a new old one :-) 16:06:16 <planetmaker> hm 16:06:24 <Ammler> as you never pushed you .devzone config, I had no idea, that you used it. 16:06:54 <planetmaker> honestly: I'd like to ask you to add the required changes. I can't test that locally here whether it works or not 16:07:09 <planetmaker> And you know in CF terms what to do :-) 16:07:39 <Ammler> yeah, but all you need it is to touch .devzone/build/nightlies/enable 16:07:45 <Ammler> why is that so complicated? 16:07:57 <Ammler> then my work was useless :-( 16:08:45 <planetmaker> it's not useless :-) 16:09:06 <planetmaker> but the diff in e.g. fish is more than just a touch .devzone/nightlies/enable 16:09:16 <planetmaker> and same with releases 16:10:49 <planetmaker> Ammler: it's like with Makefiles: it's also only me fixing them ;-) 16:11:12 <planetmaker> and if it's in the newgrf_makefile correctly: then it's easy to copy :-) 16:15:32 <Ammler> didn't I make a ticket for renaming? 16:21:52 <planetmaker> renaming the newgrf-makefile? 16:22:19 <Ammler> yes 16:25:29 <planetmaker> yes.... #950 16:26:18 <planetmaker> hm, I don't like that restriction tbh 16:27:07 <Ammler> it isn't a restriction, it works also the way it is now 16:27:13 <Ammler> but it is silly, imo 16:28:02 <planetmaker> a name can have spaces. a identifier cannot 16:28:23 <Ammler> you can create repo with spaces? 16:31:29 <planetmaker> oh, please, don't, even if possible! 16:32:15 <Ammler> so what is the issue with having identifier and repo same name? 16:35:36 <Brot6> 2cctrainset: compile of r543 failed - http://bundles.openttdcoop.org/2cctrainset/nightlies/ERROR/r543 16:36:25 <Brot6> firs: compile of r856 failed - http://bundles.openttdcoop.org/firs/nightlies/ERROR/r856 16:40:58 <Alberth> Ammler: I'd create repos called " ", " ", " " etc :p 16:45:29 <Ammler> Alberth: do you get planetmaker? 16:47:39 *** OwenS has quit IRC 16:47:50 *** OwenS has joined #openttdcoop.devzone 16:48:50 <planetmaker> Ammler: there's not really much wrong with it. But it does break people's checkouts 16:49:21 <Ammler> I mean this: "[18:27] <planetmaker> a name can have spaces. a identifier cannot" 16:49:52 <Ammler> what "name" are you refering ot? 16:49:56 <Ammler> to* 16:51:34 <planetmaker> what are you referring to? 16:51:44 <Ammler> repo 16:51:45 <planetmaker> filename and repo name e.g. don't work for base grfs 16:51:59 <planetmaker> and the repo name can be chosen locally anyway however desired 16:52:18 <Ammler> so why you need that different? 16:54:28 <planetmaker> history? 16:54:37 <planetmaker> nml is also different 16:54:52 <Ammler> no, it isn't 16:55:13 <Ammler> I guess, you confuse something 16:56:00 <Ammler> http://dev.openttdcoop.org/projects/nml and http://hg.openttdcoop.org/nml looks similar imo 16:58:27 <Ammler> http://hg.openttdcoop.org/newgrf_makefile and http://dev.openttdcoop.org/projects/newgrf-makefile doesn't 17:00:12 *** OwenS has quit IRC 17:00:56 *** OwenS has joined #openttdcoop.devzone 17:02:57 <planetmaker> well, if it bothers you, change it :-) 17:06:01 <Ammler> well, it does that is why I made the ticket, but don't change it self, as you use it, it is up2you.... 17:11:35 <frosch> call it "makegrf". no spaces :p 17:15:48 <planetmaker> :-) 17:16:36 <Ammler> we could add a tr '_' '-' 17:36:29 <planetmaker> 'tr'? 17:36:53 <Ammler> to convert repo to project 17:37:44 <planetmaker> well, basically it's only the dir name on the server, right? 17:39:18 <Ammler> no, it is only for the trigger, so redmine knows to update which project 17:39:33 <Ammler> bundles.openttdcoop.org will use repo name 18:04:41 <Brot6> test: abort: push creates new remote heads! 18:22:31 <Webster> Latest update from devactivity: FIRS Industry Replacement Set - Revision 857: Fix: Compilation failed in case sensitive systems. <http://dev.openttdcoop.org/projects/firs/repository/revisions/857> || NFO Meta Language - Revision 201: Codechange: Generalize the way built-in functions are handled. <http://dev.openttdcoop.org/projects/nml/repository/revisions/201> || NFO Meta Language - Revision 199: Fix: Error message about unsupported action2 operator contained... <http://dev.openttdcoop.org/projects/nml/repository/revisions/199> || NFO Meta Language - Revision 198: Fix: Not all operators can be resolved compile-time, even if th... <http://dev.openttdcoop.org/projects/nml/repository/revisions/198> || NFO Meta Language - Revision 200: Fix/Feature: Properly support Action2 storage operators. <http://dev.openttdcoop.org/projects/nml/repository/revisions/200> 18:28:19 <Alberth> ha ha, nml revision numbers are a complete mess, at least in the activity web page 18:37:27 <Ammler> that is because sorting is with time because of git support 18:38:00 <Ammler> oh well, in activity it needs to be with time 18:40:43 <Alberth> then the sort is broken 18:41:25 <Alberth> in my cloned copy, increasing revision numbers of the server copy have increasing time stamps 18:41:25 <Hirundo> The mess is probably caused by me using mq patches 18:43:13 <planetmaker> Hirundo: no, the announcements here are time-messed up also without mq 18:44:48 * Alberth bets on a programming error in redmine 18:47:33 <Ammler> Alberth: it is only the activity page 18:47:39 <Ammler> how else would you sort that? 18:47:51 <Alberth> planetmaker: I am looking for a nice place of the string(STR_NAMES). Would town_names(myname, STR_NAMES) { ... be useful? 18:48:51 <planetmaker> with myname = CaptialTowns | Pooper village names | awesome adjectives or alike? 18:48:56 <planetmaker> then yes 18:49:22 <Alberth> Ammler: I think it is not sorted, but events are stored in some queue or so that is handed in a non-determinstic order before storing in the DB. The Activity page probably just copies the DB order 18:49:47 <Ammler> it is sorted with time 18:50:11 <Alberth> Ammler: perhaps the server is at a place with instable time 18:50:12 <Ammler> the repository view is fine 18:50:16 <planetmaker> the commit time is transmitted and used in display 18:50:18 <Ammler> :-) 18:50:36 <Ammler> the time is in the repo, this isn't a centralized VCS 18:51:30 <Alberth> time stamps are not copied with a pull? 18:51:47 <Ammler> yep, they are, but also with push :-) 18:54:57 <Alberth> perhaps the activity page does not take seconds into account? 18:56:24 <Alberth> planetmaker: 'myname' is another problem 18:57:46 <Ammler> yes, the bug is that seconds aren't fetched from redmine 18:57:53 <Alberth> it is the name of the town_names action, so you can refer to it from another town_names action. For final town_names actions however (which have style_names), you cannot refer to them (I think, the docs are not very clear about that). 18:58:22 <Alberth> that means that you could do without myname there 19:00:04 <Alberth> Ammler: how useful :p 19:00:23 <Ammler> yes, I check if there is already a bugreport about 19:03:59 *** planetmaker has left #openttdcoop.devzone 19:04:26 <PeterT> planetmaker pressed the wrong button? 19:15:04 *** welshdragon has joined #openttdcoop.devzone 20:11:52 *** Doorslammer has quit IRC 20:15:37 *** Yexo has joined #openttdcoop.devzone 20:15:59 <Alberth> Just submitted the patch 20:16:04 <Alberth> to the tracker :) 20:16:17 <Yexo> good evening 20:16:22 <Yexo> I'll take a look in a few minutes 20:16:24 <Alberth> good evening 20:16:40 <Alberth> no hurry, I am not going to fix it in the next few days 20:17:03 <Yexo> in what way is it broken? 20:17:26 <Alberth> it is not, I fixed the issues you mentioned today 20:17:40 <Alberth> oh "fix it" 20:17:51 <Alberth> I mean, fix any issues you find 20:17:59 <Yexo> ah, ok :) 20:18:29 <Yexo> http://paste.openttd.org/225840 <- that's the syntax to add a grfid 20:19:48 <Alberth> thanks 20:20:27 <Yexo> the example you provided wil generate the names "abcpqx", "abcpqy", "abcpqz", "abcpq", "abcrstx", etc. right? 20:21:15 <Alberth> something in that direction 20:21:54 <Alberth> with some extra cases 20:21:56 <Ammler> Yexo: shouldn't nml already support "pseudo versioning" byte 0 and 1 for author/theam, byte3 for project, byte4 for version 20:22:20 <Ammler> byte3 -> 2, 4->3 20:22:35 <Yexo> Ammler: how many projects actually follow that guideline? 20:22:43 <Ammler> all I know 20:22:48 <Webster> Latest update from devactivity: NFO Meta Language - Patch #959: add action F <http://dev.openttdcoop.org/issues/959#change-2550" target="_blank">http://dev.openttdcoop.org/issues/959#change-2550> || NFO Meta Language - Patch #959 (New): add action F <http://dev.openttdcoop.org/issues/959> 20:23:05 <Ammler> some merge author/project bytes 20:23:09 <Yexo> it's already possible to do "AB" for author "AB", project 3 vresion 2 20:23:12 <Alberth> that was quick :) 20:23:18 <Ammler> but byte 3 is quite much used as version 20:23:47 <Ammler> George doesn't 20:23:51 <Yexo> I'd rather provide that info in the documentation and leave it to the users themself 20:24:00 <Yexo> I don't like to force that method 20:24:25 <Alberth> we could make an extension, and make project and version optional, or so 20:26:58 <Yexo> Alberth: skip_7/skip_9/skip_needed should return False/True/True, so they're already correct 20:28:08 <Alberth> good. first two were educated guesses, last one was plain random 20:28:13 <Yexo> meaning of those functions is: Can this action be skipped safely by an action7? / Can this .. safely by an action9? / Do we need to skip this action at all? (action10 can't be skipped by action9 and skipping it with action7 has no effect) 20:28:45 <Alberth> we need some form of documenting such stuff imho 20:28:54 <Yexo> yes, indeed 20:29:12 <Ammler> isn't action10 label? 20:29:12 <Yexo> but first I'm writing some documentation about the syntax of nml 20:29:18 <Yexo> action10 is indeed label 20:29:28 <Ammler> and you can't skip that with action9? 20:29:44 <Yexo> http://wiki.ttdpatch.net/tiki-index.php?page=Action7 according to this page not 20:29:45 * Alberth is documenting grf 20:29:59 <Ammler> Yexo: I guess, you confuse something 20:30:43 <Yexo> Ammler: why? not being able to skip a label is perfectly fine, labels are used to "skip to", not to skip 20:31:20 <Ammler> but you might have different labels and not necessary jump to next 20:32:06 <Yexo> that makes sense 20:32:10 <Yexo> but I'm just following that wiki page 20:32:13 <Ammler> maybe that was the reason some time ago, it wasn't possible to skip to end 20:32:23 <Ammler> frosch fixed that though 20:33:02 <Yexo> in openttd it's completely irrelevant 20:33:23 <Yexo> skipping an action10 with either action7 or action9 has no effect at all 20:33:45 <Yexo> but I can't be bothered to check the ttdpatch code, so I'm just following the wiki page for any possible issues with ttdpatch 20:33:46 <Ammler> well, the fact, that you can skip to end, does also mean, you can skip every cation 20:33:49 <Ammler> action* 20:33:57 <frosch> what? 20:34:22 <Ammler> 0=skip to end of grf 20:35:26 <Ammler> or is that like a "exit" in scripts? 20:36:13 <frosch> what is the difference between those statements? 20:36:39 <Ammler> which 2? 20:36:50 <frosch> http://wiki.ttdpatch.net/tiki-index.php?page=GrfLoadingStages <- anyway, that page proofed several times being more useful than the action7/9 page about skippability 20:37:09 <frosch> your two lines between by two 20:37:17 <frosch> *my 20:38:02 <frosch> the page above states that labels are resolved before any other sprites are processed 20:38:23 <frosch> as they have no meaning anymore, when action7/9 are processed 20:38:32 <Yexo> frosch: so the comment in DecodeSpecialSprite is actually wrong? 20:38:47 <Yexo> In TTDPatch, for some reason actions 1 and 2 are carried outd uring stage 1, whilst action 3 is carried out during stage 2 <- that one? 20:39:08 * frosch wonders what action6 does on the jumptarget of action7/9 20:40:44 <frosch> Yexo: i would expect "stage 1" meaning "init", and "stage 2" meaning "activation" 20:41:02 <frosch> and action1/2 are initialised during "init", whatever that means 20:41:25 <Yexo> Initialization: ... records action 1 sprite numbers, resolves action 2 references (to the sprite numbers within the grf file of action 1 sprites and references to variational/random action 2) ... 20:43:48 <frosch> well, according to that the comment in DecodeSpecialSprite seems to be correct 20:44:09 <frosch> ttdp records action3->2 and 2->1 during "init", ottd during "activation" 20:46:08 <Yexo> that table is useful, but it doesn't tell me anything about whether or not it's "safe" to skip actionF with an action9 20:46:32 <frosch> i would say it is safe 20:48:11 <frosch> the actionf page explains skippability by action9 20:48:14 <frosch> under "notes" 20:48:31 <Yexo> ehm, of course :p 20:48:36 <Yexo> I had that figured out already, sorry 20:48:50 <Yexo> the initial questoin was whether it would be safe to skip an action10 20:49:07 <Yexo> it's clear that it wouldn't work, but the action7 page claims it's not safe either 20:49:35 <Yexo> whlle according to the stages of loading a grf file page it would ignore the action10 anyway 20:49:42 <frosch> i decided to ignore the table on the action7/9 page 20:51:32 <Alberth> good night 20:52:01 *** Alberth has left #openttdcoop.devzone 20:53:14 <frosch> "You must not skip an action 2 at all unless skipping the entire rest of the file. Instead, skip the action 3 that refers to it." <- i would rather search for a testgrf than believing in that 20:54:52 <Yexo> it's a lot easier to just not allow that in nml then to find out whether or not it works in ttdpatch 20:54:59 <Yexo> I'll reconsider once I get a use case for it 21:00:10 <frosch> night 21:00:14 *** frosch has quit IRC 21:08:41 <Webster> Latest update from devactivity: FIRS Industry Replacement Set - Revision 859: Change: Update the spanish translation. <http://dev.openttdcoop.org/projects/firs/repository/revisions/859> || FIRS Industry Replacement Set - Revision 858: Fix: Add new strings to remove_defines.pnfo. <http://dev.openttdcoop.org/projects/firs/repository/revisions/858> 21:24:01 <Webster> Latest update from devactivity: NFO Meta Language - Patch #959: add action F <http://dev.openttdcoop.org/issues/959#change-2551> 21:39:17 <Webster> Latest update from devactivity: 32bpp-extra - Bug #953 (Closed): useless newgrf action in the grf <http://dev.openttdcoop.org/issues/953#change-2552> 21:54:54 <Webster> Latest update from devactivity: NFO Meta Language - Patch #959: add action F <http://dev.openttdcoop.org/issues/959#change-2554> 22:04:02 <Ammler> http://bundles.openttdcoop.org/ogfxplus/nightlies/ERROR/r19/ogfxplus-r19-build.err.log 22:07:30 <Yexo> Ammler: I just checked, but ogfx+ compiles fine here 22:09:42 <Ammler> http://bundles.openttdcoop.org/ogfxplus/nightlies/ERROR/r19/ 22:09:50 <Ammler> nml r201 22:10:57 <Webster> Latest update from devactivity: NFO Meta Language - Revision 202: Add: docs/license.txt <http://dev.openttdcoop.org/projects/nml/repository/revisions/202> 22:11:06 <Yexo> Ammler: custom_tags.txt (in the root of ogfx+) is a required file 22:11:26 <Yexo> if it's not available you'd get the error the server shows in the logs 22:12:22 <Ammler> and why don't I have such a file? 22:12:28 <Ammler> and why did it work before? 22:12:55 <Yexo> last commit to ogfx+ is "change [Makefile]: Update to Makefile r95" 22:13:40 <Ammler> http://new.dev.openttdcoop.org/projects/ogfxplus/repository 22:13:48 <Yexo> custom_tags.txt is in .hginore, it probably should be generated by the makefile 22:13:50 <Ammler> no custom_tags thre 22:14:17 <Ammler> makes sense 22:14:32 <Yexo> and indeed it isn't, I get the same error now after removing it manually 22:15:35 <Ammler> so it looks like a ogfx+ bug 22:15:36 *** ODM has quit IRC 22:16:21 <Yexo> yes, ogfx+ r18 wrote to custom_tags.txt but during the makefile update those lines where removed 22:16:40 <Yexo> hg diff -c 19 scripts/Makefile.common, first 3 removed lines 22:17:14 <Ammler> well, if you have a fix, feel free to commit it, I guess you are dev there too 22:17:25 <Ammler> as pm isn't here 22:17:57 <Yexo> I'm not yet a dev there, but planetmaker said t was ok if I committed there 22:23:18 <Ammler> well, ssh doesn't respect redmine settings :-) 22:24:59 <Yexo> indeed, it worked :) 22:25:59 <Yexo> something like this should be added to the general makeifle project, at least if it aims to support nml 22:26:00 <Webster> Latest update from devactivity: OpenGFX+ - Revision 20: Fix: the makefile didn't write custom_tags.txt anymore <http://dev.openttdcoop.org/projects/ogfxplus/repository/revisions/20> 22:33:29 <Brot6> nml: update from r201 to r202 done - http://bundles.openttdcoop.org/nml/nightlies/r202 22:33:53 <Brot6> ogfxplus: update from r18 to r20 done (8 errors) - http://bundles.openttdcoop.org/ogfxplus/nightlies/r20 22:33:57 <Brot6> Following repos didn't need a nightlies update: 2cctrainset (r543), 32bpp-extra (r36), firs (r859), fish (r372), heqs (r320), opengfx (r459), openmsx (r57), opensfx (r94), test (ERROR r20), worldairlinersset (r643) 22:34:11 <Ammler> sorry, I like testing it :-P 22:35:20 <Yexo> 8 errors, is that becaues http://bundles.openttdcoop.org/ogfxplus/nightlies/r20/ogfxplus-r20-build.err.log has 8 lines? 22:35:57 <Ammler> yes 22:36:35 <Ammler> I could add a sed filter so we could ignore those 22:37:36 <Yexo> that could be uesful for the DeprecationWarning in yacc and the "Generating LALR parsing table" line 22:37:45 <Yexo> the warnings about the tokens I'll fix soon 22:37:59 <Yexo> first more documentatino 23:25:17 *** Terkhen has quit IRC 23:25:17 *** andythenorth has quit IRC 23:25:50 *** andythenorth has joined #openttdcoop.devzone 23:26:46 *** Terkhen has joined #openttdcoop.devzone