Skip to content

Conversation

@kradalby
Copy link
Collaborator

This commit removes the official 32 bit builds.
As per #2511, they are broken at runtime and we
do not have any good way to ensure that they run over time as we cannot catch these issue at compile or
test time.

64-bit has been around for long enough that this
change should not be too disruptive.

32-bit users can compile the code themselves, and
we can still be open to accept patches for it, but not provide builds as they might not work over time.

Closes #2511

This commit removes the official 32 bit builds.
As per juanfont#2511, they are broken at runtime and we
do not have any good way to ensure that they run over
time as we cannot catch these issue at compile or
test time.

64-bit has been around for long enough that this
change should not be too disruptive.

32-bit users can compile the code themselves, and
we can still be open to accept patches for it, but
not provide builds as they might not work over time.

Signed-off-by: Kristoffer Dalby <[email protected]>
@ghost
Copy link

ghost commented Jul 16, 2025

Pull Request Revisions

RevisionDescription
r2
Remove 32-bit and zero port supportTwo breaking changes introduced: dropped 32-bit binary compatibility and disallowed zero or empty destination ports
r1
Removed certain Linux and ARM buildsSimplified build configurations by removing 32-bit Linux and specific ARM architecture builds from GitHub Actions and GoReleaser

☑️ AI review skipped for r2
Help React with emojis to give feedback on AI-generated reviews:
  • 👍 means the feedback was helpful and actionable
  • 👎 means the feedback was incorrect or unhelpful
💬 Replying to feedback with a comment helps us improve the system. Your input also contributes to shaping future interactions with the AI reviewer.

We'd love to hear from you—reach out anytime at [email protected].

Signed-off-by: Kristoffer Dalby <[email protected]>
@kradalby kradalby marked this pull request as ready for review July 16, 2025 11:10
@kradalby kradalby requested a review from juanfont July 16, 2025 11:10
@kradalby kradalby merged commit 7fce506 into juanfont:main Jul 16, 2025
83 checks passed
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 this pull request may close these issues.

[Bug] unaligned 64-bit atomic operation

2 participants