-
rzezeski
out of curiosity, what is the pain with building mattermost on illumos? Is it illumos support in go? or is it the dependencies that mm relies on not having illumos support?
-
sjorge
Well for started the current commit build.sh does not produce a build for me
-
rzezeski
for the current version it is set to or a newer version?
-
sjorge
There also a bunch of patches do not apply to newer version and I couldn't get even mmctrl to build outside of the build system
-
sjorge
The patches apply to the current version
-
sjorge
But I wasn't able to produce a copy of the package that is currently in extra with it
-
andyf
I think it's some dependencies in the chain that just don't have illumos support.
-
andyf
and patching things to work is getting harder and harder
-
rzezeski
hadfl mentioned that the deps have some issues, and it just made me wonder exactly the issue is. Like is it Linux-isms in the deps?
-
andyf
I don't know why the existing one doesn't build, it certainly did recently
-
andyf
but they might have removed some tags or releases in github or something like that.
-
sjorge
IIRC the newer go it uses does not like the current build.sh
-
sjorge
It says no can do, this is deprecated
-
sjorge
I tried hacking around it but didn't get far
-
andyf
You can always add `set_gover <ver>` near the top of the script
-
rzezeski
sjorge: yea I tried building it while waiting for some other stuff, but with an updated mm version. I saw that error. But looking at mm release notes they claim to build with an older go (for the version I set it to), so I switched to that and it got further.
-
sjorge
anyway to check what go ver the current one was build with ?
-
danmcd
-
rzezeski
I dunno, but if you look at the release notes they build 7.8.11 with Go 1.18.1. So I imagine the official mm build used something older than that. But you could try building it with the go-118 package by using `set_gover 1.18`.
-
sjorge
I might give that a go (ha) somewhere this week when I got a bit more time
-
rzezeski
Yea, looking again at mm release notes, they claim to build 7.5.x with Go 1.18.1.
-
sjorge
silly phone
-
sjorge
Why would the touch and old default to the bad action
-
sjorge
set_gover 1.18
-
sjorge
Does at least seem to have the current build.sh work
-
sjorge
Once it finished that I'll at least do a PR just for that so the current build is not broken
-
hadfl
it's a while since i had a look at mm. but iirc the main issue was a new dependency they pulled in. there is an open issue but it has stalled since...
-
hadfl
-
sjorge
Hmm yeah I that would probably also indeed be a rather big issue
-
sjorge
But with the gover set to 1.18 the current build.sh at least produce a package om bloody
-
sjorge
Which is further than I got last time
-
rzezeski
A quick glance makes me think this libc library is a "write libc in go" type project? If that's the case the better solution would be to patch mm to sidestep it completely. An alternate libc, in any language, is probably not a great idea on illumos.
-
hadfl
again iirc since it's been a while but it just gets pulled in because of sqlite support. they switched to a module from the same family that relies on this libc implementation. so patching out sqlite support would be an option
-
hadfl
however, it still boils down to someone interested and willing to keep support for a native mm.
-
sjorge
ooh it got merged! once it hits bloody, i'll just move my mattermost zone over to lx
-
sjorge
that way plugins will probably also work 😅