From 9c1d59f00eeeceb4147b4c3f225d7716e7194846 Mon Sep 17 00:00:00 2001
From: Alex Chan <alexc@tailscale.com>
Date: Tue, 31 Mar 2026 10:46:00 +0100
Subject: [PATCH] version: parse Void Linux version strings

We have ~2.5k nodes running Void Linux, which report a version string
like `1.96.2_1 (Void Linux)`. Previously these versions would fail to
parse, because we only expect a hyphen and `extraCommits` after the
major/minor/patch numbers.

Fix the version parsing logic to handle this case.

Updates #19148

Change-Id: Ica4f172d080af266af7f0d69bb31483a095cd199
Signed-off-by: Alex Chan <alexc@tailscale.com>
---
 version/cmp.go      | 5 +++++
 version/cmp_test.go | 3 +++
 2 files changed, 8 insertions(+)

diff --git a/version/cmp.go b/version/cmp.go
index 4af0aec69..6d44475e7 100644
--- a/version/cmp.go
+++ b/version/cmp.go
@@ -103,6 +103,11 @@ func parse(version string) (parsed, bool) {
 		}
 	}
 
+	// Ignore trailer like '_1 (Void Linux)'.
+	if rest[0] == '_' && strings.HasSuffix(rest, " (Void Linux)") {
+		return ret, true
+	}
+
 	// Optional extraCommits, if the next bit can be completely
 	// consumed as an integer.
 	if rest[0] != '-' {
diff --git a/version/cmp_test.go b/version/cmp_test.go
index 10fc130b7..c93df1a7c 100644
--- a/version/cmp_test.go
+++ b/version/cmp_test.go
@@ -33,6 +33,8 @@ func TestParse(t *testing.T) {
 		{"borkbork", parsed{}, false},
 		{"1a.2.3", parsed{}, false},
 		{"", parsed{}, false},
+		{"1.96.2_1 (Void Linux)", parsed{Major: 1, Minor: 96, Patch: 2}, true},
+		{"1.46.0_2 (Void Linux)", parsed{Major: 1, Minor: 46, Patch: 0}, true},
 	}
 
 	for _, test := range tests {
@@ -71,6 +73,7 @@ func TestAtLeast(t *testing.T) {
 		{"date.20200612", "date.20200612", true},
 		{"date.20200701", "date.20200612", true},
 		{"date.20200501", "date.20200612", false},
+		{"1.96.2_1 (Void Linux)", "1.42", true},
 	}
 
 	for _, test := range tests {