From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21424 invoked by alias); 14 Oct 2019 14:39:06 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 21406 invoked by uid 89); 14 Oct 2019 14:39:06 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-3.6 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.1 spammy= X-HELO: eggs.gnu.org Received: from eggs.gnu.org (HELO eggs.gnu.org) (209.51.188.92) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 14 Oct 2019 14:39:04 +0000 Received: from fencepost.gnu.org ([2001:470:142:3::e]:54596) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1iK1V0-0005XF-RQ; Mon, 14 Oct 2019 10:39:02 -0400 Received: from [176.228.60.248] (port=3750 helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1iK1Uy-0005vv-Ul; Mon, 14 Oct 2019 10:39:02 -0400 Date: Mon, 14 Oct 2019 14:39:00 -0000 Message-Id: <83r23fie80.fsf@gnu.org> From: Eli Zaretskii To: Simon Marchi CC: gdb-patches@sourceware.org In-reply-to: <1c1f820fa507243fd7a2096ec3eb2454@polymtl.ca> (message from Simon Marchi on Mon, 14 Oct 2019 10:21:50 -0400) Subject: Re: [PATCH] gdb: remove unused includes from dwarf2read.c References: <20191013045218.3261363-1-simon.marchi@polymtl.ca> <1c1f820fa507243fd7a2096ec3eb2454@polymtl.ca> X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-IsSubscribed: yes X-SW-Source: 2019-10/txt/msg00337.txt.bz2 > Date: Mon, 14 Oct 2019 10:21:50 -0400 > From: Simon Marchi > > I pushed it after Tom de Vries reviewed it on Gerrit: > > https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/24 Does this mean that now we need to look for patches in two places? Can Gerrit be set up to forward the review comments to the list? Also, if some of us decides to do the review on Gerrit, does it mean all the others need to do that as well?