| Summary: | dev-util/pkgcheck: report multilib-minimal not inherited last | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Alessandro Barbieri <lssndrbarbieri> |
| Component: | Current packages | Assignee: | Michał Górny <mgorny> |
| Status: | RESOLVED WONTFIX | ||
| Severity: | enhancement | CC: | jstein, kuraga333 |
| Priority: | Normal | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| See Also: | https://bugs.gentoo.org/show_bug.cgi?id=892255 | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
|
Description
Alessandro Barbieri
2020-02-26 22:20:35 UTC
I don't think that's a good idea. Firstly, this suggestion is kinda silly. Surely most developers know how phase function exports work, and they have the right to decide on the correct order. Making a lot of warnings and asking people to meaninglessly change inherit order is not a good idea. Secondly, a dozen other eclasses could make similar claims. The moment we start reporting this for two eclasses, we're open to impossible case where developer is asked to have two eclasses last, and can't get rid of the warning. Thirdly, there's another dozen eclasses that do not export phase functions and are entirely fine to be inherited after multilib-minimal. So no, I don't think a warning makes sense here. Closing as won't fix since I agree this isn't a great target to check for. Also, ideas for pkgcheck should be filed upstream on github unless you want to assign them as ideas to the QA team for discussion otherwise I generally won't see them. |
