Times are UTC Toggle Colours
00:36:24 *** WormnestAndroid has joined #openttd 00:47:50 *** tokai has joined #openttd 00:47:50 *** ChanServ sets mode: +v tokai 00:54:52 *** tokai|noir has quit IRC 01:04:20 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on issue #9386: Building on MacOS fails with "bus error: 10" https://git.io/Jngox 01:59:18 *** bryjen has joined #openttd 02:41:47 *** Flygon has joined #openttd 03:01:22 *** bryjen has quit IRC 03:02:32 *** D-HUND has joined #openttd 03:05:54 *** debdog has quit IRC 03:08:20 *** tokai|noir has joined #openttd 03:08:20 *** ChanServ sets mode: +v tokai|noir 03:15:23 *** tokai has quit IRC 03:26:52 *** glx has quit IRC 03:27:57 *** WormnestAndroid has quit IRC 03:29:41 *** WormnestAndroid has joined #openttd 04:37:01 *** roadt__ has joined #openttd 04:44:02 *** roadt_ has quit IRC 05:36:05 *** Gustavo6046 has joined #openttd 05:38:32 *** Gustavo6046_ has quit IRC 06:07:38 *** snail_UES_ has quit IRC 06:13:38 *** tokai has joined #openttd 06:13:38 *** ChanServ sets mode: +v tokai 06:14:01 *** Gustavo6046 has quit IRC 06:16:00 *** Gustavo6046 has joined #openttd 06:20:24 *** tokai|noir has quit IRC 06:41:50 *** sla_ro|master has joined #openttd 07:18:36 *** D-HUND is now known as debdog 07:25:11 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9390: Fix: don't allow deleting SettingDesc to prevent compilers getting confused https://git.io/Jn6KT 07:25:14 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain closed pull request #9390: Fix: don't allow deleting SettingDesc to prevent compilers getting confused https://git.io/JnwpB 07:25:38 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #9389: Fix bf500c3: reduce the amount of vars to track by using std::string_view https://git.io/Jn6Kn 07:25:41 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain closed pull request #9389: Fix bf500c3: reduce the amount of vars to track by using std::string_view https://git.io/Jnwtg 07:30:23 *** HerzogDeXtEr has joined #openttd 07:46:08 <TrueBrain> so using std::variant bumps MacOS minimum to 10.14 only because of the exception it can throw .. as that is embedded in the libc++ dylib .. 07:46:14 <TrueBrain> that is an evil reason to bump the minimum :P 07:47:06 <TrueBrain> (well, std::visit does, but who is counting) 07:47:19 <Rubidium> discrete mathematicians? 08:00:55 <peter1138> Ju 08:01:07 <peter1138> Just drop OS X support :D 08:08:24 <TrueBrain> well, according to Steam, we would lose ...... well, almost nobody :P 08:08:26 <TrueBrain> similar with Linux btw 08:08:34 <TrueBrain> funny how Steam really is a Windows platform 08:09:18 * debdog would have dropped win support, too 08:09:49 <TrueBrain> just develop for the machines we are on 08:09:55 <TrueBrain> and if it happens to work on another, good for you 08:09:55 <TrueBrain> ? 08:09:56 <TrueBrain> :D 08:17:04 *** tokai|noir has joined #openttd 08:17:04 *** ChanServ sets mode: +v tokai|noir 08:23:33 *** tokai has quit IRC 08:34:38 <peter1138> Sounds right. 08:53:21 <Xaroth> if it compiles, ship it 08:58:37 <peter1138> Well, that is the problem... 09:00:09 <LordAro> ships keep getting lost 09:02:03 <Xaroth> build more buoys 09:03:05 <TrueBrain> build bigger canal 09:05:59 <Xaroth> Flood the entire continent 09:06:20 <Xaroth> ... I think I once again find myself on some government watchlist. 09:07:30 *** WormnestAndroid has quit IRC 09:09:39 <TrueBrain> can confirm 09:14:56 *** WormnestAndroid has joined #openttd 09:19:15 *** tokai has joined #openttd 09:19:15 *** ChanServ sets mode: +v tokai 09:26:19 *** esselfe has quit IRC 09:26:20 *** tokai|noir has quit IRC 09:27:01 *** iSoSyS has joined #openttd 09:40:14 <Timberwolf> I have an urge for coffee. 09:43:45 <Timberwolf> Also, grinding the beans gives me something to do while I wait for this Terraform run. 09:43:51 <Timberwolf> Ooh, it was unexpectedly fast. 09:44:00 <LordAro> no coffee for you! 09:45:22 <Timberwolf> I realise I also need a new build of a lambda, that uses Docker-in-Docker so I have plenty of time for coffee setup now :) 09:47:34 <Eddi|zuHause2> i herd u liek containerz, so we put some containerz in ur containerz 10:20:08 *** Samu has joined #openttd 10:37:56 *** Eddi|zuHause2 has quit IRC 10:38:35 *** Eddi|zuHause2 has joined #openttd 10:40:14 *** andythenorth has joined #openttd 10:42:22 *** tokai|noir has joined #openttd 10:42:22 *** ChanServ sets mode: +v tokai|noir 10:49:20 *** tokai has quit IRC 11:42:49 <peter1138> https://twitter.com/mikeysan/status/1406730249027473410 11:47:14 *** WormnestAndroid has quit IRC 11:47:27 *** WormnestAndroid has joined #openttd 12:10:15 *** Wuzzy has joined #openttd 12:12:41 *** glx has joined #openttd 12:12:41 *** ChanServ sets mode: +v glx 12:49:15 *** snail_UES_ has joined #openttd 13:10:22 <andythenorth> well 13:10:33 <andythenorth> goes it redesign FIRS Steeltown? 13:34:17 <andythenorth> hmm 13:39:17 <FLHerne> Well, personally I'd be much more interested in the 'better West Country' economy you've mentioned, because Steeltown isn't really my thing 13:42:45 *** nielsm has joined #openttd 13:47:36 <andythenorth> yes 13:47:53 <andythenorth> I am only poking at Steeltown because it makes literally no sense currently 13:48:12 <andythenorth> everything -> vehicles cargo -> nowhere, pointless 13:48:25 <andythenorth> it's alright for the first 50 years 13:48:28 <andythenorth> then it's FAIL 13:52:40 <Samu> I like the idea of No subsidies! 13:52:46 <Samu> makes AI vs AI a bit fairer 14:00:14 *** gelignite has joined #openttd 14:00:19 <peter1138> Is that "NoSubsidies" like "NoAI"? 14:00:31 <Samu> no, it's a new game setting 14:00:36 <peter1138> Can gamescripts set up subsidies... 14:01:02 <Samu> I think they can, but haven't tested it 14:08:54 <Samu> towns building tunnels, it's still a weird thing to see 14:48:15 <nielsm> I don't think there's a subsidies API no, you can make goals and reward the player but the multiplier to all delivery payments thing isn't possible 14:51:31 <Samu> my ai does some weird stuff with aircraft, he went from coleman count to kelling k1, totally ignored the bakewell lb-9 for some reason 15:13:19 *** namad7 has joined #openttd 15:21:22 *** roadt__ has quit IRC 15:22:26 *** roadt__ has joined #openttd 15:23:17 *** namad7 has quit IRC 15:25:45 *** WormnestAndroid has quit IRC 15:27:41 *** WormnestAndroid has joined #openttd 15:46:01 *** Progman has joined #openttd 16:00:11 <Samu> AIVehicle.GetBuildWithRefitCapacity is returning -1 for some reason 16:00:46 <Samu> need to investigate better with visual studio 16:09:27 <glx> unbuildable engine, invalid cargo, or failure to build vehicle or refit it once built 16:10:17 *** WormnestAndroid has quit IRC 16:10:55 *** WormnestAndroid has joined #openttd 16:16:35 <Samu> found the problem! 16:17:01 <Samu> it can't get a free unit number 16:17:13 <glx> too many vehicles 16:17:26 <Samu> but i'm only querying for capacity 16:17:41 <Samu> GetBuildWithRefitCapacity 16:17:45 <glx> yes but it needs to build a vehicle for that 16:18:31 <glx> as capacity may depend on a newgrf callback 16:19:48 <Samu> well, the limit is 200 aircraft, he's at the limit 16:22:36 <Samu> there's a DC_QUERY_COST flag in use and a DC_AUTOREPLACE is what it needs 16:24:24 <Samu> I was so used to 5000 max vehicles, I never triggered this bug before 16:24:25 *** sla_ro|master has quit IRC 16:24:44 <glx> DC_AUTOREPLACE is added later in case of reffiting in CmdBuildVehicle 16:26:53 <glx> maybe some reordering is needed, like setting subflags, and then testing subflags instead of flags for unit_num 16:29:45 *** Taschi has joined #openttd 16:29:45 <Samu> https://i.imgur.com/5y4Wa3p.png primary capacity = -1, making calculated income totally wrong 16:30:27 <glx> incorrect handling of -1 is your bug ;) 16:31:08 <Taschi> hello 16:31:33 <Taschi> -1 should just be banned, it is the worst number and keeps creating problems for C++ devs 16:32:08 <glx> it's standard error code 16:32:29 <Taschi> i know, that's the joke 16:34:33 *** Gustavo6046_ has joined #openttd 16:37:57 *** Progman has quit IRC 16:38:29 <glx> something like https://github.com/OpenTTD/OpenTTD/compare/master...glx22:refit maybe 16:38:50 *** Gustavo6046 has quit IRC 16:38:50 *** Gustavo6046_ is now known as Gustavo6046 16:43:31 <Samu> im testing refitted to passengers, which is the default cargo 16:44:09 <glx> then it's not a refit 16:44:41 <Samu> it isn't, but still... it should handle it 16:47:20 <Samu> subflags is DC_QUERY_COST 16:48:49 <Samu> gonna test your change, but i think it won't work 16:49:11 <glx> I think it will for non default refit 16:49:24 <glx> needs more for default cargo 16:54:38 <glx> for default cargo you can use AIEngine::GetCapacity() 16:55:53 <DorpsGek> [OpenTTD/OpenTTD] rubidium42 opened pull request #9391: Fix #9386: use variant instead of unique_ptr to prevent compilers failing on the code generation https://git.io/JnPSD 16:59:04 <glx> and of course it seems there's no way to get default mail capacity for aircraft 16:59:13 <glx> (in scripts) 17:02:21 <Taschi> i'm interested in contributing code to OpenTTD but haven't worked on any C++ project of this scale before and am a tiny bit lost 17:02:32 <Taschi> would it be possible for someone to "mentor" me a bit? 17:06:09 <glx> depends on what you want to do :) 17:06:11 <Taschi> i am a professional Java EE dev with some experience in release management and build systems so i don't need to be taught how to program, but i need help to get the "lay of the land" 17:10:14 <Taschi> for example i'm having trouble finding something that's good as a first issue to work on, i have already seen the "good first issue" tag on Github but a lot of them are a) unclear on whether they actually should be fixed/implemented or b) already have pull requests 17:11:17 <glx> usually if there's a PR it's linked in the issue 17:12:35 <Taschi> yes 17:14:19 <glx> Rubidium: of course MacOS doesn't like it :) 17:16:08 <Rubidium> glx: it doesn't like 10.14; all the warnings are in mm files 17:16:34 <glx> yes, typical when you change the minimum version, it deprecates a lot 17:16:40 <glx> and it's annoying 17:16:57 <Rubidium> that's well beyond my scope ;) 17:21:02 *** Flygon has quit IRC 17:21:16 <FLHerne> Taschi: https://github.com/OpenTTD/OpenTTD/issues/9188 looks straightforward, and somewhat uncontroversial 17:22:06 <FLHerne> (TrueBrain wasn't convinced it's useful, but I do and so does whoever reported it, and there's no harm from implementing it :p) 17:23:03 <TrueBrain> you have a fun way of reading words :P 17:23:33 <Taschi> to reproduce / test that, i'd have to run a server and then somehow cause it to crash... what would be the best way to do it? Would just killing the process via Task Manager be sufficient? 17:24:09 <FLHerne> That would work I think 17:26:10 *** Wolf01 has joined #openttd 17:30:40 <glx> Rubidium: based on the compile log, some warnings should be easy to fix, but some will require bigger changes 17:31:13 <Taschi> oh, no, i misunderstood that - the netsave is done when you're playing on a server, and your client loses the network connection 17:31:31 <Taschi> which i guess would happen when killing the server 17:31:43 <glx> yes it's easy to trigger locally :) 17:37:20 <Taschi> yeah, i've managed to trigger that 17:37:53 <DorpsGek> [OpenTTD/OpenTTD] Taschi120 commented on issue #9188: `autosave/netsave.sav` is not numbered (unlike regular `autosave/autosave#.sav`) https://git.io/J3guT 17:42:10 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick opened issue #9392: Bug Report https://git.io/JnPbi 17:42:23 <glx> oh seems we don't set https://cmake.org/cmake/help/v3.9/variable/CMAKE_OSX_DEPLOYMENT_TARGET.html at the right place, and that's probably why we have MACOSX_DEPLOYMENT_TARGET in CI env 17:43:34 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #9391: Fix #9386: use variant instead of unique_ptr to prevent compilers failing on the code generation https://git.io/JnPbd 17:44:22 <glx> Samu: you again forgot to rename the issue ;) 17:44:42 <Samu> oh, sorry 17:46:43 <Samu> i dont know how to quote multiline code 17:46:48 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on issue #9392: AIVehicle.GetBuildWithRefitCapacity returns -1 when at the max num of vehicles https://git.io/JnPbi 17:46:59 <glx> with shift 17:47:13 <glx> but you can manually edit the link too 17:48:25 <glx> Lfirst-Llast 17:59:11 *** Beer has joined #openttd 18:16:50 *** Taschi has quit IRC 18:27:06 *** sla_ro|master has joined #openttd 18:27:48 *** Gustavo6046 has quit IRC 18:29:44 *** Gustavo6046 has joined #openttd 19:12:16 <Samu> im trying flags & DC_QUERY_COST at the unit_num 19:18:05 <Samu> hmm the vehicle was not undone 19:35:25 *** tokai has joined #openttd 19:35:25 *** ChanServ sets mode: +v tokai 19:40:02 *** Gustavo6046_ has joined #openttd 19:42:15 *** tokai|noir has quit IRC 19:44:40 *** Gustavo6046 has quit IRC 19:44:40 *** Gustavo6046_ is now known as Gustavo6046 19:45:50 *** Speeder__ has quit IRC 19:52:52 *** WormnestAndroid has quit IRC 19:55:03 *** WormnestAndroid has joined #openttd 20:18:55 <Samu> nevermind, there was no vehicle to undo 20:24:02 <andythenorth> yo 20:29:56 <Samu> my commit message may need editing 20:31:48 <Samu> https://github.com/OpenTTD/OpenTTD/compare/master...SamuXarick:BuildWithRefitCapacity-max-vehicles-fix?expand=1 20:32:09 <Samu> it seems a too simple fix 20:32:38 <Samu> too good to be true 20:43:45 <DorpsGek> [OpenTTD/OpenTTD] SamuXarick opened pull request #9393: Fix #9392: Return a valid value with GetBuildWithRefitCapacity when AIs are maxed out in vehicles https://git.io/JnXWb 20:48:09 *** Gustavo6046_ has joined #openttd 20:49:07 *** WormnestAndroid has quit IRC 20:51:50 *** Gustavo6046 has quit IRC 20:52:01 *** Progman has joined #openttd 20:53:08 *** gelignite has quit IRC 20:53:33 *** Gustavo6046 has joined #openttd 20:55:39 <Samu> AI now replaced all Coleman Count with Bakewell Luckett LB-9 20:55:43 <Samu> im happy 20:57:20 *** Gustavo6046_ has quit IRC 21:00:32 *** Gustavo6046 has quit IRC 21:00:59 *** Gustavo6046 has joined #openttd 21:10:24 *** Samu has quit IRC 21:19:18 *** sla_ro|master has quit IRC 21:31:07 *** Wolf01 has quit IRC 21:34:02 *** Progman has quit IRC 21:35:41 *** WormnestAndroid has joined #openttd 21:42:42 *** Wormnest has joined #openttd 21:46:24 <DorpsGek> [OpenTTD/OpenTTD] michicc opened pull request #9394: Codechange: [OSX] Use more exact enum names where introduced with the 10.12 SDK. https://git.io/JnXum 22:10:28 *** nielsm has quit IRC 22:12:22 *** jottyfan has joined #openttd 22:16:24 *** andythenorth has quit IRC 22:19:19 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #9394: Codechange: [OSX] Use more exact enum names where introduced with the 10.12 SDK. https://git.io/JnX2n 22:30:26 *** HerzogDeXtEr has quit IRC 22:37:16 *** Wormnest has quit IRC 22:42:20 *** tokai|noir has joined #openttd 22:42:20 *** ChanServ sets mode: +v tokai|noir 22:49:25 *** tokai has quit IRC 23:41:34 <DorpsGek> [OpenTTD/OpenTTD] michicc updated pull request #9394: Codechange: [OSX] Use more exact enum names where introduced with the 10.12 SDK. https://git.io/JnXum 23:50:01 *** Wuzzy has quit IRC