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

[feature] Supress Conan 2 compatibility message #15161

Closed
1 task done
mariuszwicker-cnhind opened this issue Nov 23, 2023 · 2 comments · Fixed by #15174
Closed
1 task done

[feature] Supress Conan 2 compatibility message #15161

mariuszwicker-cnhind opened this issue Nov 23, 2023 · 2 comments · Fixed by #15174
Assignees
Labels
component: ux No changes to core business logic
Milestone

Comments

@mariuszwicker-cnhind
Copy link

mariuszwicker-cnhind commented Nov 23, 2023

What is your suggestion?

When a recipe fails to build Conan 2 will print a migration note regarding v2 compatibility:

*********************************************************
Recipe 'conanfile.py (...)' cannot build its binary
It is possible that this recipe is not Conan 2.0 ready
If the recipe comes from ConanCenter check: https://conan.io/cci-v2.html
If it is your recipe, check if it is updated to 2.0
*********************************************************

This is confusing our developers though because they assume a Conan issue when in fact its a good old compiler failure right above that flashy message. Looking at the code I found now way to disable this - is there any plans to offer an option for that?

Have you read the CONTRIBUTING guide?

  • I've read the CONTRIBUTING guide
@memsharded memsharded self-assigned this Nov 23, 2023
@memsharded memsharded added this to the 2.0.15 milestone Nov 23, 2023
@memsharded
Copy link
Member

Thanks for your feedback @mariuszwicker-cnhind

That sounds right, lets remove that message, most of ConanCenter is now Conan 2.0 compatible, this message is probably more confusing at this stage than helpful.

@memsharded memsharded removed their assignment Nov 23, 2023
@memsharded memsharded added the component: ux No changes to core business logic label Nov 23, 2023
@juansblanco juansblanco self-assigned this Nov 23, 2023
@mariuszwicker-cnhind
Copy link
Author

Awesome. Thanks for the quick turnaround.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: ux No changes to core business logic
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants