IRC logs for #openttd on OFTC at 2018-12-17
⏴ go to previous day
00:17:11 *** Thedarkb1-T60 has joined #openttd
01:17:53 *** Thedarkb1-T60 has joined #openttd
05:49:16 *** HerzogDeXtEr has joined #openttd
07:01:59 *** Wacko1976 has joined #openttd
08:18:50 *** Progman has joined #openttd
10:55:03 *** WWacko1976-work has joined #openttd
13:44:34 *** Wacko1976-work has joined #openttd
13:46:28 *** WWacko1976-work has quit IRC
14:08:19 *** HerzogDeXtEr has joined #openttd
14:51:20 *** chomwitt has joined #openttd
15:24:50 *** sla_ro|master has joined #openttd
15:35:32 *** Gustavo6056 has joined #openttd
15:40:14 *** Gustavo6056 is now known as Gustavo6046
16:28:16 <nielsm> I thought about the azure CI in the shower this morning and realized, I think it's making a detached HEAD checkout which explains why the builds have no branch name detected
16:52:32 *** Wormnest has joined #openttd
17:16:47 *** Wacko1976-work has quit IRC
17:28:17 *** tokai|noir has joined #openttd
17:28:17 *** ChanServ sets mode: +v tokai|noir
17:56:21 <LordAro> istr planetmaker doing some workaround with nml to do with that
17:56:48 <LordAro> though that doesn't really help when you're trying to build a specific branch
17:57:34 *** Thedarkb1-T60 has joined #openttd
18:01:58 <nielsm> they do provide a build variable specifying the target branch name, but it's as a refs/ path so needs some munging
18:03:03 <LordAro> but i guess it only matters for "provided builds", which wouldn't be using the normal revision number anyway (tag, nightly, etc)
18:05:06 <LordAro> nielsm: tags not mentioned...
18:05:27 <nielsm> will have to experiment for those
18:05:37 <planetmaker> I haven't actually managed that yet, @ LordAro
18:06:53 <planetmaker> though... there's some version detection in that repo, yes. But no builds yet
18:20:07 *** Thedarkb1-T60 has joined #openttd
18:22:50 <Samu> that server doesn't like to serv
18:27:02 *** Eddi|zuHause2 has joined #openttd
18:53:01 *** Thedarkb1-T60 has joined #openttd
18:55:27 <planetmaker> LordAro, nielsm but it was my impression, that you need to give the revision you want to build the the CI. So possibly give the branchname via the hook from the repo to the CI
18:58:17 <nielsm> the automatic github integration gets the revision to build
18:59:02 <nielsm> yeah, it doesn't check out the named revision but the hash
18:59:03 <nielsm> git checkout --progress --force 212d1f59cf4e154eb0a56ef79c2bfb53bdb807e5
19:00:23 <nielsm> so we'd have to consume $(Build.SourceBranch) somehow and stuff it into .ottdrev for the build
19:05:15 <planetmaker> dunno how that works with the envisioned system
19:06:23 <planetmaker> for DevZone it works like: the incoming hook analyses the incoming commits. And it knows their branch, and hash. And it tells Jenkins the hash to build. And also gives the branch and / or tag name. So nothing to analyse for the CI - it just gets the version info fed - if it doesn't get the whole repo
19:07:24 <nielsm> I already have a working proof of concept, the entire build works (on windows and linux), it's just that the generated binaries don't have the branch name in the revision string
19:09:29 <planetmaker> I haven't managed to find enough time for the github webhooks. Doesn't it know the branch(es) the changesets belong to?
19:09:36 <planetmaker> Then it could simply communicate them
19:09:59 <nielsm> but azure pipelines' github integration does not use the branch name when checking out the sources
19:10:14 <nielsm> so the git checkout is left as a detached HEAD
19:10:24 <nielsm> meaning our method of detecting the current branch name fails
19:13:01 <nielsm> right now I'm still trying to figure out why the mac build fails so badly, pkg-config simply doesn't pick up on anything
19:13:43 <nielsm> actually, maybe the problem is that pkg-config itself is not found
19:14:09 <nielsm> I think perhaps configure should complain if pkg-config does not exist
19:14:21 <nielsm> (or rather, can not run)
19:14:58 <planetmaker> it should tell so, I guess
19:15:26 <nielsm> executing pkg-config zlib --modversion
19:15:42 <nielsm> and exit code 127 matches with "command not found"
19:20:36 *** frosch123 has joined #openttd
19:26:57 <nielsm> 2018-12-17T18:22:57.9268440Z checking zlib... found
19:26:57 <nielsm> 2018-12-17T18:22:57.9612610Z checking liblzma... found
19:26:57 <nielsm> 2018-12-17T18:22:57.9746510Z checking lzo2... found
19:29:00 <nielsm> to think it was something so stupid, pkg-config itself missing...
19:35:12 <nielsm> HOMEBREW_NO_AUTO_UPDATE=1 certainly shaves some time off
19:36:58 <nielsm> and the machines have two vcpus, make -j2 should also shave some time off
19:42:55 *** andythenorth has joined #openttd
19:44:17 <andythenorth> google chrome warns me it's malware :P
19:44:47 <nielsm> yes, I've added code that will hypnotize you and force you to paint pixels
19:45:07 <andythenorth> nielsm: are you actually nielsm? :P
19:45:13 <andythenorth> and not a sock puppet?
19:47:27 <andythenorth> nielsm: runs, loads a savegame
19:47:45 <nielsm> what version name does the main menu show?
19:48:20 <nielsm> so no branch name, as I suspected
19:48:27 <nielsm> probably two dashes between date and rev
19:49:25 <andythenorth> hash is ga6e4bf5d
19:52:25 <planetmaker> <nielsm> yes, I've added code that will hypnotize you and force you to paint pixels <-- then 'malware' definitely is a matter of perspective :P
19:54:48 <nielsm> ah yeah, building is much faster with -j2, worth keeping
20:02:38 *** gelignite has joined #openttd
20:31:49 <andythenorth> what goes on in the OpenTTD main menu screen?
20:32:05 <andythenorth> that test build just killed my battery in 30 minutes
20:33:07 <andythenorth> I have two other openttd instances running, paused, at about 7%
20:34:08 <planetmaker> well... game running, one core not throttled?
20:35:40 <andythenorth> if I unpause the running game in the other instances it's 50%-75%
20:35:57 <planetmaker> it depends on the actual save, ofc
20:36:19 * andythenorth remembers when ottd used to take 30% on a core 2 duo :P
20:36:31 <Wolf01> Mmmh, started netflix, starved the bandwidth
20:42:56 <planetmaker> 8461 planetm+ 20 0 2932192 113676 23752 S 16,6 1,4 0:12.29 openttd
20:42:56 <planetmaker> ... 16% for my titlegame
20:43:39 <planetmaker> model name : Intel(R) Core(TM) i7-3770 CPU @ 3.40GHz
20:44:02 <planetmaker> but harder to empty my mains "battery" :P
20:52:14 <andythenorth> you have quite a lot more CPU there :P
20:54:49 *** Wacko1976 has joined #openttd
20:58:05 *** Gustavo6046 has joined #openttd
21:00:43 *** Gustavo6046 has joined #openttd
21:03:55 <planetmaker> old CPU... 5 years at least
21:06:35 <nielsm> ad it's still perfectly adequate for most desktop usage
21:06:45 <nielsm> maybe a little slow for the latest games
21:06:48 <planetmaker> yep. And also for the games I play
21:06:56 <planetmaker> I don't play the "latest" ego-shooters
21:06:58 *** Wormnest has joined #openttd
21:07:19 <planetmaker> but works also for stuff like stellaris or pillars of eternity
21:07:36 <planetmaker> and ofc OpenTTD :)
21:19:21 *** Thedarkb1-T60 has joined #openttd
22:02:11 <frosch123> we should start a competition
22:02:40 <frosch123> find a reasonable case to use the spaceship operator in the desaster code
22:03:04 <planetmaker> spaceship operator?
22:03:27 <frosch123> returns lower/equal/larger instead of true or false
22:04:44 <frosch123> it when comparing two values is more expensive than comparing the result of <=> with the constants
22:04:59 <frosch123> you can compare two string using <=> and then switch on the result
22:05:29 <planetmaker> inventing new disasters :)
22:06:47 <frosch123> btw, my ottd-playing coworker was quite upset about the usage of ottd in the msvc cppcon talk
22:07:05 <planetmaker> in your talk about that or which?
22:07:16 <planetmaker> and why be mad about it? :)
22:07:27 <frosch123> did you see the talk?
22:07:52 <frosch123> basically msvc people wanted to showcase their new debugging features
22:08:08 <frosch123> but their 5-line testcases did not look fancy enough
22:08:32 <frosch123> so they took a random open source project (guess which) and pasted their 5 lines into some place where they looked nice
22:08:58 <frosch123> my coworker meant that would make ottd look bad, as if it were our bugs
22:10:25 <frosch123> don't bother to watch the complete talk
22:10:31 <frosch123> it's exceptionally boring
22:11:35 <frosch123> the entertaining part is how they faked their 5 line unit tests into a fancy demo
22:11:48 <frosch123> while none of the regular viewers notice :)
22:12:00 <LordAro> well, why would they?
22:12:56 <frosch123> planetmaker: actually, the most entertaining part is when they debug strgen :)
22:13:07 <planetmaker> :D that sounds fun
22:13:24 <frosch123> first time someone patches strgen :)
22:13:39 <planetmaker> did we incorporate the patches already, did they send PRs?
22:14:20 <planetmaker> The debugging feature explained on ottd code looks good, though
22:14:51 <frosch123> should we link it from the "how to compile ottd on msvc" page?
22:15:07 <planetmaker> might not be a bad idea, tbh
22:15:37 <frosch123> but you can link it to the next manager
22:24:55 <planetmaker> he is a bit... tiring to listen to after some time :P
22:25:04 <planetmaker> artificially cheerful :P
22:25:45 <frosch123> not sure whether c++ jokes are your thing, but there are some funny 5 minute talks
22:29:35 <planetmaker> not sure I'd get all of them
22:34:48 <frosch123> just be aware that ottd source is west const
22:55:24 *** Thedarkb1-T60 has joined #openttd
23:18:25 <andythenorth> dunno what to do
23:18:29 <andythenorth> about supplies cars
23:19:15 <andythenorth> they're kind of weird
23:20:38 <andythenorth> are they just for supplies or not?
23:20:53 * andythenorth mostly uses express cars for supplies, for speed
23:24:10 <planetmaker> well, then... is there a reason for special supplies cars then?
23:33:50 <andythenorth> interesting sprites
23:33:58 <andythenorth> but maybe they should carry wider range of cargos
23:34:12 <andythenorth> currently just ENSP, FMSP, vehicles
23:34:16 *** Eddi|zuHause2 is now known as Eddi|zuHause
23:49:13 <planetmaker> mail maybe, too (thinking in terms of parcels, not letters)
23:49:38 <planetmaker> other piece goods...
continue to next day ⏵