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

String updates #147

Closed
wants to merge 4 commits into from
Closed

String updates #147

wants to merge 4 commits into from

Conversation

robdockins
Copy link
Contributor

Updates regarding string theories for Z3 and CVC4.

Recent versions of Z3 (4.8.11, maybe earlier?) and CVC4 (1.8) have
changed their string solver support to use unicode codepoints as their
atomic alphabet, and changed the string escaping rules to be those
specified by the SMTLib2.6 standard.

Here, we change our string escaping rules to suit, and modify the
SMTWriter module to express problems in term of unicode sequences
instead of ASCII sequences.

Fixes #56
Reenable the prevsiouly-broken stringTest3.
Fixes #132.
@robdockins
Copy link
Contributor Author

Tests are failing here because the Z3 versions on the build machines (4.8.8 or 4.8.10) is too old. We need 4.8.11.

@robdockins robdockins requested a review from RyanGlScott August 3, 2021 22:12
@robdockins
Copy link
Contributor Author

@RyanGlScott, do you know enough Nix magic to know how to get newer Z3s for the build environments? I'm out of my depth.

@RyanGlScott
Copy link
Contributor

I'm afraid I'm out of my depth as well. Generally speaking, picking newer versions of packages than what a particular nixpkgs version offers requires constructing an appropriate Nix script, and my Nix-fu is admittedly rather weak.

I wonder instead if it would be easier to use a dedicated GitHub Action that downloads a specific Z3 release, such as actions/install-z3.

@robdockins robdockins requested a review from kquick August 3, 2021 23:07
@robdockins
Copy link
Contributor Author

In that case I'll wait for @kquick to have time to look at this.

@robdockins
Copy link
Contributor Author

Included in #177

@robdockins robdockins closed this Jan 11, 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.

2 participants