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

Select stanza broken (when using -only-packages) #5603

Closed
hhugo opened this issue Apr 21, 2022 · 5 comments
Closed

Select stanza broken (when using -only-packages) #5603

hhugo opened this issue Apr 21, 2022 · 5 comments

Comments

@hhugo
Copy link
Collaborator

hhugo commented Apr 21, 2022

Broken Behavior

I have git repo that contains multiple opam packages.
I have a library A with an optional (select stanza) dependency on a library B that is also part of the repo but marked as vendor-ed.
After installing A with opam pin (B is not installed), I see B in the requires section of the A's lib/A/dune-package.
Trying to install a third package C result in the following error.

# File ".opam/....../lib/A/dune-package", line .., characters ...:
# 87 |   B)
#        ^^^^^^^^^^^^^^^^^^
# Error: Library "B" not found.

If I remove B from the repo and reinstall A. Everything works fine

Specifications

  • Version of dune 2.9
  • Version of ocaml 4.12
  • Version of opam 2.1.0
@hhugo hhugo changed the title Select stanza broken Select stanza broken (when using -only-packages) Apr 21, 2022
@rgrinberg
Copy link
Member

Is this a bug in 3.1.x as well? If so, could you add a reproduction test case to the repo?

@hhugo
Copy link
Collaborator Author

hhugo commented Apr 22, 2022

I confirm that this affects dune.3.1.x as well

@hhugo
Copy link
Collaborator Author

hhugo commented Apr 22, 2022

There was one missing aspect in my initial report. The issue only triggers when the optional library in marked as vendor-ed.

@hhugo
Copy link
Collaborator Author

hhugo commented May 27, 2022

could be related to #4962

@rgrinberg
Copy link
Member

rgrinberg commented Feb 28, 2023

Replaced by #7197

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

No branches or pull requests

2 participants