implement better workaround for the lack of automatic interaction between MSBuild & MSBuildToolchain

Author: SpaceIm

recipes/libsodium/all/conanfile.py

@@ -76,6 +76,10 @@ def source(self):
     def generate(self):
         if is_msvc(self):
             tc = MSBuildToolchain(self)
+            tc.configuration = "{}{}".format(
+                "Debug" if self.settings.build_type == "Debug" else "Release",
+                "DLL" if self.options.shared else "LIB",
+            )
             tc.generate()
         else:
             env = VirtualBuildEnv(self)
@@ -121,68 +125,30 @@ def _msvc_sln_folder(self):
         return sln_folders.get(str(self.settings.compiler), {}).get(str(self.settings.compiler.version), default_folder)
 
     def _build_msvc(self):
-        msvc_builds_folder = os.path.join(self.source_folder, "builds", "msvc")
-        msvc_sln_folder = os.path.join(msvc_builds_folder, self._msvc_sln_folder)
-        vcxproj_path = os.path.join(msvc_sln_folder, "libsodium", "libsodium.vcxproj")
+        msvc_sln_folder = os.path.join(self.source_folder, "builds", "msvc", self._msvc_sln_folder)
 
-        # 1.0.18 only supported up to vs2019. Add support for newer toolsets.
+        #==============================
+        # TODO: to remove once https://github.com/conan-io/conan/pull/12817 available in conan client
         if self.version == "1.0.18" and self._msvc_sln_folder == "vs2019":
             toolset = MSBuildToolchain(self).toolset
             replace_in_file(
-                self, vcxproj_path,
+                self, os.path.join(msvc_sln_folder, "libsodium", "libsodium.vcxproj"),
                 "<PlatformToolset>v142</PlatformToolset>",
                 f"<PlatformToolset>{toolset}</PlatformToolset>",
             )
-
-        # FIXME: There is currently no guarantee from new MSBuild helper that props files
-        # generated by MSBuildToolchain and MSBuildDeps have precedence over custom values of project.
-        # Therefore conantoolchain.props is injected manually before Microsoft.Cpp.Default.props like it was done
-        # with /p:ForceImportBeforeCppTargets in legacy MSBuild helper.
-        # see:
-        #  - https://learn.microsoft.com/en-us/cpp/build/modify-project-properties-without-changing-project-file
-        #  - https://github.com/conan-io/conan/issues/12155
-        conantoolchain_props = os.path.join(self.generators_folder, "conantoolchain.props")
+        conantoolchain_props = os.path.join(self.generators_folder, MSBuildToolchain.filename)
         replace_in_file(
-            self, vcxproj_path,
-            "<Import Project=\"$(VCTargetsPath)\\Microsoft.Cpp.Default.props\" />",
-            f"<Import Project=\"{conantoolchain_props}\" />\n<Import Project=\"$(VCTargetsPath)\\Microsoft.Cpp.Default.props\" />",
+            self, os.path.join(msvc_sln_folder, "libsodium", "libsodium.vcxproj"),
+            "<Import Project=\"$(VCTargetsPath)\\Microsoft.Cpp.targets\" />",
+            f"<Import Project=\"{conantoolchain_props}\" /><Import Project=\"$(VCTargetsPath)\\Microsoft.Cpp.targets\" />",
         )
-
-        # Honor runtime library from profile
-        runtime_library = MSBuildToolchain(self).runtime_library
-        for prop_file, runtime_library_old in [
-            ("DebugDEXE.props", "MultiThreadedDebugDLL"),
-            ("DebugDLL.props", "MultiThreadedDebugDLL"),
-            ("DebugLEXE.props", "MultiThreadedDebug"),
-            ("DebugLIB.props", "MultiThreadedDebug"),
-            ("DebugLTCG.props", "MultiThreadedDebug"),
-            ("DebugSEXE.props", "MultiThreadedDebug"),
-            ("ReleaseDEXE.props", "MultiThreadedDLL"),
-            ("ReleaseDLL.props", "MultiThreadedDLL"),
-            ("ReleaseLEXE.props", "MultiThreaded"),
-            ("ReleaseLIB.props", "MultiThreaded"),
-            ("ReleaseLTCG.props", "MultiThreaded"),
-            ("ReleaseSEXE.props", "MultiThreaded"),
-        ]:
-            replace_in_file(
-                self, os.path.join(msvc_builds_folder, "properties", prop_file),
-                f"<RuntimeLibrary>{runtime_library_old}</RuntimeLibrary>",
-                f"<RuntimeLibrary>{runtime_library}</RuntimeLibrary>",
-            )
+        #==============================
 
         msbuild = MSBuild(self)
-        build_type = "{}{}".format(
+        msbuild.build_type = "{}{}".format(
             "Dyn" if self.options.shared else "Static",
             "Debug" if self.settings.build_type == "Debug" else "Release",
         )
-
-        platform = {
-            "x86": "Win32",
-            "x86_64": "x64",
-        }[str(self.settings.arch)]
-
-        msbuild.build_type = build_type
-        msbuild.platform = platform
         msbuild.build(os.path.join(msvc_sln_folder, "libsodium.sln"))
 
     def build(self):