windows: use WiX to make MSI installers
WiX makes .msi
installers. This drops nsis
and makes .exe
installers no longer available.
Required for !1000 (merged) due to long paths not being supported by NSIS.
Merge request reports
Activity
assigned to @ben.boeckel
- Resolved by Mathieu Westphal (Kitware)
Windows users are not accustomed to .msi as much as they are to .exe. I feel like this is a big change that may be required but should be discussed and validated.
- Resolved by Ben Boeckel
Looking at the failures, it looks like we need to enable the MSI Installer Service on our CI machines for WiX to work properly.
Grabbed from a CI machine's log:
light.exe : error LGHT0217 : Error executing ICE action 'ICE07'. The most common cause of this kind of ICE failure is an incorrectly registered scripting engine. See http://wixtoolset.org/documentation/error217/ for details and how to solve this problem. The following string format was not expected by the external UI message logger: "The Windows Installer Service could not be accessed. This can occur if the Windows Installer is not correctly installed. Contact your support personnel for assistance.".
@brad.king Did you have to do something explicitly for CMake's usage of WiX or is this perhaps an undocumented side-effect of once having WiX globally installed?
added 9 commits
- 0e574ca2 - paraview.bundle: add the suffix only to the package filename
- c51f3176 - superbuild: bump to let packaging know its generator
- cede0bb9 - paraview.bundle: avoid the suffix on WiX generators
- 146ca1cb - paraview.bundle: set the GUIDs for the WiX installers
- 9cd4f39f - windows: suppress validation on CI packaging
- aa81be29 - gitlab-ci: download and make WiX available
- 9d53cad4 - packages: use WiX on Windows instead of NSIS
- da436554 - packages: remove NSIS from packaging
- 237625cf - gitlab-ci: remove NSIS bits
Toggle commit listmentioned in merge request !1000 (merged)
requested review from @cory.quammen
mentioned in commit f42a44c2
mentioned in issue paraview#21556 (closed)