Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Switch to building with WDK nuget #232

Merged
merged 4 commits into from
Nov 18, 2024

Conversation

Alan-Jowett
Copy link
Member

@Alan-Jowett Alan-Jowett commented Nov 16, 2024

This pull request includes several updates to the project configuration files to integrate and utilize the Windows Driver Kit (WDK) and the Windows SDK via NuGet packages. The main changes involve modifying the project files to import a new wdk.props file, updating the Windows Target Platform Version, and adjusting include directories.

Key changes include:

Integration of WDK and Windows SDK via NuGet:

  • CMakeLists.txt: Added logic to install required NuGet packages for WDK and Windows SDK and set the WDK_ROOT based on the platform (arm64 or x64).
  • wdk.props: Added a new file to import the necessary properties and targets from the installed NuGet packages.
  • Project Files: Updated various project files (.vcxproj) to import the wdk.props file and set the Windows Target Platform Version to 10.0.26100.0. This includes:
    • cxplat_test.vcxproj
    • cxplat_winkernel.vcxproj
    • cxplat_winuser.vcxproj
    • sample.vcxproj
    • setup_build.vcxproj
    • usersim.vcxproj
    • tests.vcxproj
    • usersim_dll_skeleton.vcxproj

Update of Include Directories:

  • Include Directories: Adjusted the AdditionalIncludeDirectories in various project files to use the new WDK and SDK directories from the NuGet packages. This includes:

Minor Code Adjustments:

  • framework.h: Updated the include path for netioddk.h to remove the relative path.

These changes ensure that the project is correctly configured to use the latest versions of the WDK and Windows SDK, improving compatibility and maintainability.

@Alan-Jowett Alan-Jowett merged commit 30fc371 into microsoft:main Nov 18, 2024
9 checks passed
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.

2 participants