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

Add a post_package_info hook to check cpp_info.resdirs #482

Open
SpaceIm opened this issue Feb 1, 2023 · 0 comments
Open

Add a post_package_info hook to check cpp_info.resdirs #482

SpaceIm opened this issue Feb 1, 2023 · 0 comments

Comments

@SpaceIm
Copy link
Contributor

SpaceIm commented Feb 1, 2023

see conan-io/conan-center-index#15581 & conan-io/conan-center-index#15599 (comment)

When a recipe is migrated to layout(), cpp_info.resdirs is empty by default instead of ["res"]. For most recipe it's fine and a good default, but few recipes now need to set this attribute explicitly.

I propose this hook:

  • Report an error if cpp_info.resdirs miss existing res or share folder in root package folder. Since conan v1 recipes always populate resdirs, this hook shouldn't break them.
  • bonus: if a recipe uses layout() (conan v2 recipes only), report an error if cpp_info.resdirs list non-existing folders.
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

1 participant