From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx-relay86-hz2.antispameurope.com (mx-relay86-hz2.antispameurope.com [94.100.136.186]) by sourceware.org (Postfix) with ESMTPS id CF918383F84B for ; Fri, 25 Sep 2020 07:34:40 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org CF918383F84B Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=net-b.de Authentication-Results: sourceware.org; spf=none smtp.mailfrom=prvs=053093aacd=burnus@net-b.de Received: from s041.bre.qsc.de ([195.90.7.81]) by mx-relay86-hz2.antispameurope.com; Fri, 25 Sep 2020 09:34:38 +0200 Received: from [192.168.52.128] (port-92-195-151-115.dynamic.as20676.net [92.195.151.115]) by s041.bre.qsc.de (Postfix) with ESMTPSA id 4767E2C00BB; Fri, 25 Sep 2020 09:34:37 +0200 (CEST) Subject: Re: [patch, doc] Update people who can review gfortran patches To: Thomas Koenig , "fortran@gcc.gnu.org" , gcc mailing list References: <2b079e88-6065-8d81-2e7d-27b3cbf6652b@netcologne.de> From: Tobias Burnus Message-ID: <1a57b83d-6060-994f-9709-14b3ca6d5468@net-b.de> Date: Fri, 25 Sep 2020 09:34:36 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 MIME-Version: 1.0 In-Reply-To: <2b079e88-6065-8d81-2e7d-27b3cbf6652b@netcologne.de> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US X-cloud-security-sender: burnus@net-b.de X-cloud-security-recipient: fortran@gcc.gnu.org X-cloud-security-Virusscan: CLEAN X-cloud-security-disclaimer: This E-Mail was scanned by E-Mailservice on mx-relay86-hz2.antispameurope.com with C7E61D7E06C X-cloud-security-connect: s041.bre.qsc.de[195.90.7.81], TLS=1, IP=195.90.7.81 X-cloud-security: scantime:.2085 X-Spam-Status: No, score=-7.2 required=5.0 tests=BAYES_00, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, KAM_SHORT, NICE_REPLY_A, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H4, RCVD_IN_MSPIKE_WL, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: fortran@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Fortran mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 25 Sep 2020 07:34:42 -0000 On 9/25/20 8:02 AM, Thomas Koenig via Fortran wrote: > for review of its patches, gfortran relies on a group of people > who can approve patches.  Unfortuntately, many of them are not > active. Others, who have the capability and who have acted as > de facto approvers (without anybody minding) are missing. I think you want to link to https://gcc.gnu.org/git/?p=gcc.git;a=blob;f=MAINTAINERS which is the official list. — At least additionally. I note the the HTML page states "contributions will be reviewed" and not "approved" – and (nearly) any review comment is welcome, be it from an experienced maintainer or just by someone who passes by. Still, it is a bit misleading if this list includes someone who is not an official reviewer. New reviewers get added by approval of the steering committee (https://gcc.gnu.org/steering.html, typically by some existing maintainer/reviewer proposing a new one – by sending a private email to one of the SC members.) Regarding Jakub: while he is a global reviewer and very active, for Fortran, he is mostly reviewing OpenMP and – to a lesser extent – OpenACC patches and only a very few other patches. Thus, listing him under 'contributions will be reviewed' is a bit misleading. Cheers, Tobias