Skip to content

Fix opam switch list-available when given several arguments#6318

Merged
rjbou merged 3 commits intoocaml:masterfrom
kit-ty-kate:fix-list-available-multiple-args
Dec 3, 2024
Merged

Fix opam switch list-available when given several arguments#6318
rjbou merged 3 commits intoocaml:masterfrom
kit-ty-kate:fix-list-available-multiple-args

Conversation

@kit-ty-kate
Copy link
Copy Markdown
Member

Required for #6186

Giving several arguments would never have given any results before. Furthermore it now matches the behaviour of opam list

@rjbou
Copy link
Copy Markdown
Collaborator

rjbou commented Dec 2, 2024

Agree on the idea. Some history, it was introduced from the beginning as an ands (see #2650).
We need to advertise it as it is a change of behaviour. Maybe have it behind a CLI version guard ?

@kit-ty-kate
Copy link
Copy Markdown
Member Author

Maybe have it behind a CLI version guard ?

i don't think we need to do that for something that had no useful outcome

@rjbou rjbou added the KIND: BUG label Dec 2, 2024
@rjbou
Copy link
Copy Markdown
Collaborator

rjbou commented Dec 2, 2024

As it is a bug fix, lgtm

@rjbou rjbou merged commit 5fc2c29 into ocaml:master Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants