yallop / ocaml-ctypes

Library for binding to C libraries using pure OCaml
MIT License
369 stars 95 forks source link

Don't build with esy cause of circular dependency #671

Closed oteku closed 3 years ago

oteku commented 3 years ago

I can't build using esy because circular dependency found Note that lib:

ARCH : 1,2 GHz Intel Core i7 quatre cœurs OS : 11.2.3 (20D91) ESY : 0.6.10

package.json:

{ "name":"dummy",
    "dependencies": {
      "ocaml": "4.12",
      "@opam/dune": ">=2.8.5",
      "@opam/ctypes-foreign": "*"},
    "esy": {
        "build": "dune build -p #{self.name}",
        "release": { "releasedBinaries": ["conduit"] }
      }
}

shell infos

~/Projects/dummytest via 🐫 esy install
info install 0.6.10 (using package.json)
info checking https://github.com/ocaml/opam-repository for updates...
info checking https://github.com/esy-ocaml/esy-opam-override for updates...
info resolving esy packages: done
info solving esy constraints: done
info resolving npm packages: done
info fetching: done
info installing: done

~/Projects/dummytest via 🐫 took 2s esy build
info esy build 0.6.10 (using package.json)
error: found circular dependency on: @opam/ctypes-foreign@opam:0.18.0
  processing @opam/ctypes@opam:0.18.0
  processing @opam/ctypes-foreign@opam:0.18.0
  processing dummy@link-dev:./package.json
  creating task for dummy@link-dev:./package.json
esy: exiting due to errors above
avsm commented 3 years ago

Thanks for the bug report, but I'm not sure what we should with it on this repo -- it looks like an esy bug to me. Have you contacted their developers?

oteku commented 3 years ago

Thanks. So, i report it also on esy https://github.com/esy/esy/issues/1309

yallop commented 3 years ago

It seems like the problem in esy is fixed (https://github.com/esy/esy/issues/1306#issuecomment-861768374). @oteku, can we close this issue?