search

janestreet / bonsai

A library for building dynamic webapps, using Js_of_ocaml
MIT License
367 stars 39 forks source link

Bonsai_web missing from opam, bonsai has incorrect dependencies list #4

Closed TyOverby closed 4 years ago

TyOverby commented 4 years ago

I would expect “bonsai” opam package to have only the dependencies listed in src/dune, and for the dune project inside of the bonsai_web subdirectory to be published as well.

xclerc commented 4 years ago

My understanding is that bonsai_web is indeed part of the bonsai package because its public name is bonsai.web.

If we want bonsai_web to be independent, we should set its public name to e.g. bonsai_web.

TyOverby commented 4 years ago

If we want bonsai_web to be independent, we should set its public name to e.g. bonsai_web.

Can we do this and keep them both in the same github repository?