Re: Reported-by (Re: Fix a typo in the development processdocument.)

[Jonathan Corbet]

On Fri, 9 Jan 2009 15:15:35 -0800
Randy Dunlap <randy.dunlap@xxxxxxxxxx> wrote:

> BTW, Jon, if Reported-by: is supposed to have a status that is
> (nearly) as well accepted as most of the other tags, then it should
> be documented in SubmittingPatches along with the other tags and not
> only in development-process/5.Posting, no?

Yes, it probably should.  How's the following?  I'll drop it into my
docs tree if nobody gripes.

jon

--

Document Reported-by: in SubmittingPatches

Reported-by: Randy Dunlap <randy.dunlap@xxxxxxxxxx>
Signed-off-by: Jonathan Corbet <corbet@xxxxxxx>

diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
index f309d3c..6f29e29 100644
--- a/Documentation/SubmittingPatches
+++ b/Documentation/SubmittingPatches
@@ -405,7 +405,14 @@ person it names.  This tag documents that potentially interested parties
 have been included in the discussion
 
 
-14) Using Tested-by: and Reviewed-by:
+14) Using Reported-by:, Tested-by: and Reviewed-by:
+
+If this patch fixes a problem reported by somebody else, consider adding a
+Reported-by: tag to credit the reporter for their contribution.  Please
+note that this tag should not be added without the reporter's permission,
+especially if the problem was not reported in a public forum.  That said,
+if we diligently credit our bug reporters, they will, hopefully, be
+inspired to help us again in the future.
 
 A Tested-by: tag indicates that the patch has been successfully tested (in
 some environment) by the person named.  This tag informs maintainers that
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/