Log for #openttdcoop.devzone on 24th May 2010:
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... <> || NFO Meta Language - Patch #958 (Closed): Add -o option <>
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. <>
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.// will support both
09:10:55  <Ammler> :*
09:11:02  <Ammler> and yeah, moin
09:12:02  <Ammler>
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. <>
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. <>
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/
14:04:19  <Brot6> test: abort: push creates new remote heads!
14:17:42  <Alberth> Yexo: a review would be useful imho:
14:17:57  <Alberth> or should I add it to a tracker?
14:28:11  <Alberth>   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> 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. <>
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(). <>
15:17:09  <Brot6> airportsplus: update from r48 to r49 done (0 errors) -
15:17:16  <Brot6> nutracks: update from r60 to r61 done (1 errors) -
15:17:21  <Brot6> snowlinemod: update from r10 to r11 done (0 errors) -
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:
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>
15:40:28  <Ammler>
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> <-- 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 <> || OpenSFX - Revision 93: DevZone: add buildrequires mercurial and p7zip <>
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>
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 -
16:36:25  <Brot6> firs: compile of r856 failed -
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> and looks similar imo
16:58:27  <Ammler> and 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> 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. <> || NFO Meta Language - Revision 201: Codechange: Generalize the way built-in functions are handled. <> || NFO Meta Language - Revision 199: Fix: Error message about unsupported action2 operator contained... <> || NFO Meta Language - Revision 198: Fix: Not all operators can be resolved compile-time, even if th... <> || NFO Meta Language - Revision 200: Fix/Feature: Properly support Action2 storage operators. <>
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> <- 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 <" target="_blank">> || NFO Meta Language - Patch #959 (New): add action F <>
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> 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> <- 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. <> || FIRS Industry Replacement Set - Revision 858: Fix: Add new strings to remove_defines.pnfo. <>
21:24:01  <Webster> Latest update from devactivity: NFO Meta Language - Patch #959: add action F <>
21:39:17  <Webster> Latest update from devactivity: 32bpp-extra - Bug #953 (Closed): useless newgrf action in the grf <>
21:54:54  <Webster> Latest update from devactivity: NFO Meta Language - Patch #959: add action F <>
22:04:02  <Ammler>
22:07:30  <Yexo> Ammler: I just checked, but ogfx+ compiles fine here
22:09:42  <Ammler>
22:09:50  <Ammler> nml r201
22:10:57  <Webster> Latest update from devactivity: NFO Meta Language - Revision 202: Add: docs/license.txt <>
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>
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 <>
22:33:29  <Brot6> nml: update from r201 to r202 done -
22:33:53  <Brot6> ogfxplus: update from r18 to r20 done (8 errors) -
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 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

Powered by YARRSTE version: svn-trunk