-
Notifications
You must be signed in to change notification settings - Fork 993
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
[feature] Improve version conflict ERROR message #13420
Comments
Hi! Thanks for the report. After looking at it for a bit, I've come to the conclusion that this would not be trivial to solve. That "None" represents in this case the virtual node for the graph when running Maybe we can look into this at a later date when more pressing things are past us and we can focus on improving the UX with issues such as this one :) |
Did you solve this problem? |
I would like to support this issue! In the output of the first
(I could locally reproduce that message with conan version 2.0.4)
At least that would be much more helpful. |
For complicated cases, the solution would not be printing anything way smarter to terminal, but instead producing a better |
Absolutely true, @memsharded , that would be the best solution. Especially because such a version conflict could be caused by several upstreams simultaneously. E. g. two upstreams requiring version 1.0 of a package, three others requiring 1.1 and another four requiring version 2.0. Would be pretty hard to squeeze that kind of information into a concise error message. - Whereas, maybe not impossible (with a nice ascii list/table). With the html-graph feature, the conflict error message could/should then point the user to it. E. g.: "There are version conflicts in the upstream graph. Use 'conan graph ...' to generate an html overview which will show the conflicting dependencies." But what the error message should not do is give the user "wrong" or at least confusing information, like it currently does. Maybe a quick fix would be not to print (in the tutorial example) |
Yes, totally agree
Sure, lets try to improve the message. In any case, I think improving the html graph with colors shouldn't be too complicated, lets try to do it at once. |
Closed by #13946 which improves the error message a bit, but more importantly makes the html output able to highlight conflicting nodes for better debugability |
What is your suggestion?
In
libvips
recipe (not yet in conancenter, see conan-io/conan-center-index#16339) there is currently a version conflict while using conan v2 client becauseglib
still depends onpcre2/10.40
whilelibselinux
(a dependency ofglib
when OS is Linux) has been updated to depend onpcre2/10.42
.Error message is:
This
None->pcre2/10.40
message is not helpful, would it be possible to list the correct recipes having conflicting dependencies versions? Here it should be something likeERROR: Version conflict: libselinux/3.3->pcre2/10.42, glib/2.76.0->pcre2/10.40.
.Have you read the CONTRIBUTING guide?
The text was updated successfully, but these errors were encountered: