Skip to content

Lift the four-digit series ceiling at 0099 - #199

Merged
shibco merged 1 commit into
shibco:mainfrom
ClickSentinel:fix/patch-number-ceiling
Aug 13, 2026
Merged

Lift the four-digit series ceiling at 0099#199
shibco merged 1 commit into
shibco:mainfrom
ClickSentinel:fix/patch-number-ceiling

Conversation

@ClickSentinel

Copy link
Copy Markdown
Collaborator

The series globs matched 00*.patch, so a patch numbered 0100 or above was applied by nothing and audited by nothing — no error, just a runtime missing the fix. The Wine series is at 0097, leaving 0098 and 0099.

Match [0-9][0-9][0-9][0-9]-*.patch instead, in the apply loop, the stack stamp and the freeze. Zero-padded four-digit names sort the same lexically as numerically, so apply order is unchanged — verified across the 0097→0100 boundary.

The unlisted-patch check now lists every .patch on disk rather than the ones the series glob matches, so a file outside the numbering scheme is reported instead of passing as absent.

Verified with a synthetic 0100- patch present: the old glob saw 95 Wine patches, the new one 96, and the audit reports it as unlisted patches present.

The series globs matched 00*.patch, so a patch numbered 0100 or above was
applied by nothing and audited by nothing — no error, just a runtime missing
the fix. The Wine series is at 0097.

Match [0-9][0-9][0-9][0-9]-*.patch instead, in the apply loop, the stack stamp
and the freeze. Zero-padded four-digit names sort the same lexically as
numerically, so apply order is unchanged.

The unlisted-patch check now lists every .patch on disk rather than the ones
the series glob matches, so a file that falls outside the numbering scheme is
reported instead of passing as absent.
@ClickSentinel
ClickSentinel requested a review from shibco August 13, 2026 23:10
@shibco
shibco merged commit d541422 into shibco:main Aug 13, 2026
1 check 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.

2 participants