You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Alpine Linux 3.24.1 on 32-bit archs (armhf, armv7, x86)
Describe the bug / provide steps to reproduce it
Since b6b4ae4 you parse COMPOUNDMIN and COMPOUNDRULEwith strconv.Atoi, which is ParseInt(s, 10, 0) where bit size 0 is the native int. This means the boundary for absurd/unreadable values depends on the target word size. Meanwhile TestCompoundMinIsBounded encodes the 64-bit behaviour, causing the following failure on 32-bit targets:
--- FAIL: TestCompoundMinIsBounded (0.00s)
aff_test.go:382: "COMPOUNDMIN 9223372036854775807": COMPOUNDMIN stanza had "9223372036854775807" expected number
I think these values should both be parsed as int64 with ParseInt(parts[1], 10, 64) to avoid making incorrect assumptions about word size?
Activity
[-]`TestCompoundMinIsBounded` fails on 32-bit arm[/-][+]`TestCompoundMinIsBounded` fails on 32-bit[/+]fix: parse a compound stanza at a fixed width, not the target's (#1159)