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

Package libbinaryen.106.0.0 #21330

Merged
merged 2 commits into from
May 5, 2022

Conversation

grainbot
Copy link
Contributor

@grainbot grainbot commented May 5, 2022

libbinaryen.106.0.0

Libbinaryen packaged for OCaml



106.0.0 (2022-05-04)

⚠ BREAKING CHANGES

Features


🐫 Pull-request generated by opam-publish v2.0.3

@phated
Copy link
Contributor

phated commented May 5, 2022

lol, it looks like our #55 syntax is showing up as a link to an issue in this repository. Sorry about that.

@mseri
Copy link
Member

mseri commented May 5, 2022

Fails on arm32 and x86_32

bug-reports: "https://github.com/grain-lang/libbinaryen/issues"
depends: [
"conf-cmake" {build}
"dune" {>= "2.9.1" & < "3.0.0"}
Copy link
Member

Choose a reason for hiding this comment

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

Why such a strict bound is needed?

Copy link
Contributor

Choose a reason for hiding this comment

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

Dune 3 has an implicit dependency on opam on the system. In places you don't have opam, it fails. I want to ensure our package works in different contexts without needing to know that trivia.

Dune merged some patches this week that are yet-to-be-released. Once it works without opam, I'll open up the range.

Co-authored-by: Marcello Seri <mseri@users.noreply.github.com>
@mseri mseri merged commit 2de2614 into ocaml:master May 5, 2022
@mseri
Copy link
Member

mseri commented May 5, 2022

Thanks

@phated phated deleted the opam-publish-libbinaryen.106.0.0 branch May 5, 2022 21:53
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