From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 38163 invoked by alias); 20 Oct 2019 23:17:03 -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 38153 invoked by uid 89); 20 Oct 2019 23:17:02 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-2.1 required=5.0 tests=AWL,BAYES_00,FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=ham version=3.3.1 spammy=platforms, online, letter, personal X-HELO: mail-vk1-f196.google.com Received: from mail-vk1-f196.google.com (HELO mail-vk1-f196.google.com) (209.85.221.196) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Sun, 20 Oct 2019 23:17:01 +0000 Received: by mail-vk1-f196.google.com with SMTP id q25so2345702vkn.12 for ; Sun, 20 Oct 2019 16:17:01 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=bNiQBTKH5aE30qu74kJmutN3UOkNjV05/5VMlYvT/8s=; b=dYn0ea2nV0ebnDmt6zsEX7serNoYTkgUBSzFTHImuprAuGVLoxko8osEpZrh12VTPk OZmGNiKBIQyWduWAfHWRAtqRP9lwbnXKk81gKHRHejymvqtFkdYlSPM1mQCiRdwjF6TO BN2mWZN3Sd++hDcn5anCdNQa6UzW3sRymmjK+jQ5s1Dp4/qrhYxJ96GcIiuVMe9qQmZk /+oLQgJQoUT/abj9+pG3QeYf2O9aQvff2IjJWxbV3sdoP+wDCpqBapjwkHz1IhxE7Rb8 KbMVQMEZqzidQ/lK4TBMCNvUcll2tUTWw4SEoEaMULLZ0yQuoYBu/1fYHQKZSNSUo6It /nEA== MIME-Version: 1.0 References: <877e4z8ovc.fsf@tromey.com> In-Reply-To: <877e4z8ovc.fsf@tromey.com> From: Andrew Pinski Date: Sun, 20 Oct 2019 23:17:00 -0000 Message-ID: Subject: Re: some thoughts on gerrit To: Tom Tromey Cc: GDB Patches Content-Type: text/plain; charset="UTF-8" X-IsSubscribed: yes X-SW-Source: 2019-10/txt/msg00705.txt.bz2 On Sun, Oct 20, 2019 at 7:37 AM Tom Tromey wrote: > > After using gerrit for gdb for just this past week, I have mixed > feelings, which I thought I'd share. > > The upside is pretty good -- basically what I was hoping for when we > discussed this topic at Cauldron. > > The major benefits is that it's easy to see the status of patches. > > A benefit I didn't predict is that it's a bit simpler to submit patches. > In particular, my personal email host doesn't like it if I send log > series, so I have to remember to throttle when using send-email -- but > with gerrit that's a non-issue, because it is just a push. > > > So far the major downsides are related to patch series. > > [[[ > First, as an aside, my most recent patch series does show up here: > > https://sourceware.org/ml/gdb-patches/2019-10/authors.html > > (Search for "RAII class" under the name "Tom Tromey (Code Review)") > > ...but it somehow doesn't show up here: > > https://sourceware.org/ml/gdb-patches/2019-10/threads.html > ]]] Hi, First, we use gerrit internally at Marvell. I have another benefit for gerrit, which is not listed here. You can configure gerrit to hook into an automation service which will give then automated feedback. E.g. we use it to run check patch (on Linux kernel patches and others) and then if the patch builds and tests it on a few platforms. I don't know if you could use patchworks to do that though. Thanks, Andrew Pinski > > > Anyway, with series we are missing two things that email had: namely, > the cover letter, and threading. > > The cover letter is often a good guide to what is going on in the > series. See for example: > > https://sourceware.org/ml/gdb-patches/2019-10/msg00590.html > > It seems a shame to lose this. > > One counter-argument about the cover letter I thought of is that, > because it doesn't end up in the history, maybe the lack of it will > force us to make each commit message even clearer. And, we should do > that ... it's just that the roadmap is handy when reviewing. > > I wonder if there'd be a way to make "git review" prompt for a cover > letter and attach it somewhere as a comment. > > > Lack of threading means it is hard to see how patches relate when > reading in email. Maybe this can be fixed in gerrit? > > > I wonder a little if a sufficiently configured patchworks would be a > better fit for gdb. > > The major problem with the current patchworks is that it doesn't > automatically remove patches when they are checked in. However, a newer > version of patchworks can do this, especially if we augment it with a > commit hook to add a UUID to the commit message (which we've already > accepted for gerrit...). It seems easy to set this up. > > Another drawback of patchworks is that reviews aren't done online -- you > still use email. This doesn't bother me, but maybe it's an important > consideration for others. > > I'm not saying we should definitely switch -- just that I've noticed > some drawbacks to gerrit as compared to email, and I am wondering if we > can somehow preserve more of the good things. > > Tom