Improve graph conflict message when Conan can't show one of the conflicting recipes #13946
Conversation
|
@RubenRBS can we output graph to the stdout (e.g. json format) instead of asking to run |
conans/client/graph/graph_error.py
Outdated
| return f"Version conflict: " \ | ||
| f"Conflict between {self.require.ref} and {self.prev_require.ref} in the graph." \ | ||
| f"{conflicting_node_msg}" \ | ||
| f"\nRun conan graph info ... --format=html to inspect the graph errors." |
There was a problem hiding this comment.
I would print this --format=html (with redirect > graph.html) in all cases, even the "simple" part of the if
There was a problem hiding this comment.
can it print actual arguments here instead of ...? so it could be copy-pasted easily.
|
|
||
| c.run("install --requires=engine/1.0 --requires=ai/1.0", assert_error=True) | ||
| assert "Conflict between math/1.0.1 and math/1.0 in the graph" | ||
| assert "Conflict originates from ai/1.0" |
There was a problem hiding this comment.
Check that --format=html is also suggested
|
to execute this suggestion on CI server (e.g. ConanCenterIndex) I would need:
|
The idea is that not all debugging needs to happen in the CI server. Especially something that should have been detected by the developer in their machine while testing locally in the first place. It is not that a version conflict is something that only happens and can be debugged in the CI server. So the recommended approach is not start modifying the CI, but execute a |
|
The graph now looks like this |
|
Remove red from UI, change lib link to red |
Changelog: Fix: Improve graph conflict message when Conan can't show one of the conflicting recipes.
Docs: Omit
The error now looks like:
The graph output with conflict errors now looks like this:
Pinging @SSE4