search

conan-io / hooks

Official Conan client hooks
MIT License
32 stars 46 forks source link

[conan-center] Update KB-H077 to warning level #478

Closed uilianries closed 1 year ago

uilianries commented 1 year ago

Some PRs are failing due the new rule and it may affect master branch as well, forcing users to fix things which are not related to their changes.

SpaceIm commented 1 year ago

Is there a process running somewhere which could list all recipes failing to pass warnings hooks? Otherwise these recipes will never be fixed. There is an easy fix by the way when you hit this hook: fix_apple_shared_install_name(self) at the end of package(), it's less intrusive than modifications requested by conan v2 linter.

SpaceIm commented 1 year ago

Well until https://github.com/conan-io/conan/issues/12959 is fixed, it's indeed a bad idea to report an error.

uilianries commented 1 year ago

@SpaceIm the idea is fixing https://github.com/conan-io/conan-center-index/issues/2232 and showing warning level.