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

Add opam update --depexts to replace opam depext -u #4379

Merged
merged 1 commit into from
Dec 12, 2020

Conversation

AltGr
Copy link
Member

@AltGr AltGr commented Oct 6, 2020

This is not strictly an opam command, but it's useful to have here. Closes

Comment on lines 1812 to 1819
let depexts_only =
mk_flag ["depexts"]
"Request the system package manager to update its databases (skipping \
all opam packages, unless $(b,--development) or $(b,--repositories) is \
also specified). This generally requires $(b,sudo) rights." in
Copy link
Member

Choose a reason for hiding this comment

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

Just a side-note that this will want gating on the CLI version for >= 2.1

@rjbou rjbou added this to the 2.1.0~beta3 milestone Oct 7, 2020
@rjbou
Copy link
Collaborator

rjbou commented Oct 7, 2020

Note: OpamSysInteract.update update calls weren't checked nor updated, just imported as is from depext plugin.

@rjbou rjbou added the PR: QUEUED Pending pull request, waiting for other work to be merged or closed label Oct 12, 2020
@rjbou
Copy link
Collaborator

rjbou commented Oct 12, 2020

Queued until CLI versioning PR merged, otherwise, lgtm! thanks

This is not strictly an opam command, but it's useful to have here.
Closes ocaml#4355
@rjbou rjbou removed the PR: QUEUED Pending pull request, waiting for other work to be merged or closed label Dec 11, 2020
@rjbou rjbou merged commit 0f83175 into ocaml:master Dec 12, 2020
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

Successfully merging this pull request may close these issues.

[opam 2.1~beta2] opam 2.1 does not seem to have an equivalant for opam depext -u
3 participants