Skip to content
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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

talregev
Copy link
Contributor

@talregev talregev commented Sep 28, 2024

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)

  • It create platform.lib. this is a generic name that can collide with other libs. I propose to change target platform to msquic_platform
  • I export openssl targets, but there other option that it use schannel, or osx ssl lib. need to do it more generic

fix #4569

@talregev talregev requested a review from a team as a code owner September 28, 2024 03:44
@talregev talregev marked this pull request as draft September 28, 2024 03:47
@talregev talregev marked this pull request as ready for review September 28, 2024 03:48
@nibanks nibanks added Area: Build external Proposed by non-MSFT labels Sep 28, 2024
@@ -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)
Copy link
Member

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.

Copy link
Contributor Author

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?

Copy link
Contributor Author

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?

Copy link
Contributor Author

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.

@talregev
Copy link
Contributor Author

This is a draft PR and need more work done for it.
Issue in this PR (maybe more)

  • It create platform.lib. this is a generic name that can collide with other libs. I propose to change target platform to msquic_platform
  • I export openssl targets, but there other option that it use schannel, or osx ssl lib. need to do it more generic

@nibanks can you help me to solve these issue in this PR?

@talregev
Copy link
Contributor Author

@nibanks
I can work on the second issue with working ci. If you approve my other PR: #4575 then the ci will work automatic for me.

Let me know what do you think about first issue. Is it acceptable from you to change the target name platform to msquic_platform ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Build external Proposed by non-MSFT
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expose CxPlat SelfSignedCert APIs
2 participants