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

Also add a dynamic library #100

Closed
wants to merge 1 commit into from

Conversation

msoos
Copy link

@msoos msoos commented Jun 3, 2024

This also builds a dynamic library, besides a static one. I have a feeling it may have been purposefully left out, but I need it, and I think it may not be too harmful to do it? Just a suggestion.

Mate

@kotopesutility
Copy link

I would also specify soname like Fedora does.

@arminbiere
Copy link
Owner

See issue #40. The solution will be to require to use ./configure --shared which builds both. The default will remain to only build the static library. This will be ported from Kissat. Next feature update after the 2.0.0 release is planned to address this.

@arminbiere arminbiere closed this Jun 18, 2024
@msoos
Copy link
Author

msoos commented Jun 18, 2024 via email

@kotopesutility
Copy link

Next feature update after the 2.0.0 release is planned to address this.

After 2.1.0?

@arminbiere
Copy link
Owner

Thanks for bumping. We had to first improve the user propagator part and now started to work on merging other branches. We will keep an eye on this one too.

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