-
Notifications
You must be signed in to change notification settings - Fork 6.2k
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
[collada-dom] Add REMOVE_RECURSE to solve DLLs error path #16763
[collada-dom] Add REMOVE_RECURSE to solve DLLs error path #16763
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.
Please also update the patches format.
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.
collada-dom only supports shared library linkage: https://github.com/rdiankov/collada-dom/blob/master/dom/CMakeLists.txt#L31.
We either need to patch out the SHARED
keyword from the add_library() call or we need to add vcpkg_check_linkage(ONLY_DYNAMIC_LIBRARY)
at the top of this port.
Thanks! |
Describe the pull request
When install colladda-dom:x64-windows-static ,The error shows that DLL files are generated in the bin directory during static install, use REMOVE_RECURSE in the portfile to remove them,
Note: no feature need to test