Skip to content

Conversation

@maelvls
Copy link
Contributor

@maelvls maelvls commented Jun 8, 2017

QBF solving in OCaml, including bindings to solvers

Ocaml-qbf provides a unified API to several QBF solvers, along with
common types for formulas, QCNF and literals and operations such as
simplifications and reduction to CNF.

It packs:

  • a binding to quantor, which is shipped with the library
  • a binding to Depqbf


Pull-request generated by opam-publish v0.3.4

@camelus
Copy link
Contributor

camelus commented Jun 8, 2017

✅ All lint checks passed 5fd5b61
  • These packages passed lint tests: qbf.0.2

✅ Installability check (6833 → 6834)
  • new installable packages (1): qbf.0.2
@maelvls maelvls force-pushed the opam-publish/qbf.0.2 branch from a728ae2 to bbaf08b Compare June 8, 2017 16:15
@maelvls maelvls changed the title qbf.0.2 - via opam-publish Jun 8, 2017
@maelvls maelvls force-pushed the opam-publish/qbf.0.2 branch from bbaf08b to 748073e Compare June 9, 2017 11:06
@maelvls maelvls changed the title [Work in progress] qbf.0.2 - via opam-publish Jun 9, 2017
@maelvls maelvls force-pushed the opam-publish/qbf.0.2 branch from 748073e to 5fd5b61 Compare June 9, 2017 13:10
@avsm
Copy link
Member

avsm commented Jun 9, 2017

thanks for this! The unistd.h is failing on alpine linux, and it might just need the linux-headers package there. it's not a blocker to merge, but you can try to fix it if you'd like a full green light.

@maelvls
Copy link
Contributor Author

maelvls commented Jun 9, 2017

Oh... bummer. I just did a fix to the project in order to avoid this <sys/unistd.h> (I think it wasn't actually needed)

@avsm
Copy link
Member

avsm commented Jun 9, 2017

that's an even better fix, and makes it more portable to the BSDs as well. thanks!

@maelvls
Copy link
Contributor Author

maelvls commented Jun 9, 2017

Looks like it passes the Alpine 3.5 tests now! 😁

@avsm
Copy link
Member

avsm commented Jun 11, 2017

This is perfect, many thanks for all the iterations @maelvalais! You may want to announce it on https://discuss.ocaml.org, where we have a Community category and an announce tag for this purpose.

@avsm avsm merged commit 44402bb into ocaml:master Jun 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants