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
[directxmath] port updated for cmake improvements #15383
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@walbourn, thanks for the PR!
Could you also update the version in CONTROL file?
I've not updated the version of the library. It's just fixed the CMake. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@walbourn, I suggest to use the version that we referenced in portfile.cmake.
https://github.com/microsoft/DirectXMath/releases
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your updates!
Thanks for the PR! |
In PR 15210 I added directxmath as a new port using the CMake integration upstream. This works fine on Windows and installs for x64-linux, but trying to use it from Linux with
find_package
fails.This is due to case-sensitivity in the Linux filesystem combined with a CamelCase
PROJECT_NAME
, and the specific logic offind_package
behavior (it needs<PackageName>Config.cmake
or<lowercase-packagename>-config.cmake
). This is fixed upstream in this PR and this port updates to points to this version.Also moved the installed locations of DirectXMath headers into
include/DirectXMath
to avoid any accidental include order issues since those headers are in the Windows SDK as well.