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

Update bundled Z3 to z3 4.12.3. #279

Merged
merged 2 commits into from
May 29, 2024
Merged

Update bundled Z3 to z3 4.12.3. #279

merged 2 commits into from
May 29, 2024

Conversation

waywardmonkeys
Copy link
Contributor

This also changes a test to be less specific about the exact answer since it varies between versions of Z3.

@waywardmonkeys
Copy link
Contributor Author

cc: @Pat-Lafon

@Pat-Lafon
Copy link
Contributor

The changes look good to me. Are you asking about diffing the windows linking issue?

@waywardmonkeys
Copy link
Contributor Author

4.12.4 has been released ... I'll update to that and see if that fixes the Windows linking issues. Oddly enough, it links for me locally, so I don't know what's wrong yet. But I was pinging you to see if you had any comments about any part of it.

@Pat-Lafon
Copy link
Contributor

FWIW I tried building z3 using github actions with 4.12.4 and still ran into this issue. https://github.com/Pat-Lafon/z3.rs/actions/runs/7215253276/job/19659127883?pr=2

@Pat-Lafon
Copy link
Contributor

I was trying a couple different commits and I think I narrowed down the issue to this commit: Z3Prover/z3@9d3fef3 . The previous commit in the history seems to pass but upgrading to a more recent cmake probably needs something else on windows.

@waywardmonkeys
Copy link
Contributor Author

Look who did that commit!

It must be a cmake policy thing ... but I'm about to prepare for travel and fly half way around the world this weekend ...

waywardmonkeys and others added 2 commits May 29, 2024 14:13
This fixes an issue when using a more recent set of cmake policies
with linking debug builds on Windows.
@waywardmonkeys waywardmonkeys merged commit d134bb3 into master May 29, 2024
11 checks passed
@waywardmonkeys waywardmonkeys deleted the update-bundled-z3 branch May 29, 2024 07:54
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.

None yet

3 participants