Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ARM and ARM64 builds on docker hub #138

Closed
SenorSmartyPants opened this issue Feb 1, 2020 · 13 comments · Fixed by #1681
Closed

Add ARM and ARM64 builds on docker hub #138

SenorSmartyPants opened this issue Feb 1, 2020 · 13 comments · Fixed by #1681
Assignees
Labels
critical Critical issue dependencies Pull requests that update a dependency file docker Docker related issues. runtime Issues relating to the runtime requirements and behavior.
Milestone

Comments

@SenorSmartyPants
Copy link

Please add ARM and ARM64 builds on docker hub in addition to AMD64. Thanks

@PiDockMedia
Copy link
Contributor

comixed/comixed:latest should be multi-arch now. I only tested the amd64 image but it seemed to build with no errors. Give it a try and let me know how it goes for you.

@SenorSmartyPants
Copy link
Author

This seemed to be working earlier, but when I pulled latest there is only amd64 build now.

@mcpierce mcpierce assigned mcpierce and PiDockMedia and unassigned mcpierce Apr 3, 2020
@mcpierce mcpierce added the docker Docker related issues. label Apr 3, 2020
@crivote
Copy link

crivote commented Jun 11, 2020

Hi, I tried to install a container in a Raspberry pi 4 using your image on dockerhub but got the error:
exec user process caused "exec format error"
I think that means there are'nt still ARM builds. Do you plan to release them? Thanks!

@mcpierce mcpierce added this to the No Milestone milestone Jun 11, 2020
@mcpierce
Copy link
Contributor

@crivote Hi, thanks for bringing that to our attention. Do you know what exec is causing that error? We have a dependency on webp-imageio for handling WebP pages in a comic, which is the only part of the project that contains native code. In the JAR I see:

     0 Thu Apr 23 11:36:56 EDT 2020 native/
     0 Thu Apr 23 11:36:56 EDT 2020 native/win/
     0 Thu Apr 23 11:36:56 EDT 2020 native/win/32/
404992 Thu Apr 23 11:36:56 EDT 2020 native/win/32/webp-imageio.dll
     0 Thu Apr 23 11:36:56 EDT 2020 native/win/64/
504832 Thu Apr 23 11:36:56 EDT 2020 native/win/64/webp-imageio.dll
     0 Thu Apr 23 11:36:56 EDT 2020 native/mac/
     0 Thu Apr 23 11:36:56 EDT 2020 native/mac/64/
626700 Thu Apr 23 11:36:56 EDT 2020 native/mac/64/libwebp-imageio.dylib
     0 Thu Apr 23 11:36:56 EDT 2020 native/linux/
     0 Thu Apr 23 11:36:56 EDT 2020 native/linux/64/
712056 Thu Apr 23 11:36:56 EDT 2020 native/linux/64/libwebp-imageio.so

So is it one of those that's causing the error for you? If so, I'd recommend visiting their Github page and file a bug with them, and referring back to this ticket as well.

@mcpierce mcpierce removed this from the No Milestone milestone Dec 18, 2020
@FrancisKunz
Copy link

Hi,

still got the same problem installing Comixed on a Raspberry Pi4 with Ubuntu server:

standard_init_linux.go:211: exec user process caused "exec format error"

@mcpierce mcpierce assigned mcpierce and unassigned PiDockMedia Apr 7, 2021
@mcpierce mcpierce added dependencies Pull requests that update a dependency file runtime Issues relating to the runtime requirements and behavior. labels Apr 7, 2021
@mcpierce mcpierce added this to the 0.8 milestone Apr 7, 2021
mcpierce added a commit to mcpierce/comixed that referenced this issue Apr 7, 2021
mcpierce added a commit that referenced this issue Apr 7, 2021
@mcpierce
Copy link
Contributor

mcpierce commented Apr 7, 2021

@FrancisKunz Ah, and I see that webp-imageio has had a few releases since this issue was reported (we're using 0.1.2 and right now they have 0.1.6 available).

I'm cutting a new release (0.8.0-2.0) now and, if you find that it fixes this issue, please let me know and we can mark this as fixed.

mcpierce added a commit that referenced this issue Apr 7, 2021
@FrancisKunz
Copy link

@mcpierce as soon as the new docker image is up, i'll update you on the status

@FrancisKunz
Copy link

bad news ...
Comixed 0.8.0-2.0
standard_init_linux.go:211: exec user process caused "exec format error"

@mcpierce
Copy link
Contributor

mcpierce commented Apr 8, 2021

@FrancisKunz Bugger.... Sorry to say, but it's a bug outside of our codebase. That's an error in the webp-imageio library that we're using, and we can't fix bugs like that.

Would you mind opening a bug with them and referencing this bug?

@mcpierce mcpierce reopened this Sep 14, 2023
@mcpierce
Copy link
Contributor

Re-opening this so we make the Docker image itself multi-arch.

@mcpierce mcpierce modified the milestones: 0.8, 1.6 Sep 14, 2023
@eneglan
Copy link

eneglan commented Sep 14, 2023

Thanks for re-opening this. I have no real experience with building docker images but judging from this guide https://www.docker.com/blog/multi-arch-build-and-images-the-simple-way/ it should be quite straightforward for you to create a multi-arch image with buildx.

@mcpierce mcpierce added the critical Critical issue label Sep 14, 2023
@mcpierce
Copy link
Contributor

Thanks for re-opening this. I have no real experience with building docker images but judging from this guide https://www.docker.com/blog/multi-arch-build-and-images-the-simple-way/ it should be quite straightforward for you to create a multi-arch image with buildx.

Literally the link I was just reading to see what I can do to meet this request. 😄

@mcpierce
Copy link
Contributor

I wonder, since we use their Github action to publish our updates to Docker, if we can't use this feature to publish for ARM as well?

mcpierce added a commit to mcpierce/comixed that referenced this issue Sep 14, 2023
Also added semver tagging of the docker releases.
mcpierce added a commit that referenced this issue Sep 14, 2023
Also added semver tagging of the docker releases.
mcpierce added a commit that referenced this issue Sep 14, 2023
Also added semver tagging of the docker releases.
mcpierce added a commit that referenced this issue Sep 14, 2023
Also added semver tagging of the docker releases.
mcpierce added a commit to mcpierce/comixed that referenced this issue Sep 15, 2023
mcpierce added a commit to mcpierce/comixed that referenced this issue Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
critical Critical issue dependencies Pull requests that update a dependency file docker Docker related issues. runtime Issues relating to the runtime requirements and behavior.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants