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

Temporarily remove arm7 build from downloads #6948

Open
mrsdizzie opened this issue May 14, 2019 · 7 comments
Open

Temporarily remove arm7 build from downloads #6948

mrsdizzie opened this issue May 14, 2019 · 7 comments
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile type/bug

Comments

@mrsdizzie
Copy link
Member

Since the arm7 build stopped working back at #6339, should we remove it from the build/downloads page to avoid people downloading it and then reporting that it doesn't work?

arm6 builds will work fine on arm7+ since it arm backwards compatible (golang itself only offers an arm6 build). I tried to debug why the arm7 build was segfaulting, but it was happening at a pretty low level and I wasn't able to get anywhere (I don't think it would be code in Gitea that needs to change even if so). I did see that glibc has deprecated statically linked applications that use dlopen, so something will probably have to change in the future anyway:

https://sourceware.org/ml/libc-announce/2018/msg00000.html

@techknowlogick
Copy link
Member

techknowlogick commented May 14, 2019

I think it could be code in Gitea that is causing this, as I have another project that uses the same xgo image and build scripts as Gitea, but still manages to work with the arm7 build (it even uses go-sqlite3).

Perhaps this might be helpful in tracing where in xgo could be the issue: mattn/go-sqlite3#504

Edit: there is also: http://dayo.com.ng/cross-compiling-go-sqlite3-for-arm7-from-debian-9/ and http://jensd.be/800/linux/cross-compiling-for-arm-with-ubuntu-16-04-lts

@techknowlogick techknowlogick added type/bug topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile labels May 14, 2019
@mrsdizzie
Copy link
Member Author

The gitea arm7 binary as-is fails somehow with the combination of bindata and sqlite3 on arm7-- it works fine if you omit either one but segfaults when using both together (which is required of course). So it isn't just that sqlite3 has an issue on arm7, but the combination of those two things triggers some sort of code path that leads to a segfault very early on in loading the various libraries.

Gitea works fine on arm7 if you cross compile manually as well...and I suspect the issue is related to a specific way of --static compiling it. Also given that the code works fine on arm6 but not arm7 makes me feel like it is related to the code that is generated by the compiler itself (I tried different versions of GCC, and it still works/fails under the same options).

@stale
Copy link

stale bot commented Jul 13, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added the issue/stale label Jul 13, 2019
@lunny lunny added the issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented label Jul 14, 2019
@stale stale bot removed the issue/stale label Jul 14, 2019
@ricardoboss
Copy link

So, what does the workaround look like? Is there an image tag supporting arm/v6?

@zeripath
Copy link
Contributor

yes just use the arm-6 build

@eblanshey
Copy link

@zeripath is there a solution for installing via Docker by any chance? I don't see any tags with arm v6 on Docker Hub.

@zeripath
Copy link
Contributor

zeripath commented Sep 4, 2022

I'm afraid we don't provide arm6 dockers. We would need to create a drone-docker buildchain for this.

We'd be open for PRs to help provide this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
issue/confirmed Issue has been reviewed and confirmed to be present or accepted to be implemented topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile type/bug
Projects
None yet
Development

No branches or pull requests

6 participants