Times are UTC Toggle Colours
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 02:40:52 *** Flygon has quit IRC 02:43:27 *** Wormnest has quit IRC 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 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 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 13:18:36 *** tokai has joined #openttd 13:18:36 *** ChanServ sets mode: +v tokai 13:26:53 *** 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:43 <truebrain> but no more AWS services directly 14:11:50 <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:20 <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 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 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 21:07:52 *** keikoz has quit IRC 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 21:47:06 *** gelignite has quit IRC 21:47:16 *** 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