Lucas Amaral <[email protected]> writes: > Alex Bennée <[email protected]> writes: > >> I don't see the r-b on the patch. > > It's on the v2 thread for the same patch (then 1/3, unchanged content > in v5 patch 1/4): > > > https://patchew.org/QEMU/[email protected]/[email protected]/ > > I'm not familiar with the convention here -- should the Reviewed-by > be re-stated in the cover letter of each respin, or carried only in > the patch trailer of the next version? Will do whichever is standard > in v6.
Generally you apply the tags to your commits so they are carried into the next version: https://qemu.readthedocs.io/en/master/devel/submitting-a-patch.html#proper-use-of-reviewed-by-tags-can-aid-review There is tooling that can do this automatically for you, for example b4 will gather review tags and apply them when it pulls a series. > > Thanks, > Lucas -- Alex Bennée Virtualisation Tech Lead @ Linaro
