Times are UTC Toggle Colours
06:49:56 *** LSky` has joined #openttdcoop.devzone 08:10:04 *** LSky` has quit IRC 09:42:25 *** LSky` has joined #openttdcoop.devzone 10:50:33 *** avdg has joined #openttdcoop.devzone 10:50:33 *** SmatZ has joined #openttdcoop.devzone 15:14:22 *** Alberth has joined #openttdcoop.devzone 15:51:21 *** frosch123 has joined #openttdcoop.devzone 15:54:11 <frosch123> is someone already rebooting devzone? 16:00:29 <frosch123> https://paste.openttdcoop.org/pvcko8qz5?/pvcko8qz5 <- implement sub parameter positions 16:00:39 <frosch123> and enhance the plural/gender validation 16:00:47 <frosch123> it now accepts all base language strings 16:01:02 <frosch123> and prints a lot of valid errors about translations, which neither wt3 or strgen detect :) 16:07:13 <Alberth> hi hi 16:07:59 <frosch123> https://paste.openttdcoop.org/pfmzadhgu <- Alberth, Rubidium: any opinion, i prefer (c) 16:11:51 <Alberth> I wonder if {CARGO_LONG}{G 0 x y} would ever happen 16:12:24 <frosch123> i would expect it already being used 16:12:25 <frosch123> :p 16:12:44 <Alberth> yeah, I agree with that 16:13:03 <frosch123> latin.txt:STR_TOWN_VIEW_CARGO_FOR_TOWNGROWTH_DELIVERED :{ORANGE}{CARGO_TINY} / {CARGO_LONG}{GREEN} (tradit{G 1 us a um i ae a}) 16:13:05 <Alberth> the proposal is for the cargo primitives + P 16:13:16 <frosch123> italian.txt:STR_TOWN_VIEW_CARGO_FOR_TOWNGROWTH_REQUIRED :{ORANGE}{CARGO_TINY} / {CARGO_LONG}{RED} (ancora richiest{G 1 o o a}) 16:13:48 <frosch123> Alberth: G works different to P, G must refer to 0 in all cases, but STRINGn 16:14:44 <Alberth> ok 16:15:32 <frosch123> G constructs the substring for a parameter position, and then scans for {G=} 16:15:45 <frosch123> P otoh just takes the parameters and casts it to int :p 16:16:16 <Alberth> hmm, too long ago :( 16:16:34 <Alberth> {P 0:1 ...} would continue to work with c ? 16:17:01 <frosch123> yes, "0" and "0:1" would work, "0:0" would give error in eints 16:17:22 <frosch123> possibly also in strgen, if someone cares :p 16:17:23 <Alberth> nobody will even figure out "0:0" :p 16:18:44 <Alberth> so, c includes a, and b isn't feasible anyway. I agree c is best, both for the unsuspecting user and as technical solution 16:29:34 <Alberth> very nice work with the patch 16:30:00 <Alberth> what happens with this extension in context of GS and NwGRFs? 16:30:25 <frosch123> it's vaild for both 16:30:31 <^Spike^> hmmm devzone dead... 16:30:45 <frosch123> ^Spike^: i already started the first part of the 30+min thingie 16:30:53 <^Spike^> ah :) 16:30:56 <^Spike^> then i'm impatient :D 16:31:14 <frosch123> i killed it 17:58 or so 16:31:22 <frosch123> 18:25 was too early :p 16:31:36 <^Spike^> hehe 16:32:15 <Alberth> :O so there is just no NewGRF / GS project that uses it :) 16:32:20 <frosch123> Alberth: except that newgrf likely do not have the CARGO_xxx codes 16:32:27 <frosch123> i am quite sure a gs uses it 16:32:45 <frosch123> sv and ncg for a start :p 16:32:55 <frosch123> but whether the translations use plural forms there, is questionable :p 16:32:55 <Alberth> I was expecting that :) 16:33:31 <Alberth> just the documentation needs extending 16:34:17 <frosch123> well, if we add the default position, we can save that work :p 16:34:36 <frosch123> old gs remain broken, noone noticed that coal is singular 16:34:39 <frosch123> while the rest is plural :p 16:35:18 <Alberth> :) 16:40:33 <^Spike^> frosch123 just run the for loop to check if all pids are gone from that container :) 16:40:37 <^Spike^> then do the restart :) 16:41:02 <frosch123> ok, i thought it actually tried to kill them 16:41:52 <^Spike^> nop :) 16:42:41 <^Spike^> it get's all pids runs vzpid on them 16:42:50 <^Spike^> and if column 2 matches 203 it prints the line 16:49:15 <DevZone> Project trams-2cc build #34-push: SUCCESS in 18 sec: https://jenkins.openttdcoop.org/job/trams-2cc/34/ 16:52:54 <DevZone> Project xussrset - Trains from Russia build #961-push: SUCCESS in 5 min 5 sec: https://jenkins.openttdcoop.org/job/xussrset/961/ 16:59:44 <frosch123> https://devs.openttd.org/~frosch/diffs/strgen_plural.diff <- strgen patch and lang fixes :/ 17:16:29 <Alberth> {P was just ignored in those cases? :) 17:17:12 <frosch123> no, it just used it anyway 17:17:21 <frosch123> and used the stringid or cargoid or whatever for the plural 17:17:36 <Alberth> haha :) 17:18:59 <frosch123> btw. i am not making strgen as good as eints, because then i would have to fix 50+ strings :p 17:20:43 <Alberth> yeah, I wonder if you should fix strings other than trivially correct ones 17:21:48 <Alberth> and let eints and time do its job :) 17:26:56 <Alberth> patch looks fine, but I can't really judge the approach itself 18:28:47 <frosch123> Error: {G=..} may only be used for the default string (that is, without case extension) <- i am still unsure about that one 18:29:02 <frosch123> openttd only uses {G=} from the default case 18:29:10 <frosch123> but is it an error if translators specify it for more? 18:32:00 <frosch123> almost all remaining "translation bug" are of that type 18:54:41 <DevZone> Project World Airliner Set build #1038-push: FAILURE in 39 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1038/ 18:59:54 <DevZone> Project World Airliner Set build #1039-push: STILL FAILING in 29 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1039/ 19:03:22 <DevZone> Project World Airliner Set build #1040-push: STILL FAILING in 30 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1040/ 19:05:32 <DevZone> Project World Airliner Set build #1041-push: STILL FAILING in 31 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1041/ 19:09:18 <DevZone> Project World Airliner Set build #1042-push: STILL FAILING in 30 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1042/ 19:10:34 <DevZone> Project eints build #33-push: SUCCESS in 17 sec: https://jenkins.openttdcoop.org/job/eints/33/ 19:12:24 <DevZone> Project World Airliner Set build #1043-push: STILL FAILING in 15 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1043/ 19:14:48 <DevZone> Project World Airliner Set build #1044-push: STILL FAILING in 30 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1044/ 19:16:29 <DevZone> Project World Airliner Set build #1045-push: STILL FAILING in 30 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1045/ 19:22:16 <DevZone> Yippee, build fixed! 19:22:17 <DevZone> Project World Airliner Set build #1046-push: FIXED in 3 min 39 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1046/ 19:54:48 <Alberth> frosch123: If {G= } is the same everywhere, it shouldn't be a problem, but you get problems of consistency then ("this G= is different from the default translation, it must be the same everywhere"), which is somewhat silly. Not allowing it is mostly the same as the sane one ends up with eventually 19:55:43 <Alberth> s/sane one/sabe solution/ 19:55:47 <Alberth> *sane 19:56:06 <frosch123> good point :) 20:03:25 <DevZone> Project World Airliner Set build #1047-push: SUCCESS in 3 min 36 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1047/ 20:07:00 <DevZone> Project World Airliner Set build #1048-push: SUCCESS in 3 min 34 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1048/ 20:10:31 <DevZone> Project World Airliner Set build #1049-push: SUCCESS in 3 min 30 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1049/ 20:14:05 <DevZone> Project World Airliner Set build #1050-push: SUCCESS in 3 min 34 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1050/ 20:17:37 <DevZone> Project World Airliner Set build #1051-push: SUCCESS in 3 min 31 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1051/ 20:23:12 *** Alberth has left #openttdcoop.devzone 20:41:29 <DevZone> Project World Airliner Set build #1052-push: SUCCESS in 3 min 53 sec: https://jenkins.openttdcoop.org/job/worldairlinersset/1052/ 20:53:58 *** frosch123 has quit IRC 21:57:46 <DevZone> Project xussrset - Trains from Russia build #962-push: SUCCESS in 4 min 4 sec: https://jenkins.openttdcoop.org/job/xussrset/962/ 23:16:12 *** LSky` has quit IRC