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

Merged

brenno merged 1 commit from fix/iscc-versietoets-banner into main

2026-08-20 07:13:46 +00:00

brenno commented

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

Owner

rc3 werd rood op een correcte installatie: de vondst wees keurig naar C:\Program Files\Inno Setup 7\ISCC.exe (de gepinde 7.1.0, niet de Chocolatey-shim — de doorgifte uit #1601 werkt dus), maar ISCC.exe draagt geen bruikbare VersionInfo en meldde ProductVersion 0.0.0.0. De fail-closed toets deed zijn werk, op een verkeerde meting.

Reparatie: de versie komt nu uit de banner die de compiler zelf afdrukt (Compiler engine version: Inno Setup 7.1.0 — het rc2-log toonde die vorm al voor 6.7.1). Bij een mismatch staat de volledige banneruitvoer in de foutmelding, zodat een volgende verrassing zichzelf documenteert in plaats van een extra rc te kosten.

Poort volgt de nieuwe vorm; mutatie (toets uitgezet) betrapt. check-static + check-registrations groen; raakt geen lib-code.

Na de merge: v0.4.7-rc4 naar de spiegel. Verwacht bewijs in het log: de vondstregel met het Inno Setup 7-pad én een compileregel die niet meer naar /c/ProgramData/Chocolatey/... wijst.

rc3 werd rood op een correcte installatie: de vondst wees keurig naar `C:\Program Files\Inno Setup 7\ISCC.exe` (de gepinde 7.1.0, niet de Chocolatey-shim — de doorgifte uit #1601 werkt dus), maar ISCC.exe draagt geen bruikbare `VersionInfo` en meldde `ProductVersion 0.0.0.0`. De fail-closed toets deed zijn werk, op een verkeerde meting. **Reparatie:** de versie komt nu uit de banner die de compiler zelf afdrukt (`Compiler engine version: Inno Setup 7.1.0` — het rc2-log toonde die vorm al voor 6.7.1). Bij een mismatch staat de volledige banneruitvoer in de foutmelding, zodat een volgende verrassing zichzelf documenteert in plaats van een extra rc te kosten. Poort volgt de nieuwe vorm; mutatie (toets uitgezet) betrapt. `check-static` + `check-registrations` groen; raakt geen lib-code. Na de merge: `v0.4.7-rc4` naar de spiegel. Verwacht bewijs in het log: de vondstregel met het `Inno Setup 7`-pad én een compileregel die niet meer naar `/c/ProgramData/Chocolatey/...` wijst.

brenno added 1 commit

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

fix(release): de ISCC-versietoets leest de banner, niet een lege VersionInfo

scans / scans (pull_request) Successful in 3m2s

Details

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

Details

63f513163e

rc3 werd rood op een volkomen correcte installatie: de vondst wees keurig naar
C:\Program Files\Inno Setup 7\ISCC.exe — de gepinde 7.1.0, niet de
Chocolatey-shim — maar ISCC.exe draagt geen bruikbare VersionInfo en meldde
ProductVersion 0.0.0.0. De fail-closed toets deed dus zijn werk, alleen op een
verkeerde meting.

De versie komt nu uit de banner die de compiler zelf afdrukt ("Compiler engine
version: Inno Setup 7.1.0", zoals het rc2-log al liet zien voor 6.7.1). Faalt
de match, dan staat de volledige banneruitvoer in de foutmelding, zodat er
geen extra rc nodig is om te raden wat er stond.

Poort volgt de nieuwe vorm; mutatie (toets uitgezet) betrapt.

Verwijst naar #1583.

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

brenno merged commit f52bd9b7c5 into main

2026-08-20 07:13:46 +00:00

brenno referenced this pull request from a commit

2026-08-20 07:13:47 +00:00

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

brenno referenced this pull request

2026-08-20 07:44:20 +00:00

Vraag ISCC zijn versie via --version, de enige bron die hem noemt #1603

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