Fix: Dangerzone installed using an msi built with WiX Toolset 3 is not uninstalled by an msi built with WiX Toolset 5

Work around the issue by adding some extra functionality to the "Next" button on the welcome screen of the installer. When the user clicks it to proceed with the installation this: 1. Flips the install scope to "perUser" which is the default in WiX 3 2. Finds the older installation 3. And finally flips the scope back to "perMachine" which is the default in WiX 4 and newer TODO: Revert this once we are reasonably certain there are no affected Dangerzone Installations?
Change: Wrap installer ui related things in a UI element
2025-05-03 12:11:50 +02:00 · 2024-10-30 22:08:36 +02:00 · 2024-10-30 22:08:23 +02:00 · 2024-10-27 17:43:59 +02:00 · 2024-10-27 17:43:59 +02:00 · 2024-10-27 17:43:59 +02:00
2 changed files with 36 additions and 31 deletions
--- a/install/windows/build-app.bat
+++ b/install/windows/build-app.bat
@ -28,6 +28,7 @@ REM validate Dangerzone.msi
 wix msi validate Dangerzone.msi

 REM code sign Dangerzone.msi
+wix msi inscribe Dangerzone.msi
 signtool.exe sign /v /d "Dangerzone" /a /n "Freedom of the Press Foundation" /fd sha256 /t http://time.certum.pl/ Dangerzone.msi

 REM verify the signature of Dangerzone.msi
--- a/install/windows/build-wxs.py
+++ b/install/windows/build-wxs.py
@ -84,7 +84,9 @@ def main():
    with open(version_filename) as f:
        # Read the Dangerzone version from share/version.txt, and remove any potential
        # -rc markers.
-        version = f.read().strip().split("-")[0]
+        dangerzone_version = f.read().strip().split("-")[0]
+
+    dangerzone_product_upgrade_code = "12B9695C-965B-4BE0-BC33-21274E809576"

    build_dir = os.path.join(
        os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__)))),
@ -122,17 +124,18 @@ def main():
        "Package",
        Name="Dangerzone",
        Manufacturer="Freedom of the Press Foundation",
-        UpgradeCode="12B9695C-965B-4BE0-BC33-21274E809576",
+        UpgradeCode=dangerzone_product_upgrade_code,
        Language="1033",
        Compressed="yes",
        Codepage="1252",
-        Version=version,
+        Version=dangerzone_version,
    )
+
    ET.SubElement(
        package_el,
        "SummaryInformation",
        Keywords="Installer",
-        Description="Dangerzone " + version + " Installer",
+        Description="Dangerzone " + dangerzone_version + " Installer",
        Codepage="1252",
    )
    ET.SubElement(package_el, "MediaTemplate", EmbedCab="yes")
@ -152,40 +155,22 @@ def main():
        Id="ARPURLINFOABOUT",
        Value="https://freedom.press",
    )
-    ET.SubElement(
-        package_el,
-        "WixVariable",
-        Id="WixUILicenseRtf",
-        Value="..\\install\\windows\\license.rtf",
-    )
-    ET.SubElement(
-        package_el,
-        "WixVariable",
-        Id="WixUIDialogBmp",
-        Value="..\\install\\windows\\dialog.bmp",
-    )
-    ET.SubElement(
-        package_el,
-        "MajorUpgrade",
-        AllowSameVersionUpgrades="yes",
-        DowngradeErrorMessage="A newer version of [ProductName] is already installed. If you are sure you want to downgrade, remove the existing installation via Programs and Features.",
-    )

-    # Add The UI element
    ui_el = ET.SubElement(package_el, "UI")
+
    ET.SubElement(
        ui_el, "ui:WixUI", Id="WixUI_InstallDir", InstallDirectory="INSTALLFOLDER"
    )
    ET.SubElement(ui_el, "UIRef", Id="WixUI_ErrorProgressText")

-    # Workaround for an issue after upgrading from WiX Toolset v3 to v5 where the older
+    # Workaround for an issue after upgrading from WiX Toolset 3 to 5 where the older
    # version of Dangerzone is not uninstalled during the upgrade
    #
-    # Fix the issue by adding some extra functionality to the "Next" button on the welcome screen
+    # Work around the issue by adding some extra functionality to the "Next" button on the welcome screen
    # of the installer. When the user clicks it to proceed with the installation this:
-    # 1. Flips the install scope to "perUser" which is the default in WiX v3
+    # 1. Flips the install scope to "perUser" which is the default in WiX 3
    # 2. Finds the older installation
-    # 3. And finally flips the scope back to "perMachine" which is the default in WiX v4 and newer
+    # 3. And finally flips the scope back to "perMachine" which is the default in WiX 4 and newer
    #
    # Adapted from this stack overflow answer: https://stackoverflow.com/a/35064434
    #
@ -215,13 +200,32 @@ def main():
        Value="1",
    )

+    ET.SubElement(
+        package_el,
+        "WixVariable",
+        Id="WixUILicenseRtf",
+        Value="..\\install\\windows\\license.rtf",
+    )
+    ET.SubElement(
+        package_el,
+        "WixVariable",
+        Id="WixUIDialogBmp",
+        Value="..\\install\\windows\\dialog.bmp",
+    )
+    ET.SubElement(
+        package_el,
+        "MajorUpgrade",
+        AllowSameVersionUpgrades="yes",
+        DowngradeErrorMessage="A newer version of [ProductName] is already installed. If you are sure you want to downgrade, remove the existing installation via Programs and Features.",
+    )
+
    # Add the ProgramMenuFolder StandardDirectory
    programmenufolder_el = ET.SubElement(
        package_el,
        "StandardDirectory",
        Id="ProgramMenuFolder",
    )
-    # Add a shortcut for Dangerzone in the Start menu
+
    shortcut_el = ET.SubElement(
        programmenufolder_el,
        "Component",
@ -255,14 +259,14 @@ def main():
        Id="ProgramFiles64Folder",
    )

-    # Create the directory structure for the installed product
+    # Generate the directory layout for the installed product
    build_directory_xml(programfilesfolder_el, data)

-    # Create a component group for application components
    applicationcomponents_el = ET.SubElement(
        package_el, "ComponentGroup", Id="ApplicationComponents"
    )
-    # Populate the application components group with components for the installed package
+
+    # Generate the components for the installed product
    build_components_xml(applicationcomponents_el, data)

    # Add the Feature element
Author	SHA1	Message	Date
jkarasti	223fb0f1b9	Fix: Dangerzone installed using an msi built with WiX Toolset 3 is not uninstalled by an msi built with WiX Toolset 5 Work around the issue by adding some extra functionality to the "Next" button on the welcome screen of the installer. When the user clicks it to proceed with the installation this: 1. Flips the install scope to "perUser" which is the default in WiX 3 2. Finds the older installation 3. And finally flips the scope back to "perMachine" which is the default in WiX 4 and newer TODO: Revert this once we are reasonably certain there are no affected Dangerzone Installations?	2024-10-30 22:08:36 +02:00
jkarasti	8603cd3b86	Change: Wrap installer ui related things in a `UI` element fix	2024-10-30 22:08:23 +02:00
JKarasti	7e4346a306	Change: Build 64bit installer	2024-10-27 17:43:59 +02:00
JKarasti	dba5b7a3ac	Docs: Documentation for WiX Toolset 5	2024-10-27 17:43:59 +02:00
JKarasti	a0f99f89f7	Change: Make `build-app.bat` script work with WiX Toolset v5 - WiX Toolset v3 used to validate the msi package by default. In v5 that has moved to a new command, so add a new validation step to the script.	2024-10-27 17:43:59 +02:00
JKarasti	c1dc2490b5	Change: Use WiX Toolset v5 to build the msi in CI	2024-10-27 17:43:59 +02:00
JKarasti	6e2a95326d	Change: Write Dangerzone.wxs inside the script directly Also reduce duplication slightly by definig `build_dir`, `cx_freeze_dir` and `dist_dir`	2024-10-27 17:43:58 +02:00
JKarasti	5d47984e45	Fix: Make GUIDs uppercase See [1] [1] https://learn.microsoft.com/en-us/windows/win32/msi/guid	2024-10-27 17:43:58 +02:00
JKarasti	d61f8667ae	Change: Write dangerzone version and upgradecode into Package and SummaryInformation elements directly	2024-10-27 17:43:57 +02:00
JKarasti	4f97ed4177	Refactor: `build_dir_xml()` function - rename for clarity - remove unnecessary checks	2024-10-27 17:43:57 +02:00
JKarasti	9035497da3	Change: Wrap all files to be included in the .msi in a `ComponentGroupRef` With this, all the files are organised into Components, each of which points to a Directory defined in the StandardDirectory element. This simplifies the Feature element considerable as only thing it needs to include everything in the built msi is a reference to `ApplicationComponents`	2024-10-27 17:43:57 +02:00
JKarasti	ea3f4c88a5	Refactor: Simplify `build_data()` function - Rename variables to be more clear about what they do: - reorganise code - simplify a few checks	2024-10-27 17:43:57 +02:00
JKarasti	a1dba4a098	Change: Swap Media element with MediaTemplate This is a new default and makes authoring slightly simpler without any functional changes.	2024-10-27 17:43:56 +02:00
JKarasti	aab5bd08d7	Change: Convert Wix UI extension authoring to WiX Toolset v5 Due to limitations of the xml.etree.ElementTree library, add the items in the root element as a dictionary	2024-10-27 17:43:56 +02:00
JKarasti	965210dee5	Change: Wrap `ProgramFilesFolder` component with a `StandardDirectory` component	2024-10-27 17:43:56 +02:00
JKarasti	f7f81c5960	Change: Wrap `ProgramMenuFolder` component with a `StandardDirectory` component	2024-10-27 17:43:55 +02:00
JKarasti	8c26f3512b	Change: Merge Product into Package element - The Keywords and Description items move under a new SummaryInformation element. - Shuffle things around so that elements previously under the product element are now under the Package element. - Rename SummaryCodepage in SummaryInformation to Codepage and remove a duplicate Manufacturer item. - Remove InstallerVersion and let WiX set it to default value. (500 a.k.a Windows 7)	2024-10-27 17:43:55 +02:00