Closed Legogris closed 4 years ago
I don't believe a dependency on cffi has changed between 1.17 and 1.18. It seems that you're missing some headers though, the directions for installing cffi might help: https://cffi.readthedocs.io/en/latest/installation.html
(Looks like you need the python-dev and libffi-dev packages?)
@clokep Note that this is just building the container image from /docker/Dockerfile
, which I assume should be self-contained. I'd wager this may be related to https://github.com/matrix-org/synapse/pull/7839. Busy now but may look into making a PR at some point in the future unless anyone else addresses it.
I somehow totally missed the word "Docker" 😄
Yes, it is likely related to that.
I think this has been addressed in https://github.com/matrix-org/synapse/pull/7921, although that PR seems to add more libraries than are necessary. A PR which fixes specifically this problem (independently of multiarch support) would be appreciated.
I think this has been addressed in #7921, although that PR seems to add more libraries than are necessary. A PR which fixes specifically this problem (independently of multiarch support) would be appreciated.
This is precisely multi-arch support I'm raising here, actually. Can't comment on if either of those specific libraries are required, though.
Yeah ill raise a pr to get this working on other archs whilst im getting the rest of the multi-arch docker stuff through.
So ive raise https://github.com/matrix-org/synapse/pull/8144, with all the packages I needed to build the docker image on my pi. I'm not sure why they arent needed on amd64, I assume the base image is different somehow.
My multi-arch branch needed even more dependencies, so could be aarch64 might still fail, i could put them all in to be on the safe side or @Legogris , could you try the dockerfile and confirm if it works or not before we merge?
actually, since the delta between this and the multi-arch pr was just libxml, ive added it in to be on the safe side, its only the build image and a binary so cost is negligible in terms of time.
I'm pretty confident this was working for 1.17.
Getting the following when building docker image for
arm64
.