From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 32026 invoked by alias); 14 Oct 2019 17:31:40 -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 32018 invoked by uid 89); 14 Oct 2019 17:31:40 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-3.6 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.1 spammy=territory, posting, resulted, emails X-HELO: smtp.polymtl.ca Received: from smtp.polymtl.ca (HELO smtp.polymtl.ca) (132.207.4.11) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 14 Oct 2019 17:31:38 +0000 Received: from simark.ca (simark.ca [158.69.221.121]) (authenticated bits=0) by smtp.polymtl.ca (8.14.7/8.14.7) with ESMTP id x9EHVU4N015062 (version=TLSv1/SSLv3 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT) for ; Mon, 14 Oct 2019 13:31:35 -0400 DKIM-Filter: OpenDKIM Filter v2.11.0 smtp.polymtl.ca x9EHVU4N015062 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=polymtl.ca; s=default; t=1571074296; bh=LTzK6iBbNY+qH5QgA69X1pIRh/nb1ycytMGilCmL5N4=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=qHoK2SihvuZuyM1vK/p0XZ2EnbItAB+mAsjWjb79DDL0dHj0TwubIlvY6Wi6+Happ D2Tv8ys3k+HThtSWEliIJx+al4w1ySTazAOkRwYsNWdRv2DfCufDcyAR1daqLX6ezA 7/tsiWdgOrDmdNfSiwoYOOwE3342yx7r+GiihdhY= Received: by simark.ca (Postfix, from userid 112) id 2FBA21E79E; Mon, 14 Oct 2019 13:31:30 -0400 (EDT) Received: from simark.ca (localhost [127.0.0.1]) by simark.ca (Postfix) with ESMTP id E0ADC1E059; Mon, 14 Oct 2019 13:31:28 -0400 (EDT) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Mon, 14 Oct 2019 17:31:00 -0000 From: Simon Marchi To: Eli Zaretskii Cc: gdb-patches@sourceware.org Subject: Re: Gerrit In-Reply-To: <83k197i74c.fsf@gnu.org> References: <20191013045218.3261363-1-simon.marchi@polymtl.ca> <1c1f820fa507243fd7a2096ec3eb2454@polymtl.ca> <83r23fie80.fsf@gnu.org> <83k197i74c.fsf@gnu.org> Message-ID: <8c18a33556daaad6d35a0d2a4f987899@polymtl.ca> X-Sender: simon.marchi@polymtl.ca User-Agent: Roundcube Webmail/1.3.10 X-IsSubscribed: yes X-SW-Source: 2019-10/txt/msg00374.txt.bz2 On 2019-10-14 13:12, Eli Zaretskii wrote: > I see some emails from Gerrit, does it mean you already set that up? > Because those emails leave a lot to be desired, IMO. Yes, I just did. The content of the emails is fully configurable. We can work on improving them if you have specific pain points or suggestions. The page here describes which emails are sent, and what variable information about each change is available to include in the email: https://gerrit-review.googlesource.com/Documentation/config-mail.html > Anyway, seeing the beginning of a patch was the only way for me to > know that a patch needs me to review the documentation parts. Now I > wonder how I can do that when the patch is posted on Gerrit. What do you mean by "beginning of a patch", do you mean the diff stat that shows the changed files? If so, I believe this information is available in the notification sent for a new change. This patch, for example: https://gnutoolchain-gerrit.osci.io/r/c/binutils-gdb/+/29 resulted in this message being sent: https://sourceware.org/ml/gdb-patches/2019-10/msg00371.html And it contains: M gdb/block.c M gdb/testsuite/gdb.dwarf2/varval.exp 2 files changed, 55 insertions(+), 3 deletions(-) Does that help? >> As long as we use Gerrit and mail patches in parallel, people are free >> to send patches using the system they prefer. I think it's simpler if >> reviewers use the system that was chosen by the patch author (reply on >> Gerrit if the patch is on Gerrit, reply by email if the patch is by >> email). > > But I cannot reply on Gerrit without registering there, can I? I can only guess, but probably not. > I'm also somewhat bothered by what I've read on the wiki. I > understand that anyone can register on Gerrit, and after that push > patches for review, independently of their write access to the > sourceware repository. Then, if gnutoolchain-gerrit.osci.io is > associated with or operated by FSF/GNU, it would mean we provide a way > for random people to push changes to GDB to a public repository > affiliated with us, without having any control on what is being pushed > ahead of the push. Suppose someone pushes there changes that violate > the GPL, or do something else that is against the GNU policies -- > wouldn't that appear as if we are "authorizing" those just by having > that code in the repository, even though it's on a branch and haven't > yet been admitted to sourceware? That goes into lawyer territory, so I can't give a definitive answer. The way I see it is that it's not really different than that person posting a patch with the same content on the mailing list. It's the same content, just a different format. Simon