Fix MSBuild PreprocessorDefinitions

DefineConstants is a C# property. For C++. it must be PreprocessorDefinitions under a ClCompile item. Also, the logic was twisted
canonical · Sep 18, 2023 · 23084ab · 23084ab
1 parent ed6b228
commit 23084ab
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/msix/storeapi/storeapi.vcxproj b/msix/storeapi/storeapi.vcxproj
@@ -43,9 +43,11 @@
     <Import Project="$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props" Condition="exists('$(UserRootDir)\Microsoft.Cpp.$(Platform).user.props')" Label="LocalAppDataPlatform" />
   </ImportGroup>
   <PropertyGroup Label="UserMacros" />
-  <PropertyGroup Label="Mock API" Condition="'$(UP4W_TEST_WITH_MS_STORE_MOCK)' == ''">
-    <DefineConstants>UP4W_TEST_WITH_MS_STORE_MOCK</DefineConstants>
-  </PropertyGroup>
+  <ItemDefinitionGroup Label="Mock API" Condition="'$(UP4W_TEST_WITH_MS_STORE_MOCK)' != ''">
+    <ClCompile>
+      <PreprocessorDefinitions>UP4W_TEST_WITH_MS_STORE_MOCK;%(PreProcessorDefinitions)</PreprocessorDefinitions>
+    </ClCompile>
+  </ItemDefinitionGroup>
   <ItemDefinitionGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">
     <ClCompile>
       <WarningLevel>Level3</WarningLevel>