From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 65990 invoked by alias); 6 Dec 2019 16:19:12 -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 65978 invoked by uid 89); 6 Dec 2019 16:19:11 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-5.8 required=5.0 tests=AWL,BAYES_00,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.1 spammy=automation, shine, fond 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; Fri, 06 Dec 2019 16:19:10 +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 xB6GJ2dW014486 (version=TLSv1/SSLv3 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 6 Dec 2019 11:19:06 -0500 DKIM-Filter: OpenDKIM Filter v2.11.0 smtp.polymtl.ca xB6GJ2dW014486 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=polymtl.ca; s=default; t=1575649147; bh=4zOreXlT/MHKadSaH8WAzbU8At/i/5OIIUHCIjM3+v8=; h=Subject:To:References:From:Date:In-Reply-To:From; b=TdyKwmixH/29UsqLz1etG3PKvk3q7GlL5XeJV/fj097JO0nz8xycij/hR80HLANXR Om+dfwEgqBrF8ESSX7cbG9rbxOiFrvxiWgIU89nDM1k8GZjhSrIJOYpq53OUHK8LT/ NV70O1/WlzuhgvcnqAmMg/8dIq4Rfm5l0BCFi30s= Received: from [172.16.0.95] (192-222-181-218.qc.cable.ebox.net [192.222.181.218]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by simark.ca (Postfix) with ESMTPSA id D098B1E5FA; Fri, 6 Dec 2019 11:19:01 -0500 (EST) Subject: Re: Gerrit status update To: Luis Machado , "gdb-patches@sourceware.org" References: <93b1726e-d5d9-f264-4efc-79b0cdb034aa@polymtl.ca> <7bacce91-fcb6-a096-dda4-1b77be539320@linaro.org> From: Simon Marchi Message-ID: <65299de1-3733-5cce-35f7-77a6ec3944bb@polymtl.ca> Date: Fri, 06 Dec 2019 16:19:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.1 MIME-Version: 1.0 In-Reply-To: <7bacce91-fcb6-a096-dda4-1b77be539320@linaro.org> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-IsSubscribed: yes X-SW-Source: 2019-12/txt/msg00230.txt.bz2 On 2019-12-05 7:14 p.m., Luis Machado wrote: > On 11/22/19 6:58 PM, Simon Marchi wrote: >> Hi all, >> >> We had a discussion on IRC yesterday about the shortcomings of Gerrit >> for our workflow.  I'd like to share what was discussed, but also give >> a chance to those who are not on IRC to have a voice. > > Thanks. > >> >> The two main problems of Gerrit, when applied to our current workflow, >> are related to patch series.  While it is possible to upload a sequence >> of changes, it is not possible to treat a sequence of changes as a >> logical patch series.  And therefore: >> >> - It is not possible to attach a cover letter to a series. >> - The email notifications are not threaded like a series would.  It's >>    basically impossible to follow a series by email, as each patch will >>    have its own little thread. >> > I went through yours and Tom's e-mail, but also found glibc's take on it (https://sourceware.org/ml/libc-alpha/2019-11/msg00774.html). Oh, thanks for pointing our that thread on the libc list, I didn't know about it. I also went through it, I think overall it summarizes the situation pretty well. You could s/glibc/gdb/ in the discussion and it would still apply. > Carlos pointed out a possible way to have a cover letter, and apparently that is more or less the recommended way of doing it. Yes, I agree that having an empty commit at the end of the chain would probably work well enough. > > It seems to me the biggest issue is having gerrit translated properly to e-mail format, both in terms of interaction and organization of replies (threaded series etc). > > On the positive side i think gerrit provides quite a few useful features to speed up the process of submission and reviewing of patches, which people have already pointed out. So i won't list those here. Agreed. > Considering the drawbacks of gerrit seem to be mostly concentrated around series of patches, maybe bigger series in particular, should we try and do those through e-mail instead. Maybe as a recommendation? > > For other smaller contributions, gerrit still seems to be fairly useful. Hmm, I'm not sure I'd like to keep two parallel systems, with a vague rule to tell which one to use. It will be confusing for us and newcomers. And don't forget that the initial goal of this effort is to improve the tracking of patches, to avoid patches falling between the cracks. If some patches are sent by email and are not tracked, we are partially missing our goal. Also, I think the benefits of Gerrit shine particularly when dealing with patch series, especially the diff-between-versions. For a single patch, I can easily apply the two patches locally and do a simple diff. I just learned about git range-diff, reading the glibc thread. I'll try to use it in the future to compare patch series versions. Although it doesn't seem to be able to show the diffs using one's favorite diff viewer, and I'm not particularly fond of reading raw diffs. > As for patchwork, my experience using it in the past wasn't that great. It may have improved since then, but i remember it simply being forgotten over time, and patches piling up anyway. Yes, that's what the experience has shown. The amount of manual work required to maintain it made it not worth it. So I would not consider using Patchwork without some accurate patch tracking based on some kind Change-Id, like we have with Gerrit, that would allow us to do some automation. We can start to do this outside Patchwork, using some script that use the Patchwork API. If that works well enough, we could even attempt to contribute it upstream. One of the Patchwork developer is open to include this kind of feature: https://github.com/getpatchwork/patchwork/issues/327#issuecomment-561573729 Simon