Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: wixv4 msi build (fixes #1) #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JPHutchins
Copy link
Contributor

@JPHutchins JPHutchins commented Nov 10, 2024

This commit adds a WiX v4 MSI build to the project. The build is run from the msi/build.ps1 script. A GitHub workflow is added to run the build and upload the MSI as an artifact.

MSI artifact here: https://github.com/JPHutchins/wsl-usb-manager/actions/runs/11768450149

Closes #1

This commit adds a WiX v4 MSI build to the project. The build is run
from the msi/build.ps1 script. A GitHub workflow is added to run the
build and upload the MSI as an artifact.
@nickbeth
Copy link
Owner

I've only briefly looked at this, I'll take a deeper look in the next days. I'm wondering if it makes sense to have CI build an msi for every commit on master and for every PR? Personally I don't think so, an msi package only needs to be built for releases which are created manually.

@JPHutchins
Copy link
Contributor Author

I've only briefly looked at this, I'll take a deeper look in the next days. I'm wondering if it makes sense to have CI build an msi for every commit on master and for every PR? Personally I don't think so, an msi package only needs to be built for releases which are created manually.

It's not implemented yet, but it would also be triggered by a release action and then uploaded to the release.

The reason for doing it "a lot" is coverage. It's nice to see it break on the PR rather than whenever the release is made.

Comment on lines +4 to +14
<Project Sdk="WixToolset.Sdk/4.0.4">
<PropertyGroup>
<DefineConstants> ApplicationFiles=$(PORTABLE_PATH)/
</DefineConstants>
<OutputName>$(MSI_NAME)</OutputName>
</PropertyGroup>

<ItemGroup>
<PackageReference Include="WixToolset.Heat" Version="4.0.4" />
<PackageReference Include="WixToolset.UI.wixext" Version="4.0.4" />
</ItemGroup>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Update to 4.0.6 or 5.0.2

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.

Add MSI Installer
2 participants