Skip to content

Conversation

@mheily
Copy link
Owner

@mheily mheily commented May 1, 2022

Relates to issue#135

@mheily mheily requested a review from arr2036 May 1, 2022 01:21
@arr2036
Copy link
Collaborator

arr2036 commented May 1, 2022

That seems to break the Windows build. I remember in the original CMake config we explicitly didn't attempt to build static archives for Windows, maybe we need a similar restriction still?

@mheily
Copy link
Owner Author

mheily commented Jun 6, 2022

I tried disabling the static library on Windows, but it still failed.

I'll try setting up a Windows development machine, and see if I can figure out how to fix the build.

@arr2036 arr2036 force-pushed the master branch 2 times, most recently from 090e0fc to a8d7805 Compare June 17, 2022 20:26
@mheily mheily force-pushed the issue135/unified_shared_static branch from c690637 to 8b826c4 Compare June 20, 2022 23:38
@mheily mheily marked this pull request as draft June 20, 2022 23:52
@mheily mheily force-pushed the issue135/unified_shared_static branch from e24621f to 35849fa Compare June 21, 2022 00:31
@mheily mheily marked this pull request as ready for review June 21, 2022 00:43
Copy link
Collaborator

@arr2036 arr2036 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@arr2036 arr2036 merged commit 80933d2 into master Jun 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants