From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hartmut Goebel Subject: Re: Adopt a patch! Date: Wed, 20 Sep 2017 13:48:54 +0200 Message-ID: <0cd911c2-f64a-c60b-ef70-749c135a58e9@crazy-compilers.com> References: <877ex5d555.fsf@gnu.org> <4fecd5dd.AEQAQDR72NkAAAAAAAAAAAOzWv8AAAACwQwAAAAAAAW9WABZudnG@mailjet.com> <87d16pf5x5.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:35590) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1duejc-00065w-TP for guix-devel@gnu.org; Wed, 20 Sep 2017 09:08:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1duejX-00048N-Lc for guix-devel@gnu.org; Wed, 20 Sep 2017 09:08:12 -0400 Received: from mail-out.m-online.net ([2001:a60:0:28:0:1:25:1]:43153) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1duejX-00047F-EW for guix-devel@gnu.org; Wed, 20 Sep 2017 09:08:07 -0400 Received: from frontend01.mail.m-online.net (unknown [192.168.8.182]) by mail-out.m-online.net (Postfix) with ESMTP id 3xxygr2BHhz1qtP6 for ; Wed, 20 Sep 2017 13:48:55 +0200 (CEST) Received: from localhost (dynscan1.mnet-online.de [192.168.6.70]) by mail.m-online.net (Postfix) with ESMTP id 3xxygq5bjKz1qql4 for ; Wed, 20 Sep 2017 13:48:55 +0200 (CEST) Received: from mail.mnet-online.de ([192.168.8.182]) by localhost (dynscan1.mail.m-online.net [192.168.6.70]) (amavisd-new, port 10024) with ESMTP id FLNS7ciCH5qi for ; Wed, 20 Sep 2017 13:48:55 +0200 (CEST) Received: from hermia.goebel-consult.de (ppp-93-104-72-123.dynamic.mnet-online.de [93.104.72.123]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.mnet-online.de (Postfix) with ESMTPS for ; Wed, 20 Sep 2017 13:48:55 +0200 (CEST) Received: from [192.168.110.2] (lenashee.goebel-consult.de [192.168.110.2]) by hermia.goebel-consult.de (Postfix) with ESMTP id 03620603A4 for ; Wed, 20 Sep 2017 13:48:55 +0200 (CEST) In-Reply-To: Content-Language: en-US List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: "Guix-devel" To: guix-devel@gnu.org Am 19.09.2017 um 16:22 schrieb Arun Isaac: > Just thinking out loud: Maybe, we need more people with commit > access. Theoretically, anyone can review a patch, but ultimately it is > people with commit access who will have to finally apply and push the > patch. As the rate of submission of patches grows, this increases the > work load on those with commit access. It is not only the work load, but also the work-flow which makes it hard for occasional reviewers and committers. The mail-interface might be great for those who are used to it, but it requires one to subscribe to the patch-mainling-list, keep an eye on review, download the patch, lint the patch, reply to the mail. If the patch is okay, I need to pull--rebase on the current master, push, write a mail for closing the bug-entry. This means switching forth and back between mail, shell, and browser. These are far too much manual steps. An if I only have little time, patches are piling up. The mailbox=C2=A0 get cluttered by patches I do no= t follow. This woes! And this is why I'm not regularly reviewing patches. Compare it with an integrated workflow on e.g. Gitlab or github: The list of patches to review is always up to date, same for the state and comments. Using CI (gitlabs integrated pipeline are great!) saves me a lot of work, e.g. linting. If the patch is okay, it is a single click (okay, maybe 5 clicks) to rebase i on top of master. The patch is closed automatically, the submitter is notified, bookkeeping (referencing to the commit) is done. We already discussed using e.g. goks last year with no result. Maybe it's time to restart the approach. --=20 Regards Hartmut Goebel | Hartmut Goebel | h.goebel@crazy-compilers.com | | www.crazy-compilers.com | compilers which you thought are impossible |