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

Cannot build for armv7 #3

Closed
gpasqualetti opened this issue Mar 19, 2023 · 3 comments · Fixed by #4
Closed

Cannot build for armv7 #3

gpasqualetti opened this issue Mar 19, 2023 · 3 comments · Fixed by #4

Comments

@gpasqualetti
Copy link

When I try to build for armv7 in a Docker container, I receive the following error:

[7/8] RUN cd gpodder2go && ./gpodder2go init:
#10 0.770 panic: suint(-301924841) expects input >= 0
#10 0.771
#10 0.771 goroutine 24 [running]:
#10 0.772 github.com/golang-migrate/migrate/v4.suint(...)
#10 0.773 /go/pkg/mod/github.com/golang-migrate/migrate/v4@v4.15.2/util.go:46
#10 0.774 github.com/golang-migrate/migrate/v4.(*Migrate).readUp(0x25426c0, 0xffffffff, 0xffffffff, 0x2542880)
#10 0.776 /go/pkg/mod/github.com/golang-migrate/migrate/v4@v4.15.2/migrate.go:582 +0x7b8
#10 0.779 created by github.com/golang-migrate/migrate/v4.(*Migrate).Up
#10 0.779 /go/pkg/mod/github.com/golang-migrate/migrate/v4@v4.15.2/migrate.go:283 +0x10c

The problem seems to be some option of migrate already reported here, together with a possible solution: golang-migrate/migrate#213

@oxtyped
Copy link
Owner

oxtyped commented Mar 23, 2023

Thanks for reporting this and linking the upstream issue too.

Seems like changing the migration format with migrate create -format or migrate create -seq would be a quick workaround to this issue on armv7.

@oxtyped
Copy link
Owner

oxtyped commented Mar 23, 2023

Any reviews will be appreciated!

@gpasqualetti
Copy link
Author

Thank you for the patch! I tested it and the problem seems to be solved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants