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

conan remove --outdated works with a pattern too #1987

Closed
Croydon opened this issue Jan 17, 2021 · 1 comment · Fixed by #1988
Closed

conan remove --outdated works with a pattern too #1987

Croydon opened this issue Jan 17, 2021 · 1 comment · Fixed by #1988

Comments

@Croydon
Copy link
Contributor

Croydon commented Jan 17, 2021

According to the documentation conan remove --outdated works only with references, which implies that it does not work with patterns

https://docs.conan.io/en/latest/reference/commands/misc/remove.html

  -o, --outdated        Remove only outdated from recipe packages. This flag
                        can only be used with a reference

Coming from bincrafters/community#1320, it seems to work perfectly fine with a pattern.

See e.g. https://github.com/bincrafters/bincrafters-conventions/blob/0ec9c9f5052a00bbcdb962d8dae0117b131ac323/.github/workflows/remove-obsolete-packages.yml#L29

@solvingj
Copy link
Contributor

It was probably trying to distinguish the usage from the common string path_or_reference which is featured in many commands. But yes, explicitly adding that it accepts a pattern seems reasonable.

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 a pull request may close this issue.

2 participants