release.IsNewer: сравнивать все части версии — номер сборки за день значим
Раньше 2026.09.24.11 и 2026.09.24.5 считались одной версией, и обновление в пределах одного дня не предлагалось. Исправление выпускается как 2026.09.25.1, чтобы его увидели установки со старым сравнением. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_014FsEKyJrYF69Cj9ZXzFLRi
This commit is contained in:
+18
-12
@@ -6,31 +6,37 @@ import (
|
|||||||
)
|
)
|
||||||
|
|
||||||
// IsNewer возвращает true, если версия remote строго новее local.
|
// IsNewer возвращает true, если версия remote строго новее local.
|
||||||
// Понимает semver вида "MAJOR.MINOR.PATCH" (с опциональным префиксом "v"
|
// Сравнивает все числовые части: "MAJOR.MINOR.PATCH" и наши "ГГГГ.ММ.ДД.N" — четвёртая часть
|
||||||
// и суффиксом "-beta" — суффикс игнорируется при сравнении чисел).
|
// (номер сборки за день) значима: 2026.09.24.11 новее 2026.09.24.5. Префикс "v" и суффиксы
|
||||||
// Некорректные/пустые версии считаются «старыми» (0.0.0).
|
// "-beta", "+build" игнорируются. Недостающие части — нули; некорректные/пустые версии — «старые».
|
||||||
func IsNewer(remote, local string) bool {
|
func IsNewer(remote, local string) bool {
|
||||||
rm := parseVer(remote)
|
rm := parseVer(remote)
|
||||||
lc := parseVer(local)
|
lc := parseVer(local)
|
||||||
for i := 0; i < 3; i++ {
|
for i := 0; i < len(rm) || i < len(lc); i++ {
|
||||||
if rm[i] != lc[i] {
|
var r, l int
|
||||||
return rm[i] > lc[i]
|
if i < len(rm) {
|
||||||
|
r = rm[i]
|
||||||
|
}
|
||||||
|
if i < len(lc) {
|
||||||
|
l = lc[i]
|
||||||
|
}
|
||||||
|
if r != l {
|
||||||
|
return r > l
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return false
|
return false
|
||||||
}
|
}
|
||||||
|
|
||||||
func parseVer(s string) [3]int {
|
func parseVer(s string) []int {
|
||||||
s = strings.TrimSpace(s)
|
s = strings.TrimSpace(s)
|
||||||
s = strings.TrimPrefix(s, "v")
|
s = strings.TrimPrefix(s, "v")
|
||||||
if i := strings.IndexAny(s, "-+"); i >= 0 {
|
if i := strings.IndexAny(s, "-+"); i >= 0 {
|
||||||
s = s[:i] // отбрасываем -beta, +build
|
s = s[:i] // отбрасываем -beta, +build
|
||||||
}
|
}
|
||||||
var out [3]int
|
var out []int
|
||||||
parts := strings.Split(s, ".")
|
for _, p := range strings.Split(s, ".") {
|
||||||
for i := 0; i < 3 && i < len(parts); i++ {
|
n, _ := strconv.Atoi(p)
|
||||||
n, _ := strconv.Atoi(parts[i])
|
out = append(out, n)
|
||||||
out[i] = n
|
|
||||||
}
|
}
|
||||||
return out
|
return out
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,24 @@
|
|||||||
|
package release
|
||||||
|
|
||||||
|
import "testing"
|
||||||
|
|
||||||
|
func TestIsNewerBuildNumber(t *testing.T) {
|
||||||
|
cases := []struct {
|
||||||
|
r, l string
|
||||||
|
want bool
|
||||||
|
}{
|
||||||
|
{"2026.09.24.11", "2026.09.24.5", true},
|
||||||
|
{"2026.09.24.5", "2026.09.24.11", false},
|
||||||
|
{"2026.09.24.1", "2026.09.24", true},
|
||||||
|
{"2026.09.24", "2026.09.24.1", false},
|
||||||
|
{"2026.09.25", "2026.09.24.11", true},
|
||||||
|
{"2026.09.18-beta1", "2026.09.18", false},
|
||||||
|
{"1.2.3", "1.2.3", false},
|
||||||
|
{"v1.3.0", "1.2.9", true},
|
||||||
|
}
|
||||||
|
for _, c := range cases {
|
||||||
|
if got := IsNewer(c.r, c.l); got != c.want {
|
||||||
|
t.Errorf("IsNewer(%q, %q) = %v, want %v", c.r, c.l, got, c.want)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
Reference in New Issue
Block a user