IRC logs for #openttd on OFTC at 2020-09-02
⏴ go to previous day
01:00:01 <TrueBrain> That didn't take long
01:00:12 <TrueBrain> Anonymous access too
01:00:26 <TrueBrain> Guess that solves the Docker Hub issue for now :p
02:13:19 *** snail_UES_ has joined #openttd
03:10:29 *** Webster has joined #openttd
03:59:57 *** TrueBrain has joined #openttd
04:22:05 *** Speeder has joined #openttd
05:31:54 *** Smedles has joined #openttd
06:24:51 *** Smedles has joined #openttd
06:24:51 *** daspork has joined #openttd
06:31:18 *** snail_UES_ has joined #openttd
07:33:50 *** sla_ro|master has joined #openttd
08:06:48 *** andythenorth has joined #openttd
08:29:43 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz updated pull request #8285: Rewrite `OverflowSafeInt` to avoid undefined behaviour https://git.io/JJgcH
08:31:02 <DorpsGek_III> [OpenTTD/OpenTTD] techgeeknz commented on pull request #8285: Rewrite `OverflowSafeInt` to avoid undefined behaviour https://git.io/JUY4I
08:45:55 *** andythenorth has joined #openttd
09:53:44 *** Flygon_ has joined #openttd
10:12:12 *** WormnestAndroid has quit IRC
10:12:24 *** WormnestAndroid has joined #openttd
10:29:42 <DorpsGek_III> [OpenTTD/DorpsGek] TrueBrain opened pull request #28: Codechange: update to latest black coding style https://git.io/JUYzf
10:32:46 <LordAro> TrueBrain: curious set of changes
10:33:18 <TrueBrain> well, it was one of the things I did not like about black; happy they now changed it :)
10:33:23 <LordAro> looks like it's changed to always putting 1 parameter per line, rather than trying to make them fit?
10:33:36 <TrueBrain> no, IF it doesn't fit, put 1 per line
10:33:46 <TrueBrain> instead of making a newline, owh, now it does fit, leave it like that
10:33:54 <TrueBrain> (all except the "if" btw)
10:34:07 <TrueBrain> I really did not like what it did there, so this is nice :)
10:34:14 <TrueBrain> once you hit \n, do it for all parameters
10:34:18 <TrueBrain> the "if" is a bit weird however
10:34:27 <TrueBrain> it is more readable like this for sure
10:34:30 <TrueBrain> but I am curious why it did this
10:35:12 <TrueBrain> right, I also need to change packages are now (also?) going to be pushed to the GHCR .. that needs some experimenting first, I gues
10:35:42 <DorpsGek_III> [OpenTTD/DorpsGek] LordAro approved pull request #28: Codechange: update to latest black coding style https://git.io/JUYzG
10:40:02 <TrueBrain> meh, that is an annoying change; many moving parts that needs changing at once ..
10:41:29 <DorpsGek_III> [OpenTTD/DorpsGek] TrueBrain merged pull request #28: Codechange: update to latest black coding style https://git.io/JUYzf
10:42:25 <DorpsGek_III> [OpenTTD/DorpsGek] TrueBrain updated pull request #27: Scheduled monthly dependency update for September https://git.io/JUmp9
10:43:39 <DorpsGek_III> [OpenTTD/DorpsGek] TrueBrain approved pull request #27: Scheduled monthly dependency update for September https://git.io/JUYzX
10:43:57 <DorpsGek_III> [OpenTTD/DorpsGek] TrueBrain merged pull request #27: Scheduled monthly dependency update for September https://git.io/JUmp9
10:44:41 <TrueBrain> hmmmm ... spammmmmm :)
10:47:00 <TrueBrain> okay, black is going to fail on all repos ... time to do the boring work :)
10:48:39 *** DorpsGek_III has joined #openttd
10:48:39 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain opened pull request #66: Codechange: update to latest black coding style https://git.io/JUYzA
10:49:31 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #66: Codechange: update to latest black coding style https://git.io/JUYzA
10:50:16 <DorpsGek_III> [OpenTTD/bananas-server] TrueBrain opened pull request #28: Codechange: update to latest black coding style https://git.io/JUYzh
10:50:41 <DorpsGek_III> [OpenTTD/bananas-api] LordAro approved pull request #66: Codechange: update to latest black coding style https://git.io/JUYgf
10:51:09 <DorpsGek_III> [OpenTTD/bananas-server] LordAro approved pull request #28: Codechange: update to latest black coding style https://git.io/JUYgU
10:52:11 <DorpsGek_III> [OpenTTD/bananas-frontend-web] TrueBrain opened pull request #39: Codechange: update to latest black coding style https://git.io/JUYgI
10:52:51 <DorpsGek_III> [OpenTTD/bananas-frontend-web] LordAro approved pull request #39: Codechange: update to latest black coding style https://git.io/JUYgL
10:53:05 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain merged pull request #66: Codechange: update to latest black coding style https://git.io/JUYzA
10:53:22 <DorpsGek_III> [OpenTTD/bananas-server] TrueBrain merged pull request #28: Codechange: update to latest black coding style https://git.io/JUYzh
10:53:38 <DorpsGek_III> [OpenTTD/bananas-frontend-web] TrueBrain merged pull request #39: Codechange: update to latest black coding style https://git.io/JUYgI
10:53:51 <TrueBrain> and now pyup on those repos .. that will be spammy too :P
10:54:25 <TrueBrain> it is not like it interrupts any conversation :P
10:56:44 <andythenorth> why all this spam :(
10:56:52 <andythenorth> you ruined my conversation :(
10:57:45 <TrueBrain> next spam won't be from me, okay? :P
10:58:52 <TrueBrain> hmm, how do I force pyup to run again, I wonder ..
11:02:19 <TrueBrain> hmm, seems black is not always putting all parameters on a new line .. lol
11:15:27 <TrueBrain> seems to be related if it ends with a comma or not
11:16:05 <LordAro> your client bolded it for me :p
11:16:15 <TrueBrain> client is stupid :(
11:16:32 <TrueBrain> did my client really do it, or yours? :P
11:16:51 <LordAro> well it allowed me to send it, didn't it? :p
11:17:05 <LordAro> and i'd be surprised if irssi would do something so modern :p
11:17:28 <TrueBrain> yeah, it is the comma at the end
11:18:57 <LordAro> "re-implemented support for explicit trailing commas: now it works consistently within any bracket pair, including nested structures (#1288 and duplicates)"
11:19:51 <TrueBrain> and it looks better too!
11:20:05 <LordAro> as long as you don't have a trailing comma in a bad place!
11:20:13 <TrueBrain> or missing it in the right place!
11:40:33 <TrueBrain> drwxr-xr-x 0/0 0 1970-01-01 01:00 test-1/ <- almost correct information for a root-folder in tarballs for BaNaNaS :P
11:44:01 <TrueBrain> drwxr-xr-x patric/patric 33369 2020-09-02 11:43 test-1/
11:44:04 <TrueBrain> oops, size is wrong :P
11:44:45 <TrueBrain> drwxr-xr-x patric/patric 0 2020-09-02 11:44 test-1/
11:46:29 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain opened pull request #67: Fix #59: OpenTTD's client expects a folder as first entry in tarballs https://git.io/JUYVn
11:54:04 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain opened pull request #68: Fix #65: ignore errors in tarballs/zipfiles https://git.io/JUYV9
12:00:47 <TrueBrain> now how to force PyUp .. hmmm
12:04:02 <TrueBrain> lets see if this helps ..
12:06:24 <TrueBrain> lol .. nope .. it doesn't make a new "initial update" :P
12:07:19 <LordAro> just going to wait until October? :p
12:07:50 <TrueBrain> I removed the project from pyup, created it again
12:07:53 <TrueBrain> it does make the configuration thingy
12:12:04 <TrueBrain> and everything is working, so this seems to be fine
12:13:13 <TrueBrain> I will wait with a release till the other 2 PRs are approved :D
12:21:15 <TrueBrain> that seems all to work just fine .. :D
12:21:25 <TrueBrain> I love it when dep-updates don't break anything
12:25:59 <TrueBrain> deployments to staging worked too, so yeah .. if you would be so kind to review the 2 remaining PRs LordAro , means I can deploy this to production :)
12:26:27 <TrueBrain> 4 more repos that will annoy us every month .. but okay, at least it means a lot less chance of CVEs etc :P
12:26:55 <TrueBrain> next up ... fixing the CVE warning on "website" .. ugh, that is going to be painful
12:27:00 <TrueBrain> first lunch, I guess
12:27:53 <DorpsGek_III> [OpenTTD/bananas-api] LordAro approved pull request #67: Fix #59: OpenTTD's client expects a folder as first entry in tarballs https://git.io/JUYoi
12:29:05 <DorpsGek_III> [OpenTTD/bananas-api] LordAro commented on pull request #68: Fix #65: ignore errors in tarballs/zipfiles https://git.io/JUYoQ
12:32:31 *** andythenorth has joined #openttd
12:35:32 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain commented on pull request #68: Fix #65: ignore errors in tarballs/zipfiles https://git.io/JUYKm
12:36:01 <TrueBrain> hmm ... branch is out-of-date,but ... it really is not
12:36:06 <TrueBrain> well, kinda, but nothing that is an issue
12:36:19 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #67: Fix #59: OpenTTD's client expects a folder as first entry in tarballs https://git.io/JUYVn
12:36:34 <TrueBrain> *tries the "update branch"* button
13:06:03 *** Smedles has joined #openttd
13:10:59 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain closed issue #59: Base-music tarballs created are incompatible with OpenTTD client https://git.io/JJuy0
13:11:01 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain merged pull request #67: Fix #59: OpenTTD's client expects a folder as first entry in tarballs https://git.io/JUYVn
13:24:37 <TrueBrain> Go is a weird language
13:24:38 <TrueBrain> "if hookErr, ok := err.(hooks.HookError); ok {"
13:24:41 <TrueBrain> wtf is this saying?
13:25:47 *** Eddi|zuHause has joined #openttd
13:28:45 <andythenorth> if there's no error, do this function?
13:28:57 <andythenorth> FML I am glad I am not an actual programmer
13:29:07 <andythenorth> that looks like except/try to me
13:36:10 <Timberwolf> It's trying to cast err (which I guess is an interface) to type HookError, from package hooks. If that succeeds, execute the code in the { ... } scope.
13:36:57 <Timberwolf> hookErr will also be in scope at that point.
13:41:12 <LordAro> easentially a dynamic_cast, that returns a tuple of the result, and whether it succeeded
13:42:32 <TrueBrain> but ... how is that readable?
13:43:26 <andythenorth> so it's 'efficient'
13:43:43 <TrueBrain> tus-protocol is also "funny" .. I decline an upload in the pre-finish, yet it finishes just fine :P
13:46:45 <TrueBrain> hmm, no, it seems to be the tusd implementation that is this weird :P
14:00:16 <LordAro> TrueBrain: not particularly different to any other language syntax :p
14:06:45 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #68: Fix #65: ignore errors in tarballs/zipfiles https://git.io/JUYV9
14:06:47 <TrueBrain> LordAro: okay, this is a bit of a compromise between not-the-cleanest-code and reporting the error to the user
14:07:51 <Eddi|zuHause> i'm not sure i missed part of the conversation, but "do thing if no error" i would do like "try: potential_error(); do_thing_if_no_error(); finally: do_the_rest();"
14:13:45 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #68: Fix #65: ignore errors in tarballs/zipfiles https://git.io/JUYV9
14:13:46 <TrueBrain> now including regression checks :)
14:16:18 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain updated pull request #68: Fix #65: ignore errors in tarballs/zipfiles https://git.io/JUYV9
14:16:21 <TrueBrain> and even with better regression checks :D
14:27:40 <Timberwolf> Go has no native exceptions, which is one of those things that sounds nice as an ideology and not so nice when you've just written your 5th `if thing, err := Whatever(); err != nil {` of what was supposed to be a simple method.
14:34:23 <Timberwolf> I'd say it's more resistant to that style which wraps everything in a single try/catch and the only error you get is "oh no, something happened" but I'm pretty sure the kind of developers who program like that will just discard the errors and ignore them.
14:47:34 <TrueBrain> Rust has a similar issue; I always start applications with propagating errors correctly .. but after a certain amount of time you forget a few times, and after that you just unwrap() and be done with it
14:47:39 <TrueBrain> it is difficult to be consistent with it
14:47:48 <TrueBrain> but .. try/catch blocks are VERY difficult to validate for a compiler
14:48:00 <TrueBrain> so I get it .. exceptions are annoying :P
14:52:03 <Eddi|zuHause> exceptions have been annoying since before they were even called exceptions
14:52:29 <Eddi|zuHause> "don't use GOTO" ... "but exceptions!!!"
14:53:31 <Eddi|zuHause> that was like 50 years ago
14:53:32 <glx> on error resume next <- the nice BASIC way :)
17:05:12 *** snail_UES_ has joined #openttd
17:22:09 *** sla_ro|master has joined #openttd
17:34:57 *** Mahjong2 has joined #openttd
17:42:20 *** Mahjong2 has joined #openttd
17:45:10 *** Mahjong2 has joined #openttd
18:04:35 *** Mahjong has joined #openttd
18:07:32 *** gelignite has joined #openttd
18:08:01 *** Mahjong has joined #openttd
18:13:25 *** Mahjong has joined #openttd
18:13:56 *** Progman has joined #openttd
18:18:02 *** Mahjong has joined #openttd
18:26:01 *** Mahjong has joined #openttd
18:26:25 *** frosch123 has joined #openttd
18:27:09 *** sla_ro|master has joined #openttd
18:47:13 <DorpsGek_III> [OpenTTD/bananas-api] LordAro approved pull request #68: Fix #65: ignore errors in tarballs/zipfiles https://git.io/JUYxm
18:50:58 <LordAro> TrueBrain: sorry for the delay, i got busy
18:51:55 *** HerzogDeXtEr has joined #openttd
19:19:06 *** Gustavo6046 has joined #openttd
19:27:13 *** otetede has joined #openttd
19:45:51 <DorpsGek_III> - Update: Translations from eints (by translators)
19:53:04 <andythenorth> is it lunch time?
20:33:23 <Wolf01> So... youtube evening? Starting with "how potholes work?"
20:36:08 *** y2kboy23_ has joined #openttd
20:37:17 *** Wormnest has joined #openttd
21:08:08 *** tokai|noir has joined #openttd
21:08:09 *** ChanServ sets mode: +v tokai|noir
21:17:52 <glx> ahah I watched it yesterday Wolf01
21:19:42 *** jottyfan has joined #openttd
21:24:11 <TrueBrain> LordAro: no worries, tnx for review :)
21:24:21 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain merged pull request #68: Fix #65: ignore errors in tarballs/zipfiles https://git.io/JUYV9
21:24:22 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain closed issue #65: ReadError: file could not be opened successfully https://git.io/JUqnC
21:35:08 <DorpsGek_III> [OpenTTD/bananas-api] TrueBrain commented on issue #59: Base-music tarballs created are incompatible with OpenTTD client https://git.io/JJuy0
continue to next day ⏵