De packager compileert met de gepinde ISCC, niet met een toevallige shim #1601

Merged

brenno merged 1 commit from fix/pin-iscc-path into main

2026-08-20 06:41:54 +00:00

brenno commented

2026-08-20 06:34:52 +00:00

Owner

Het rc2-log betrapte een omzeilde pin: de job installeerde en verifieerde Inno Setup 7.1.0, waarna de packager compileerde met /c/ProgramData/Chocolatey/bin/iscc — een ongepinde 6.7.1 die op het runner-image blijkt te staan. find_iscc probeert command -v iscc vóór de zoekpaden, dus de pin was decoratie.

Reparatie: de installatiestap zoekt na afloop de zojuist geïnstalleerde ISCC op, toetst ProductVersion tegen de pin (fail-closed), en geeft het pad via GITHUB_ENV als OCIDECK_ISCC door. De packager normaliseert dat Windows-pad met cygpath.

Poort: export én de StartsWith-vergelijking gepind; twee mutaties, twee betrapt (waaronder één die een te losse regex ving). make check-static + check-registrations groen; wijziging raakt geen lib-code.

Na de merge: v0.4.7-rc3 naar de spiegel om op de echte runner te bewijzen dat de compileregel nu de gepinde 7.1.0 noemt.

Het rc2-log betrapte een omzeilde pin: de job installeerde en verifieerde Inno Setup **7.1.0**, waarna de packager compileerde met `/c/ProgramData/Chocolatey/bin/iscc` — een ongepinde **6.7.1** die op het runner-image blijkt te staan. `find_iscc` probeert `command -v iscc` vóór de zoekpaden, dus de pin was decoratie. **Reparatie:** de installatiestap zoekt na afloop de zojuist geïnstalleerde ISCC op, toetst `ProductVersion` tegen de pin (fail-closed), en geeft het pad via `GITHUB_ENV` als `OCIDECK_ISCC` door. De packager normaliseert dat Windows-pad met `cygpath`. **Poort:** export én de `StartsWith`-vergelijking gepind; twee mutaties, twee betrapt (waaronder één die een te losse regex ving). `make check-static` + `check-registrations` groen; wijziging raakt geen lib-code. Na de merge: `v0.4.7-rc3` naar de spiegel om op de echte runner te bewijzen dat de compileregel nu de gepinde 7.1.0 noemt.

brenno added 1 commit

2026-08-20 06:34:52 +00:00

fix(release): de packager compileert met de gepinde ISCC, niet met een toevallige shim

scans / scans (pull_request) Successful in 3m20s

Details

static-gate / static-gate (pull_request) Successful in 6m54s

Details

f6c13f4ca7

Het rc2-log betrapte het: de stap installeerde en sha256-verifieerde Inno Setup
7.1.0, waarna de packager compileerde met /c/ProgramData/Chocolatey/bin/iscc —
een ongepinde 6.7.1 die op het runner-image blijkt te staan. find_iscc probeert
`command -v iscc` vóór de zoekpaden, dus de pin was decoratie: als het image
morgen een andere versie shimt, drijft de bouw stil mee.

De installatiestap zoekt nu na afloop de zojuist geïnstalleerde ISCC op, toetst
diens ProductVersion tegen de pin (fail-closed: een verkeerde vondst gooit),
en geeft het pad via GITHUB_ENV als OCIDECK_ISCC aan de packager door — die
daarmee geen keus meer heeft. De packager normaliseert dat Windows-pad met
cygpath voordat -x en de aanroep het zien; "Git Bash kan meestal wel met
backslashes overweg" is geen fundament.

Poort: export én de StartsWith-vergelijking gepind (het woord alleen was te
zwak — de throw-boodschap noemt de pin ook; de mutatietoets betrapte dat).
Twee mutaties, twee betrapt. Override-route rookgetoetst.

Verwijst naar #1583.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

brenno merged commit 8c65c35815 into main

2026-08-20 06:41:54 +00:00

brenno referenced this pull request from a commit

2026-08-20 06:41:54 +00:00

De packager compileert met de gepinde ISCC, niet met een toevallige shim (#1601)

brenno referenced this pull request

2026-08-20 07:06:45 +00:00

De ISCC-versietoets leest de banner, niet een lege VersionInfo #1602

brenno referenced this pull request

2026-08-20 09:20:00 +00:00

Windows-installer meenemen in de bouwlijn op de spiegel — of bewust handwerk laten #1583