Skip to content

Bug: Cannot build on 32-bit system #935

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

Closed
adlternative opened this issue Apr 6, 2025 · 3 comments · Fixed by #936
Closed

Bug: Cannot build on 32-bit system #935

adlternative opened this issue Apr 6, 2025 · 3 comments · Fixed by #936

Comments

@adlternative
Copy link

adlternative commented Apr 6, 2025

I tried to incorporate zoekt into gitea and encountered an error on i386:

# github.com/sourcegraph/zoekt
Error: ../../../go/pkg/mod/github.com/sourcegraph/[email protected]/matchtree.go:675:9: cannot use maxUInt32 (untyped int constant 4294967295) as int value in assignment (overflows)
Error: Process completed with exit code 1.

This seems like a uint32 overflow into int on a 32-bit machine. Does anyone know how to fix this?

Zoekt Version: github.com/sourcegraph/zoekt v0.0.0-20240410142517-ab1b8f09199e
See: https://github.com/go-gitea/gitea/actions/runs/14281855193/job/40032337115?pr=33850

Copy link

linear bot commented Apr 6, 2025

@jtibshirani
Copy link
Contributor

Thanks for reporting this @adlternative! I opened #935 to fix this. With that fix, I am able to build for 32-bit x86 with no compile failures. I tested on my 64 bit machine through this command:

GOOS=linux GOARCH=386 go test ./...

@adlternative
Copy link
Author

Thanks for reporting this @adlternative! I opened #935 to fix this. With that fix, I am able to build for 32-bit x86 with no compile failures. I tested on my 64 bit machine through this command:

GOOS=linux GOARCH=386 go test ./...

Great! Thanks for fix this bug!

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