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
arm64 build randomly crashes with varying stack traces #113
Comments
It's non-deterministic
|
Compiled it with go1.18beta1 in the hopes that that'll magically fix it. No dice.
|
Hi |
Yeah I actually think it may have been related to |
Thanks for testing! |
Running into the same problem. It seems the arm64 docker images contain the x86_64 version?! Running inside binwiederhier/ntfy:v1.14.0-arm64v8: $ file /usr/bin/ntfy
/usr/bin/ntfy: ELF 64-bit LSB executable, x86-64, version 1 (GNU/Linux), statically linked, for GNU/Linux 3.2.0, Go BuildID=C-k_QQ5tGpfq0jf2PkzM/g1fJZSinrAJaIbeaILA8/2zJMzfzFaQjc9TW67dR-/TCEZaAXwb380n2_Eu-wc, BuildID[sha1]=3da21a972eee54975e3c908388c495eff5ce8d28, stripped UPDATE: I can't find any CI build configurations in the repo. Are the images built manually? |
I'm basically stuck because I can't test the arm stuff at all. I tried with qemu but I'm not sure of it the emulator is broken, because it shows "not implemented" exceptions. The builds are defined here: https://github.com/binwiederhier/ntfy/blob/main/.goreleaser.yml#L27 You can check the makefile to see how it's called. |
Alright let me dig a little there:
So that's correct ^^ But you are right, the Docker image does contain the wrong architecture binary:
This must be an issue with GoReleaser or my config then. I'll look. |
This line seems off:
|
Seems you are missing |
Yup I just saw that too. I'll release v1.14.1 now. Here's the fix: 8187b49 Thank you very much for helping find this. I don't know how that ever worked, or maybe it got removed 🤷 |
Seems you had it removed accidentally :) 093154f#diff-42e26dc67aed8aa3edb2472b4403288c1699fb6dc47419b9a475f0f224fe4689L92 |
Indeed ... :-) |
The new arm64 docker image seems to work. Thanks for your work! |
The text was updated successfully, but these errors were encountered: