From eebffb157146a67dda0f24588337fe21c8f7aea5 Mon Sep 17 00:00:00 2001
From: yuyoyuppe <a.yuyoyuppe@gmail.com>
Date: Fri, 12 Mar 2021 15:49:06 +0300
Subject: [PATCH] win32 -> x86, separate logging, fix(?) CI

---
 .pipelines/pipeline.user.windows.yml                      | 7 +++++--
 installer/PowerToysSetup/PowerToysSetup.wixproj           | 3 +--
 installer/PowerToysSetup/Product.wxs                      | 6 +++---
 installer/PowerToysSetup/build_vcm_x86.cmd                | 3 ---
 .../VideoConferenceModule/Video Conference.vcxproj        | 3 +++
 .../VideoConferenceModule/build_vcm_x86.cmd               | 3 +++
 .../VideoConferenceProxyFilter.vcxproj                    | 5 +++--
 .../videoconference/VideoConferenceShared/Logging.cpp     | 8 ++++++--
 .../VideoConferenceShared/VideoConferenceShared.vcxproj   | 4 ++--
 9 files changed, 26 insertions(+), 16 deletions(-)
 delete mode 100644 installer/PowerToysSetup/build_vcm_x86.cmd
 create mode 100644 src/modules/videoconference/VideoConferenceModule/build_vcm_x86.cmd

diff --git a/.pipelines/pipeline.user.windows.yml b/.pipelines/pipeline.user.windows.yml
index c3e3aea321..764cffb68e 100644
--- a/.pipelines/pipeline.user.windows.yml
+++ b/.pipelines/pipeline.user.windows.yml
@@ -61,6 +61,10 @@ build:
             - 'x64/**/*.pdb'
           exclude:
             - 'x64/Release/obj/**/*.pdb'
+        - from: 'x86/Release'
+        - to: 'Build_Output'
+          include:
+            - 'modules\VideoConference\VideoConferenceProxyFilter_x86.dll'
         - from: 'x64/Release'
           to: 'Build_Output'
           include:
@@ -147,8 +151,7 @@ build:
             - 'modules\PowerRename\PowerRenameExt.dll'
             - 'modules\ShortcutGuide\ShortcutGuide.dll'
             - 'modules\VideoConference\VideoConferenceModule.dll'
-            - 'modules\VideoConference\VideoConferenceProxyFilter.dll'
-            - 'modules\VideoConference\VideoConferenceProxyFilterx86.dll'
+            - 'modules\VideoConference\VideoConferenceProxyFilter_x64.dll'
             - 'Notifications.dll'
             - 'os-detection.dll'
             - 'PowerToys.exe'
diff --git a/installer/PowerToysSetup/PowerToysSetup.wixproj b/installer/PowerToysSetup/PowerToysSetup.wixproj
index b837586276..cab87ecbf9 100644
--- a/installer/PowerToysSetup/PowerToysSetup.wixproj
+++ b/installer/PowerToysSetup/PowerToysSetup.wixproj
@@ -77,8 +77,7 @@
 call "$([MSBuild]::GetVsInstallRoot())\Common7\Tools\VsDevCmd.bat" -arch=amd64 -host_arch=amd64 -winsdk=10.0.18362.0
 SET PTRoot=..\..\..\..
 call "..\..\publish.cmd"
-)
-call "..\..\build_vcm_x86.cmd"</PreBuildEvent>
+)</PreBuildEvent>
   </PropertyGroup>
   <!--
 	To modify your build process, add your task inside one of the targets below and uncomment it.
diff --git a/installer/PowerToysSetup/Product.wxs b/installer/PowerToysSetup/Product.wxs
index 9be1e160bf..90c393cdc5 100644
--- a/installer/PowerToysSetup/Product.wxs
+++ b/installer/PowerToysSetup/Product.wxs
@@ -12,7 +12,7 @@
 
   <?define RepoDir="$(var.ProjectDir)..\..\" ?>
   <?define BinX64Dir="$(var.RepoDir)x64\$(var.Configuration)\" ?>
-  <?define BinX32Dir="$(var.RepoDir)Win32\$(var.Configuration)\" ?>
+  <?define BinX32Dir="$(var.RepoDir)x86\$(var.Configuration)\" ?>
 
 	<Product Id="*"
        Name="PowerToys (Preview)"
@@ -597,10 +597,10 @@
     <DirectoryRef Id="VideoConferenceInstallFolder" FileSource="$(var.BinX64Dir)modules\$(var.VideoConferenceProjectName)\">
      <Component Id="Module_VideoConference" Guid="5996527a-40fc-432e-b3ac-abc0b4bd3887" Win64="yes">
       <Condition>WINDOWSBUILDNUMBER >= 18362</Condition>
-      <File SelfRegCost="1" Source="$(var.BinX64Dir)modules\$(var.VideoConferenceProjectName)\VideoConferenceProxyFilter.dll" KeyPath="yes">
+      <File SelfRegCost="1" Source="$(var.BinX64Dir)modules\$(var.VideoConferenceProjectName)\VideoConferenceProxyFilter_x64.dll" KeyPath="yes">
         <Class Id="{31AD75E9-8C3A-49C8-B9ED-5880D6B4A764}" Context="InprocServer32" ThreadingModel="apartment" Description="PowerToys VideoConference Mute" />
       </File>
-      <File SelfRegCost="1" Source="$(var.BinX32Dir)modules\$(var.VideoConferenceProjectName)\VideoConferenceProxyFilterx86.dll">
+      <File SelfRegCost="1" Source="$(var.BinX32Dir)modules\$(var.VideoConferenceProjectName)\VideoConferenceProxyFilter_x86.dll">
         <Class Id="{31AD75E9-8C3A-49C8-B9ED-5880D6B4A732}" Context="InprocServer32" ThreadingModel="apartment" Description="PowerToys VideoConference Mute" />
       </File>
       <File Source="$(var.BinX64Dir)modules\$(var.VideoConferenceProjectName)\VideoConferenceModule.dll" />
diff --git a/installer/PowerToysSetup/build_vcm_x86.cmd b/installer/PowerToysSetup/build_vcm_x86.cmd
deleted file mode 100644
index a5a47fddbf..0000000000
--- a/installer/PowerToysSetup/build_vcm_x86.cmd
+++ /dev/null
@@ -1,3 +0,0 @@
-msbuild ..\..\..\..\src\modules\videoconference\VideoConferenceProxyFilter\VideoConferenceProxyFilterx86.sln -p:Configuration="Release" -p:Platform="Win32"
-
-exit 0
\ No newline at end of file
diff --git a/src/modules/videoconference/VideoConferenceModule/Video Conference.vcxproj b/src/modules/videoconference/VideoConferenceModule/Video Conference.vcxproj
index a3b38fbd71..eaacc8cccd 100644
--- a/src/modules/videoconference/VideoConferenceModule/Video Conference.vcxproj	
+++ b/src/modules/videoconference/VideoConferenceModule/Video Conference.vcxproj	
@@ -79,6 +79,9 @@
       <OutputFile>$(OutDir)$(TargetName)$(TargetExt)</OutputFile>
       <AdditionalDependencies>mfplat.lib;mf.lib;mfreadwrite.lib;mfuuid.lib;shlwapi.lib;gdiplus.lib;dwmapi.lib;uxtheme.lib;shcore.lib;Wtsapi32.lib;%(AdditionalDependencies)</AdditionalDependencies>
     </Link>
+    <PreBuildEvent>
+      <Command>call "$(ProjectDir)build_vcm_x86.cmd"</Command>
+    </PreBuildEvent>
     <PostBuildEvent>
       <Command>xcopy /y /I  "$(ProjectDir)Icons\*" "$(OutDir)Icons"
 xcopy /y /I  "$(ProjectDir)black.bmp*" "$(OutDir)"</Command>
diff --git a/src/modules/videoconference/VideoConferenceModule/build_vcm_x86.cmd b/src/modules/videoconference/VideoConferenceModule/build_vcm_x86.cmd
new file mode 100644
index 0000000000..1ebc12eb4f
--- /dev/null
+++ b/src/modules/videoconference/VideoConferenceModule/build_vcm_x86.cmd
@@ -0,0 +1,3 @@
+msbuild ..\VideoConferenceProxyFilter\VideoConferenceProxyFilterx86.sln -p:Configuration="Release" -p:Platform="Win32"
+
+exit 0
\ No newline at end of file
diff --git a/src/modules/videoconference/VideoConferenceProxyFilter/VideoConferenceProxyFilter.vcxproj b/src/modules/videoconference/VideoConferenceProxyFilter/VideoConferenceProxyFilter.vcxproj
index a6b57ccffa..735b954d8e 100644
--- a/src/modules/videoconference/VideoConferenceProxyFilter/VideoConferenceProxyFilter.vcxproj
+++ b/src/modules/videoconference/VideoConferenceProxyFilter/VideoConferenceProxyFilter.vcxproj
@@ -63,10 +63,11 @@
   </ItemDefinitionGroup>
   <PropertyGroup Condition="'$(Platform)'!='Win32'">
     <OutDir>$(SolutionDir)$(Platform)\$(Configuration)\modules\VideoConference\</OutDir>
+    <TargetName>VideoConferenceProxyFilter_x64</TargetName>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Platform)'=='Win32'">
-    <OutDir>..\..\..\..\$(Platform)\$(Configuration)\modules\VideoConference\</OutDir>
-    <TargetName>VideoConferenceProxyFilterx86</TargetName>
+    <OutDir>..\..\..\..\x86\$(Configuration)\modules\VideoConference\</OutDir>
+    <TargetName>VideoConferenceProxyFilter_x86</TargetName>
   </PropertyGroup>
   <Import Project="$(VCTargetsPath)\Microsoft.Cpp.props" />
   <ImportGroup Label="ExtensionSettings">
diff --git a/src/modules/videoconference/VideoConferenceShared/Logging.cpp b/src/modules/videoconference/VideoConferenceShared/Logging.cpp
index 8befc89d54..382233f974 100644
--- a/src/modules/videoconference/VideoConferenceShared/Logging.cpp
+++ b/src/modules/videoconference/VideoConferenceShared/Logging.cpp
@@ -9,7 +9,7 @@
 
 #include <mfapi.h>
 
-#pragma warning(disable: 4127)
+#pragma warning(disable : 4127)
 
 static std::mutex logMutex;
 constexpr inline size_t maxLogSizeMegabytes = 10;
@@ -37,7 +37,11 @@ void LogToFile(std::wstring what, const bool verbose)
 
     std::lock_guard lock{ logMutex };
     std::wstring logFilePath = tempPath;
-    logFilePath += L"\\PowerToysVideoConference.log";
+#if defined(_WIN64)
+    logFilePath += L"\\PowerToysVideoConference_x64.log";
+#elif defined(_WIN32)
+    logFilePath += L"\\PowerToysVideoConference_x86.log";
+#endif
     size_t logSizeMBs = 0;
     try
     {
diff --git a/src/modules/videoconference/VideoConferenceShared/VideoConferenceShared.vcxproj b/src/modules/videoconference/VideoConferenceShared/VideoConferenceShared.vcxproj
index 2b8b3bcb8c..1b628c7e89 100644
--- a/src/modules/videoconference/VideoConferenceShared/VideoConferenceShared.vcxproj
+++ b/src/modules/videoconference/VideoConferenceShared/VideoConferenceShared.vcxproj
@@ -53,8 +53,8 @@
     <IntDir>$(SolutionDir)$(Platform)\$(Configuration)\obj\$(ProjectName)\</IntDir>
   </PropertyGroup>
   <PropertyGroup Condition="'$(Platform)'=='Win32'">
-    <OutDir>..\..\..\..\$(Platform)\$(Configuration)\modules\VideoConference\</OutDir>
-    <IntDir>..\..\..\..\$(Platform)\$(Configuration)\obj\$(ProjectName)\</IntDir>
+    <OutDir>..\..\..\..\x86\$(Configuration)\modules\VideoConference\</OutDir>
+    <IntDir>..\..\..\..\x86\$(Configuration)\obj\$(ProjectName)\</IntDir>
   </PropertyGroup>
   <PropertyGroup>
     <LinkIncremental>true</LinkIncremental>