From 444fbe0c8f5db9d20272a7659f85bd56087d77c5 Mon Sep 17 00:00:00 2001
From: Yair Morgenstern <yairm210@hotmail.com>
Date: Sun, 28 Jul 2024 15:57:10 +0300
Subject: [PATCH] Version correct

---
 .github/workflows/CI.yml | 2 +-
 unciv.wxs                | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
index 4342b8a..0e84a72 100644
--- a/.github/workflows/CI.yml
+++ b/.github/workflows/CI.yml
@@ -28,7 +28,7 @@ jobs:
         run: dotnet tool install --global wix
         
       - name: Build WiX on Windows
-        run: |
+        run: | # Trying to set env variables for a simple command is like ???, now I truly understand the "Windows Sucks" mentality
           cmd /C "set UNCIV_VERSION=4.12.14 && wix build .\unciv.wxs"
           dir
         
diff --git a/unciv.wxs b/unciv.wxs
index d5d9087..23199aa 100644
--- a/unciv.wxs
+++ b/unciv.wxs
@@ -5,8 +5,9 @@
     <Package Language="1033"
              Manufacturer="Yair Morgenstern"
              Name="$(var.Name)"
-             Version="$(env.UNCIV_VERSION)"
+             Version="$(env.UNCIV_VERSION).0"
             UpgradeCode="74d42802-054a-4869-a703-658c361fe94c">
+             <!-- MSI versions require a 4-part, so we add a ".0" to the end of every version -->
         <Icon Id="icon.ico" SourceFile="files/Unciv.ico"/>
         <MajorUpgrade DowngradeErrorMessage="A newer version of [ProductName] is already installed." Schedule="afterInstallInitialize" />