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

opam install --deps-only skips the installation reasons #4578

Closed
avsm opened this issue Mar 4, 2021 · 1 comment · Fixed by #4975
Closed

opam install --deps-only skips the installation reasons #4578

avsm opened this issue Mar 4, 2021 · 1 comment · Fixed by #4975
Assignees
Milestone

Comments

@avsm
Copy link
Member

avsm commented Mar 4, 2021

With opam 2.1beta5, opam install . shows the reason a package is included.

$ opam install .            
Package base-images does not exist, create as a NEW package? [Y/n] y
base-images is now pinned to git+file:///Users/avsm/src/git/ocurrent/docker-base-images#nnpchecker (version ~dev)
The following actions will be performed:
  ∗ install seq                     base         [required by re, fmt, tyxml]
  ∗ install conf-m4                 1            [required by ocamlfind]
  ∗ install conf-gmp                3            [required by zarith]
  ∗ install mirage-no-xen           1            [required by mirage-crypto-pk, nocrypto]
  ∗ install conf-capnproto          1            [required by capnp-rpc-net]
  ∗ install camlp4                  4.10+system  [required by extunix]
  ∗ install ocamlbuild              0.14.0       [required by extunix, astring, bos]
  ∗ install conf-pkg-config         2            [required by mirage-crypto]
  ∗ install dune                    2.8.2        [required by base-images]
  ∗ install mirage-no-solo5         1            [required by mirage-crypto-pk, nocrypto]
  ∗ install cmdliner                1.0.4        [required by current_github, capnp-rpc-unix, dockerfile-opam]
  ∗ install jbuilder                1.0+beta20.2 [required by irmin-watcher]
  ∗ install conf-which              1            [required by conf-graphviz]
  ∗ install ocamlfind               1.8.1        [required by extunix, astring, bos]
  ∗ install conf-gmp-powm-sec       3            [required by mirage-crypto-pk]

But opam install . --deps-only does not

The following actions will be performed:
  ∗ install seq                     base
  ∗ install conf-m4                 1
  ∗ install conf-gmp                3
  ∗ install mirage-no-xen           1
  ∗ install conf-capnproto          1
  ∗ install camlp4                  4.10+system
  ∗ install ocamlbuild              0.14.0
  ∗ install conf-pkg-config         2
  ∗ install dune                    2.8.2
  ∗ install mirage-no-solo5         1
  ∗ install cmdliner                1.0.4
  ∗ install jbuilder                1.0+beta20.2
  ∗ install conf-which              1
  ∗ install ocamlfind               1.8.1
  ∗ install conf-gmp-powm-sec       3
  ∗ install uchar                   0.0.2
  ∗ install conf-sqlite3            1
  ∗ install stdlib-shims            0.3.0
  ∗ install stdint                  0.7.0
  ∗ install sexplib0                v0.14.0

It would be handy to always see the installation reason, irrespective of if the local packages are being installed or not.
(Severity of this is minor, not a blocker for anything)

@AltGr
Copy link
Member

AltGr commented Mar 4, 2021

My guess is that the reasons are computed from the requested packages ; in the case of deps-only these were removed from the list before the printing, and without a root to start from, the whole "reasons" computing is skipped.

@dra27 dra27 added this to the 2.1.1 milestone Mar 12, 2021
@dra27 dra27 modified the milestones: 2.1.1, 2.1.2 Sep 17, 2021
@dra27 dra27 modified the milestones: 2.1.2, 2.1.3 Nov 19, 2021
@kit-ty-kate kit-ty-kate linked a pull request Jan 7, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants