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] validate() should work for conan install #10723

Closed
1 task done
mpusz opened this issue Mar 5, 2022 · 2 comments
Closed
1 task done

[feature] validate() should work for conan install #10723

mpusz opened this issue Mar 5, 2022 · 2 comments

Comments

@mpusz
Copy link

mpusz commented Mar 5, 2022

validate() was advertised as a new solution that should replace configuration verification that was previously handled in the configure() method. However, it turns up that even though configure() properly triggers an error during conan install, validate() is not checked at all at this stage.

I personally think that such behavior is highly misleading and if possible validate() should be used to validate the configuration for all workflows (both package creation and package consumption).

Otherwise, the documentation should be very clear that configure() is the only possible way to verify a configuration error in the package consumption workflow.

@samuel-emrys
Copy link
Contributor

Duplicate of #10602 - looks like this is slated for 1.47.0

@memsharded
Copy link
Member

Thats right, lets continue on #10602

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

No branches or pull requests

3 participants