From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from eggs.gnu.org (eggs.gnu.org [IPv6:2001:470:142:3::10]) by sourceware.org (Postfix) with ESMTPS id 41B053858C83; Tue, 16 May 2023 16:04:55 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 41B053858C83 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=gnu.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=gnu.org Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pyxAR-0006EZ-Mk; Tue, 16 May 2023 12:04:51 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date: mime-version; bh=UmZQV1FsrAeeN+2D8/7WPujCn4rrA0ayA+NmshrxCLY=; b=DHsP1EUSPIYD ax4M+rsWy3VSColCO8ZmuzU+vsKS2ED2qvuIVWeCRbKtmAw5xSSSCwKJUBzMQIh/8eOWaIPMSkOkf dmmB6CjxX65KbGHjj3485uILNatmJEoXujGNDSR4J3QBeLg7yZVjlyYH/d4skYwnQsra84EHPrrKq pjXn3LtG78T891QxmQu37zvnGUH59bbem891s8DzR2ui/sfRqcokXmeVco/OVADXwv/cN5paRlxTV RBjKNeLQHKZsLTdiLaBPwxeriBHVtQq+MAAm1oqqpvuZT00bkYakz6I+6HWop4XESHi+JjrNcoU9G 6hw+IOJvEpzewZ9+HNjtjw==; Received: from [87.69.77.57] (helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1pyxAG-00084e-MU; Tue, 16 May 2023 12:04:51 -0400 Date: Tue, 16 May 2023 19:04:48 +0300 Message-Id: <83pm70z2hr.fsf@gnu.org> From: Eli Zaretskii To: Bruno Larsen Cc: gdb-patches@sourceware.org, gdb@sourceware.org In-Reply-To: <20230516143826.3431583-2-blarsen@redhat.com> (message from Bruno Larsen via Gdb-patches on Tue, 16 May 2023 16:38:27 +0200) Subject: Re: [PATCH 1/1] [gdb]: add git trailer information on gdb/MAINTAINERS References: <20230516143826.3431583-1-blarsen@redhat.com> <20230516143826.3431583-2-blarsen@redhat.com> X-Spam-Status: No, score=1.7 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_BARRACUDACENTRAL,SPF_HELO_PASS,SPF_PASS,TXREP,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Level: * X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: > Cc: gdb@sourceware.org, > Bruno Larsen > Date: Tue, 16 May 2023 16:38:27 +0200 > From: Bruno Larsen via Gdb-patches > > + - Tested-by: > + > + Used when a contributor does not want to comment on the quality > + of the code in the patch, but has tested and sees no regressions on their > + hardware. > + > + - Reviewed-by: > + > + Used when a contributor has looked at code and agrees with the changes, > + but either does not have the authority or doesn't feel comfortable > + approving the patch (usually due to unfamiliarity with a certain > + part of the code). Reviewed-by is used by responsible maintainers as well. > + > + - Approved-by: > + > + Used by responsible mainainers or global maintainers when ^^^^^^^^^^ Typo. > + 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. > + I think the above list is incomplete, because there appears to be no "git trailer" (why do we have to call it "git" trailer, btw? will that change if we ever switch to a different VCS?) for the situation where the responsible maintainer does approve some part of the patch, but not all of it (e.g., because the other parts are not in the expertise domain of that maintainer). I thought Reviewed-by is such a trailer, but based on the above I'm beginning to think I was confused.