Skip to content

Conversation

@yeelam-gordon
Copy link
Contributor

Summary of the Pull Request

This pull request updates the build system for several native and managed projects, modernizing NuGet package management and improving code analysis configuration. The main changes involve switching from legacy packages.config and manual .props/.targets imports to PackageReference style for native projects, updating package versions, and streamlining code analysis settings.

Build system modernization and package management:

  • Migrated native projects (PowerToys.MeasureToolCore.vcxproj, FindMyMouse.vcxproj) from legacy packages.config and manual .props/.targets imports to NuGet PackageReference style, simplifying dependency management and build configuration. This includes removing the packages.config file and related import/error logic, and introducing PackageReference items for required packages. [1] [2] [3] [4] [5]
  • Updated the centralized package versions in Directory.Packages.props, adding new entries for boost, boost_regex-vc143, Microsoft.Windows.ImplementationLibrary, and Microsoft.WindowsAppSDK.Foundation to support the new build system and dependencies. [1] [2]

Code analysis improvements:

  • Added configuration to both native and managed projects (PowerToys.MeasureToolCore.vcxproj, MeasureToolUI.csproj) to suppress specific warnings (81010002) and exclude NuGet cache files from code analysis, reducing noise and improving build performance. [1] [2]

Project reference and output handling:

  • Updated the managed project (MeasureToolUI.csproj) to handle native project outputs more robustly, ensuring the WinMD and DLL files are available at runtime and configuring the project reference to avoid assembly reference issues.

Compiler configuration:

  • Enhanced C++ compiler settings in Cpp.Build.props to treat angle-bracket includes as external, disable warnings and analysis for external headers, and optimize build performance.

@yeelam-gordon yeelam-gordon requested a review from a team as a code owner November 27, 2025 05:33
@github-actions

This comment has been minimized.

@yeelam-gordon
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@yeelam-gordon yeelam-gordon changed the title Draft on using centralized package management for vcxproj Using centralized package management for vcxproj Dec 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants