From patchwork Thu Jul 13 10:56:53 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Guinevere Larsen X-Patchwork-Id: 72637 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id A8BBD3858C74 for ; Thu, 13 Jul 2023 11:09:46 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org A8BBD3858C74 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1689246586; bh=H7G4qnSar7NRSKqd9TPUoVeKxMzSOhxFeAsEJHM8fJI=; h=To:Cc:Subject:Date:In-Reply-To:References:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=TqA/AXQv+wdkagLviMofbKJnF0NaH881cQu7Ba5J2xgT2ideB8x4FkDpBtatFJWlr id5k2YJ8oRVwTTeQgQtbJU44Z9fBNu1LlqUIqYCZxHMTXSUwUH4AEMmLD2n2VVTpOk t+ebGx14rIUpSjrq+eG63JFZ32EcZLzW9mAhPS3E= X-Original-To: gdb-patches@sourceware.org Delivered-To: gdb-patches@sourceware.org Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id C7F9E3857033 for ; Thu, 13 Jul 2023 11:09:21 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org C7F9E3857033 Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-671-he412yFjNUyEVvdb8E-f8Q-1; Thu, 13 Jul 2023 07:09:17 -0400 X-MC-Unique: he412yFjNUyEVvdb8E-f8Q-1 Received: from smtp.corp.redhat.com (int-mx03.intmail.prod.int.rdu2.redhat.com [10.11.54.3]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 15A8D93B760; Thu, 13 Jul 2023 11:09:17 +0000 (UTC) Received: from fedora.redhat.com (unknown [10.45.226.146]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 8096F111E3EA; Thu, 13 Jul 2023 11:09:15 +0000 (UTC) To: gdb-patches@sourceware.org Cc: pedro@palves.net, aburgess@redhat.com, kevinb@redhat.com, brobecker@adacore.com, simon.marchi@polymtl.ca, tom@tromey.com, tdevries@suse.de, ulrich.weigand@de.ibm.com, eliz@gnu.org, Bruno Larsen Subject: [PATCH v4 1/1] [gdb]: add git trailer information on gdb/MAINTAINERS Date: Thu, 13 Jul 2023 12:56:53 +0200 Message-ID: <20230713105651.2281574-4-blarsen@redhat.com> In-Reply-To: <20230713105651.2281574-2-blarsen@redhat.com> References: <20230713105651.2281574-2-blarsen@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 3.1 on 10.11.54.3 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com X-Spam-Status: No, score=-11.5 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, RCVD_IN_DNSWL_NONE, RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: gdb-patches@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gdb-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: Bruno Larsen via Gdb-patches From: Guinevere Larsen Reply-To: Bruno Larsen Errors-To: gdb-patches-bounces+patchwork=sourceware.org@sourceware.org Sender: "Gdb-patches" The project has been using Tested-By (tb), Reviewed-By (rb) and Approved-By (ab) for some time, but there has been no information to be found in the actual repository. This commit changes that by adding information about all git trailers to the MAINTAINERS file, so that it can be easily double-checked. The upstream discussion also brought up the use of Acked-by, which is better defined in this commit. Finally, for completeness sake, the trailers Co-Authored-By and Bug were added, even though they have been in use for some time already Reviewed-by: Kevin Buettner --- gdb/MAINTAINERS | 72 +++++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 64 insertions(+), 8 deletions(-) diff --git a/gdb/MAINTAINERS b/gdb/MAINTAINERS index 7fa608fd82c..609f740f21d 100644 --- a/gdb/MAINTAINERS +++ b/gdb/MAINTAINERS @@ -43,14 +43,9 @@ patch without review from another maintainer. This especially includes patches which change internal interfaces (e.g. global functions, data structures) or external interfaces (e.g. user, remote, MI, et cetera). -The term "review" is used in this file to describe several kinds of feedback -from a maintainer: approval, rejection, and requests for changes or -clarification with the intention of approving a revised version. Review is -a privilege and/or responsibility of various positions among the GDB -Maintainers. Of course, anyone - whether they hold a position but not the -relevant one for a particular patch, or are just following along on the -mailing lists for fun, or anything in between - may suggest changes or -ask questions about a patch! +The word "contributor" is used in this document to refer to any GDB +developer listed above as well as folks who may have suggested some +patches but aren't part of one of those categories for any reason. There's also a couple of other people who play special roles in the GDB community, separately from the patch process: @@ -78,6 +73,67 @@ consensus among the global maintainers and any other involved parties. In cases where consensus can not be reached, the global maintainers may ask the official FSF-appointed GDB maintainers for a final decision. +The term "review" is used in this file to describe several kinds of +feedback from a maintainer: approval, rejection, and requests for changes +or clarification with the intention of approving a revised version. +Approval is a privilege and/or responsibility of various positions among +the GDB Maintainers. Of course, anyone - whether they hold a position, but +not the relevant one for a particular patch, or are just following along on +the mailing lists for fun, or anything in between - may suggest changes, ask +questions about a patch or say if they believe a patch is fit for upstreaming! + +To ensure that patches are only pushed when approved, and to properly credit +the contributors who take the time to improve this project, the following +trailers are used to identify who contributed and how. All patches pushed +upstream should have at least one Approved-By patches (with the exception of +obvious patches, see below). The trailers (or tags) currently in use are: + + - Acked-By: + + Used when a contributor has taken a quick glance at a patch and agrees + with the direction outlined in the commit message, but hasn't evaluated + the code for correctness or regressions. + Usage: "Acked-By: Your Name " + + - Tested-by: + + Used when a contributor has tested the patch and finds that it + fixes the claimed problem. It may also be used to indicate that + the contributor has performed regression testing. By itself, this + tag says nothing about the quality of the fix implemented by the + patch, nor the amount of testing that was actually performed. + Usage: "Tested-By: Your Name " + + - Reviewed-by: + + Used when a contributor has looked at the code and agrees with + the changes, but either doesn't have the authority or doesn't + feel comfortable approving the patch. + Usage: "Reviewed-By: Your Name " + + - Approved-by: + + Used by responsible maintainers or global maintainers when a patch is + ready to be upstreamed. Some patches may touch multiple areas and + require multiple approvals before landing (such as a maintainer only + approving documentation), it is up to the maintainer giving the approval + tag to make it clear when that a tag is not sufficient. Responsible, + Global and Official FSF-appointed maintainers may approve their own + patches, but it is recommended that they seek external approval before + doing so. + Usage: "Approved-By: Your Name " + + - Co-Authored-By: + + Used when the commit includes meaningful conrtibutions from multiple people. + Usage: "Co-Authored-By: Contributor's Name " + + - Bug: + + This trailer is added with a link to the GDB bug tracker bug for + added context on relevant commits. + Usage: "Bug: " + The Obvious Fix Rule --------------------