Times are UTC Toggle Colours
00:47:20 *** stevenh has joined #openttdcoop.devzone 00:47:23 <stevenh> Morning all... 00:47:38 <stevenh> Does anyone know how to get a GRF on devzone to start being built as a nightly? 00:47:48 <Supercheese> Apply for membership (lemme dig the link) 00:48:09 <stevenh> i'm manager of the grf 00:48:16 <Supercheese> Ah, nevermind then 00:48:35 <Supercheese> You should have a folder called ".devzone" 00:48:43 <Supercheese> within that a "build" folder 00:48:52 <Supercheese> and "nightlies" 00:49:10 <Supercheese> such as: https://dev.openttdcoop.org/projects/heqs/repository/show/.devzone/build/nightlies 00:49:31 <Supercheese> just a blank file titled "enable" in there should do the trick 00:52:38 <stevenh> thanks! 00:58:46 <stevenh> Any pointers to speeding up the repository? (Just in case I'm doing something wrong...) 00:59:24 <Supercheese> There's a bit of lag, you have to wait for pushes to build. Of course nightlies won't happen right away 01:01:00 <stevenh> yup yup... no idea what timezone they're kicking off in 01:32:32 <stevenh> thanks again Supercheese! 01:32:53 <Supercheese> You're welcome :) 02:25:19 *** stevenh has quit IRC 06:11:03 *** phatmatt has quit IRC 06:11:07 *** phatmatt has joined #openttdcoop.devzone 06:11:59 *** phatmatt is now known as Guest3952 06:12:30 *** phatmatt has joined #openttdcoop.devzone 06:12:54 *** phatmatt has quit IRC 06:19:12 *** Guest3952 is now known as phatmatt 08:05:46 <DevZone> Project jptrains build #1-push: SUCCESS in 23 sec: https://jenkins.openttdcoop.org/job/jptrains/1/ 10:04:08 *** DevZone has quit IRC 10:05:19 *** DevZone has joined #openttdcoop.devzone 10:11:12 <DevZone> Yippee, build fixed! 10:11:13 <DevZone> Project OpenGFX+ Airports build #25-push: FIXED in 49 sec: https://jenkins.openttdcoop.org/job/airportsplus/25/ 11:01:53 *** frosch123 has joined #openttdcoop.devzone 12:32:28 *** Supercheese has quit IRC 12:33:01 *** Supercheese has joined #openttdcoop.devzone 12:45:38 *** phatmatt has quit IRC 12:45:38 *** KenjiE20 has quit IRC 12:45:38 *** Ammler has quit IRC 12:45:38 *** Supercheese has quit IRC 12:45:38 *** XeryusTC has quit IRC 12:57:02 *** Supercheese has joined #openttdcoop.devzone 12:57:02 *** phatmatt has joined #openttdcoop.devzone 12:57:02 *** KenjiE20 has joined #openttdcoop.devzone 12:57:02 *** Ammler has joined #openttdcoop.devzone 12:57:02 *** XeryusTC has joined #openttdcoop.devzone 15:27:51 *** andythenorth has joined #openttdcoop.devzone 16:12:05 *** Alberth has joined #openttdcoop.devzone 16:51:13 <DevZone> Project xussrset - Trains from Russia build #75-push: SUCCESS in 3 min 45 sec: https://jenkins.openttdcoop.org/job/xussrset/75/ 16:52:08 <DevZone> Project OpenGFX+ Industries build #63-push: FAILURE in 54 sec: https://jenkins.openttdcoop.org/job/ogfx-industries/63/ 16:52:41 <DevZone> Project Dutch Tramset build #15-push: SUCCESS in 33 sec: https://jenkins.openttdcoop.org/job/dutchtramset/15/ 16:55:00 <DevZone> Project OpenGFX+ BigGUI build #24-push: SUCCESS in 2 min 19 sec: https://jenkins.openttdcoop.org/job/ogfx-biggui/24/ 16:56:57 <DevZone> Project ISR Industrial Station Renewal build #54-push: SUCCESS in 1 min 56 sec: https://jenkins.openttdcoop.org/job/isr/54/ 17:05:56 <DevZone> Project OpenGFX+ Trains build #12-push: SUCCESS in 18 min: https://jenkins.openttdcoop.org/job/ogfx-trains/12/ 17:14:26 <planetmaker> hm... eints unfixes scotish gaelic in ogfx-industries 17:27:36 <DevZone> Project NML - NewGRF Meta Language build #120-nightlies: SUCCESS in 9 min 29 sec: https://jenkins.openttdcoop.org/job/nml/120/ 17:52:52 <frosch123> no errors logged 17:53:04 <frosch123> but some crawler is just trying to read all strings from eints 17:53:11 <planetmaker> he 17:53:11 <frosch123> getting 401 everywhere 17:53:15 <planetmaker> :D 17:53:43 <planetmaker> I wonder why eints re-committed the thing I fixed in gd_GB yesterday 17:54:19 <frosch123> most lkely it did consider them the same 17:54:50 <planetmaker> it's the ... {COMMA} {P a b c} vs. ... {COMMA} {P 1 a b c} issue 17:55:08 <frosch123> well, you could try fixing it again 17:55:16 <frosch123> than we can see what eints reads on upload 17:55:19 <planetmaker> did something change? 17:55:34 <frosch123> what should have changed? 17:56:36 <Alberth> planetmaker: that's the same thing to eints 17:56:46 <planetmaker> I see 17:56:58 <Alberth> ie it follows openttd conventions 17:57:47 <planetmaker> STR_ERR_TOWN_TOO_SMALL 17:57:48 <planetmaker> (Correct) STR_ERR_TOWN_TOO_SMALL :... tha am baile seo ro bheag airson {COMMA} gnìomhachasan dhen t-seòrsa seo (chan fhaod ach aon dhe na gnìomhachasan seo a bhith ann airson gach {COMMA} {P taigh thaigh taighean taigh}) 17:57:57 <planetmaker> is the string in question 17:58:14 <Alberth> I was pondering about it, and maybe we should make eints output that number always 17:58:43 <Alberth> if we change nml, all current {P ... } without numbers will break 17:59:09 <Alberth> planetmaker: openttd says that {P} refers to the previous number 18:07:06 <planetmaker> why would it break? 18:07:42 <frosch123> i kind of doubt any newgrf would break 18:07:54 <planetmaker> or what change do you mean? You mean requiring a position parameter always? 18:07:59 <frosch123> it's very unlikely that {P} is used in front of a number 18:08:10 <planetmaker> I'd risk that :-) 18:08:14 <frosch123> so, i think we should just change nml 18:08:23 <frosch123> it' more likely that if fixes some sets, than it breaks some 18:08:36 <frosch123> (like having a {P} between two numbers) 18:08:38 <planetmaker> All NewGRFs which use it are on this server. So we can also advice. Or even fix 18:09:10 <planetmaker> Well... of course, there others, but they very likely don't use so many languages 18:09:32 <planetmaker> and the weired ones break it 18:11:05 <planetmaker> the update from 0.2.x to 0.3.0 breaks anyway some other stuff ;-) 18:11:15 <planetmaker> so if we break something, better before 0.3.0 18:11:37 <frosch123> there is no 0.3.0 ? :o 18:11:41 <planetmaker> And the two bugs you fixed yesterday, day before yesterday were the two ones I wanted to have seen fixed 18:11:53 <planetmaker> Now I would feel comfortable to actually release 0.3.0 18:12:03 <planetmaker> The changelog is... meeeh 18:12:06 <planetmaker> huge 18:12:12 <frosch123> well, we should fix the {P} :p 18:19:26 <DevZone> Project finnishtrams build #11-nightlies: SUCCESS in 13 min: https://jenkins.openttdcoop.org/job/finnishtrams/11/ 18:25:29 *** oskari89 has joined #openttdcoop.devzone 18:27:21 <planetmaker> so, I think we need not check this out as we know the result anyway, yes? 18:27:31 <planetmaker> or should I still commit a revert of it, frosch123 ? 18:27:42 <frosch123> nah, better fix nml :p 18:33:51 <DevZone> Project Nutracks build #59-nightlies: SUCCESS in 3 min 45 sec: https://jenkins.openttdcoop.org/job/nutracks/59/ 18:33:54 <DevZone> Project Japanese Trains build #2-nightlies: SUCCESS in 12 min: https://jenkins.openttdcoop.org/job/jptrains/2/ 18:45:48 *** zooks has joined #openttdcoop.devzone 18:55:05 <DevZone> Project Iron Horse build #354-nightlies: SUCCESS in 7 min 58 sec: https://jenkins.openttdcoop.org/job/iron-horse/354/ 19:10:37 <DevZone> Project OpenGFX build #89-nightlies: SUCCESS in 12 min: https://jenkins.openttdcoop.org/job/opengfx/89/ 19:22:26 <frosch123> Alberth: would you like to push your grfstring.py doc thingie? 19:23:02 <frosch123> http://paste.openttdcoop.org/show/2763/ <- i wonder whether that would just do 19:23:17 <Alberth> it isn't committed yet 19:25:14 <Alberth> I wouldn't know, I haven't gotten to understanding that bit of code 19:25:47 <frosch123> well, i would just try it :p 19:26:31 <Alberth> I assume you didn't hire an infinite number of monkeys to come up with that patch :p 19:26:44 <frosch123> but with your docs, i think i kind of know what it does, and why 19:27:17 <frosch123> well, i know the input and in know the output :p 19:27:33 <Alberth> it needs understanding of newgrf text specs, is my guess :) 19:27:35 <frosch123> so i can surround it :p 19:27:51 <frosch123> engage the code from both ends :) 19:27:59 <Alberth> :D 19:28:31 <frosch123> i think the most useful part of python docs is the type information 19:29:09 <frosch123> you can skip a lot of c++ docs by just naming parameters useful 19:29:25 <frosch123> but in python you still have a hard time to figure out a parameter, if you do not know the type of it 19:30:13 <Alberth> yes the type is the crucial information 19:30:27 <Alberth> you need my docs? 19:30:33 <frosch123> i found them in paste 19:30:42 <frosch123> else i could not have written that diff :p 19:30:50 <Alberth> I mean you need it pushed? 19:30:55 *** andythenorth has joined #openttdcoop.devzone 19:30:59 <planetmaker> docs hardly ever hurt 19:31:02 <frosch123> well, would allow me to make a proper patch 19:31:09 <Alberth> k 19:31:10 <frosch123> and somehow force pm to test it or so :p 19:31:10 <planetmaker> unless you're the fly which is squatted by the book 19:32:56 <Alberth> pushed 19:33:28 <Alberth> books are not so useful for catching flies, they're too heavy :) 19:33:39 <frosch123> hmm, i still need to get used to python's ternary operator 19:33:51 <Alberth> I don't like it 19:34:11 <Alberth> and usually use an if/else :) 19:34:34 <Alberth> o/ andy 19:36:08 <DevZone> Project NML - NewGRF Meta Language build #121-push: SUCCESS in 3 min 16 sec: https://jenkins.openttdcoop.org/job/nml/121/ 19:36:18 <andythenorth> Alberth: just pm-ed you something 19:36:28 <Alberth> firs doesn't have {P [^0-9] 19:36:44 <andythenorth> ? 19:37:11 <Alberth> I don't seem to have any pms 19:37:25 <frosch123> so, do i have a ogfx+industies checkout? 19:37:31 <Alberth> {P ...} commands without number 19:38:53 <andythenorth> Alberth: forgot to include you : 19:38:54 <andythenorth> :) 19:38:54 <frosch123> pff, ogfx-industries doesn't even use {P} in english 19:38:55 <andythenorth> fixed 19:39:12 <andythenorth> python ternary operator is wierd 19:39:17 <andythenorth> I hate ternary operators anyway 19:39:25 <andythenorth> if-else is easier to scan 19:40:41 <planetmaker> frosch123, yes, that's why it only happened when people started to translate :D 19:40:52 <andythenorth> I guess it's ternary is just a convenient shorthand :P 19:41:00 <frosch123> english is plural 0 , right? 19:41:13 <frosch123> well, at least it did compile 19:41:25 <planetmaker> yes, english is plural 0 19:41:27 <Alberth> yes it is 19:42:16 <Alberth> andythenorth: planetmaker did some eints in nfo stuff afaik 19:42:22 <Alberth> not sure of its status 19:42:29 <frosch123> how do i trigger that message? 19:42:53 <planetmaker> which message do you mean, frosch123 ? 19:42:56 <Alberth> bla bla {COMMA} {P x x} 19:43:05 <planetmaker> failure to compile ogfx-industries? Just try to compile tip 19:43:06 <Alberth> or {SIGNED_WORD} perhaps 19:43:31 <frosch123> planetmaker: no, in game 19:43:36 <frosch123> i compiled ogfx-industries 19:43:40 <frosch123> and added a plural to english 19:44:00 <planetmaker> oh :-) Then which message? STR_XXX? 19:44:06 <frosch123> TOWN_TOO_SMALL 19:44:21 <planetmaker> fund a bank, I think 19:44:27 <frosch123> btw. we need an error message if unpausing a game is not possible due to game script 19:45:16 <frosch123> i guess i need some special parameters then 19:45:47 <planetmaker> let me find out :-) 19:46:59 <planetmaker> frosch123, water towers, banks and shops exhibit that behaviour 19:47:08 <planetmaker> you can only have n per m citizens 19:47:33 <planetmaker> thus try to build many in a (small) town 19:47:36 <frosch123> yay, works 19:48:32 <frosch123> though i am not sure whether the message is correct by itself 19:48:37 <frosch123> one bank per 150 houses? 19:48:42 <planetmaker> Alberth, andythenorth nfo support for lang files is... well, rather meh. But works for ISR 19:48:48 <frosch123> does that only apply for the second bank? 19:48:57 <frosch123> this town has nowhere near 150 houses :p 19:48:59 <planetmaker> they have a well-structured nfo lang file which contains all strings. Thus easy script 19:49:24 <planetmaker> frosch123, yes, if there's 151 houses, you can build 2 19:49:38 <planetmaker> I *think* :-) 19:49:54 <Alberth> would be consistent with observed behavior :p 19:50:02 <frosch123> now, there is certainly a bug 19:50:18 <frosch123> i try to build a second one, and it says, there can not be 18444 industries :p 19:50:33 <Alberth> lol 19:50:33 <planetmaker> :D 19:50:42 <planetmaker> that may even be true, frosch123 ;-) 19:50:45 <frosch123> but the plural works :p 19:51:08 <planetmaker> switch (FEAT_INDUSTRIES, PARENT, name ## _num_houses_check_switch, [STORE_TEMP(LOAD_TEMP(0), 0x100), STORE_TEMP(NUM_HOUSES_PER_INDUSTRY, 0x101), var[0xB6] - NUM_HOUSES_PER_INDUSTRY * LOAD_TEMP(0)]) { \ 19:51:08 <planetmaker> 0..0xFFFF: return CB_RESULT_LOCATION_ALLOW; \ 19:51:08 <planetmaker> return string(STR_ERR_TOWN_TOO_SMALL); \ 19:51:41 <frosch123> @base 10 16 18444 19:51:41 <Webster> frosch123: 480C 19:51:53 <planetmaker> strange number 19:52:05 <frosch123> it does not change when funding more of them 19:52:18 <planetmaker> I wonder whether it's a missing initialization 19:52:25 <frosch123> seems to occur for everything bigger than 0 19:53:32 <frosch123> hmm, now it happens every time 19:53:42 <frosch123> yeah, maybe uninitalised then 19:54:17 <frosch123> http://paste.openttdcoop.org/show/2765/ <- anyway, seems to work 19:54:33 <frosch123> is there an issue for that on devzone? 19:54:44 <planetmaker> no. Just checked 19:55:12 <Alberth> not for NML 19:55:54 <planetmaker> I mean ogfx-industries, but for the {P thing...} it probably could be moved from eints to nml 20:11:47 <frosch123> so, shall i commit as Fix for the eints-issue? 20:12:10 <planetmaker> I could move the issue between the projects ;-) (or you could :-P ) 20:12:17 <planetmaker> or even copy 20:12:50 <DevZone> Project Iron Horse build #355-push: SUCCESS in 54 sec: https://jenkins.openttdcoop.org/job/iron-horse/355/ 20:13:25 <planetmaker> http://dev.openttdcoop.org/issues/6501 <--- that, yes? 20:13:42 <frosch123> no 20:13:45 <frosch123> 6503 20:14:00 <frosch123> hmm 20:14:03 <frosch123> are they the same? 20:14:25 <planetmaker> Dunno. Alberth considers them different, I think. Not sure 20:14:31 <planetmaker> To me they look the same 20:16:02 <Alberth> 6501 was not clear in intention what to report 20:16:48 <Alberth> ie it could also be interpreted as {P} without number would be too ambiguous, and always required a number 20:17:34 <Alberth> but if the OP considers them equal who am I to argue :p 20:17:49 <planetmaker> :D 20:18:41 <planetmaker> Well, the issue is that we a) need valid strings and b) the strings must have a syntax which is clear and simple in what parameter refers to which number 20:19:29 <planetmaker> when I posted that issue, I did not want to impose a solution but note that there's an issue. Which possibly needs discussion 20:19:33 <planetmaker> which we had 20:20:29 <planetmaker> I think we agreed on 'as close as possible to OpenTTD behaviour'. Which means eints is correct. And NML should adopt 20:20:58 <Alberth> I agree on that 20:21:17 <Alberth> so, issue solved thus :) 20:21:27 <planetmaker> both? :D 20:21:41 <Alberth> yeah 6503 and 6501 at least 20:21:45 <planetmaker> (on eints side) 20:21:56 <DevZone> Project Iron Horse build #356-push: SUCCESS in 59 sec: https://jenkins.openttdcoop.org/job/iron-horse/356/ 20:21:57 <Alberth> I think I added another one though 20:22:32 <planetmaker> http://dev.openttdcoop.org/issues/6504 20:22:38 <planetmaker> but... isn't that NML, too? 20:23:04 <DevZone> Project Iron Horse build #357-push: SUCCESS in 1 min 7 sec: https://jenkins.openttdcoop.org/job/iron-horse/357/ 20:23:13 <frosch123> 6504 looks like eints fault 20:23:15 <frosch123> nml is correct 20:23:26 <frosch123> there is only one pluaral parameter 20:23:39 <frosch123> though i have no idea what the translator intended 20:23:52 <Alberth> s/,/ /g 20:23:55 <DevZone> Project Iron Horse build #358-push: SUCCESS in 42 sec: https://jenkins.openttdcoop.org/job/iron-horse/358/ 20:24:06 <Alberth> he thinks he is using ##plural 6 :) 20:24:16 <Alberth> which by now is true 20:24:38 <Alberth> I agree eints is behaving weird there 20:25:14 <Alberth> which is not fixed afaik 20:25:31 <Alberth> I fixed the ability to upload ##plural 6 :) 20:26:23 <frosch123> so, close 6501 as duplicate of 6503? 20:26:30 <frosch123> move 6503 to nml? 20:26:36 <frosch123> commit fix for 6503? 20:26:44 <Alberth> yes 20:26:46 <planetmaker> +1 20:33:03 <DevZone> Project NML - NewGRF Meta Language build #122-push: SUCCESS in 3 min 50 sec: https://jenkins.openttdcoop.org/job/nml/122/ 20:38:02 <DevZone> Project Iron Horse build #359-push: SUCCESS in 8 min 31 sec: https://jenkins.openttdcoop.org/job/iron-horse/359/ 21:12:32 <DevZone> Project Iron Horse build #360-push: SUCCESS in 42 sec: https://jenkins.openttdcoop.org/job/iron-horse/360/ 21:13:09 *** Zuu has joined #openttdcoop.devzone 21:38:56 *** zooks has quit IRC 21:47:26 <DevZone> Project Iron Horse build #361-push: SUCCESS in 1 min 48 sec: https://jenkins.openttdcoop.org/job/iron-horse/361/ 21:53:46 <DevZone> Project Japanese Trains build #3-push: SUCCESS in 1 min 18 sec: https://jenkins.openttdcoop.org/job/jptrains/3/ 21:58:44 <DevZone> Project ISR Industrial Station Renewal build #55-push: SUCCESS in 18 sec: https://jenkins.openttdcoop.org/job/isr/55/ 22:00:56 *** andythenorth has quit IRC 22:09:01 *** Alberth has left #openttdcoop.devzone 22:11:20 *** oskari89 has quit IRC 22:41:28 *** Zuu has quit IRC 22:42:14 <DevZone> Project Japanese Buildings build #1-push: FAILURE in 56 sec: https://jenkins.openttdcoop.org/job/jpbuild/1/ 22:44:26 <DevZone> Project Japanese Landscape build #1-push: FAILURE in 1 min 31 sec: https://jenkins.openttdcoop.org/job/jpland/1/ 22:46:24 <DevZone> Project Japanese Stations build #1-push: FAILURE in 1.3 sec: https://jenkins.openttdcoop.org/job/jpstations/1/