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

Update remaining references to base packages #4623

Merged
merged 5 commits into from
Apr 6, 2021
Merged

Conversation

dra27
Copy link
Member

@dra27 dra27 commented Mar 31, 2021

This fixes the remaining issues in #4572 - see commits for notes.

The distinction for base packages no longer makes sense - if the user
choose to mark an invariant package as automatically installed then so
be it. opam remove -a will behave correctly (i.e. it will fail unless
it's allowed to update the invariant).
@dra27 dra27 added this to the 2.1.0~rc milestone Mar 31, 2021
@dra27 dra27 requested a review from AltGr March 31, 2021 14:54
@dra27 dra27 added this to PR in Progress in Opam 2.1.x via automation Mar 31, 2021
Copy link
Collaborator

@rjbou rjbou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Opam 2.1.x automation moved this from PR in Progress to PR Finalised Apr 1, 2021
@dra27 dra27 merged commit c6be7c8 into ocaml:master Apr 6, 2021
Opam 2.1.x automation moved this from PR Finalised to Done Apr 6, 2021
@dra27 dra27 deleted the base-messages branch April 6, 2021 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Opam 2.1.x
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants