From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from albireo.enyo.de (albireo.enyo.de [37.24.231.21]) by sourceware.org (Postfix) with ESMTPS id 9154A387702F; Thu, 19 Mar 2020 14:00:22 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 9154A387702F Received: from [172.17.203.2] (helo=deneb.enyo.de) by albireo.enyo.de with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) id 1jEvie-0003Bi-2D; Thu, 19 Mar 2020 14:00:20 +0000 Received: from fw by deneb.enyo.de with local (Exim 4.92) (envelope-from ) id 1jEvhH-0000jC-H8; Thu, 19 Mar 2020 14:58:55 +0100 From: Florian Weimer To: Richard Biener Cc: Tom Tromey , overseers@gcc.gnu.org, Jonathan Wakely via Gcc , Segher Boessenkool , Alexander Monakov , Overseers mailing list , "Frank Ch. Eigler" , "Frank Ch. Eigler" Subject: Re: Not usable email content encoding References: <20200317194613.GH22482@gate.crashing.org> <20200317195158.GC112952@elastic.org> <874kumt0bh.fsf@mid.deneb.enyo.de> <20200318110109.GA5496@redhat.com> <20200318142239.GF112952@elastic.org> <3af9771e-e577-f2a1-843e-c2b078bfc4ea@t-online.de> <20200318162250.GG112952@elastic.org> <87zhccsdfd.fsf@tromey.com> <87imj0pjbr.fsf@mid.deneb.enyo.de> Date: Thu, 19 Mar 2020 14:58:55 +0100 In-Reply-To: (Richard Biener's message of "Thu, 19 Mar 2020 14:41:05 +0100") Message-ID: <87blosphsw.fsf@mid.deneb.enyo.de> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Spam-Status: No, score=0.0 required=5.0 tests=KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: overseers@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Overseers mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 19 Mar 2020 14:00:29 -0000 * Richard Biener: > On Thu, Mar 19, 2020 at 2:28 PM Florian Weimer wrote: >> >> * Tom Tromey: >> >> > Also, gerrit was pretty bad about threading messages, so it became qui= te >> > hard to follow progress in email (but following all patches in the web >> > interface is very difficult, a problem shared by all these web UIs). >> >> What I found most disappointing was that the web interface doesn't >> guide you to the next reasonable step for your reviews and patches, >> like showing comments which need addressing. Tagging messages in an >> email client for later action actually works better than that, I think. > > I guess if anything we'd want something git-centric now like github > or gitlab pull requests & reviews. The only complication is approval > then which would still mean manual steps. Gitlab has a =E2=80=9Cmerge if CI passes=E2=80=9D button, I think. There a= re similar third-party solutions for Github. > Patch review would also not be publicly visible and archived(?) so > both chiming in late after visible progress and archeology would be > harder. The comments could be archived on a mailing list. But there is only some threading. Maybe this can be compensated to some extent by producing smaller patches, so that there's less reason for mega-review-threads. I don't know if Gitlab supports patch reviews, or if you only can review all the squashed changes in a merge request. Github and Gerrit support comments on individual patches, but Pagure does not, so this is not something that has universal support in all such tools. (I would consider such a feature critical for glibc development.) I haven't figured out yet under what circumstances Github links commits to pull requests in the web UI. I'm not sure if this relationship is available outside the web UI. (Obviously, this is missing from the current gcc-patches feed as well.)