-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
[gainput] imporve cmake search gainput library #11000
Conversation
17043a6
to
a91e301
Compare
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.
gainput*.cmake
is still generated in share/gainput. We should generate unofficial-gainput*.cmake
in share/unofficial-gainput.
And set vcpkg_fixup_cmake_targets()
to vcpkg_fixup_cmake_targets(CONFIG_PATH unofficial-gainput TARGET_PATH unofficial-gainput)
.
See example.
Thanks for giving a examples. I have modified the relevant code. |
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.
Dynamic build generated gainputstatic.lib
. Can you confirm this?
This behavior existed before I modified the code. Should I modify it? |
@L-Sun Yes, dynamic libraries can only be generated in dynamic triples, as are static libraries. |
It has passed on x64_windows_static pipeline, so should we need update ci.baseline.txt with |
@L-Sun Just remove |
|
I don’t have an Apple device, so I can’t test in an osx environment. I have reset osx related code in |
In SOURCE/lib/CMakeLists.txt line 54:
When using the triple |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
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.
No, the baseline seems incorrect and not related with this PR.
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
LGTM, thanks for this PR! |
Alright, this is cool! Thanks @L-Sun :) |
Describe the pull request
Add a patch to improve CMake import [gainput]