Skip to content

Conversation

@maelvls
Copy link
Contributor

@maelvls maelvls commented Jan 11, 2021

CHANGES:

0.3 - 2021-01-11

Added

  • Moved to Dune which has become the
    de-facto OCaml build tool. We were previously using Oasis, which was
    using Ocamlbuild under the hood and made the builds slightly slower.

Removed

  • depqbf support has been temporarily removed due to the move to Dune. A
    PR to re-introduce depqbf in Dune would be welcome!

Changed

  • OCaml 4.08.x is now the minimum version required to run this library.

Fixed

  • Fixed the x86_64 build on Windows + Cygwin64 + the mingw64 toolchain.
  • Fixed building ocaml-qbf using opam 2. Opam 2 is sandboxing builds and
    do not accept messing with /tmp.
@camelus
Copy link
Contributor

camelus commented Jan 11, 2021

Commit: 58a022a

@maelvls has posted 11 contributions.

☀️ All lint checks passed 58a022a
  • These packages passed lint tests: qbf.0.3, touist.3.4.0, touist.3.4.1

��️ Installability check (+1)
  • new installable packages (1): qbf.0.3
@maelvls maelvls force-pushed the release-qbf-0.3 branch 2 times, most recently from 268eb02 to e110d60 Compare January 11, 2021 11:15
CHANGES:

All notable changes to this project will be documented in this file.

The format is based on [Keep a Changelog][Keep a Changelog] and this project adheres to [Semantic Versioning][Semantic Versioning].

## [Unreleased]

## [0.3][] - 2021-01-11

### Added

- Moved to [Dune](https://github.com/ocaml/dune) which has become the
  de-facto OCaml build tool. We were previously using Oasis, which was
  using Ocamlbuild under the hood and made the builds slightly slower.

### Removed

- `depqbf` support has been temporarily removed due to the move to Dune. A
  PR to re-introduce `depqbf` in Dune would be welcome!

### Changed

- OCaml 4.08.x is now the minimum version required to run this library.

### Fixed

- Fixed the `x86_64` build on Windows + Cygwin64 + the mingw64 toolchain.
- Fixed building ocaml-qbf using opam 2. Opam 2 is sandboxing builds and
  do not accept messing with /tmp.

## [0.2][] - 2016-06-09

### Added

- ocaml-qbf is now [available on
  opam](https://opam.ocaml.org/packages/qbf)! You can install it with:

  ```sh
  opam install qbf
  ```

### Changed

- Updated picosat to version [960](http://fmv.jku.at/picosat/)
- Updated quantor to version [3.2](http://fmv.jku.at/quantor/)

### Fixed

- The library can now be cross-compiled on Cygwin64 and Cygwin32 using the
  `x86_64-w64-mingw32` toolchain.
- The macOS build was also fixed.

[Keep a Changelog]: https://keepachangelog.com/
[Semantic Versioning]: https://semver.org/
[Unreleased]: c-cube/ocaml-qbf@0.3...HEAD
[0.3]: https://github.com/c-cube/ocaml-qbf/compare/0.2..0.3
[0.2]: https://github.com/c-cube/ocaml-qbf/releases/0.2
@maelvls
Copy link
Contributor Author

maelvls commented Jan 11, 2021

Build failed for an older version of touist that apparently did not specify cppo as a build dependency 😓

Should I send a PR to fix the 3.4.1 opam file of touist? The opam file looked like this for 3.4.1:

("cppo_ocamlbuild" {build & >= "1.6.0"} |
"cppo" {build & >= "0.9.4" & <= "1.5.0"})

Should have I put instead:

 "cppo_ocamlbuild" {build & >= "1.6.0"} 
 "cppo" {build & >= "0.9.4" & <= "1.5.0"}

?

Copy link
Member

@kit-ty-kate kit-ty-kate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should have I put instead:

 "cppo_ocamlbuild" {build & >= "1.6.0"} 
 "cppo" {build & >= "0.9.4" & <= "1.5.0"}

?

Indeed. I just fixed that.

build: [
["dune" "build" "-p" name "-j" jobs]
["dune" "build" "@doc" "-p" name ] {with-doc}
["dune" "runtest" "-p" name "-j" jobs] {with-test & ocaml:version < "4.11"}
Copy link
Member

@kit-ty-kate kit-ty-kate Jan 11, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the reason behind not testing on 4.11? I couldn't find anything in c-cube/ocaml-qbf#7

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure, @c-cube might know more about that 😅

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, let's try. I'll remove my change if there is an issue

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems to be passing without any issues. Could you return the fix upstream?

Copy link
Contributor Author

@maelvls maelvls Jan 20, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will do!
Done in c-cube/ocaml-qbf#11

@kit-ty-kate
Copy link
Member

All green. Thanks!

@kit-ty-kate kit-ty-kate merged commit c933722 into ocaml:master Jan 13, 2021
@maelvls maelvls deleted the release-qbf-0.3 branch January 20, 2021 09:12
maelvls added a commit to c-cube/ocaml-qbf that referenced this pull request Jan 20, 2021
c-cube pushed a commit to c-cube/ocaml-qbf that referenced this pull request Jan 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants