+tech-pkg (I hope you don't mind) On Mon, 22 Nov 2021, Bartek Krawczyk wrote:
W dniu 22.11.2021 o 14:13, Benny Siegert pisze:do you see any problems with wip/adguardhome? May it be imported to pkgsrc?It can probably be imported, I just haven't had the time to do it. I am worried about it not starting with Go 1.17. Did you report this upstream? What happens when you do build it with 1.17?I checked with upstream and they have a target of removing Go 1.16 support with 0.108.x. We're now on 0.106.3 so 2 major versions from now. I'll try to debug it further but maybe you could import it when you have time so we have more eyes looking at it and testing?
I took a close look at adguardhome today, and I am uncomfortable about importing it into pkgsrc. It just seems very brittle.
Two points that I think express this brittleness:1. The package inherently only builds with a single Go version, due to the way the qtls package does unsafe TLS-related things assuming a certain struct definition. (Never mind that unsafely manipulating TLS-related data seems like a really bad idea!) As a result, this would break each time we upgrade Go versions. At the moment, we remove Go versions as they become unsupported, i.e. we would remove go116 when go118 becomes stable. Having adguardhome in pkgsrc itself would mean that we have to keep it in mind whenever we deprecate a Go version.
2. The way you have to regenerate frontend assets by hand and make a local distfile means additional steps and likely additional breakage.
From my point of view, I would rather keep this package in wip for now. Sorry.
But perhaps one of the other pkgsrc developers has a different opinion? -- Benny