From f843db26c50c981d3864cc08ba2dc3535ea44a37 Mon Sep 17 00:00:00 2001
From: Carlos Fernandez <carlos@ccextractor.org>
Date: Thu, 19 Feb 2026 17:45:06 -0800
Subject: [PATCH] Fix WiX installer: use preprocessor variable for conditional
 directory

The <?if>/<else> around <StandardDirectory> opening tags produced
invalid XML because WiX parses XML structure before evaluating
preprocessor directives. Use a <?define> variable for the ID instead.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 windows/installer.wxs | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/windows/installer.wxs b/windows/installer.wxs
index 05574861..7f5fd82f 100644
--- a/windows/installer.wxs
+++ b/windows/installer.wxs
@@ -40,10 +40,11 @@
         </Component>
     </StandardDirectory>
 		<?if $(sys.BUILDARCH) = x86 ?>
-		<StandardDirectory Id="ProgramFilesFolder">
+		<?define ProgramFilesDir = "ProgramFilesFolder" ?>
 		<?else ?>
-		<StandardDirectory Id="ProgramFiles64Folder">
+		<?define ProgramFilesDir = "ProgramFiles64Folder" ?>
 		<?endif ?>
+		<StandardDirectory Id="$(var.ProgramFilesDir)">
 			<Directory Id="INSTALLFOLDER" Name="CCExtractor">
 				<Directory Id="CCX_tessdata" Name="tessdata"/>
 				<Directory Id="CCX_data" Name="data">