On 2016-02-09 19:59, Robin H. Johnson wrote:
On Tue, Feb 09, 2016 at 07:03:28PM -0500, Tim Harder wrote:Just to note, devs that make these mistakes should already get emails from the CI test setup [1]. If those emails need to be more specific about the severity I'm sure that could be changed so you don't need to act like a CI bot if you don't want to. ;) [1]: https://archives.gentoo.org/gentoo-automated-testing/message/79a7b4fe2afaa71de9ef1189464f8324Well then raise the severity, because it's still blocking rsync as nobody fixed it.I have reverted it now, so that rsync continues to propagate (it would be nice if repoman manifest-check could be non-fatal about missing DIST entries, but strict about everything else).
I imagine in the end it would be better to put some of the faster checks into the git hook pipeline itself. For example, reject commits at pushtime that are missing DIST entries.
Since I'm not familiar with the infra setup all I can do is provide support if it's feasible and someone wants to implement this using pkgcore/pkgcheck similar to what the CI stuff already does. Thanks, Tim
signature.asc
Description: PGP signature
