Skip to content

remove broken Windows ARM builds#7309

Merged
neilalexander merged 1 commit intomainfrom
remove_broken_arm_windows_build
Sep 15, 2025
Merged

remove broken Windows ARM builds#7309
neilalexander merged 1 commit intomainfrom
remove_broken_arm_windows_build

Conversation

@alexbozhenko
Copy link
Copy Markdown
Member

@alexbozhenko alexbozhenko commented Sep 12, 2025

ARM (but not arm64) is broken on windows.

Latest Goreleaser(2.12) started to complain about this:

# go clean -cache && goreleaser  release --skip=announce,publish,validate --clean -f .goreleaser.yml
...
    ⨯ broken target, use at your own risk            target=windows_arm_6
    ⨯ broken target, use at your own risk            target=windows_arm_7

Support is to be removed from Golang. Stop building for ARM on Windows
golang/go#70705
https://go.dev/wiki/GoArm
goreleaser/goreleaser#5951
https://golang.google.cn/doc/go1.24#windows
https://golang.google.cn/doc/go1.25#windows

# GOTOOLCHAIN=go1.25.1 go tool dist list -broken -json   | jq '.[] | select(.GOOS == "windows") | select(.Broken == true)'
{
  "GOOS": "windows",
  "GOARCH": "arm",
  "CgoSupported": false,
  "FirstClass": false,
  "Broken": true
}

Also, judging by the number of downlads,
it looks like only automated scripts and not real people download those builds.
https://somsubhra.github.io/github-release-stats/?username=nats-io&repository=nats-server

Test plan:

# go clean -cache && goreleaser  release --skip=announce,publish,validate --clean -f .goreleaser.yml

# diff -u <(ls -1 dist_before/) <(ls -1 dist/)
--- /dev/fd/63  2025-09-12 12:43:38.071393996 -0700
+++ /dev/fd/62  2025-09-12 12:43:38.072393998 -0700
@@ -61,13 +61,7 @@
 nats-server-v2.12.0-RC.3-windows-amd64.zip
 nats-server-v2.12.0-RC.3-windows-arm64.sbom.spdx.json
 nats-server-v2.12.0-RC.3-windows-arm64.zip
-nats-server-v2.12.0-RC.3-windows-arm6.sbom.spdx.json
-nats-server-v2.12.0-RC.3-windows-arm6.zip
-nats-server-v2.12.0-RC.3-windows-arm7.sbom.spdx.json
-nats-server-v2.12.0-RC.3-windows-arm7.zip
 nats-server_windows_386_sse2
 nats-server_windows_amd64_v1
-nats-server_windows_arm_6
 nats-server_windows_arm64_v8.0
-nats-server_windows_arm_7
 SHA256SUMS

Signed-off-by: Alex Bozhenko alexbozhenko@gmail.com

Signed-off-by: Alex Bozhenko <alexbozhenko@gmail.com>
@alexbozhenko alexbozhenko requested a review from a team as a code owner September 12, 2025 19:46
@alexbozhenko alexbozhenko force-pushed the remove_broken_arm_windows_build branch from 12d980c to 841b4e3 Compare September 12, 2025 19:56
Copy link
Copy Markdown
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@neilalexander neilalexander merged commit 7975217 into main Sep 15, 2025
48 checks passed
@neilalexander neilalexander deleted the remove_broken_arm_windows_build branch September 15, 2025 13:42
neilalexander added a commit that referenced this pull request Sep 29, 2025
Includes the following:

- #7290
- #7295
- #7291
- #7287
- #7299
- #7300
- #7297
- #7303
- #7304
- #7305
- #7309
- #7307
- #7320
- #7337
- #7344
- #7345
- #7348
- #7349
- #7350
- #7357
- #7356
- #7358
- #7367
- #7293

Signed-off-by: Neil Twigg <neil@nats.io>
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.

2 participants