-
Notifications
You must be signed in to change notification settings - Fork 40
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
Enable shared libs on win32 #59
Conversation
“temporarily remove other tests” Please remember to get them back before we merge the PR :) Thank you. |
This comment has been minimized.
This comment has been minimized.
I tested this PR with the cross-compiler and shared libraries (dll) and the import libraries (dll.a) are now generated on Windows.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Hi, @jschueller and @amontoison , Thank @jschueller for the efforts! Is this PR ready? If yes, I will merge it after @jschueller gets the deleted tests back (remember to do this :) ). Thank you. Best regards, |
this is good to go |
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 description provided.