Make Acked/Reviewd/Tested-by tags visible

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Wed Jun 11 17:07:48 EST 2014


Dear Jeremy Kerr,

On Wed, 11 Jun 2014 10:00:54 +1000, Jeremy Kerr wrote:

> They should be parsed when the original patch is received, and when any
> follow-ups are appended to the patch. However:
> 
> > I noticed that (in the Buildroot patchwork [0]) some tags were not
> > accounted for:
> >     http://patchwork.ozlabs.org/patch/354654/  (missing reviewed-by)
> >     http://patchwork.ozlabs.org/patch/354590/  (missing acked-by)
> >     [...]
> 
> .. it looks like I may be missing a case there. I'll take a look.

Another thing that is missing IMO is a way of sorting the patches by
the number of A/R/T tags. Probably not needed to be able to sort them
for each individual value, but being able to sort them by the sum of
A+R+T would be useful so that maintainers can spot immediately which
patches have received Acked-by, Tested-by or Reviewed-by tags.

Thanks for this very useful new feature!

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com


More information about the Patchwork mailing list