From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-3.9 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_PASS shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dcvr.yhbt.net (Postfix) with ESMTPS id 52AE31F4B5 for ; Tue, 12 Nov 2019 19:21:46 +0000 (UTC) DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:subject:to:cc:references:from:message-id:date :mime-version:in-reply-to:content-type :content-transfer-encoding; q=dns; s=default; b=TGgUXL5JAbp7VLtn V0W44VJO4P9EAvggQVmrg6ZRSfBOIHzP0yN7HldAdpVcWc+r7j4U4wIVXhDrGfdr h/P/8LnV+UuIrOrz6+r1V81ZvfIyWW9TfIOPik7s9y5QTodUdWtCg8Etu6Bkvfac lI0WkYbhDdaCXHcHt8hd1MTuZ+Y= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:subject:to:cc:references:from:message-id:date :mime-version:in-reply-to:content-type :content-transfer-encoding; s=default; bh=rR5j5u5sSep91i8Pecv3Qt gV5XM=; b=GF0BCQxxJi6GE8mkUeyK3n91bmtPCU2kVebTr5/YEj7Uxr6wkheude ccmY27RlPHM19JIBYGJruVfX3pfuNj89Aj804sTJnE/VFIuvdrLvUsyfVfeGAhMO EQ0/AUAKWiFuLB+oVOwSB+71fmSk4ET2Ycmh1g4OmnYWGXgN5eTf0= Received: (qmail 47309 invoked by alias); 12 Nov 2019 19:21:44 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Received: (qmail 47295 invoked by uid 89); 12 Nov 2019 19:21:43 -0000 Authentication-Results: sourceware.org; auth=none X-HELO: us-smtp-delivery-1.mimecast.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1573586500; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=PTVaPt/McJ3I7DjCzda6UJOaiCOB0Urlpam/wnHlDfE=; b=afWZfISh0IB5uua1q+Ymzcq4fDueiHMYoJG/k+KfW3bwQdQjwMa2UD3vHrtQ3ZK2FQmDHa dUrhJH/5rTX2gimBrWyPT/GMSqRJPLFzelvlZVl7EPlaI4o1Ww/Kw4DIdKAl/CwbZSSyek NcZzq4uXRtrD7gvFuYrZed/aew4ykzU= Subject: Re: Setup non-pushing gerrit instance for glibc. To: "Gabriel F. T. Gomes" Cc: libc-alpha , Sergio Durigan Junior References: <2e93ece9-386b-c587-9355-33a4695a3f02@redhat.com> <20191112155303.2215a667@tereshkova.br.ibm.com> From: Carlos O'Donell Message-ID: <1f1bbb12-4121-7243-35f7-dd082fc1c4fd@redhat.com> Date: Tue, 12 Nov 2019 14:21:34 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 MIME-Version: 1.0 In-Reply-To: <20191112155303.2215a667@tereshkova.br.ibm.com> X-Mimecast-Spam-Score: 0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On 11/12/19 1:53 PM, Gabriel F. T. Gomes wrote: > Hi, community, >=20 > I finally used gerrit today. Afterwards, I re-read this thread and here > are my impressions and questions... >=20 > First of all, thanks for doing this. :) >=20 > (CC'ing Sergio, because I don't know if he follows the list and I have a > question for him) >=20 > 1. On Reviewed-by statements, >=20 > On Tue, 12 Nov 2019, Florian Weimer (Code Review) wrote: >> >> (I'm not sure if we can keep up the practice of adding Reviewed-By: line= s >> with Gerrit, because doing so creates a new patch version that needs >> review.) >=20 > Florian noticed [1] that Reviewed-by statements might break our use of > gerrit, because changing the commit message creates a new version of the > patch. Carlos mentioned that having the statements is useful for his > metrics (I also like these fields, even though I don't use them as > professionally). >=20 > If we are going to make gerrit push automatically one day, could we make > it add the Reviewed-by statement automatically (based on the Reviewers > field) when it pushes the change? Maybe it doesn't even need to create a > new version of the patch with the new commit message (although it would b= e > nice if it could create some sort of link between the patch that actually > gets commit and the patch displayed in the web interface (the same that i= s > sent to the mailing list)). >=20 > If that is possible, then the patch author (or even the reviewers) won't > need to change the commit message *during patch review*, which will avoid > the creation of the new patch version. >=20 > [1] https://sourceware.org/ml/libc-alpha/2019-11/msg00404.html I try to explain the consequences and tradeoffs here: https://www.sourceware.org/ml/libc-alpha/2019-11/msg00439.html Yes, gerrit can automatically add Reviewed-by. Yes it will work if we stick to one workflow for patch submission. > 3. On quote selection freedom, >=20 > When writing reviews by email, I like to select the precise amount of > quotes (stitching different quotes together, and snipping unimportant > parts) that I judge relevant for the conversation. This is, obviously, > a subjective judgment and I don't expect any tool to be able to do the > same ever (famous last words?). My point here is, I didn't find a way to > do that (adjust the amount of quote and snipping) on the web interface. Highlight with your cursor the text you want, and hit 'c' to comment. > On Fri, 08 Nov 2019, Simon Marchi wrote: >> >> So please try to use "Quote" instead of "Reply" and quote >> the relevant portion of what you are replying to (just like you would by >> email), so that it appears in the notification email. >=20 > Simon mentioned [2] a "Quote" (button?). Maybe that is what I wanted and > couldn't find. Highlight, and hit 'c'? > Like others in the community, I think I prefer using email, which gives m= e > freedom to select the amount of quoting and snipping I want, so maybe thi= s > will never be a problem to anyone. Does highlighting and pressing 'c' resolve this for you? > [2] https://sourceware.org/ml/libc-alpha/2019-11/msg00306.html >=20 > 4. On the "patches go missing" problem, >=20 > On Sun, 27 Oct 2019, Sergio Durigan Junior wrote: >> >> When the GDB community decided to give gerrit a try, the main problem it >> was looking to solve was the "patches go missing" issue. Basically, we >> have a lot of patches being sent to the project and not so many >> reviewers, so it's not uncommon for a patch to get "buried" waiting for >> a review, and if the author doesn't ping it, it gets lost. With gerrit, >> you can get a pretty good view of all the patches that were submitted so >> far, and you can always check if a patch is too old or if it's been too >> long since the last interaction on it. >=20 > Can you (GDB community) already tell if gerrit actually helped with those > patches that went missing, or is it to soon? While I understand that > gerrit has the potential to help with that, I also get the impression tha= t > patches will still pile up, but on the web interface, instead of on the > mailing list. I get this impression because our patchwork instance has > piles of patches (btw, this is not an inquisitorial kind of question - my > heart is still open for the new tool - I just want to know if you think > gerrit is helping). The problem with Patchwork is that it never helped with patch review. Patchwork was simply another way to view the mailing list. Gerrit today lets me actually complete reviews from start-to-finish and can act as an assitance to me. For example Gerrit already tells me if patches have merge conflicts, so I can ignore those or tell people to rebase. Eventually if we get some ci/cd tied to gerrit it should be really really helpful for reviewers. --=20 Cheers, Carlos.