-
Notifications
You must be signed in to change notification settings - Fork 530
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
export platform #4574
base: main
Are you sure you want to change the base?
export platform #4574
Conversation
cdc2238
to
36ca8b5
Compare
src/bin/CMakeLists.txt
Outdated
@@ -269,7 +269,7 @@ endif() | |||
file(GLOB PUBLIC_HEADERS "../inc/*.h" "../inc/*.hpp") | |||
|
|||
if(BUILD_SHARED_LIBS) | |||
install(TARGETS msquic EXPORT msquic DESTINATION lib) | |||
install(TARGETS msquic platform inc logging_inc warnings main_binary_link_args OpenSSL OpenSSLQuic EXPORT msquic DESTINATION lib) |
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.
OpenSSL isn't always used.
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.
I know. I write that on the description on this PR.
Can you suggest me how to do it more general?
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.
There is other targets in other tls options?
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.
I think I can work with this with the ci.
This is a draft PR and need more work done for it.
@nibanks can you help me to solve these issue in this PR? |
36ca8b5
to
9cbe47d
Compare
9cbe47d
to
e615e0f
Compare
Description
For support compile msquic with external msh3 version 0.7.0. It require symbol that found on platform target.
I export these symbols
Other info
This is a draft PR and need more work done for it.
Issue in this PR (maybe more)
fix #4569