On 10/23/23 16:02, Sean Christopherson wrote:
+Mateusz
On Mon, Oct 23, 2023, Przemek Kitszel wrote:
Additional tags between Co-developed-by and corresponding Signed-off-by
could include Reviewed-by tags collected by Submitter, which is also
a Co-developer, but should sign-off at the very end of tags provided by
the Submitter.
...
Mateusz Polchlopek <[email protected]> has reported this to me.
Heh, there's a tag for that...
Reported-by: Mateusz Polchlopek <[email protected]>
And it's usually a good idea to Cc the reporter in case there are questions they
can help answer.
Heh ;) then I would get a checkpatch warning for not providing Link: to
the report, somehow I wanted to avoid those for checkpatch contrib :)
@@ -509,16 +509,18 @@ Example of a patch submitted by the From: author::
Signed-off-by: Second Co-Author <[email protected]>
Signed-off-by: From Author <[email protected]>
-Example of a patch submitted by a Co-developed-by: author::
+Example of a patch submitted by a Co-developed-by: author, who also collected
+a Reviewed-by: tag posted for earlier version::
From: From Author <[email protected]>
<changelog>
Co-developed-by: Random Co-Author <[email protected]>
Signed-off-by: Random Co-Author <[email protected]>
Signed-off-by: From Author <[email protected]>
Co-developed-by: Submitting Co-Author <[email protected]>
+ Reviewed-by: Some Reviewer <[email protected]>
Signed-off-by: Submitting Co-Author <[email protected]>
This is silly. Allowing tags in-between Co-developed-by with Signed-off-by
unnecessarily complicates things, e.g. people already miss/forget the rule about
tightly coupling Co-developed-by with Signed-off-by.
Meh, I see that as a pure process simplification with proposed rule
being almost the same as the current one, thus as easy to remember or
forget.
And if we're being super pedantic about chronological history, arguably the
Reviewed-by should come before the Co-developed-by as adding the Reviewed-by is
a (trivial) modification to the patch that was done by the submitter.
Tagging patches is not considered co-development by most people.
