IRC logs for #openttd on OFTC at 2023-07-08
            
00:39:03 <DorpsGek> [OpenTTD/OpenTTD] glx22 updated pull request #11109: Add: [Emscripten] support for bootstrapping https://github.com/OpenTTD/OpenTTD/pull/11109
02:21:12 *** D-HUND has joined #openttd
02:24:35 *** debdog has quit IRC (Ping timeout: 480 seconds)
02:40:52 *** Flygon has quit IRC (Quit: A toaster's basically a soldering iron designed to toast bread)
02:43:27 *** Wormnest has quit IRC (Quit: Leaving)
03:40:30 *** keikoz has joined #openttd
04:16:56 *** tokai has joined #openttd
04:16:56 *** ChanServ sets mode: +v tokai
04:23:38 *** tokai|noir has quit IRC (Ping timeout: 480 seconds)
05:25:36 *** nielsm has joined #openttd
06:06:58 *** HerzogDeXtEr has joined #openttd
09:36:24 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain updated pull request #11116: Change: [CI] rework preview flow and use Cloudflare Pages to publish https://github.com/OpenTTD/OpenTTD/pull/11116
09:59:20 <truebrain> right, #11116 is ready for review πŸ™‚ Once merged, I will flip the backend to serve the new previews πŸ™‚
10:01:57 <truebrain> previews use some light caching, max of 60s, to help with the reloads of the upcoming PR of glx πŸ™‚
10:02:47 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #11109: Add: [Emscripten] support for bootstrapping https://github.com/OpenTTD/OpenTTD/pull/11109#pullrequestreview-1520682922
10:23:57 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #11109: Add: [Emscripten] support for bootstrapping https://github.com/OpenTTD/OpenTTD/pull/11109#pullrequestreview-1520688897
10:24:43 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #11116: Change: [CI] rework preview flow and use Cloudflare Pages to publish https://github.com/OpenTTD/OpenTTD/pull/11116#pullrequestreview-1520688899
10:26:00 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #11116: Change: [CI] rework preview flow and use Cloudflare Pages to publish https://github.com/OpenTTD/OpenTTD/pull/11116#pullrequestreview-1520689115
10:29:43 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain commented on pull request #11109: Add: [Emscripten] support for bootstrapping https://github.com/OpenTTD/OpenTTD/pull/11109#pullrequestreview-1520689432
10:29:53 <truebrain> wauw, the `isCancelled` part in the content download is ... thin ..
10:29:56 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #11116: Change: [CI] rework preview flow and use Cloudflare Pages to publish https://github.com/OpenTTD/OpenTTD/pull/11116#pullrequestreview-1520689453
10:30:13 <truebrain> it is really weird to use the disconnect of the TCP as indicator you shouldn't retry the fallback πŸ˜›
10:30:17 <truebrain> tnx _glx_
10:30:31 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #11116: Change: [CI] rework preview flow and use Cloudflare Pages to publish https://github.com/OpenTTD/OpenTTD/pull/11116
10:37:24 <truebrain> let's test with your PR if I can restore most previews without them being rebased .. it should work, as long as they are recent enough (read: have the nlohmann-json patch in their codebase)
10:38:23 <truebrain> still can't believe more than 1/3rd of the open PRs has a preview label .. clearly people like this πŸ˜›
10:55:51 <truebrain> okay, #11109 now has a working preview .. what to do about the rest, hmm
10:59:28 <truebrain> oops ... previews cancel other previews
11:06:55 <truebrain> so in a concurrency tag you cannot use the `github.event` .. that is not nice
11:20:41 <truebrain> everyone seems to think it should just work .. but it really is empty πŸ˜›
11:25:07 <truebrain> it fails only when put in a reusing workflow .. haha, lol
11:25:09 <truebrain> found a bug! πŸ˜›
11:26:27 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain opened pull request #11121: Fix: [CI] preview for one PR could cancel the preview of another https://github.com/OpenTTD/OpenTTD/pull/11121
11:28:34 <DorpsGek> [OpenTTD/OpenTTD] glx22 approved pull request #11121: Fix: [CI] preview for one PR could cancel the preview of another https://github.com/OpenTTD/OpenTTD/pull/11121#pullrequestreview-1520694827
11:32:29 <truebrain> right, removed a bunch of preview labels, as they couldn't build a preview anymore .. the rest I will try to rebuild πŸ™‚
11:32:37 <truebrain> once PRs rebase, we can slap the preview label on it again πŸ™‚
11:33:59 <Eddi|zuHause> you mean i need to revive diagonal crossings?
11:35:29 <truebrain> now I get to click 100+ times to remove all the preview envs we had ... not sure what I was thinking, giving them all their own env ..
11:41:02 <talltyler> Yay to diagonal crossings
11:42:08 <emperorjake> It's been done before after all
11:43:03 <truebrain> 192 clicks later ... pffff
11:43:30 *** D-HUND is now known as debdog
12:02:18 <andythenorth> πŸ’―πŸ’―
12:10:18 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain merged pull request #11121: Fix: [CI] preview for one PR could cancel the preview of another https://github.com/OpenTTD/OpenTTD/pull/11121
12:15:38 <truebrain> okay, all remaining previews are building; another thing migrated \o/ w00p
12:15:52 <truebrain> all that is left, are some silly redirect domains .. PARTY!! πŸ˜„
12:17:42 <andythenorth> \o/
12:27:04 *** Extrems` has joined #openttd
12:31:41 *** Extrems has quit IRC (Ping timeout: 480 seconds)
12:31:41 *** Extrems` is now known as Extrems
12:32:19 <_glx_> hmm and now I need a new image
13:08:14 *** tokai has quit IRC (Quit: c('~' )o)
13:18:36 *** tokai has joined #openttd
13:18:36 *** ChanServ sets mode: +v tokai
13:26:54 *** gelignite has joined #openttd
13:32:28 <DorpsGek> [OpenTTD/OpenTTD] glx22 commented on pull request #11109: Add: [Emscripten] support for bootstrapping https://github.com/OpenTTD/OpenTTD/pull/11109#pullrequestreview-1520706857
13:33:18 <truebrain> oooeeehhh, you found a bug hiding in the crowd? πŸ˜„
13:34:32 <_glx_> at least it probably explains the crash with the invalid url
13:35:49 <_glx_> I think it's because we don't check `success` parameter in OnConnect()
13:36:20 <_glx_> but as said I need to confirm in debugger
13:40:02 <_glx_> https://gist.github.com/glx22/2db3d27783e0f5c45485c35076429f5f <-- it's a "vector erase iterator outside range" assert
13:40:40 <truebrain> owh, so not even something I did! πŸ˜›
13:40:48 <truebrain> one of those recent C++-ify things I guess πŸ˜› πŸ˜„
13:41:55 <_glx_> anyway checking success is a good idea, as connection failure here means no download is possible
13:48:09 <truebrain> oof .. I need to look back into grfsearch .. I always forget that syntax .. you can trigger it from in-game I think .. let's see
13:50:54 <truebrain> which requires finding a server that has GRFs that are not on BaNaNaS
13:50:55 <truebrain> tricky πŸ˜„
13:53:28 <Eddi|zuHause> i made a big mistake.
13:54:00 <Eddi|zuHause> i took a bar of chocolate from the kitchen, ate a few pieces and put the rest on my desk
13:54:06 <Eddi|zuHause> and now that rest is liquid
13:55:09 <talltyler> Oh no 😦
14:02:29 <truebrain> lol, I forgot we had a complete read-only copy of Flyspray still running πŸ˜„
14:02:35 <truebrain> as in, it is just flat HTML
14:02:47 <truebrain> but I thought they were redirects to github
14:03:02 <LordAro> probably could be at this point
14:03:10 <truebrain> I think so too
14:03:12 <truebrain> has been 4 years
14:03:18 <truebrain> 5 even
14:03:28 <truebrain> 2018-04-07
14:03:38 <LordAro> :o
14:05:17 <truebrain> yes, you are getting old
14:07:12 *** D-HUND has joined #openttd
14:09:20 <truebrain> okay, old flyspray copy is gone .. they now point to GitHub instead
14:10:52 *** Flygon has joined #openttd
14:11:28 <truebrain> okay ... that removes the last of DNS entries to AWS
14:11:38 <truebrain> everything now either routes via Cloudflare or via our Nomad cluster (which is hosted on AWS)
14:11:42 <truebrain> but no more AWS services directly
14:11:49 <DorpsGek> [OpenTTD/OpenTTD] glx22 opened pull request #11122: Fix: [Bootstrap] don't crash when failing to connect to content server https://github.com/OpenTTD/OpenTTD/pull/11122
14:12:17 <DorpsGek> [OpenTTD/OpenTTD] TrueBrain approved pull request #11122: Fix: [Bootstrap] don't crash when failing to connect to content server https://github.com/OpenTTD/OpenTTD/pull/11122#pullrequestreview-1520711840
14:13:00 <truebrain> lot of cleaning up to do on the AWS side, but .. that is less important πŸ™‚
14:17:47 <_glx_> on pc it's simple to cause failures, but for emscripten testing that implies a rebuild
14:25:15 <truebrain> manipulate your hosts file might help there
14:25:21 <truebrain> or your firewall
14:25:33 <_glx_> I can block Ips
14:25:59 <truebrain> bit few IPs to block, but that works .. but easier to alter your hosts file in that case πŸ™‚
14:28:06 <_glx_> just a pain to remember where this file is on windows πŸ™‚
14:34:19 <truebrain> yes .. system32 ... drivers ... somewhere πŸ˜›
14:34:25 <truebrain> it auto-types it for me these days πŸ˜„
14:35:31 <Eddi|zuHause> is that still where in system32 there's the 64bit files, and in syswow64 there's the 32bit files?
14:50:40 <_glx_> etc/hosts in system32/drivers yes
14:54:43 <DorpsGek> [OpenTTD/OpenTTD] glx22 merged pull request #11122: Fix: [Bootstrap] don't crash when failing to connect to content server https://github.com/OpenTTD/OpenTTD/pull/11122
15:21:46 <ahyangyi> Eddi|zuHause: Probably
15:22:07 <ahyangyi> syswow64 is "windows(32) on windows 64", so it's always for 32 bit stuff
15:22:26 <ahyangyi> the name is... so wow
15:34:07 *** Wormnest has joined #openttd
16:11:32 <DorpsGek> [OpenTTD/website] TrueBrain opened pull request #267: Add: post about the most recent migration we just finished https://github.com/OpenTTD/website/pull/267
16:11:36 <truebrain> LordAro: you asked for this! Not me! πŸ˜›
16:12:56 <truebrain> it really is a rough draft, to be clear; I wrote it 3 times in the description, but people tend to not read πŸ˜›
16:13:13 <truebrain> and I see I need to fix some HTML styling too; this looks odd πŸ˜›
16:30:55 *** gelignite has quit IRC (Quit: Stay safe!)
16:37:16 <LordAro> :)
17:27:06 *** gelignite has joined #openttd
18:25:40 <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267
18:29:28 <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267
18:29:34 <truebrain> okay, that should be a lot easier to read now .. ready for review πŸ˜„
18:33:27 <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267
18:35:10 <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267
18:35:23 <truebrain> typos typos typos ... well, now I leave the typo-finding to others πŸ˜„
18:36:46 <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267
18:36:49 <truebrain> okay, one more, wrong link for Nomad .. fixed now πŸ˜„
18:37:25 <truebrain> https://pr-267.openttd-website-2fx.pages.dev/news/2023/07/08/our-infrastructure
18:37:25 <truebrain> https://pr-267.openttd-website-2fx.pages.dev/news/2023/07/09/infra-migration
18:37:25 <truebrain> For those that like reading the result more than the source πŸ™‚
18:39:49 <DorpsGek> [OpenTTD/OpenTTD] eints-sync[bot] pushed 1 commits to master https://github.com/OpenTTD/OpenTTD/commit/a54c28f2c3f720831a58cfcfc5d0d034efe91b22
18:39:50 <DorpsGek> - Update: Translations from eints (by translators)
18:44:32 <ahyangyi> Wow, the demolish visual effect looks impressive in OpenGFX2!
18:45:05 <ahyangyi> https://cdn.discordapp.com/attachments/1008473233844097104/1127309399111762000/image.png
19:02:07 <DorpsGek> [OpenTTD/website] James103 commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520765031
19:17:04 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
19:28:12 <DorpsGek> [OpenTTD/OpenTTD] niklasva commented on issue #10125: Allow user to change location of OpenTTD data directory from ~/Documents https://github.com/OpenTTD/OpenTTD/issues/10125
19:40:19 <DorpsGek> [OpenTTD/website] stormcone commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520774539
19:44:50 <DorpsGek> [OpenTTD/website] TrueBrain commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520775048
19:48:05 <DorpsGek> [OpenTTD/website] TrueBrain commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520775294
19:49:09 <DorpsGek> [OpenTTD/website] TrueBrain commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520775366
19:52:47 <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267
19:52:50 <DorpsGek> [OpenTTD/website] TrueBrain commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#issuecomment-1627477298
20:08:39 *** Wormnest has joined #openttd
20:12:56 <DorpsGek> [OpenTTD/website] stormcone commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520774969
20:21:41 *** keikoz has quit IRC ()
20:28:47 *** keikoz has joined #openttd
21:01:16 *** Wormnest has quit IRC (Ping timeout: 480 seconds)
21:07:52 *** keikoz has quit IRC (Ping timeout: 480 seconds)
21:18:29 *** Wormnest has joined #openttd
21:21:53 <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267
21:37:54 *** nielsm has quit IRC (Ping timeout: 480 seconds)
21:47:06 *** gelignite has quit IRC (Read error: Connection reset by peer)
21:47:17 *** gelignite has joined #openttd
22:01:01 <DorpsGek> [OpenTTD/website] James103 commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520816465
22:02:55 <andythenorth> https://cdn.discordapp.com/attachments/1008473233844097104/1127359187295404032/image.png
22:02:55 <andythenorth> goes it delete subtypes?
22:03:01 <andythenorth> what about variants for this? πŸ˜›
22:04:57 <DorpsGek> [OpenTTD/website] TrueBrain commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520819838
22:06:48 <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267
22:14:05 <talltyler> Yes, boat variants!
22:26:35 <kamnet> TrueBrainviaGitHub: This was a fine read. A group I'm a part of was recently discussing their individual experiences with AWS and ponderings about moving away from it. I think they will find this information worth reading.
22:28:38 <truebrain> tnx πŸ™‚
22:28:46 <truebrain> now if someone can approve it .. like LordAro or something ... πŸ˜„ πŸ˜›
22:29:32 <talltyler> I have a pending comment or two, working on a review
22:29:52 <talltyler> I think I have website approval rights once I’m done reading though πŸ™‚
22:29:54 <truebrain> sweet!
22:54:02 <DorpsGek> [OpenTTD/website] 2TallTyler commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520820701
22:56:23 <truebrain> I appreciate you let me have my voice πŸ™‚
22:58:54 <talltyler> You have a distinctive writing voice. I laughed when you described how Cloudflare’s access β€œsucks balls” -- that’s so unexpected as I’m used to formal professional communications
22:59:08 <truebrain> πŸ˜„
22:59:10 <truebrain> good, so that worked πŸ˜›
22:59:32 <truebrain> it is a balance .. you don't want to have anyone working at Cloudflare getting mad at you πŸ˜› But I like doing these things from time to time πŸ˜„
23:01:26 <DorpsGek> [OpenTTD/website] TrueBrain commented on pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520823027
23:03:31 <DorpsGek> [OpenTTD/website] TrueBrain updated pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267
23:18:17 <DorpsGek> [OpenTTD/website] 2TallTyler approved pull request #267: Add: two posts, one about our infrastructure, and one about our recent migration to it https://github.com/OpenTTD/website/pull/267#pullrequestreview-1520823836
23:20:17 <truebrain> will merge tomorrow, and make the appropiate announcement. Tnx for review πŸ™‚
23:32:25 *** HerzogDeXtEr has quit IRC (Read error: Connection reset by peer)