Times are UTC Toggle Colours
00:43:39 *** Supercheese has quit IRC 01:00:33 *** Laedek has joined #openttd 01:36:06 *** snail_UES_ has joined #openttd 01:44:40 *** Supercheese has joined #openttd 01:53:06 *** Supercheese has quit IRC 01:53:12 *** glx has quit IRC 01:53:23 *** Supercheese has joined #openttd 02:22:07 *** Laedek has quit IRC 02:22:43 *** Laedek has joined #openttd 02:32:30 *** Laedek_ has joined #openttd 02:32:32 *** Laedek has quit IRC 02:52:25 *** Flygon has joined #openttd 03:07:39 *** Laedek has joined #openttd 03:10:52 *** Laedek_ has quit IRC 03:17:27 *** sla_ro|master has joined #openttd 03:35:08 *** sla_ro|master has quit IRC 03:43:49 *** ToBeFree has joined #openttd 03:45:00 *** ToBeFree has quit IRC 04:00:33 *** imdak has joined #openttd 04:06:24 *** snail_UES_ has quit IRC 05:12:08 *** nielsm has joined #openttd 05:56:09 *** nielsm has quit IRC 06:23:12 *** Supercheese has quit IRC 06:23:29 *** Supercheese has joined #openttd 07:14:42 *** Mahjong1 has quit IRC 08:48:31 *** Supercheese has quit IRC 09:28:08 <TrueBrain> meh; bit stuck ... when I install Debian Stretch on OpenTTD, it simple doesn't want to boot correctly .. 09:28:12 <TrueBrain> ugh 09:32:38 <peter1138> Debian... on OpenTTD...? 09:36:37 <TrueBrain> on the OpenTTD infrastructure 09:38:11 <TrueBrain> tomorrow a call with Google Cloud Platform ... lets hope they can offer something 09:38:43 <TrueBrain> as I simply don't have time anymore to do it myself, all this infrastructure blabla .. 09:41:07 <SpComb> it's fun to write a custom CI/CD system, though? :P 09:41:25 <TrueBrain> meh; I tried really hard to avoid that too 09:41:32 <TrueBrain> but given the constraints ... 09:41:40 <SpComb> what does GCP offer... GCE or even GKE? 09:41:45 <TrueBrain> nothing yet 09:41:48 <TrueBrain> I asked them what they could offer 09:41:55 <TrueBrain> but I want GKE 09:42:11 <SpComb> you'd expect that having kube would make e.g. CI/CD easier to implement 09:42:14 <TrueBrain> as that would mean a lot more people can help out, without understanding most part of it 09:42:30 <TrueBrain> the CI/CD is done up till the part it needs to deploy something to somewhere 09:42:36 <TrueBrain> and yeah, kubernetes would make that near-trivial 09:42:41 <TrueBrain> (as the configuration is already in there) 09:43:04 <TrueBrain> I was looking if I could do it in docker, which means I have to write my own configuration layer ... which really sucks balls 09:43:20 <TrueBrain> setting up your own kubernetes is .. euh ... yeah, lets not go there 09:49:13 <SpComb> my day job is automating kube setup and maintenance :) 09:49:36 <SpComb> https://kontena.io/pharos/ insert employer sponsored open-source content 09:50:11 <TrueBrain> so set it up for OpenTTD already :P 09:50:22 <SpComb> no, we offer the tooling that lets you do it yourself :P 09:50:24 <TrueBrain> I am really annoyed CoreOS is no longer really "stable" .. bit uncertain in which state it is .. 09:50:42 <TrueBrain> I looking into kubespray ... I slowly walked away 09:50:57 <SpComb> pharos would be somewhat similar 09:51:00 <SpComb> what was wrong with kubespray? 09:51:12 <SpComb> but yeah, if you can get free GKE resources from google, then it's certainly a good option 09:51:17 <TrueBrain> the complexity .. and the amount of required maintainance 09:51:33 <TrueBrain> mainly, we would need to set it up on-premise 09:51:35 <TrueBrain> we have a single machine 09:51:45 <SpComb> doable 09:51:47 <TrueBrain> all stuff kinda demands you have at least 3 nodes in kubernetes 09:51:56 <TrueBrain> so ... I have to spin up 3 VMs? And another for ansible? 09:52:02 <TrueBrain> I dont have that memory available 09:52:22 <SpComb> https://pharos.sh/docs/usage/#using-node-taints 09:52:54 <SpComb> you can have a single master host without the NoSchedule taint 09:52:59 <TrueBrain> yeah, I saw you can do that too with kubespray, but than people claim that is not "production ready" 09:53:07 <TrueBrain> and I got just really annoyed of me not having enough domain knowledge :D 09:53:32 <SpComb> dunno why it wouldn't be production-ready assuming you have enough resources to cover the kube overhead, and don't consider the lack of HA to be a problem 09:53:37 <TrueBrain> I guess it is mainly that ... I know shit about this stuff .. and it is a pretty steep learning curve 09:53:49 <TrueBrain> lack of HA is never been an issue :D 09:53:59 <TrueBrain> but okay .. so now I run into the next issue .. that I would need to upgrade the dom0 09:54:14 <SpComb> physical host running some ancient xen? 09:54:20 <TrueBrain> which means renting a second machine, linking them together (with a GRE and some black magic on network level), migrating to one machine, upgrading, migrating back .. 09:54:46 <TrueBrain> it runs XenServer .. upgrading a single-host XenServer is a real pita 09:54:51 <TrueBrain> I regret ever making that choice 09:55:42 <SpComb> I've used ganeti for kvm with drbd and live-migration, but it lacks support for a lot of kvm knobs and it's not really maintained anymore, which is a pity 09:55:47 <TrueBrain> and when you look what it costs to get a kube in cloud and kube on permise ...... cloud is so expensive 09:56:28 <SpComb> does the old version of xen prevent you from running newer domU kernels/distros? 09:56:31 <TrueBrain> so yeah .. what I am trying to create is an infrastructure that is as simple as possible, so more people can support it 09:56:40 <TrueBrain> yeah, latest kernel just flat-out crashes 09:56:43 <TrueBrain> (4.9) 09:56:45 <TrueBrain> "latest" 09:56:53 <TrueBrain> latest Debian Stretch kernel flat-out crashes 09:57:05 <SpComb> is it still some xen-PV mode without any bootloader support? 09:57:22 <TrueBrain> its PV, with pygrub bootloader 09:57:43 <TrueBrain> I can put it in HVM .. but .. I care about my performance 09:57:57 <SpComb> kvm with virtio... 09:58:14 <TrueBrain> the migration is the issue, to any other technoligy 09:58:26 <TrueBrain> so that is why I have the plan to migrate everything inside Dockers 09:58:28 <TrueBrain> which is a lot easier 09:58:32 <SpComb> but yeah, maintaining a single physical machine like that is a challenge once you get to the point where you have to start doing a major upgrade on it... tends to turn into a full rebuild 09:58:34 <TrueBrain> and run Dockers "somewhere" 09:58:55 <TrueBrain> I upgraded the machine twice now .. both times took a month, including planning and execution 09:59:09 <TrueBrain> OVH is a good provider to have in these cases 09:59:40 <TrueBrain> maybe we should rent a second machine, make that a kube-cluster, and move everything in dockers to there or something 09:59:55 <TrueBrain> best thing about Docker, we can just migrate even the shitty-old-stuff we have 10:00:11 <TrueBrain> (some parts still run in Django 1.2) 10:00:17 <SpComb> btw if a single machine is enough, then `kubeadm init` is pretty trivial, all you really need is an appropriate kubeadm yaml 10:00:39 <TrueBrain> people kept telling me I should never do that? 10:00:44 <SpComb> why not? 10:00:51 <TrueBrain> (again, I know very little about kube, so I just go with what people tell me) 10:00:54 <TrueBrain> long long rants 10:01:06 <TrueBrain> something about hard to maintain, no upgrade path, etc 10:01:21 <SpComb> `kubeadm upgrade` has worked fine so far for 1.9 -> 1.10 -> 1.11 10:01:30 <TrueBrain> good to know :) 10:01:58 <SpComb> given, the systemd units changed in 1.11, and there were some things like `kubelet --cluster-dns` that you had to tweakin the 1.10 systemd units, and those tweaks no longer work in 1.11... but they're no longer necessary either 10:02:37 <TrueBrain> hmm .. so I guess I could give that a try .. 10:02:49 <SpComb> okay, and you also need to do a two-step dance where you first download the raw kubeadm binary and use that to run the upgrade before upgrading the kubeadm package, or it breaks 10:03:01 <SpComb> but those are documented clearly enough in https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/kubeadm-upgrade-1-11/#upgrade-the-control-plane 10:03:30 <TrueBrain> so ideal I have the kube cluster hosted externally, as that would mean no support from our side .. 10:03:38 <TrueBrain> but for now I can spin one up and see if that works 10:03:39 <TrueBrain> I guess 10:04:00 <TrueBrain> just installing a new debian stretch fails horribly .. hmm ... lets clone an existing :D 10:04:47 <TrueBrain> btw, random question, do you happen to know: can you in a kube cluster have persistant storage? 10:04:55 <TrueBrain> for example, what if I want to run mediawiki in it? 10:05:05 <TrueBrain> (uploads etc ar enot in a database, but on a filesystem) 10:05:38 <SpComb> it took me a couple years, but I finally managed to upgrade our puppet infra from 3.x -> 4.x/5.x this month... means we can finally start migrating from squeeze/wheezy/jessie -> stretch now, because puppet 3.x is no longer available/supported :) 10:06:09 <TrueBrain> ugh, puppet .... I started to really hate puppet :D 10:06:11 <TrueBrain> but gratz :) 10:06:15 <SpComb> there's various approaches to it... trivial approach is to bind the pod to a specific node and use a hostPath volume, of course 10:06:45 <TrueBrain> sneaky :D 10:07:43 <SpComb> after that it starts getting more complicated with persistentVolumes... you can use local storage if you create local persistentVolumes and bind them to specific nodes, but the approches to that which I've seen barely make any sense, it's just more complex than the hostPath approach and I'm not aware of any automagic sync/replication functionality 10:08:27 <SpComb> and then there's all kinds of custom storage providers like OpenEBS, buuuuut I'm not exactly convinced that they would be production-ready yet either 10:08:39 <SpComb> it's a solved problem if you have some external storage like EBS, though 10:08:40 <TrueBrain> do cloud providers solve this? 10:08:54 <SpComb> yeah, then you just use the integrated kube support for those cloud provider volumes 10:09:16 <TrueBrain> more reason to use that :D 10:10:32 <TrueBrain> so you say, just download kubeadm, and run kubeadm init :) 10:10:54 <SpComb> sure, it's the easiest way to get started, and it *does* have an upgrade path 10:11:09 <SpComb> main limitation with kubeadm is that it doesn't really do multi-master/HA at all, at least not yet 10:11:25 <TrueBrain> when we run kube on premise, that is also never going to happen 10:11:27 <TrueBrain> so that is fine 10:11:59 <TrueBrain> first, let me see what happens if I upgrade this clone .. see if it does want to boot .. 10:13:33 <TrueBrain> the page on kubernetes.io that tells how to install kubeadm is a 404 :D 10:13:35 <TrueBrain> lovely 10:14:10 <SpComb> so it is 10:15:15 <TrueBrain> owh right .. and than you get to "picking a network plugin" .. yeah .... eeuuuuhuhhhhh ... *closes eyes* 10:15:17 <TrueBrain> :D 10:15:21 <TrueBrain> you happen to have a suggestion? 10:16:56 <SpComb> you can try https://kontena.io/pharos/#get-started with the download from https://github.com/kontena/pharos-cluster/releases 10:17:35 <TrueBrain> okay, 4.9.0-r6 works, 4.9.0-r7 crashes 10:17:43 <SpComb> but we only support Ubuntu... not Debian 10:17:52 <TrueBrain> revert to snapshot 10:17:58 <TrueBrain> is there really a difference, in terms of support? 10:19:52 <TrueBrain> holy crap at these prices .. people pay that? Big business .... :D 10:20:12 *** tokai has joined #openttd 10:20:12 *** ChanServ sets mode: +v tokai 10:20:22 <SpComb> yes :) 10:20:39 <TrueBrain> kubernetes 1.10? What about 1.11! 10:21:05 <SpComb> https://github.com/kontena/pharos-cluster/pull/473 10:21:22 <SpComb> answer: it's july, everybody except me is on holidays :P 10:21:41 <TrueBrain> :D 10:21:50 <TrueBrain> and it is warmmmmm 10:23:00 <TrueBrain> okay, now first upgrading while holding the kernel back .. 10:23:57 <TrueBrain> SpComb: why do I read a lot that they want kubernetes to run on Docker 17.03? 18.N has been out for months now? 10:25:02 <SpComb> partly political, kube does not use any of the newer Docker features past 1.11 or so, so the newer Docker releases really only bring more bugs 10:25:25 <TrueBrain> lolz 10:25:37 <SpComb> long-term plan is to replace Docker with some other CRI like containerd or cri-o, but it hasn't happened yet 10:26:37 <SpComb> I mean, it's an option, but not the default 10:27:14 *** tokai|noir has quit IRC 10:28:13 <TrueBrain> can a host be: this current host? 10:28:15 <TrueBrain> without ssh :D 10:30:28 <TrueBrain> seems it always goes over ssh 10:30:30 <TrueBrain> pfft 10:31:47 <TrueBrain> awh, it really refuses the host because it is not ubuntu :D 10:32:02 <TrueBrain> Ubuntu is not a server distribution ffs :P 10:32:26 <SpComb> I'd guess that Ubuntu server is more commol than Debian, tbh 10:32:56 <TrueBrain> it still makes me puke everytime I notice that 10:33:02 <TrueBrain> somewhere something went terribly wrong :D 10:34:48 <SpComb> I'm pretty sure that canonical makes more money from Ubuntu server/cloud than desktop 10:35:54 <TrueBrain> hmm .. kubeadm is not in the google repo, what documentation suggests 10:35:56 <TrueBrain> grrrr 10:37:36 <TrueBrain> they too, only added it for Ubuntu 10:37:38 <TrueBrain> lolz 10:38:00 <SpComb> you can probably install the ubuntu dist on debian just fine 10:38:04 <TrueBrain> just did :) 10:38:15 <SpComb> the kube packages only contain a sinhle static binary + systemd units 10:39:50 <TrueBrain> so many preflight errors :D 10:39:56 <TrueBrain> it even refuses newer Dockers 10:39:57 <TrueBrain> lol 10:41:47 *** RafiX has joined #openttd 10:43:56 <TrueBrain> I love the: swap is not allowed, part 10:46:02 <TrueBrain> well, that seems to run..ish 10:52:56 <TrueBrain> tnx SpComb, kubeadm seems to work .. now I need to figure out how it all works, but I had to do that anyway :D 10:54:16 <SpComb> yeah, and if nothing else a single-node kubeadm setup is a good step towards something like GKE 10:54:36 <TrueBrain> this is a bit weird ... I told it to use 10.100.0.0/16 .. but the sock-shop now has 10.104.N.N 10:54:52 <SpComb> I wouldn't be surprised if GCP were willing to offer the OpenTTD project free GKE 10:55:15 <SpComb> are you sure it osn't 10.100.0.0/12? 10:56:24 <SpComb> the service cidr is also separate 10:56:45 <TrueBrain> I wrote /16 .. but what-ever ... as long as it stays away from the lower-end of that range, we will be fine 10:57:02 <TrueBrain> OpenTTD infra runs very isolated in terms of networks .. so it cannot do much harm 10:57:19 <TrueBrain> and I hope GCP can do something for OpenTTD .. not having to maintain this ... pfftt .. that would be very nice :) 10:57:29 <TrueBrain> and having everything run in pods ..... even nicer 10:57:57 <SpComb> the default service cidr is 10.96.0.0/12 and that covers 10.104.x.x 10:58:13 <TrueBrain> funny .. I really use another address :P Owh well, that range is fine 10:58:14 <TrueBrain> tnx :) 10:58:22 <SpComb> odd if kubeadm allows the two to overlap 10:58:22 <TrueBrain> sock-shop works! 10:58:49 <TrueBrain> and I can even reach it from our web-proxy :P 10:58:56 <SpComb> having the pod cidr within the service cidr sounds like a bad idea 10:59:02 <SpComb> nodePort service? 10:59:18 <TrueBrain> no clue .. 10:59:23 <TrueBrain> I just loaded the sock-shop demo 10:59:29 <TrueBrain> it gave me a service on a port 10:59:31 <TrueBrain> which worked 10:59:32 <TrueBrain> :P 11:00:01 <TrueBrain> so now all I have to find out is how to start my test image 11:00:13 <TrueBrain> and a deployment is than as simple as settng a new image for it 11:00:27 <TrueBrain> images are like: truebrain/testing:sha-of-commit 11:00:50 <TrueBrain> first lunch! 11:01:06 <TrueBrain> tnx a bunch SpComb :) 11:01:36 <SpComb> using nodeport services with an externally configured frontend proxy is probably the common initial setup, but there's also native kube ingress-nginx and letsencrypt stuff 11:01:56 <SpComb> np... where do I have sales send the consulting bill to? :P 11:14:41 <TrueBrain> try /dev/null :D 12:42:56 *** imdak has quit IRC 12:48:05 *** sim-al2 has quit IRC 12:56:11 *** snail_UES_ has joined #openttd 13:06:33 *** Erato has joined #openttd 13:06:44 <Erato> o/ 13:07:51 <Erato> I had a question about BaNaNaS and was told that the people here could help. 13:08:36 *** snail_UES_ has quit IRC 13:09:34 <Erato> I would like to change the name of one of my newgrfs, because I incorrectly entered it and it seems like I can't change it without reuploading under a different GRF-ID. 13:22:33 *** Wacko1976 has joined #openttd 13:45:21 <FLHerne> TrueBrain: ^ (?) 13:51:59 <TrueBrain> renaming is a bit of a pita indeed :D 13:55:02 <TrueBrain> Erato: what is the grfid? 13:55:14 <Erato> 454E2001 13:55:50 <TrueBrain> it already has 4 versions? 13:56:02 <Erato> yes 13:56:08 <TrueBrain> that makes it even harder :D 13:56:11 <TrueBrain> what would you like changed? 13:56:21 <Erato> "RIMS" should be "Real International Maglev Set" 13:56:59 *** Mahjong has joined #openttd 13:57:24 <TrueBrain> the main issue with renames is that it takes a lot of time and effort 13:57:34 <TrueBrain> the name is part of the filename as it ends up on the mirrors 13:57:53 <TrueBrain> it is also how the files end up at the user on his disk 13:58:23 <TrueBrain> so I am wondering what the easiest approach is .. as renaming 4 versions will take way too much time to make it reasonable 13:58:37 <planetmaker> dumdidei... it would at worst mean users download it again... but grfid is checked before, IIRC? 13:58:39 <TrueBrain> Erato: do you have a new version ready to upload? 13:58:42 <Erato> I'm going to be updating it relatively soon 13:58:54 <Erato> It's not ready yet, unfor2018-07-19T15:31:19 <SpComb> blocked? You mean they have to go through the same PR review as any other plebs? 15:31:26 <andythenorth> yes 15:31:46 <andythenorth> previously, often, they kind of didn't 15:36:35 <Alberth> not really, anything non-trivial was reviewed too 15:37:07 <Alberth> ie technically being able to bypass doesn't mean it's a good idea to do so :P 15:38:57 <Alberth> and in my experience, a review does improve a solution, it's beneficial too thus 15:39:22 <andythenorth> I am more laissez faire :P 15:39:47 <andythenorth> experience people usually want to avoid being the one wearing clown shoes 15:39:52 <andythenorth> experienced * 15:40:19 <Alberth> it's also a form of learning 15:40:52 <Alberth> you get new insights in how to solve things beyond what you know or considered 15:57:01 *** HerzogDeXtEr has joined #openttd 15:58:39 <SpComb> it's also possible to do really bad reviews and get stuck on some inconsequential opinionated aspects 15:58:46 <SpComb> reviewing it also a skill that you need to learn 16:00:22 *** cHawk has quit IRC 16:02:40 <andythenorth> we are figuring out reviews at work 16:02:56 <andythenorth> reviews seem like they should be about risk 16:02:57 <andythenorth> not taste 16:03:18 <nielsm> yeah 16:03:22 <andythenorth> taste is *really* important when coding 16:03:34 <andythenorth> but if I'm coding it, and I've complied with all/any rules 16:03:44 <andythenorth> then *your* taste shouldn't creep into the review 16:04:01 <andythenorth> just wastes everyone's time 16:04:23 <Alberth> work is simple, as the boss decides the direction 16:04:27 <andythenorth> I should have used Bob and Alice in my example :P 16:04:38 <andythenorth> Alberth: in my case, why would I decide the direction? 16:04:42 <andythenorth> I pay other people to do that 16:04:50 <andythenorth> my job is mostly spreadsheets and signing stuff 16:05:03 <Alberth> open source has the additional problem of holder of the direction of development 16:05:18 <andythenorth> Guido is having an interesting moment with that 16:05:23 <Alberth> in that case, the people that you pay decide the direction :) 16:05:42 <andythenorth> so we look at risks 16:05:47 <nielsm> IMO reviews should also look at the context of the changes and consider if all relevant documentation/comments are being updated to match the new state 16:06:05 <andythenorth> - infosec risk (confidentiality, availability, integrity) 16:06:16 <andythenorth> - and stasis risk: does this tie our hands in future 16:06:17 <Alberth> suppose I submit a patch to remove some in my view obsolete part 16:06:37 <nielsm> I had a bit of an argument on a review a while ago on a pull request here, about whether/how much comments should be changed 16:06:47 <Alberth> even if I follow all rules, there has to be agreement that the part is indeed obsolete 16:07:49 <andythenorth> well yes 16:07:54 <andythenorth> I'm not disputing that :) 16:08:00 <andythenorth> the map array PR is a good case for that 16:08:10 <andythenorth> it's a fundamental change, even if it's not dramatic 16:09:23 <Alberth> map array is not a real change, it's just an expansion of possibilities, we can disagree on how much cpu time it may cost, but that's about it 16:09:41 <Alberth> shunting would be bigger in my view 16:10:24 <andythenorth> consent and consensus are always interesting 16:10:39 <Alberth> but people just submit patches, without making sure it's acceptable, so that comes with the review 16:10:45 <andythenorth> I tend to apply consensus as 'people have stopped disagreeing with me' 16:10:51 <Alberth> while work has already set that direction 16:10:56 <andythenorth> this is kind of interesting http://www.sociocracy.info/consent-versus-consensus/ 16:11:30 <andythenorth> "Consent means that one has an opportunity to present a “reasoned no.” It does not require “a yes,” an affirmation, nor does it imply unanimity or full agreement. It is also not a veto; it must be explained." 16:12:26 <Alberth> oh, there are many level of agreement I agree :p 16:12:58 <andythenorth> the upside of my projects is instant 100% consent :P 16:13:09 <andythenorth> the downside is, I have to do nearly everything :P 16:13:36 <Alberth> but at some point it's about direction, and if some guy doesn't want, it's not going to happen with that guy 16:14:10 <andythenorth> +1 16:14:48 *** Progman has joined #openttd 16:17:52 <Alberth> but we may be deviating from not getting code reviewed :p 16:18:19 <andythenorth> it's all related :P 16:18:39 <Alberth> I am sure FIRS is the center of it :p 16:20:27 <SpComb> how common is it to review your own code with `git diff` before each `git commit`? 16:20:51 <Alberth> 99.5% of the time for me 16:20:55 <SpComb> yeah 16:21:33 <Alberth> but that's just technical correctness, is it all there, nothing weird? 16:22:02 <SpComb> I wouldn't necessarily assume that to always be the case 16:22:17 <Alberth> not "he, if you take this and this approach, it's much simpler, and you can do that and that too" 16:23:11 <Alberth> nope, I usually find 1 or 2 things every 3 commits or so 16:23:28 <Alberth> just an additional empty line usually 16:24:35 <Alberth> on the other hand, when I revisit code, I do sometimes think "oh man this is all wrong" :p 16:24:56 <Alberth> broken in some obscure way :) 16:27:28 <peter1138> Well 16:27:37 <Eddi|zuHause> the "which idiot coded THAT?!?" problem :p 16:27:52 <peter1138> With reviews, you now have "which idiot approved THAT?!?" 16:28:15 <Eddi|zuHause> i meant, the "idiot" is you from 6 months/5 years/whatever ago 16:28:39 <andythenorth> 'former us' 16:28:47 <andythenorth> 'former us' were stupid 16:29:03 <Eddi|zuHause> s/you/yourself/ 16:31:51 <Alberth> everything is broken, for most parts we're still working out how :) 16:32:06 <peter1138> So maybe I should make maintain a branch with everything merged in my forked. 16:32:11 <peter1138> -ed 16:32:33 <peter1138> But maybe I won't bother, because that way lies madness. 16:32:58 <peter1138> As it is, I've effectively given up coding anything because of the review system. 16:33:11 <SpComb> :( 16:33:17 <peter1138> Great principle, it's just not working in practice. 16:33:45 <peter1138> Map array changes. No comment. Nothing. Not even a "holy shit you shouldn't do this" 16:33:46 <SpComb> the worst part of review systems is when you have open PRs that never get reviews, either negative or positive 16:33:51 <SpComb> yeah 16:34:21 <Alberth> number of people that can give meaning full comments on that is close to 0 16:34:24 <peter1138> GUI timing changes. Reviewed briefly, changed made, not looked at again. Stagnant. 16:34:27 <SpComb> because reviewing also takes effort, maybe more effort than writing the code itself sometimes 16:35:44 <peter1138> NRT, I may just merge this later, becuase it's had zero objections. 16:35:55 <Eddi|zuHause> understanding other people's code is a pain 16:36:17 <peter1138> However, I wouldn't be able to fix any issues myself, because it'd just get stuck. 16:36:25 <andythenorth> NRT, we already had TB review 16:36:36 <andythenorth> all the issues TB found were fixed afaik 16:36:36 <SpComb> well, the motivation for reviewing has to also come from somewhere 16:36:40 <andythenorth> then you reviewed it peter1138 16:37:19 <andythenorth> I am +1 to reviews, but I like to know what risk they're preventing them 16:37:32 <andythenorth> shitloads of things are done in software because 'someone said it's correct' 16:37:37 <andythenorth> that's not engineering 16:37:58 <SpComb> I personally appreciate having someone else review my code and give comments 16:38:37 <SpComb> doesn't answer your question, though 16:40:49 <nielsm> map array changes is the nuclear plant the famous bike shed belongs to 16:40:58 <Eddi|zuHause> maybe for "trusted devs" the review should be more optional? like you make a pull request, and if no other dev has said "hold on, i want to review that first" within a certain time, then you get to just commit it? 16:41:27 <andythenorth> yes. the argument isn't "are reviews good or bad" 16:41:35 <andythenorth> it's about gates to progress 16:42:09 <SpComb> it certainly requires someone to also commit to reviewing stuff 16:44:09 * nielsm is downloading an actual linux iso via bittorrent 16:44:51 <Eddi|zuHause> how edgy 16:45:59 <nielsm> huh apparently I've had a linux vm running in HyperV for possibly years here and totally forgotten about it 16:46:08 <nielsm> and I mean actually running 16:46:10 <peter1138> :D 16:46:41 <nielsm> VM created april 2016 16:47:09 *** frosch123 has joined #openttd 16:48:09 <nielsm> sudo dhutdown 16:48:13 <nielsm> without typos 16:48:19 <peter1138> Bit out of date then? 16:48:19 <nielsm> -h now too 17:02:49 *** Flygon has quit IRC 17:04:18 *** debdog has quit IRC 17:06:25 *** debdog has joined #openttd 17:09:21 <Alberth> safe to say it works :p 17:09:53 <peter1138> Not for very long. 17:25:03 <peter1138> Current Game Speed Factor: 1.02x 17:25:04 <peter1138> Ooer 17:26:25 <LordAro> peter1138: what's the future like? 17:26:56 <peter1138> Orange? :S 17:29:48 *** Wacko1976_ has joined #openttd 17:30:14 *** Wolf01 has joined #openttd 17:30:33 <Wolf01> o/ 17:35:34 *** Wacko1976 has quit IRC 17:38:16 <peter1138> \o 17:44:21 <TrueBrain> peter1138: you always "had a patch for that", even when there was no review system; are you sure the review system is to blame? :) 17:44:22 <andythenorth> 30 days, no objections, clear to merge? 17:44:29 <andythenorth> ^ prone to being gamed 17:44:32 <andythenorth> oh it's TrueBrain :) 17:45:06 <TrueBrain> I always wonder why, especially full-fletched-devs, can be so pessistic yet action-less 17:45:11 <TrueBrain> if you don't like something, change it 17:45:25 <TrueBrain> don't sit on the side of the road like a grumpy old man holding your stick up high 17:45:29 <TrueBrain> do something about it! 17:45:34 <TrueBrain> don't expect someone else to do it or something 17:45:54 <TrueBrain> this community has the huge tentency to go back to his hibernation when someone is not yelling loud enough to move forward 17:46:06 *** gelignite has joined #openttd 17:46:32 <TrueBrain> is your PR not getting a review? Tell people and "force" them to review, how-ever nice you can do that (not by passive-aggresively making statements in this channel btw; that is not a good option :D) 17:46:39 <Wolf01> Is game dead yet? 17:46:46 <TrueBrain> is your PR not being merged? Ask yourself why not! Talk about it! Come up with ideas to solve it! 17:46:55 <TrueBrain> fix your own problem! This is Open Source! 17:47:04 <TrueBrain> be active! Even better: be PROACTIVE! 17:47:27 <TrueBrain> but most of all: grow some balls and take action 17:47:32 <TrueBrain> if you consider NRT good enough: fucking merge it 17:47:36 <TrueBrain> if it blows up, it blows up 17:47:38 <TrueBrain> boo-fucking-hoo 17:47:46 <TrueBrain> OpenTTD has a history of shit being blown up 17:47:48 <andythenorth> there's always revert 17:47:51 <andythenorth> no kittens die 17:48:09 <TrueBrain> no movement is always worse than any movement, in which ever direction 17:48:17 <TrueBrain> but worse than no movement, complaining about the lack of moment 17:48:21 <TrueBrain> moment = movement 17:48:50 <TrueBrain> the only (!!) thing that is currently enforced, is that the CI approves a PR, and that a dev merges the PR 17:48:57 <TrueBrain> there is no restriction on WHO reviewed it 17:49:25 <TrueBrain> and if you think that system sucks: come with a suggestion! 17:49:43 <TrueBrain> (am I rattling this cage hard enough? Or should I continue?) 17:50:01 <TrueBrain> Take PR 6828 17:50:06 <TrueBrain> why isn't this accepted yet? 17:50:08 <peter1138> I'm not complaining about the system! 17:51:53 <TrueBrain> [18:32] <peter1138> As it is, I've effectively given up coding anything because of the review system. 17:51:59 <TrueBrain> sorry, you could have fooled me :D How should I have read it? :) 17:53:40 <TrueBrain> (and just to be clear, the above was aimed at any person in this channel; get active! Stop being so passive!) 17:56:29 <TrueBrain> peter1138: PR#6780 -> there are some comments that are not addressed yet; if they are fixed, that seems mergable? 17:59:30 <nielsm> lol, just discovered that the configure script doesn't actually check the user has make installed 17:59:55 <TrueBrain> :D 18:09:23 <TrueBrain> no comments about the content of PR#6821 .. that is a bit sad 18:09:33 <TrueBrain> who to slap for that .... hmmmmmmmmmm 18:10:00 <TrueBrain> frosch123: does it make sense to, on load of a savegame in Single Player, convert all other companies to AIs? 18:11:23 <frosch123> i somewhat doubt that 18:11:43 <frosch123> likely ais won't make sense of the already built stuff 18:11:48 <TrueBrain> it feels a bit weird, but I dont know why 18:12:01 <frosch123> and some people like to play multiplayer with themself 18:12:27 <frosch123> by switching companies 18:12:42 <frosch123> or playing a bit on their own and then continue in multiplayer or whatnot 18:12:46 <nielsm> is there an (easy) way to convert an existing player company to an AI if you ever want to? 18:13:02 <frosch123> so. in essence, i can imagine more people complaining about conversion than prasiing :p 18:13:21 <frosch123> nielsm: i don't think so 18:13:27 <frosch123> but maybe adding a console command makes sense 18:13:44 <frosch123> force ai onto existing company, and remove an ai without klling the company 18:13:45 <TrueBrain> so we are going to reject the PR than :) 18:13:56 <Alberth> simplest option for the ai is to sell everything and start anew 18:14:04 <frosch123> currently we have "startai" and "stopai". maybe people want "attachai" and "detachai" 18:18:20 <nielsm> https://0x0.st/sODf.jpg well there's some sort of test environment at least 18:19:25 <TrueBrain> frosch123: tnx; updated tickets, closed PRs 18:19:47 <nielsm> also looks fine https://0x0.st/sODV.png 18:20:14 <nielsm> interesting it runs at slightly faster than 33.33 here 18:22:02 <TrueBrain> lol 18:23:38 <TrueBrain> nielsm: yeah, tri-state is the bare minimum; it could be a lot better. But I am already happy if someone adds tri-state tbh :D 18:26:26 <TrueBrain> frosch123: tldr, #6846, did he fix all your mentioned points? 18:26:53 <frosch123> i'll check on the weekend 18:26:58 <frosch123> did not manage last weekend 18:27:30 <TrueBrain> no worries 18:27:35 <TrueBrain> was mostly wondering, and didnt feel like reading :D 18:28:09 <TrueBrain> and ugh, I did so little of what I planned .. the stupid weather is really draining me :( 18:28:14 <TrueBrain> I like the forest PR :) 18:30:21 <TrueBrain> tempted to merge it, but I should validate it first .. 18:30:29 <TrueBrain> wish I had a bot I could tell to make me a binary 18:30:33 <TrueBrain> owh wait, that was what I was building 18:32:38 <TrueBrain> I wonder about 6784 .. feels like someone needs to say "left" or "right" on it .. 18:33:04 <Wolf01> Go ahead? 18:33:35 <nielsm> unless anyone has objections for fps-meter I think it's also ready to merge now? 18:33:44 <nielsm> or someone thinks it needs re-testing on mac 18:33:46 <Wolf01> Ships need to be fixed in many ways 18:34:04 <nielsm> bbl food 18:35:11 <TrueBrain> Wolf01: there is a bit of politics involved; if we can make J0anJosep happy by adding a simple statement, it makes his work in his patchpack easier 18:35:43 <Wolf01> So? 18:36:33 <TrueBrain> something about an upwards spiral of hapiness 18:36:40 <TrueBrain> if you help someone else, he is more likely to help you :) 18:36:57 <Wolf01> Fine 18:37:13 <TrueBrain> so what I wonder, how much code is it 18:37:17 <TrueBrain> one line? 18:37:18 <TrueBrain> 20? 18:41:13 <andythenorth> TrueBrain: I am -1 to the forest patch 18:41:25 <andythenorth> I would rather tree algorithm was newgrf content 18:41:37 <Wolf01> ^ 18:42:06 <LordAro> perhaps, but if "tree placement as newgrf" is going to take years and years, why not add this now? 18:42:08 <TrueBrain> which is unlikely to happen ... that is a thoughy! 18:42:14 <LordAro> it can always be taken out later 18:42:44 <TrueBrain> it depends .. do we want "master" to be as close to original as reasonable, and extend by addons (read: newgrfs) 18:42:50 <TrueBrain> or do we extend in "master" too 18:42:59 <TrueBrain> as if we add forest, having tree placement in newgrf is even more unlikely 18:43:05 <Wolf01> I'm for extending with newgrfs 18:43:12 <andythenorth> content all the way 18:43:13 <TrueBrain> (lets put away if newgrfs are the correct solution for this btw) 18:43:19 <andythenorth> just content 18:43:27 <TrueBrain> currently there are two ways to extend OpenTTD: NewGRFs and Squirrel 18:43:29 <andythenorth> extensible, smaller core 18:43:32 <andythenorth> more patchpacks 18:43:34 <TrueBrain> rest of the world uses lua :D 18:43:49 <andythenorth> fewer conflicts, fewer tarpits 18:43:56 <andythenorth> more state = more tarpit 18:44:08 <TrueBrain> I really like forest, but I do agree ... but do we want to become the project that rejects all new ideas? :D 18:44:08 <andythenorth> more tarpit = slower development, or total stasis 18:44:30 <andythenorth> if only there was a really active patchpack :( 18:44:33 <andythenorth> oh wait 18:45:02 <TrueBrain> if only OpenTTD still had GPMI in its core ... :P (that allows binary extension via addons) 18:45:12 <andythenorth> https://github.com/JGRennison/OpenTTD-patches/commits/jgrpp 18:45:36 <TrueBrain> frosch123: do you agree, should we reject forest patch (refering him to the above URL)? 18:45:58 <andythenorth> also https://wiki.openttd.org/FAQ_development#What_are_the_goals_of_the_offical_branch.3F 18:46:21 <frosch123> TrueBrain: i have not looked at the tree patch 18:46:38 <TrueBrain> in very simple words: it adds another tree-placement algorithm, that favors forests 18:46:41 <frosch123> but newgrf- or squirrel-based tree stuff is the argument, that is not going to happen 18:46:50 <frosch123> *if 18:46:51 <TrueBrain> (YATP - Yet Another Tree Placement) 18:47:15 <TrueBrain> no, the argument is: should 'master' have YATP, or should it be by extension (either by an addon or by a patchpack) 18:47:57 <frosch123> i don't see the latter as a valid option 18:48:05 <TrueBrain> care to explain why? 18:48:11 <TrueBrain> (sounds more rude than intended :D) 18:48:17 <TrueBrain> could you elaborate? :D 18:48:17 <frosch123> there are many trees on the map 18:48:25 <frosch123> neither newgrf nor scripts are good with managing that 18:48:42 <andythenorth> do we really need so many trees? 18:48:42 <TrueBrain> isn't that irrelevant for the choice? HOW the latter should be solved, is a second talk? 18:48:46 <frosch123> so i am quite sure there will never be "addons" for tree placement 18:49:02 <TrueBrain> but if we choice when to add stuff to 'master' based on how difficult it is to do it in an addon, line becomes very blurry? 18:49:11 <TrueBrain> patchpacks are also addons, in some sense? 18:49:19 <frosch123> anyway, i ignore trees with invisiblity. so i have no opinion on the tree patch itself 18:49:20 <andythenorth> I would prefer to continue forcing choices by preferring content 18:49:24 <frosch123> if it is good, it should be added 18:49:35 *** Progman_ has joined #openttd 18:49:53 <andythenorth> I don't want to discuss an implementation, but tree tiles could be done with much less variety and nobody would notice 18:49:56 <frosch123> i assume it does not affect savegames 18:50:10 <frosch123> so changing/removing tree placement is fine any time 18:50:29 <TrueBrain> but do we want to bother with that in 'master', or is that something for patchpacks? 18:51:01 <andythenorth> patchpacks, possibly port a winning implementation if there is one 18:51:15 <andythenorth> growing more half-assed variants is not appealing 18:51:24 <andythenorth> and we don't seem to delete old dead implementations 18:51:25 <TrueBrain> well, you do want to prevent another PBS, yes :P 18:51:45 <TrueBrain> that last one, is the tricky part; once added to master, removing it is very difficult, as people start to depend on it 18:51:52 <andythenorth> or another New Economy 18:51:53 <TrueBrain> people still complain that the original AI is removed 18:52:06 <TrueBrain> but OpenTTD with TGP is .. ughhhhh, sucky 18:52:14 <TrueBrain> so there are really two sides to this coin :) 18:52:15 <frosch123> reading it, i would complain about the description :p 18:52:18 <andythenorth> 'people complain' doesn't actually mean very much 18:52:22 <frosch123> "'Original' plants trees uniformly scattered, 'Improved' plants them in groups, 'Forest' plants them in forests" 18:52:34 <frosch123> ^^ that says nothing 18:52:45 <frosch123> maybe it should say "large groups"? 18:52:47 <andythenorth> 'people complain' is easily circumvented by not reading tt-forums or the german forum :P 18:52:56 <TrueBrain> you nitpick on that in the PR, I was trying to see what we are at in the more high-level picture of this all :) 18:53:26 <TrueBrain> when are a group of trees a forest :) 18:53:37 <TrueBrain> for me the description reads fine btw :) 18:53:38 <andythenorth> when nobody hears them fall 18:54:09 <frosch123> anyway, everything the patch adds can be removed without breaking anything 18:54:31 *** Progman has quit IRC 18:54:41 <TrueBrain> okay; than I am going to merge it 18:54:44 *** Progman_ is now known as Progman 18:54:46 <TrueBrain> or do you want the description changed? 18:55:00 <frosch123> so, if anyone has tried it, and subjectively likes the appearance, i am +1 to it 18:55:14 <TrueBrain> ugh, this "trying" again .... 18:55:18 <TrueBrain> andythenorth: can you try that PR? :P :P 18:55:21 <andythenorth> yes 18:55:25 <andythenorth> give me 5-10 mins 18:55:29 <TrueBrain> \o/ 18:56:05 <TrueBrain> okay ... FPS meter ... think everyone pissed over that enough ... I see no blockers .... 18:56:09 <frosch123> changing the text can be a separate pr 18:56:11 <TrueBrain> merging time? (going once ... goince twice ...) 18:56:15 * andythenorth compiling 18:56:20 <frosch123> "improved" is just as stupid :) 18:56:23 * andythenorth needs the new quad core mac 18:58:02 <TrueBrain> now to squash this FPS ..... 18:58:03 <andythenorth> wow my fans span up :P 18:58:11 <TrueBrain> as 49 commits is a bit much 18:58:34 <andythenorth> I know Apple analogies are like cat sick 18:58:41 <TrueBrain> nielsm: only the first commit-message is sufficient? Or does it need more? 18:58:42 <andythenorth> but I read this today, re 'people will complain' 18:58:43 <andythenorth> https://forums.macrumors.com/threads/true-tone-on-external-displays-doesnt-work-with-macbook-pro-in-clamshell-mode-updated.2128138/#post-26259454 18:58:59 <TrueBrain> lolz 18:59:27 <TrueBrain> C++11 stuff .. oef .. that is a difficult one 18:59:47 <andythenorth> ok the forest patch is bad 18:59:50 <andythenorth> don't merge 18:59:56 <andythenorth> I need to comment on PR :P 19:00:02 <TrueBrain> all yours 19:00:04 <andythenorth> just thinking how to do it kindly 19:00:13 <TrueBrain> screenshot? :) 19:00:14 <andythenorth> it makes random diamond shaped groups of treees 19:00:40 <andythenorth> it k 19:00:44 <andythenorth> screenshotting 19:01:26 <TrueBrain> and yes: BE NICE! :) 19:01:47 <andythenorth> https://github.com/OpenTTD/OpenTTD/pull/6848#issuecomment-406381059 19:01:57 <andythenorth> so problems 19:01:58 <andythenorth> - looks bad 19:02:05 <andythenorth> - leaves no trees anywhere else 19:02:09 <TrueBrain> diamond-shapes are to be expected :) 19:02:15 <andythenorth> - conflates with forest industry 19:02:15 <TrueBrain> and that was the idea: forest-only 19:02:46 <andythenorth> - options makes no sense in world-gen menu 'none', 'original', 'improved', 'forest' 19:02:49 <TrueBrain> feels like it needs a settings between "improved" and "forest" :D 19:02:51 <andythenorth> UX: which do I pick? 19:03:05 <andythenorth> why don't we just fix on one good, and remove the bullshit? 19:03:06 <TrueBrain> yeah, as frosch123 said, "improved" is now silly 19:03:25 <TrueBrain> okay, make this into a nice reply, state a clear opinion and direction for the author to execute 19:03:41 <TrueBrain> (please don't leave a: you figure it out, reply; that is not nice for a first-time contributor) 19:03:49 <andythenorth> on the plus side, reminds me of http://everygamegoing.com/ills/bbc_b/acornsoft/tapes/Jcb_Digger_002.gif 19:03:59 <andythenorth> yeah, these replies demand the most care 19:05:17 <TrueBrain> frosch123: to bother you some more (sorry :D), adding 2 bytes to _map, your thoughts? How is memory etc these days? 19:06:07 <frosch123> that's no issue 19:06:21 <frosch123> you can also add 8 just fine :p 19:06:23 <TrueBrain> percentage-wise it is less and less :D 19:06:40 <frosch123> map array is neglible to the rest 19:06:50 <TrueBrain> "back in my day", it was a huge impact, as adding bytes often meant some players no longer could play big maps 19:06:59 <frosch123> esp. on clients with 32bpp 19:07:14 <TrueBrain> is there any "policy" in place for assignments? Or is a PR like peter1138's fine? 19:08:39 <Eddi|zuHause> but... map array has become this mythical catch-all for rejecting any major feature. you can't just expand it on a whim and undermine that :p 19:08:53 <frosch123> we try to put the same stuff into the same bits for different tile types 19:09:12 <frosch123> so, unless the patch contains switches on tiletype to read on different bits, it should be fine 19:09:22 <TrueBrain> *picks up what-ever Eddi|zuHause says, opens the door, puts it next to the trash*, anything else? :D 19:09:57 <nielsm> newgrf tree planting mechanism ideas: https://0x0.st/sOkT.txt 19:10:13 <TrueBrain> that only leaves me to wonder ... he added a m8 as 16bit .. while most (not all) are 8bit 19:10:22 <TrueBrain> better to add 2 8bit, or are we just going all-the-way here 19:10:58 <frosch123> i think we talked about that before, and i prefered 16bit 19:10:59 <TrueBrain> (only 6 bits are used, of the 16bit) 19:11:37 <frosch123> iirc peter had multiple versions of the patch, one with 2x8, and one with 1x16 19:11:43 <frosch123> and the favour was for 1x16 19:11:52 <TrueBrain> ack; that means someone just has to review the moving (and expanding) of railtypes 19:12:14 <TrueBrain> and don't share that favour, but I don't care enough (only care enough to mention it :P) 19:13:01 <frosch123> the idea was that reading a subset of bits is easier than merging multiple bytes into something bigger 19:13:27 <frosch123> we have some things, like objects, where we store ids over multiple bytes 19:13:40 <frosch123> like m2 and m7 bits 2 to 5 or something 19:13:55 <andythenorth> nielsm: re trees, it does just need someone to pick it up and drive it through 19:14:01 <andythenorth> many things have been declared impossible before 19:14:02 <andythenorth> then done :P 19:14:17 <andythenorth> 'performance reasons' is ~50% bullshit excuse :) 19:14:30 <TrueBrain> nielsm: but is the first commit message of your FPS sufficient? (I am about to squash the whole damn thing into a single commit, and want to be sure that is okay) 19:14:37 <nielsm> https://www.tt-forums.net/viewtopic.php?f=31&t=83507#p1209912 <-- fps-meter already being used to diagnose issues! 19:14:48 <nielsm> TrueBrain let me just check a bit 19:14:56 <TrueBrain> yeah ... maps not being 256x256 was impossible! 19:15:07 <TrueBrain> (and that was my first patch for OpenTTD .. .14 years ago :P) 19:15:39 <TrueBrain> tnx btw frosch123! Always helps to know in a PR what was already talked over :) 19:16:06 <Eddi|zuHause> most of my patches were 1-3 lines, of things other people were apparently not bothered enough to do them 19:16:30 <TrueBrain> I added some labels in Pull Request 19:16:32 * andythenorth looking for the 'most of savegame is trees' issue 19:16:34 <nielsm> TrueBrain, if you can make commit message the title of PR + first paragraph in PR initial post 19:16:34 <andythenorth> can't find it 19:16:44 <nielsm> i.e.: Frame rate and various game loop/graphics timing measurements and graphs. Accessible via the Help menu, and can print some stats in the console via the `fps` command. 19:16:54 <Eddi|zuHause> i think my biggest patch that got in (line-wise) was the screenshot thing 19:17:09 <TrueBrain> nielsm: done! 19:17:30 <TrueBrain> seriously nicely done for such a huge patch 19:17:32 <Eddi|zuHause> where it took the area that is currently visible, but applied the max zoom to it 19:17:36 <TrueBrain> how we say it in dutch: "petje af" 19:17:43 *** Thedarkb has joined #openttd 19:17:50 *** gelignite has quit IRC 19:18:16 <TrueBrain> so that leaves NRT and std:vector 19:18:17 <Eddi|zuHause> which turned out not very difficult to do, just take the whole-map screenshot routine, and add some limits to it 19:18:59 <Eddi|zuHause> all of the infrastructure for it was already in place, just had to be combined the right way 19:19:07 <TrueBrain> frosch123: I assume you agree'd withi the std::vector stuff? (last quesiton of the evening, I promise) 19:19:46 <TrueBrain> nielsm: your comment in #6817, is a bit vague to me; are those two changes in there, or should they be added? 19:20:03 <andythenorth> so LordAro is ok to review this one? o_O https://github.com/OpenTTD/OpenTTD/pull/6805 19:20:06 <nielsm> I think the std::vector patch requires C++11 but not sure 19:20:20 <TrueBrain> LordAro: related to #6817, do we need to enforce build systems to use C++11? With that PR, won't they simply fail? 19:20:54 <nielsm> TrueBrain the GUIBrideList is not being fixed iirc 19:21:05 <TrueBrain> can you make that more clear in your comment, so the author knows if he has any actions left? 19:21:26 <TrueBrain> (in general in pull requests, please make it very obvious to the author if it is an action on his side, or just a remark :D) 19:22:08 <TrueBrain> and as it is a first-time contributor, make it something nice :) "Would you mind to also fix ... as they are really important to fix ..." :D 19:22:09 <TrueBrain> tnx! 19:22:55 <TrueBrain> ah, that introduces more of that 'auto' stuff ... 'auto' is dutch for 'car', so in my head I always go: BROOM BROOM 19:23:58 <frosch123> TrueBrain: it has some codestyle issues, but the principle is fine 19:24:02 <andythenorth> so the forest patch....is contentish trees ruled out or no? 19:24:05 <andythenorth> before I mislead Sylvain 19:24:21 <andythenorth> or it's a non-issue either way? 19:24:50 <nielsm> also just before I was so busy looking at code I almost forgot my pizza in the oven :( 19:25:02 <nielsm> got it out while it was still edible 19:25:03 <andythenorth> :o 19:25:09 <andythenorth> I cook using timer on my phone 19:25:12 <andythenorth> only way 19:25:24 <andythenorth> primary benefit of smartphone 19:25:25 <nielsm> yeah I usually set a timer, forgot it this one time 19:25:34 <TrueBrain> frosch123: ack, tnx :) 19:25:49 <frosch123> i think noone read all the diffs :p 19:26:11 <TrueBrain> my oven just stops after the time I put in 19:26:14 <TrueBrain> avoids any mistakes :D 19:26:23 <andythenorth> poka-yoke 19:26:33 <frosch123> https://github.com/OpenTTD/OpenTTD/pull/6817/commits/445bcdbb3a9bf6f440560cf492692da13bc8073d#diff-0b90577e7f946552f0ac4285ea1b9ae6 <- no idea whether that is done 19:26:37 <andythenorth> mistake of forgetting to eat the food? o_O 19:27:06 <TrueBrain> frosch123: lolz :D 19:27:10 <nielsm> ah yeah leaving commented-out code is a bad smell 19:27:57 <TrueBrain> so, it needs a review; that is fine :) 19:28:19 <TrueBrain> we closed 124 PRs already :o 19:28:21 <TrueBrain> damn 19:28:50 *** glx has joined #openttd 19:28:50 *** ChanServ sets mode: +v glx 19:28:52 <TrueBrain> happy the FPS stuff is in :D 19:28:55 <TrueBrain> owh hi glx! 19:29:10 <glx> hi 19:29:12 <andythenorth> TrueBrain: damn yes 19:29:16 <andythenorth> I counted at the weekend 19:29:18 <andythenorth> shitloads 19:29:25 <andythenorth> just need binaries now :) 19:29:25 <TrueBrain> the number... is just there 19:29:27 <TrueBrain> no need to count 19:29:32 <TrueBrain> YES! Working on it .. 19:29:43 <TrueBrain> tomorrow call with Google Cloud Platform .. I so hope they can offer something 19:29:49 <TrueBrain> would make shit so much easier (and faster) 19:30:01 <andythenorth> oof I counted commits manually (by counting pages) 19:30:11 <andythenorth> I should github better 19:35:25 <Alberth> better call microsoft nowadays :) 19:35:34 <TrueBrain> to learn GitHub? 19:36:51 <andythenorth> hmm 19:37:07 <andythenorth> in the tank game we say 'git gud' when players are lame 19:37:12 <andythenorth> I need to 'github gud' :P 19:37:37 <TrueBrain> rrreeeeeeeel gud 19:40:33 <nielsm> okay I'm impressed how MS managed to hide Safe Mode in win10 behind a maze of buttons 19:40:50 <nielsm> it seems to prefer you reinstall windows before trying to boot to safe mode 19:41:29 <Alberth> instead of calling google 19:44:31 <TrueBrain> Azure will be fine too; but for kubernetes, GCP is better :) 19:47:06 <LordAro> TrueBrain: ideally the build system should be modified indeed 19:47:35 <TrueBrain> a bit more snippets of information would be greatly appreciated :D 19:47:38 <LordAro> otherwise someone will try compiling with something ancient and get some obscure error 19:47:50 <TrueBrain> modified in what sense? 19:47:54 <TrueBrain> just 'modified'? :D 19:47:59 *** sim-al2 has joined #openttd 19:48:20 <LordAro> some sort of test program probably 19:48:36 <TrueBrain> owh, you want to add a check if you are using a C++11 capable compiler! 19:48:41 <TrueBrain> see, that is more sensible :) 19:48:46 <LordAro> sorry, wasn't clear :p 19:48:52 <TrueBrain> no, it was very vague :) 19:48:55 <TrueBrain> very :D 19:49:00 <TrueBrain> I guess porting to cmake is easier :P 19:49:05 <LordAro> quite possibly! 19:50:51 <Alberth> autotools! :p 19:50:55 <LordAro> https://cmake.org/cmake/help/latest/manual/cmake-compile-features.7.html#manual:cmake-compile-features(7) & https://cmake.org/cmake/help/latest/prop_gbl/CMAKE_CXX_KNOWN_FEATURES.html#prop_gbl:CMAKE_CXX_KNOWN_FEATURES 19:51:07 <LordAro> Alberth: i tried that, TB shouted at me :p 19:51:17 <Alberth> hah :) 19:52:11 <TrueBrain> wait, who needs to be "corrected? :P 19:52:47 <TrueBrain> LordAro: in CMake you just say the lowest allowed compiler, and it will do its thing correctly :D Which is really nice 19:57:03 <TrueBrain> andythenorth: also a note on the forest patch: theyALL have EXACTLY the same size 19:57:05 <TrueBrain> bit boring 19:58:03 <andythenorth> I am writing a comment 19:58:15 <andythenorth> kind of hard because I can never approve it 19:58:19 <andythenorth> unless I can? :P 19:58:21 * andythenorth checks 19:58:28 <TrueBrain> sure you can 19:58:36 <TrueBrain> otherwise by proxy! :P 19:58:40 <andythenorth> k 19:58:54 <andythenorth> all is well 20:01:31 <andythenorth> done that comment 20:03:57 *** Alberth has left #openttd 20:04:16 <TrueBrain> and I am off! nn! 20:24:58 <Eddi|zuHause> there are seriously chemicals called "Alite" and "Belite"? 20:27:19 <andythenorth> sounds like a FIRS economy 20:27:40 <Eddi|zuHause> apparently they're components in cement 20:28:05 <andythenorth> o_O 20:28:05 <Eddi|zuHause> with different ratio of Ca to Si 20:28:09 <andythenorth> worth knowinng 20:28:50 <Eddi|zuHause> one is Ca3SiO5 and the other Ca2SiO4 20:28:57 <Eddi|zuHause> and i instantly forgot which is which 20:29:52 <frosch123> there is also cerite 20:31:16 <andythenorth> cement economy 20:31:23 <andythenorth> super extra detail 20:31:53 <andythenorth> clinker 20:31:57 <andythenorth> gypsum 20:31:58 <andythenorth> slag 20:39:43 *** Laedek has quit IRC 20:58:46 <Wolf01> 'night 20:58:50 *** Wolf01 has quit IRC 20:58:57 *** andythenorth has quit IRC 21:26:37 *** Progman has quit IRC 21:26:39 *** frosch123 has quit IRC 21:43:50 *** sim-al2 has quit IRC 21:52:36 *** sim-al2 has joined #openttd 22:03:00 *** Laedek has joined #openttd 22:23:01 *** Thedarkb has quit IRC 22:30:12 *** Thedarkb has joined #openttd 22:30:35 *** Lejving_ has joined #openttd 22:34:09 *** Lejving has quit IRC 22:41:32 *** Wormnest has quit IRC 22:47:33 *** RafiX has quit IRC 23:33:17 *** snail_UES_ has joined #openttd 23:51:14 *** HerzogDeXtEr has quit IRC