-
Notifications
You must be signed in to change notification settings - Fork 113
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
Support dynamic SPM product #295
base: master
Are you sure you want to change the base?
Support dynamic SPM product #295
Conversation
bb20c90
to
dcea5f7
Compare
Hi @andrii-vysotskyi-cko, thanks for your contribution and apologies for the late response. Could you clarify a bit on the need for declaring this separate library product? According to SPM documentation, leaving the linking type empty lets SPM decide which one to use—effectively supporting both linkage types. I might be missing something here but, to me, this sounds like there is no need to explicitly declare a separate dynamic-only library product:
📚 SPM Docs |
Hey @daniel-moh, thank you for the response. The use case we are facing is that there is a binary only dynamic framework that is distributed via SPM. This framework depends on Having an ability to use "dynamic" version of |
Thanks for the clarification @andrii-vysotskyi-cko. I understand that adding an additional explicit dynamic library product represents the workaround you need, but it does introduce some complexity that feels a bit unconventional to me. I am still wondering why SPM doesn't automatically pick the correct product like it seems to suggest in its docs. To better understand the specific issue you’re encountering, could you provide a minimal reproducible example or a more detailed explanation of the problem? That way we can rule out that this is a bug in SPM or potentially a misconfiguration on your end and make sure that this is in fact something we need to change on the library end. 🙏 |
2689d99
to
4f060ee
Compare
This PR adds a dynamic product to
JOSESwift
.