From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:470:142:3::10]:37933) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ioms7-0006DW-8l for guix-patches@gnu.org; Tue, 07 Jan 2020 06:18:04 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ioms5-0002bN-UG for guix-patches@gnu.org; Tue, 07 Jan 2020 06:18:02 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:41414) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1ioms5-0002bF-Qv for guix-patches@gnu.org; Tue, 07 Jan 2020 06:18:01 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1ioms5-0007gK-NY for guix-patches@gnu.org; Tue, 07 Jan 2020 06:18:01 -0500 Subject: [bug#38846] [PATCH 0/4] Move 'HACKING' to the manual, and a proposal for commit access Resent-Message-ID: From: Ludovic =?UTF-8?Q?Court=C3=A8s?= References: <20200101162945.4946-1-ludo@gnu.org> Date: Tue, 07 Jan 2020 12:17:31 +0100 In-Reply-To: (zimoun's message of "Mon, 6 Jan 2020 22:44:38 +0100") Message-ID: <87h817jz1w.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: zimoun Cc: GNU Guix maintainers , 38846@debbugs.gnu.org Hi! zimoun skribis: > However, let point 2 minor weak points for further discussions: > > a- if the number of committers with access is more or less fixed, > then access could be transferred (less active, less time, less > motivation, etc.); Sure, we should (and do) remove people who =E2=80=9Cdisappeared=E2=80=9D or= retired, and of course welcome new people willing to participate. > and b- the bottleneck is the patch review (even if it should be > improved in the future with the Guix Data Service). > Well, a- is more about human relationship, hard to fix. However, we > should work on b- but how? What does it mean "encourage patch review"? > Candy or beer at Guix Days? ;-) Heh. :-) This is the hard part for all free software projects. As you noticed, there=E2=80=99s no shortage of patches that have fallen through the cracks, unfortunately. The =E2=80=9Cencourage patch review=E2=80=9D bit in this series, in my view= , is to tell newcomers that the deal is not just that they can push their own stuff more quickly, but also that they can help others get their stuff in in return. It may sound obvious to some, but it=E2=80=99s probably better if explicitly stated. Also, perhaps we=E2=80=99ve had the problem where commi= tters did not feel =E2=80=9Centitled=E2=80=9D to review and push other people=E2= =80=99s patches; this paragraph is also a way to tell them that yes, they _are_ entitled to that. It won=E2=80=99t solve the problem overnight; offering a beverage at the Gu= ix Days won=E2=80=99t solve it either but it=E2=80=99s a good idea anyway. :-) Ludo=E2=80=99.