obsproject / obs-plugintemplate

GNU General Public License v2.0
307 stars 141 forks source link

powershell: Fixes Inno Setup installation and iscc lookup #128

Closed NPatch closed 1 week ago

NPatch commented 1 month ago

Inno Setup package name in .WingetFile is wrong and doesn't install. Inno Setup installs in LocalAppData\Programs which Install-BuildDependencies does not look at.

Description

Updates build and packaging powershell scripts for Windows.

Motivation and Context

Inno Setup's winget package name is incorrect in winget which results in failing to install it. Used winget search to search for the package and found the package by JRSoftware.

Additionally, Inno Setup installs under [LOCALAPPDATA]\Programs\Inno Setup 6, so iscc is never found by Get-Program as the $Prefixes array in the Install-BuildDependencies.ps1 does not contain any AppData directory as viable install locations. This PR only deals with Inno's [LOCALAPPDATA]\Programs install directory, but, since many programs these days use AppData which requires less permissions than ProgramFiles, it might be better to add more options.

How Has This Been Tested?

Both changes were tested on Windows 11 through the integrated powershell terminal on VSCode. In both cases, the changes allowed the scripts to install, find and run Inno Setup and at the end the -BuildInstaller switch actually produced an Installer.

Types of changes

Checklist: