Repository navigation
Upgrade the installer to WiX v7 - #35
Merged
Merged
Conversation
Rewrite Product.wxs as Package.wxs on WiX 7.0.1's defaults: the default feature, generated component GUIDs, MajorUpgrade and an exe picked from publish/<arch> by -arch. Downgrades are refused, the shortcut moves to the top of the Start menu and the widget starts through Explorer after a non-silent install.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Rewrites
Product.wxsfor WiX 7.0.1 (from 6.0.2). It leans on the defaults WiX has added since v4 instead of spelling out the feature, media, upgrade table and component GUIDs.MajorUpgradereplaces the hand-writtenUpgradetable andRemoveExistingProducts. The old version is still removed before the new one installs, in the same place in the sequence.Programs\Network Monitor.lnkinstead ofPrograms\Network Monitor\Network Monitor.lnk, so All apps lists the widget directly rather than as a folder holding one item. Upgrading removes the old folder.Software\Microsoft\Daniel Chalmers\Network MonitortoSoftware\Daniel Chalmers\Network Monitor, the key Radial Actions and Sentry Deck use. Upgrading removes the old key. The app doesn't read either key.-d MainExeSource. The package picks the exe frompublish/<arch>by-arch, and still takes its version from the exe. WiX v7 won't build until its OSMF EULA is accepted, so the build passes-acceptEula wix7, which accepts it for that run only.Also dropped, with no change in behavior:
Language="1033", so the package is language-neutral.InstallerVersion="500"andSummaryInformation, both WiX's defaults now.Secure="yes"on the ARP properties, which matters only for elevated installs.Icon, since the exe has the same icon.Validation
dotnet test: 179 passed.Built the MSIs locally with WiX 7.0.1 from Release publishes, using the commands in
create-release-artifacts:master'sProduct.wxsinto the same tables as WiX 6.0.2.master's package shows only the changes above.wix msi validateonmaster's package reports ICE38, ICE61 and ICE64 errors and ICE69 and ICE91 warnings. On this one it reports only ICE38, which every per-user package with a file key path gets, and the ICE61 and ICE91 warnings.Tested in Windows Sandbox with 4.1.1 packages built from this branch, against the 4.1.0 release:
master's package does.Tested this PR's CI artifacts in the same sandbox:
Not included
Arm64 itself (an Arm64 smoke test in CI) is a separate follow-up.