-
Notifications
You must be signed in to change notification settings - Fork 46
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
- forbid CONAN_PKG and CONAN_LIBS #355
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: SSE4 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have concerns with this in cases where upstream does not provide official targets. This will force CCI to invent targets and make downstream depend on them. Having it somehow namespaced to Conan makes it clearer
@Croydon, this is aligned with Conan milestones and it is strategic. You can find a longer rationale here: conan-io/conan-center-index#7153 TL;DR.- Conan v2.0 will rely on Of course, like everything, if you have good reasons to propose something different, it is time to suggest and provide feedback in the conan repository or the tribe. Thanks for your understanding. |
(Executing a dry run for this branch. I'll post output as soon as it is available) Failing ones:
|
closes: #354
we will need to run hooks against current conan-center to understand the scale of the destruction