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

Cyclic dependencies between alcotest/tyxml/odoc #16663

Closed
Khady opened this issue Jun 18, 2020 · 2 comments
Closed

Cyclic dependencies between alcotest/tyxml/odoc #16663

Khady opened this issue Jun 18, 2020 · 2 comments

Comments

@Khady
Copy link
Contributor

Khady commented Jun 18, 2020

recently getting this in the CI of a project. I suspect that there is a problem since a recent release (might be odoc 1.5.1?)

Run opam depext notabot --yes --with-doc --with-test
# Detecting depexts using vars: arch=x86_64, os=linux, os-distribution=ubuntu, os-family=debian
[ERROR] No solution for notabot: The actions to process have cyclic dependencies:
          - ∗ alcotest.1.1.0 → ∗ tyxml.4.4.0 → ∗ odoc.1.5.1 → ∗ alcotest.1.1.0

The list of deps in my project:

 "ocaml" {>= "4.03.0"}
 "dune" {>= "2.5.0"}
 "atdgen"
 "base"
 "cmdliner"
 "devkit"
 "hex"
 "lwt"
 "lwt_ppx"
 "nocrypto"
 "stdio"
 "uri"
 "ocamlformat" {dev & = "0.13.0"}
@kit-ty-kate
Copy link
Member

I believe this is fixed in opam 2.1. I reported the same kind of issue here: ocaml-opam/opam-depext#121

The workaround here would be to call opam depext without argument, then with just --with-test, then with just --with-doc, all separately. In your case however the --with-test and --with-doc arguments are not necessary since you don't have any doc or test dependencies

Khady added a commit to ahrefs/monorobot that referenced this issue Jun 22, 2020
Khady added a commit to ahrefs/monorobot that referenced this issue Jun 22, 2020
Khady added a commit to Khady/modern-ocaml that referenced this issue Jun 22, 2020
@Khady
Copy link
Contributor Author

Khady commented Jun 22, 2020

thanks for the tip

@Khady Khady closed this as completed Jun 22, 2020
yasunariw pushed a commit to ahrefs/monorobot that referenced this issue Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants