IRC logs for #openttd on OFTC at 2020-06-05
⏴ go to previous day
00:00:50 <supermop_Home_> well all my screen filler had congealed so i couldn't print a t shirt
00:01:10 <supermop_Home_> and ended up going to a protest in an aloha shirt
02:32:28 *** innocenat_ has joined #openttd
03:21:49 *** gelignite_ has joined #openttd
03:23:38 *** blathijs has joined #openttd
04:18:16 *** Wormnest has joined #openttd
04:44:53 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz opened pull request #8187: Cleanup: Fix typo in settings.h.preamble ... https://git.io/JfXhQ
04:54:27 *** blathijs has joined #openttd
06:07:50 *** blathijs has joined #openttd
06:30:25 *** WormnestAndroid has quit IRC
06:32:13 *** WormnestAndroid has joined #openttd
07:50:45 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro approved pull request #8187: Cleanup: Fix typo in settings.h.preamble ... https://git.io/Jf1JX
08:02:13 *** sla_ro|master has joined #openttd
08:11:13 <DorpsGek_III> [OpenTTD/OpenTTD] andythenorth commented on pull request #8186: Fix: unbreak building with ICU on macOS https://git.io/Jf1UG
08:12:42 *** andythenorth has joined #openttd
08:22:23 <supermop_Home_> hi andythenorth
08:24:03 <andythenorth> Horse finished yet? :)
08:24:26 <supermop_Home_> I've played with the trailers
08:24:38 <supermop_Home_> haven't noticed a bug yet
08:24:49 <andythenorth> they're a little bit overkill
08:25:10 <supermop_Home_> tbh I've been modelling banquette seating for the past 6 hours
08:25:24 <supermop_Home_> didn't spend too much time with them
08:25:27 <andythenorth> there's a grf for that? o_O
08:36:34 <Eddi|zuHause> i can't find mouse sensitivity in these settings...
08:38:43 <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh merged pull request #8187: Cleanup: Fix typo in settings.h.preamble ... https://git.io/JfXhQ
08:39:56 <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh approved pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/Jf1Ts
08:41:35 <Eddi|zuHause> now that path of exile got a vulkan renderer and i don't get killed by microstutter on every boss fight, i notice how truely horrible i am at that game
10:24:07 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8187: Cleanup: Fix typo in settings.h.preamble ... https://git.io/Jf1LN
10:58:42 *** arikover has joined #openttd
11:00:01 *** andythenorth has joined #openttd
11:28:06 *** Sacro_ is now known as Sacro
11:41:06 *** gelignite has joined #openttd
11:57:22 *** WormnestAndroid has quit IRC
11:57:35 *** WormnestAndroid has joined #openttd
12:09:55 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/Jf13G
12:13:48 <DorpsGek_III> [OpenTTD/OpenTTD] nielsmh approved pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/Jf13l
12:14:20 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/Jf134
12:16:33 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/Jf13g
14:35:12 <DorpsGek_III> [OpenTTD/OpenTTD] FLHerne commented on pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/Jf1cO
14:36:07 <glx> andythenorth: #156 is fixed (was a stupid mistake I made)
14:43:08 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro commented on pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/Jf1ca
14:44:58 *** Gustavo6046 has joined #openttd
14:45:27 <andythenorth> ok var 61 checks reduced by factor of 16
14:45:41 <glx> now both versions should have the same result
14:45:51 <andythenorth> 1778 checks per vehicle in consist now
14:46:08 <andythenorth> I can cut that down by factor 7 I think
14:47:24 * andythenorth wonders how vehicles cache frequently accessed results
14:47:40 <glx> for now you always do the comparisons even if the var61 result is "invalid" I think
14:48:15 <andythenorth> yes, I could optimise quite a lot of that out
14:48:31 <andythenorth> even for weird players, most consists will be < 16 of these vehicles I reckon
14:48:40 <andythenorth> but I check 127 both ways just in case
14:49:17 <andythenorth> also the result gets % 6 called on it elsewhere, so maybe I only need to check 6 each way
14:49:39 <andythenorth> it's a nice test case in NOT extending the newgrf spec :)
14:55:44 <Eddi|zuHause> andythenorth: some 40+ vars might be cached, but everything else isn't
14:55:47 <andythenorth> it should actually return once it's compared one invalid result
15:23:50 *** virtualrandomnumber has joined #openttd
15:27:13 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro merged pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/JfXlC
15:27:47 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro commented on pull request #8186: Fix: unbreak building with ICU on macOS https://git.io/Jf1W0
15:28:14 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro commented on pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/Jf1Wz
15:31:08 <DorpsGek_III> [OpenTTD/OpenTTD] danchr dismissed a review for pull request #8186: Fix: unbreak building with ICU on macOS https://git.io/Jf1J1
16:51:53 <_dp_> great, running 1.10.2 for just a few hours and got first desync already...
16:52:07 <_dp_> doesn't seem to be subsidy related though
16:53:15 *** virtualrandomnumber has quit IRC
16:59:16 *** Wormnest has joined #openttd
18:27:00 *** Hobbyboy has joined #openttd
18:27:49 *** gelignite has joined #openttd
18:38:00 <Samu> my poe char finally died, so i'm back here
18:43:44 *** WormnestAndroid has quit IRC
18:43:57 *** WormnestAndroid has joined #openttd
18:55:30 <andythenorth> is it Friday now?
18:55:37 <andythenorth> where is TrueBrain to advise me on life?
18:58:04 <LordAro> andythenorth: is it alcohol time?
18:58:13 <andythenorth> we have Friday zoom beers
18:58:21 <andythenorth> rounds off the week
19:03:56 <happpy> hi question the mac problem mite be fixs if the so if that 10.12 was 10.15 then the problem be fixs
19:05:13 <happpy> its say 10.12 for mac if that's was 10.15 problem be fixs
19:06:37 <LordAro> upgrading to 10.15 will fix the issue, yes
19:07:09 <LordAro> unless you're referring to what the download provides
19:07:19 <LordAro> that's a separate thing
19:10:06 <happpy> is there a way to tel people on openttd website to change or update that 10.12 to 10.15
19:13:28 <happpy> or if someone got a mac and update the software on the mac word that fixs it
19:14:17 <glx> it's not always possible to upgrade
19:15:26 <happpy> so the only fixs is 10.12 to 10.15
19:15:37 <LordAro> happpy: 10.13 or newer should be fine
19:16:01 <LordAro> glx: niels suggested(?) that anyone with 10.12 should be able to upgrade to at least 10.13#
19:16:35 <happpy> and how do there do that
19:16:47 <glx> hardware wise it should be ok I think, but I guess they may have software issues
19:17:08 <LordAro> happpy: probably by googling "upgrade mac os"
19:17:23 <glx> and IIRC the software case is worse when going from 10.14 to 10.15
19:17:45 <LordAro> i know several people who stuck with 10.14
19:17:57 <andythenorth> you can pretty much guarantee for macos 10.x+1 is worse than 10.x
19:18:07 <andythenorth> it's a business based on selling emojis these days
19:18:32 <andythenorth> it tends towards 'about the same as the other platforms', at a price premium
19:19:24 <LordAro> andythenorth: we're trying to help here, not confuse :p
19:20:59 <andythenorth> I am just rambling
19:21:17 * andythenorth is a very satisfied mac user, can you tell?
19:22:11 <happpy> i don't have a mac i used windows but i just trying to think on how to fix the mac problem
19:23:06 <LordAro> i wouldn't worry about problems you're not experiencing :p
19:23:19 <happpy> its sound to me the mac software need update or the update for openttd for mac only
19:23:46 <happpy> i just try to help out a player
19:24:49 <LordAro> right now, the best thing to tell people experiencing this issue is to look into upgrading their own system
19:24:55 <LordAro> currently, OTTD has no solution
19:25:05 * andythenorth closes all work tabs until Monday
19:25:21 <andythenorth> that's jinxed me, probably get a big outage overnight now :P
19:25:35 <LordAro> i'm sure they weren't important
19:25:52 <happpy> will thsnks for the help have a nice day
19:28:26 <andythenorth> eh I should test that :)
19:29:30 <glx> sooo should I press the green button on #7270 ?
19:29:47 <LordAro> oh yes, i was going to look through it
19:30:17 <LordAro> might want to consider a dev blog post?
19:31:26 <LordAro> glx: have you tested the release pipeline?
19:32:08 <glx> not personally but it used to work and I don't think I touched anything in this area
19:35:08 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro approved pull request #7270: Introduce CMake (and removing all other project-related code) https://git.io/Jf1ge
19:35:12 *** virtualrandomnumber has joined #openttd
19:36:09 <DorpsGek_III> [OpenTTD/OpenTTD] glx22 closed issue #6884: Installer reports success even if not all files were copied https://git.io/Jf1gv
19:36:12 <DorpsGek_III> [OpenTTD/OpenTTD] glx22 merged pull request #7270: Introduce CMake (and removing all other project-related code) https://git.io/fhbqc
19:36:39 <glx> nighly will be a good test of release
19:36:40 <LordAro> glx: #8089 can be closed as well
19:38:14 <andythenorth> they are likely 4400hp each
19:38:21 <andythenorth> that's a lot of power
19:38:25 <DorpsGek_III> [OpenTTD/OpenTTD] glx22 commented on issue #8089: Byte order CPP definition not defined on FreeBSD https://git.io/JffEs
19:40:17 <glx> oh #8182 can be closed too
19:42:04 <glx> ok now let's update my cmake_api branch
19:46:54 <TrueBrain> <andythenorth> where is TrueBrain to advise me on life? <- which live? :D I need beer ...
19:47:12 <andythenorth> now I need sleep
19:47:38 <TrueBrain> I wanted to "just lay down for a bit"... yeah, lets not try that again
19:51:10 <TrueBrain> means I can close those PRs from my fork
19:54:27 <glx> TrueBrain: #11 is already included
19:54:39 <TrueBrain> glx: I just closed them all with the same message :P
20:03:08 *** Gustavo6046 has joined #openttd
20:14:51 <DorpsGek_III> [OpenTTD/OpenTTD] danchr opened pull request #8189: Update: generate Mac bundle on install rather than packaging https://git.io/Jf12c
20:16:18 <DorpsGek_III> [OpenTTD/OpenTTD] danchr updated pull request #8189: Update: generate Mac bundle on install rather than packaging https://git.io/Jf12c
20:17:21 <Wolf01> V453000: the new mining drill looks gorgeous
20:18:47 <DorpsGek_III> [OpenTTD/OpenTTD] andythenorth commented on pull request #8189: Update: generate Mac bundle on install rather than packaging https://git.io/Jf12E
20:22:14 <FLHerne> (apparently it's barely playable, because it's half-finished and assumes you're a process chemist)
20:27:31 *** Wolf01 is now known as Guest82
20:33:00 <TrueBrain> welcome to 20 minutes ago :)
20:33:18 <LordAro> my connection cut out, didn't read back all the way
20:33:58 <Eddi|zuHause> hey, i just learned news: the nightly failed :p
20:34:12 <TrueBrain> you ... had to wait 20 minutes Eddi|zuHause
20:37:14 <LordAro> looks like one of the variables has a newline in it?
20:37:50 <LordAro> or possibly a carriage return
20:37:55 <LordAro> that would explain why only windows fails
20:38:14 <LordAro> hrm, except mac has failed in the same way
20:40:30 *** Progman has joined #openttd
20:46:59 <LordAro> nope, can't see a way that cmake could've generated "EV_I\n as the revision year
20:47:04 <LordAro> andythenorth: run cmake build pls
20:49:16 <DorpsGek_III> [OpenTTD/OpenTTD] danchr commented on pull request #8189: Update: generate Mac bundle on install rather than packaging https://git.io/Jf1aO
20:55:27 <TrueBrain> I am guessing COMMITDATE is detected wrong
20:55:32 <TrueBrain> in result REV_YEAR gets 4 random chars
20:55:39 <TrueBrain> but yeah, needs a debugger :D
20:57:13 <andythenorth> cmake is just master now?
20:57:57 <LordAro> interestingly, running `make` on mingw now starts a cmd
20:58:25 *** otetede has joined #openttd
20:59:22 <DorpsGek_III> [OpenTTD/OpenTTD] andythenorth commented on pull request #8189: Update: generate Mac bundle on install rather than packaging https://git.io/Jf1az
20:59:48 <andythenorth> is there are any ./configure step?
21:00:25 <LordAro> has the documentation not been updated? :p
21:00:51 <LordAro> `mkdir build && cd build && cmake .. && make`
21:01:10 <EER> Hi guys, I saw the mac crash issue for 10.12, right now I have "borrowed" an old mac running 10.11 from my office, which I could try and update and investigate the crash issue over the weekend. But I read on the coop discord that perhaps it's time to just drop support for < 10.13? In that case I don't want to waste too much of my time, especially since mac is not my native platform anyway
21:01:10 <EER> ;-) (I am the same person that happpy may have talked about earlier tonight).
21:02:01 <andythenorth> cmake more colourful eh
21:02:14 <LordAro> EER: we'd like to fix it, but none of us have a Mac old enough
21:02:29 <LordAro> so any help would be quite welcome
21:02:36 <EER> Ok in that case I'll see if I can upgrade it to 10.12 and set up a build environment
21:02:43 <LordAro> i imagine you don't need to upgrade it
21:02:51 <LordAro> i imagine the problem is evident on 10.11 as well
21:02:59 <EER> It also crashes on 10.11, but I am not sure if it is the same issue
21:03:04 <andythenorth> right, is there a paste of the expected cmake failures?
21:03:14 <andythenorth> but I didn't see :P
21:03:15 <LordAro> andythenorth: only when you make one!
21:03:18 <EER> especially since the mac build on the website says 10.12+
21:03:25 <EER> so I wasn't sure if there is some dependency
21:03:36 <LordAro> EER: not as far as far as i know
21:03:52 <LordAro> i'm not sure if the builds we provide are actually only 10.12+
21:04:10 <EER> cool, in that case, I'll get on that :)
21:06:28 <LordAro> that's a big ol' list of errors
21:06:55 <LordAro> ...and it's coming from your system headers
21:07:04 <LordAro> andythenorth: best make a new issue
21:08:41 <LordAro> andythenorth: preferably with `make VERBOSE=1`
21:09:27 <andythenorth> I'll paste verbose in a bit, so hungry
21:10:27 <LordAro> i'm being slackish, for no real reason
21:10:30 <LordAro> i don't even use slack
21:21:30 <glx> nightly failure is weird, because REV_YEAR is a substring of REV_ISODATE, and CI works
21:21:46 <glx> hmm EV_I looks like a substring
21:23:05 <otetede> hello for the first time
21:24:18 <glx> const char _openttd_revision_year[] = "EV_I"; <-- that's what I have locally too, how could the CI not detect that ?
21:25:18 <LordAro> glx: release appears to have a newline on the end too
21:25:46 <glx> anyway there are 2 issues :)
21:25:52 <LordAro> oh yes, i also have EV_I
21:26:10 <glx> wrong info, and newline in .ottrev
21:26:44 <LordAro> string(SUBSTRING REV_ISODATE 1 4 REV_YEAR)
21:27:17 <glx> yeah sometimes you can just pass a variable name, sometimes not
21:27:30 <glx> cmake is not always friendly
21:29:45 <glx> but it's my fault, I should have checked better
21:34:50 <andythenorth> can't make progress without breaking eggs
21:44:29 <DorpsGek_III> [OpenTTD/OpenTTD] glx22 opened pull request #8191: Fix: Incorrect REV_YEAR, and trailing \n in .ottdrev https://git.io/Jf1V4
21:46:05 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro commented on pull request #8191: Fix: Incorrect REV_YEAR, and trailing \n in .ottdrev https://git.io/Jf1VE
21:54:56 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro commented on pull request #8189: Update: generate Mac bundle on install rather than packaging https://git.io/Jf1V5
21:58:35 <andythenorth> do I need to do more on this cmake bug?
21:58:46 <andythenorth> otherwise I try the procedure params :P
21:58:54 <andythenorth> I only have one brain so one thing at once
22:00:37 <DorpsGek_III> [OpenTTD/OpenTTD] danchr commented on pull request #8189: Update: generate Mac bundle on install rather than packaging https://git.io/Jf1Vj
22:00:43 <LordAro> andythenorth: i don't think so (yet)
22:07:22 <andythenorth> glx any limit to number of procedure params ?
22:07:34 * andythenorth read the regressions as docs
22:08:13 <glx> the only limit is the available registers I think
22:08:38 <glx> it's using the same mechanism as spritelayout params
22:10:23 <andythenorth> presumably params are not optional?
22:12:22 <glx> but if you want to pass a value to all the subcalls it's better to set a temporary yourself
22:14:51 <andythenorth> I'm just using it to stuff a temporary once
22:14:59 <andythenorth> it's quite a nice use case, simplifies vehicle switches
22:15:17 <andythenorth> procedures make nml so much better
22:16:00 *** keenriser has joined #openttd
22:20:50 <DorpsGek_III> [OpenTTD/OpenTTD] glx22 updated pull request #8191: Fix: Incorrect REV_YEAR, and trailing \n in .ottdrev https://git.io/Jf1V4
22:25:20 <DorpsGek_III> [OpenTTD/OpenTTD] LordAro approved pull request #8191: Fix: Incorrect REV_YEAR, and trailing \n in .ottdrev https://git.io/Jf1wK
22:32:10 <DorpsGek_III> [OpenTTD/OpenTTD] glx22 merged pull request #8191: Fix: Incorrect REV_YEAR, and trailing \n in .ottdrev https://git.io/Jf1V4
22:36:40 <andythenorth> oh that's interesting :D
22:36:53 <andythenorth> I've crashed the graphics chain :)
22:37:17 <glx> returned somthing not expected ?
22:40:26 <andythenorth> failed to chain a switch :)
22:52:32 <glx> LordAro: progress, windows nightly builds fine, macos builds but fail at bundle step
22:55:35 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8184: Cleanup: More code comment and doxygen fixes. https://git.io/Jf1rW
23:03:49 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8185: Cleanup: StationCargoList::AreMergable doxygen comment references Veh… https://git.io/Jf1ra
23:09:22 <glx> andythenorth: updated your comment in #8190, it was unreadable ;)
23:23:35 <DorpsGek_III> [OpenTTD/nml] andythenorth commented on pull request #155: Add: Support for parameters in switches https://git.io/Jf1rN
23:27:06 <DorpsGek_III> [OpenTTD/OpenTTD] glx22 commented on pull request #8189: Update: generate Mac bundle on install rather than packaging https://git.io/Jf1rp
23:31:15 <andythenorth> hmm, I still need to reduce var 61 use by factor of 7 :P
23:31:24 <andythenorth> by storing a result
continue to next day ⏵
