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

Yices2 MacOS Support #400

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

xeren
Copy link

@xeren xeren commented Sep 25, 2024

This PR extends the compile script for the Yices2j JNI-wrapper to support a Darwin-based system. The wrapper is statically linked with its dependencies to produce the single file libyices2j.dylib.

I tested it on an Apple M2 with Sonoma 14.6.1. The targets were arm64, as well as x86_64 (Rosetta 2, with arch x86_64 ./compile.sh). I compiled with clang. For this test, I built Yices 2.6.4 with ./configure --enable-thread-safety. Dependencies were installed using brew: gmp 6.3.0, gperf 3.1, openjdk@21 21.0.4. The build may deviate from what you use on Linux.

The modifications on compile.sh should not interfere with the Linux build, but this must be tested. I also modified the wrapper code to fix usages from potentially uninitialized variables.

This may conflict with #215.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants