Skip to content

Conversation

@walbourn
Copy link
Member

@walbourn walbourn commented Nov 13, 2025

Updates the gameinput port to use the latest NuGet release.

This port update also addressed two issues:

  • The latest gameinput.lib is not compatible with MinGW, but there's a way to build the lib from source so I do that in this case.
  • The "detecting GDKX" variables now support the 'new layout' scenario.
  • Changes comply with the maintainer guide.
  • SHA512s are updated for each updated download.
  • The "supports" clause reflects platforms that may be fixed by this new version.
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

The GameDKXboxLatest variable would need passed through to work properly in the xbox scenario, but this is being dealt with via 1834.

@walbourn
Copy link
Member Author

Verified locally:

.\vcpkg install gameinput:x64-windows
.\vcpkg install gameinput:x64-windows-static
.\vcpkg install gameinput:x64-windows-static-md
.\vcpkg install directxtk[gameinput]:x64-windows
.\vcpkg install directxtk12[gameinput]:x64-windows

.\vcpkg install gameinput:x64-mingw-static
.\vcpkg install directxtk[gameinput]:x64-mingw-static
.\vcpkg install directxtk12[gameinput]:x64-mingw-static

.\vcpkg install gameinput:x64-mingw-dynamic
.\vcpkg install directxtk[gameinput]:x64-mingw-dynamic
.\vcpkg install directxtk12[gameinput]:x64-mingw-dynamic

Copy link
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since you're depending on "vcpkg-cmake" the function call needs to be vcpkg_cmake_configure rather than vcpkg_configure_cmake. Sorry :(

@BillyONeal BillyONeal marked this pull request as draft November 13, 2025 23:42
@walbourn walbourn marked this pull request as ready for review November 14, 2025 18:36
@walbourn walbourn requested a review from BillyONeal November 14, 2025 19:58
Copy link
Member

@BillyONeal BillyONeal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@BillyONeal BillyONeal enabled auto-merge (squash) November 14, 2025 20:27
@BillyONeal BillyONeal merged commit d6b4d40 into microsoft:master Nov 14, 2025
18 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