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

Invariant syntax in opam config report doesn't match the expected syntax by --invariant #5491

Open
kit-ty-kate opened this issue Mar 24, 2023 · 2 comments · May be fixed by #5619
Open

Invariant syntax in opam config report doesn't match the expected syntax by --invariant #5491

kit-ty-kate opened this issue Mar 24, 2023 · 2 comments · May be fixed by #5619
Labels

Comments

@kit-ty-kate
Copy link
Member

opam config report since #5480 returns the invariant for the current switch:

# invariant            ocaml-base-compiler >= 5.0 & ocaml-base-compiler < 5.1

However this syntax isn't accepted by other parts of opam dealing with invariants (e.g. --invariant)
It should be something like:

# invariant            "ocaml-base-compiler" >= "5.0" & "ocaml-base-compiler" < "5.1"

instead

@kit-ty-kate kit-ty-kate added this to To do in Opam 2.2.0 via automation Mar 24, 2023
@kit-ty-kate kit-ty-kate moved this from To do to After ~alpha; before release in Opam 2.2.0 Mar 24, 2023
@rjbou
Copy link
Collaborator

rjbou commented Jul 6, 2023

It is displayed in a human readable way, it is not purpose of opam config report to be given as input of other commands.

@kit-ty-kate
Copy link
Member Author

I disagree. I think having something consistent is more human readable than having 3 different incompatible syntaxes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants