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

[QoL] Allow conan remove -t * #4478

Closed
Minimonium opened this issue Feb 6, 2019 · 6 comments · Fixed by #5176
Closed

[QoL] Allow conan remove -t * #4478

Minimonium opened this issue Feb 6, 2019 · 6 comments · Fixed by #5176

Comments

@Minimonium
Copy link
Contributor

In the CI with the cache, it's needed to reinstall system requirements when the machine has been changed. Currently, it's only supported with a full reference.

@Minimonium
Copy link
Contributor Author

Minimonium commented Feb 6, 2019

Consider #4370

@lasote
Copy link
Contributor

lasote commented Feb 11, 2019

I agree. But I would say the command should be conan remove "*" -t

(An internal note for the dev team or any other collaborator that develop this feature:
Calling directly the "cache" from the command.py is not correct and it should have a dedicated function in the conan_api. self._cache.remove_package_system_reqs(ref))

@uilianries
Copy link
Member

Should it remove global system requirements, or only from all packages?

@Minimonium
Copy link
Contributor Author

What tools do we have to remove only global system requirements from conan cli?
I'm not sure it makes sense to remove global system requirements too. Or better to say - applying to this use case it makes sense, but I feel like the usage would be a bit confusing and a little bit unexpected.

@uilianries
Copy link
Member

So maybe we could create a new option for global requirements

@lasote
Copy link
Contributor

lasote commented Feb 13, 2019

Anyway, that would be a different feature. Let's keep things clear.

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

Successfully merging a pull request may close this issue.

5 participants