From patchwork Wed Jan 1 16:34:43 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Ludovic_Court=C3=A8s?= X-Patchwork-Id: 19503 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id BBDFB179D5; Wed, 1 Jan 2020 16:36:14 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id 5AFE6179D3 for ; Wed, 1 Jan 2020 16:36:14 +0000 (GMT) Received: from localhost ([::1]:59838 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgyj-00018X-JA for patchwork@mira.cbaines.net; Wed, 01 Jan 2020 11:36:13 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:34312) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgya-00017e-HJ for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1imgyY-000664-VG for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:04 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:59743) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1imgyY-00065g-SD for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1imgyX-0002Dq-Q4 for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:01 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#38846] [PATCH 1/4] doc: Add "Tracking Bugs and Patches" section. References: <20200101162945.4946-1-ludo@gnu.org> In-Reply-To: <20200101162945.4946-1-ludo@gnu.org> Resent-From: Ludovic =?utf-8?q?Court=C3=A8s?= Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Wed, 01 Jan 2020 16:36:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 38846 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 38846@debbugs.gnu.org Cc: Ludovic =?utf-8?q?Court=C3=A8s?= , guix-maintainers@gnu.org Received: via spool by 38846-submit@debbugs.gnu.org id=B38846.15778965048430 (code B ref 38846); Wed, 01 Jan 2020 16:36:01 +0000 Received: (at 38846) by debbugs.gnu.org; 1 Jan 2020 16:35:04 +0000 Received: from localhost ([127.0.0.1]:37474 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1imgxY-0002BK-GQ for submit@debbugs.gnu.org; Wed, 01 Jan 2020 11:35:03 -0500 Received: from eggs.gnu.org ([209.51.188.92]:51650) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1imgxW-0002Az-BI for 38846@debbugs.gnu.org; Wed, 01 Jan 2020 11:34:59 -0500 Received: from fencepost.gnu.org ([2001:470:142:3::e]:38142) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1imgxR-0003To-5n; Wed, 01 Jan 2020 11:34:53 -0500 Received: from [2a01:e0a:1d:7270:af76:b9b:ca24:c465] (port=47372 helo=gnu.org) by fencepost.gnu.org with esmtpsa (TLS1.2:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1imgxQ-00045M-Ol; Wed, 01 Jan 2020 11:34:52 -0500 From: Ludovic =?utf-8?q?Court=C3=A8s?= Date: Wed, 1 Jan 2020 17:34:43 +0100 Message-Id: <20200101163446.5132-1-ludo@gnu.org> X-Mailer: git-send-email 2.24.1 MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches * doc/contributing.texi (Tracking Bugs and Patches): New section. (Submitting Patches): Refer to it. * doc/guix.texi: Update copyright line. * HACKING (Using emacs-debbugs): Remove. --- HACKING | 11 +------- doc/contributing.texi | 62 ++++++++++++++++++++++++++++++++++++++----- doc/guix.texi | 2 +- 3 files changed, 58 insertions(+), 17 deletions(-) diff --git a/HACKING b/HACKING index 2f0f93f896..eeea2f6311 100644 --- a/HACKING +++ b/HACKING @@ -2,7 +2,7 @@ #+TITLE: Hacking GNU Guix and Its Incredible Distro -Copyright © 2012, 2013, 2014, 2016, 2017, 2019 Ludovic Courtès +Copyright © 2012, 2013, 2014, 2016, 2017, 2019, 2020 Ludovic Courtès Copyright © 2015, 2017 Mathieu Lirzin Copyright © 2017 Leo Famulari Copyright © 2017 Arun Isaac @@ -63,12 +63,3 @@ after two weeks, and if you’re confident, it’s OK to commit. That last part is subject to being adjusted, allowing individuals to commit directly on non-controversial changes on parts they’re familiar with. - -* Using emacs-debbugs - -Bug reports and patches are tracked using debbugs. If you are on emacs, you -can use emacs-debbugs. - -List all open bug reports on guix-patches with - -C-u M-x debbugs-gnu guix-patches n y diff --git a/doc/contributing.texi b/doc/contributing.texi index e656676c0f..fd27f037e9 100644 --- a/doc/contributing.texi +++ b/doc/contributing.texi @@ -26,6 +26,7 @@ choice. * Packaging Guidelines:: Growing the distribution. * Coding Style:: Hygiene of the contributor. * Submitting Patches:: Share your work. +* Tracking Bugs and Patches:: Using Debbugs. @end menu @node Building from Git @@ -827,12 +828,12 @@ Thus, access to the repository is not strictly necessary. We welcome contributions in the form of patches as produced by @code{git format-patch} sent to the @email{guix-patches@@gnu.org} mailing list. -This mailing list is backed by a Debbugs instance accessible at -@uref{https://bugs.gnu.org/guix-patches}, which allows us to keep track -of submissions. Each message sent to that mailing list gets a new -tracking number assigned; people can then follow up on the submission by -sending email to @code{@var{NNN}@@debbugs.gnu.org}, where @var{NNN} is -the tracking number (@pxref{Sending a Patch Series}). +This mailing list is backed by a Debbugs instance, which allows us to +keep track of submissions (@pxref{Tracking Bugs and Patches}). Each +message sent to that mailing list gets a new tracking number assigned; +people can then follow up on the submission by sending email to +@code{@var{NNN}@@debbugs.gnu.org}, where @var{NNN} is the tracking +number (@pxref{Sending a Patch Series}). Please write commit logs in the ChangeLog format (@pxref{Change Logs,,, standards, GNU Coding Standards}); you can check the commit history for @@ -1040,3 +1041,52 @@ they are kept together. See for more information. You can install @command{git send-email} with @command{guix install git:send-email}. @c Debbugs bug: https://debbugs.gnu.org/db/15/15361.html + +@node Tracking Bugs and Patches +@section Tracking Bugs and Patches + +@cindex bug reports, tracking +@cindex patch submissions, tracking +@cindex issue tracking +@cindex Debbugs, issue tracking system +Bug reports and patch submissions are currently tracked using the +Debbugs instance at @uref{https://bugs.gnu.org}. Bug reports are filed +against the @code{guix} ``package'' (in Debbugs parlance), by sending +email to @email{bug-guix@@gnu.org}, while patch submissions are filed +against the @code{guix-patches} package by sending email to +@email{guix-patches@@gnu.org} (@pxref{Submitting Patches}). + +A web interface (actually @emph{two} web interfaces!) are available to +browse issues: + +@itemize +@item +@url{https://bugs.gnu.org/guix} lists bug reports; +@item +@url{https://bugs.gnu.org/guix-patches} lists patch submissions. +@end itemize + +You can also access both of these @i{via} the (nicer) +@url{https://issues.guix.gnu.org} interface@footnote{The web interface +at @url{https://issues.guix.gnu.org} is powered by Mumi, a nice piece of +software written in Guile, and you can help! See +@url{https://git.elephly.net/gitweb.cgi?p=software/mumi.git}.}. To view +discussions related to issue number @var{n}, go to +@indicateurl{https://issues.guix.gnu.org/issue/@var{n}} or +@indicateurl{https://bugs.gnu.org/@var{n}}. + +If you use Emacs, you may find it more convenient to interact with +issues using @file{debbugs.el}, which you can install with: + +@example +guix install emacs-debbugs +@end example + +For example, to list all open issues on @code{guix-patches}, hit: + +@example +@kbd{C-u} @kbd{M-x} debbugs-gnu @kbd{RET} @kbd{RET} guix-patches @kbd{RET} n y +@end example + +@xref{Top,,, debbugs-ug, Debbugs User Guide}, for more information on +this nifty tool! diff --git a/doc/guix.texi b/doc/guix.texi index 70e3dfea6a..999b7c9421 100644 --- a/doc/guix.texi +++ b/doc/guix.texi @@ -21,7 +21,7 @@ @set SUBSTITUTE-URL https://@value{SUBSTITUTE-SERVER} @copying -Copyright @copyright{} 2012, 2013, 2014, 2015, 2016, 2017, 2018, 2019 Ludovic Courtès@* +Copyright @copyright{} 2012, 2013, 2014, 2015, 2016, 2017, 2018, 2019, 2020 Ludovic Courtès@* Copyright @copyright{} 2013, 2014, 2016 Andreas Enge@* Copyright @copyright{} 2013 Nikita Karetnikov@* Copyright @copyright{} 2014, 2015, 2016 Alex Kost@* From patchwork Wed Jan 1 16:34:44 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Ludovic_Court=C3=A8s?= X-Patchwork-Id: 19502 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id 4B93B179D5; Wed, 1 Jan 2020 16:36:14 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id DC473179D3 for ; Wed, 1 Jan 2020 16:36:13 +0000 (GMT) Received: from localhost ([::1]:59836 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgyj-00017v-0f for patchwork@mira.cbaines.net; Wed, 01 Jan 2020 11:36:13 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:34336) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgyb-00017g-53 for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1imgyY-00064q-Em for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:05 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:59741) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1imgyY-00064f-BL for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1imgyY-0002Dw-8L for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:02 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#38846] [PATCH 2/4] doc: Move "Commit Access" section from 'HACKING' to the manual. Resent-From: Ludovic =?utf-8?q?Court=C3=A8s?= Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Wed, 01 Jan 2020 16:36:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 38846 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 38846@debbugs.gnu.org Cc: Ludovic =?utf-8?q?Court=C3=A8s?= , guix-maintainers@gnu.org Received: via spool by 38846-submit@debbugs.gnu.org id=B38846.15778965048437 (code B ref 38846); Wed, 01 Jan 2020 16:36:02 +0000 Received: (at 38846) by debbugs.gnu.org; 1 Jan 2020 16:35:04 +0000 Received: from localhost ([127.0.0.1]:37477 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1imgxb-0002Bu-Uv for submit@debbugs.gnu.org; Wed, 01 Jan 2020 11:35:04 -0500 Received: from eggs.gnu.org ([209.51.188.92]:51680) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1imgxX-0002B1-Am for 38846@debbugs.gnu.org; Wed, 01 Jan 2020 11:34:59 -0500 Received: from fencepost.gnu.org ([2001:470:142:3::e]:38143) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1imgxS-0003XN-5C; Wed, 01 Jan 2020 11:34:54 -0500 Received: from [2a01:e0a:1d:7270:af76:b9b:ca24:c465] (port=47372 helo=gnu.org) by fencepost.gnu.org with esmtpsa (TLS1.2:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1imgxR-00045M-Ju; Wed, 01 Jan 2020 11:34:53 -0500 From: Ludovic =?utf-8?q?Court=C3=A8s?= Date: Wed, 1 Jan 2020 17:34:44 +0100 Message-Id: <20200101163446.5132-2-ludo@gnu.org> X-Mailer: git-send-email 2.24.1 In-Reply-To: <20200101163446.5132-1-ludo@gnu.org> References: <20200101163446.5132-1-ludo@gnu.org> MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches * HACKING (Commit Access): Remove. (Contributing): Update URL of the manual. * doc/contributing.texi (Commit Access): New section. (Submitting Patches): Add cross reference. --- HACKING | 47 +--------------------------------- doc/contributing.texi | 59 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 60 insertions(+), 46 deletions(-) diff --git a/HACKING b/HACKING index eeea2f6311..36373278f9 100644 --- a/HACKING +++ b/HACKING @@ -17,49 +17,4 @@ See the manual for useful hacking informations, either by running info -f doc/guix.info "Contributing" -or by checking the [[http://www.gnu.org/software/guix/manual/guix.html#Contributing][web copy of the manual]]. - -* Commit Access - -For frequent contributors, having write access to the repository is -convenient. When you deem it necessary, feel free to ask for it on the -mailing list. When you get commit access, please make sure to follow the -policy below (discussions of the policy can take place on guix-devel@gnu.org.) - -Non-trivial patches should always be posted to guix-patches@gnu.org (trivial -patches include fixing typos, etc.). This mailing list fills the -patch-tracking database at [[https://bugs.gnu.org/guix-patches]]; see -"Contributing" in the manual for details. - -For patches that just add a new package, and a simple one, it’s OK to commit, -if you’re confident (which means you successfully built it in a chroot setup, -and have done a reasonable copyright and license auditing.) Likewise for -package upgrades, except upgrades that trigger a lot of rebuilds (for example, -upgrading GnuTLS or GLib.) We have a mailing list for commit notifications -(guix-commits@gnu.org), so people can notice. Before pushing your changes, -make sure to run ‘git pull --rebase’. - -All commits that are pushed to the central repository on Savannah must be -signed with an OpenPGP key, and the public key should be uploaded to your user -account on Savannah and to public key servers, such as -‘keys.openpgp.org’. To configure Git to automatically sign commits, -run: - - git config commit.gpgsign true - git config user.signingkey CABBA6EA1DC0FF33 - -You can prevent yourself from accidentally pushing unsigned commits to -Savannah by using the pre-push Git hook called located at ‘etc/git/pre-push’: - - cp etc/git/pre-push .git/hooks/pre-push - -When pushing a commit on behalf of somebody else, please add a Signed-off-by -line at the end of the commit log message (e.g. with ‘git am --signoff’). -This improves tracking of who did what. - -For anything else, please post to guix-patches@gnu.org and leave time for a -review, without committing anything. If you didn’t receive any reply -after two weeks, and if you’re confident, it’s OK to commit. - -That last part is subject to being adjusted, allowing individuals to commit -directly on non-controversial changes on parts they’re familiar with. +or by checking the [[https://guix.gnu.org/manual/devel/en/html_node/Contributing.html][web copy of the manual]]. diff --git a/doc/contributing.texi b/doc/contributing.texi index fd27f037e9..e6e6ab36c2 100644 --- a/doc/contributing.texi +++ b/doc/contributing.texi @@ -27,6 +27,7 @@ choice. * Coding Style:: Hygiene of the contributor. * Submitting Patches:: Share your work. * Tracking Bugs and Patches:: Using Debbugs. +* Commit Access:: Pushing to the official repository. @end menu @node Building from Git @@ -827,6 +828,8 @@ Development is done using the Git distributed version control system. Thus, access to the repository is not strictly necessary. We welcome contributions in the form of patches as produced by @code{git format-patch} sent to the @email{guix-patches@@gnu.org} mailing list. +Seasoned Guix developers may also want to look at the section on commit +access (@pxref{Commit Access}). This mailing list is backed by a Debbugs instance, which allows us to keep track of submissions (@pxref{Tracking Bugs and Patches}). Each @@ -1090,3 +1093,59 @@ For example, to list all open issues on @code{guix-patches}, hit: @xref{Top,,, debbugs-ug, Debbugs User Guide}, for more information on this nifty tool! + +@node Commit Access +@section Commit Access + +@cindex commit access, for developers +For frequent contributors, having write access to the repository is +convenient. When you deem it necessary, feel free to ask for it on the +mailing list. When you get commit access, please make sure to follow +the policy below (discussions of the policy can take place on +@email{guix-devel@@gnu.org}). + +Non-trivial patches should always be posted to +@email{guix-patches@@gnu.org} (trivial patches include fixing typos, +etc.). This mailing list fills the patch-tracking database +(@pxref{Tracking Bugs and Patches}). + +For patches that just add a new package, and a simple one, it's OK to +commit, if you're confident (which means you successfully built it in a +chroot setup, and have done a reasonable copyright and license +auditing). Likewise for package upgrades, except upgrades that trigger +a lot of rebuilds (for example, upgrading GnuTLS or GLib). We have a +mailing list for commit notifications (@email{guix-commits@@gnu.org}), +so people can notice. Before pushing your changes, make sure to run +@code{git pull --rebase}. + +All commits that are pushed to the central repository on Savannah must +be signed with an OpenPGP key, and the public key should be uploaded to +your user account on Savannah and to public key servers, such as +@code{keys.openpgp.org}. To configure Git to automatically sign +commits, run: + +@example +git config commit.gpgsign true +git config user.signingkey CABBA6EA1DC0FF33 +@end example + +You can prevent yourself from accidentally pushing unsigned commits to +Savannah by using the pre-push Git hook called located at +@file{etc/git/pre-push}: + +@example +cp etc/git/pre-push .git/hooks/pre-push +@end example + +When pushing a commit on behalf of somebody else, please add a +@code{Signed-off-by} line at the end of the commit log message---e.g., +with @command{git am --signoff}. This improves tracking of who did +what. + +For anything else, please post to @email{guix-patches@@gnu.org} and +leave time for a review, without committing anything (@pxref{Submitting +Patches}). If you didn’t receive any reply after two weeks, and if +you're confident, it's OK to commit. + +That last part is subject to being adjusted, allowing individuals to commit +directly on non-controversial changes on parts they’re familiar with. From patchwork Wed Jan 1 16:34:45 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Ludovic_Court=C3=A8s?= X-Patchwork-Id: 19505 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id C16E9179D5; Wed, 1 Jan 2020 16:36:20 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id 81EBD179D3 for ; Wed, 1 Jan 2020 16:36:20 +0000 (GMT) Received: from localhost ([::1]:59846 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgyq-0001Mx-24 for patchwork@mira.cbaines.net; Wed, 01 Jan 2020 11:36:20 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:34426) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgyd-00018S-Vy for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:08 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1imgyY-00065P-Os for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:07 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:59742) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1imgyY-000659-MI for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:02 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1imgyY-0002E3-Kn for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:02 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#38846] [PATCH 3/4] doc: Encourage patch review. Resent-From: Ludovic =?utf-8?q?Court=C3=A8s?= Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Wed, 01 Jan 2020 16:36:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 38846 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 38846@debbugs.gnu.org Cc: Ludovic =?utf-8?q?Court=C3=A8s?= , guix-maintainers@gnu.org Received: via spool by 38846-submit@debbugs.gnu.org id=B38846.15778965058445 (code B ref 38846); Wed, 01 Jan 2020 16:36:02 +0000 Received: (at 38846) by debbugs.gnu.org; 1 Jan 2020 16:35:05 +0000 Received: from localhost ([127.0.0.1]:37479 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1imgxd-0002C2-Fl for submit@debbugs.gnu.org; Wed, 01 Jan 2020 11:35:05 -0500 Received: from eggs.gnu.org ([209.51.188.92]:51707) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1imgxY-0002B3-9o for 38846@debbugs.gnu.org; Wed, 01 Jan 2020 11:35:00 -0500 Received: from fencepost.gnu.org ([2001:470:142:3::e]:38144) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1imgxT-0003Yu-4i; Wed, 01 Jan 2020 11:34:55 -0500 Received: from [2a01:e0a:1d:7270:af76:b9b:ca24:c465] (port=47372 helo=gnu.org) by fencepost.gnu.org with esmtpsa (TLS1.2:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1imgxS-00045M-Jp; Wed, 01 Jan 2020 11:34:54 -0500 From: Ludovic =?utf-8?q?Court=C3=A8s?= Date: Wed, 1 Jan 2020 17:34:45 +0100 Message-Id: <20200101163446.5132-3-ludo@gnu.org> X-Mailer: git-send-email 2.24.1 In-Reply-To: <20200101163446.5132-1-ludo@gnu.org> References: <20200101163446.5132-1-ludo@gnu.org> MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches * doc/contributing.texi (Commit Access): Add note about patch review. --- doc/contributing.texi | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/doc/contributing.texi b/doc/contributing.texi index e6e6ab36c2..74b4718eb3 100644 --- a/doc/contributing.texi +++ b/doc/contributing.texi @@ -1149,3 +1149,9 @@ you're confident, it's OK to commit. That last part is subject to being adjusted, allowing individuals to commit directly on non-controversial changes on parts they’re familiar with. + +One last thing: the project keeps moving forward because committers not +only push their own awesome changes, but also offer some of their time +@emph{reviewing} and pushing other people's changes. As a committer, +you're welcome to use your expertise and commit rights to help other +contributors, too! From patchwork Wed Jan 1 16:34:46 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: =?utf-8?q?Ludovic_Court=C3=A8s?= X-Patchwork-Id: 19504 Return-Path: X-Original-To: patchwork@mira.cbaines.net Delivered-To: patchwork@mira.cbaines.net Received: by mira.cbaines.net (Postfix, from userid 113) id 0661F179D5; Wed, 1 Jan 2020 16:36:16 +0000 (GMT) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mira.cbaines.net X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mira.cbaines.net (Postfix) with ESMTP id B6431179D3 for ; Wed, 1 Jan 2020 16:36:15 +0000 (GMT) Received: from localhost ([::1]:59842 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgyl-0001B8-1V for patchwork@mira.cbaines.net; Wed, 01 Jan 2020 11:36:15 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:34386) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1imgyc-00017o-K9 for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:07 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1imgya-00067T-6A for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:06 -0500 Received: from debbugs.gnu.org ([209.51.188.43]:59744) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1imgya-000675-3B for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:04 -0500 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1imgyZ-0002EC-1I for guix-patches@gnu.org; Wed, 01 Jan 2020 11:36:03 -0500 X-Loop: help-debbugs@gnu.org Subject: [bug#38846] [PATCH 4/4] DRAFT doc: Add a cooption policy for commit access. Resent-From: Ludovic =?utf-8?q?Court=C3=A8s?= Original-Sender: "Debbugs-submit" Resent-CC: guix-patches@gnu.org Resent-Date: Wed, 01 Jan 2020 16:36:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 38846 X-GNU-PR-Package: guix-patches X-GNU-PR-Keywords: patch To: 38846@debbugs.gnu.org Cc: Ludovic =?utf-8?q?Court=C3=A8s?= , guix-maintainers@gnu.org Received: via spool by 38846-submit@debbugs.gnu.org id=B38846.15778965068451 (code B ref 38846); Wed, 01 Jan 2020 16:36:02 +0000 Received: (at 38846) by debbugs.gnu.org; 1 Jan 2020 16:35:06 +0000 Received: from localhost ([127.0.0.1]:37481 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1imgxd-0002CA-Ov for submit@debbugs.gnu.org; Wed, 01 Jan 2020 11:35:06 -0500 Received: from eggs.gnu.org ([209.51.188.92]:51737) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1imgxZ-0002B5-Js for 38846@debbugs.gnu.org; Wed, 01 Jan 2020 11:35:01 -0500 Received: from fencepost.gnu.org ([2001:470:142:3::e]:38145) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1imgxU-0003cC-EV; Wed, 01 Jan 2020 11:34:56 -0500 Received: from [2a01:e0a:1d:7270:af76:b9b:ca24:c465] (port=47372 helo=gnu.org) by fencepost.gnu.org with esmtpsa (TLS1.2:DHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.82) (envelope-from ) id 1imgxT-00045M-JQ; Wed, 01 Jan 2020 11:34:56 -0500 From: Ludovic =?utf-8?q?Court=C3=A8s?= Date: Wed, 1 Jan 2020 17:34:46 +0100 Message-Id: <20200101163446.5132-4-ludo@gnu.org> X-Mailer: git-send-email 2.24.1 In-Reply-To: <20200101163446.5132-1-ludo@gnu.org> References: <20200101163446.5132-1-ludo@gnu.org> MIME-Version: 1.0 X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: guix-patches@gnu.org List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+patchwork=mira.cbaines.net@gnu.org Sender: "Guix-patches" X-getmail-retrieved-from-mailbox: Patches DRAFT: Subject to discussion! * doc/contributing.texi (Commit Access): Draft a cooptation policy. --- doc/contributing.texi | 48 +++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 46 insertions(+), 2 deletions(-) diff --git a/doc/contributing.texi b/doc/contributing.texi index 74b4718eb3..6e2cc1a369 100644 --- a/doc/contributing.texi +++ b/doc/contributing.texi @@ -1099,8 +1099,52 @@ this nifty tool! @cindex commit access, for developers For frequent contributors, having write access to the repository is -convenient. When you deem it necessary, feel free to ask for it on the -mailing list. When you get commit access, please make sure to follow +convenient. When you deem it necessary, consider applying for commit +access by following these steps: + +@enumerate +@item +Find three committers who would vouch for you, emailing a signed +statement to @email{guix-maintainers@@gnu.org} (a private alias for the +collective of maintainers). You can view the list of committers at +@url{https://savannah.gnu.org/project/memberlist.php?group=guix}. + +Committers are expected to have had some interactions with you as a +contributor and to be able to judge whether you are sufficiently +familiar with the project's practices. It is @emph{not} a judgment on +the quality of your work, so a refusal should rather be interpreted as +``let's try again later''. + +@item +Send @email{guix-maintainers@@gnu.org} a signed message stating your +intent, listing the three committers who support your application, and +giving the fingerprint of the OpenPGP key you will use to sign commits +(see below). + +@item +Once you've been given access, please send a message to +@email{guix-devel@@gnu.org} to say so, again signed with the OpenPGP key +you will use to sign commits. That way, everyone can notice and ensure +you control that OpenPGP key. + +@c TODO: Add note about adding the fingerprint to the list of authorized +@c keys once that has stabilized. + +@item +Make sure to read the rest of this section and... profit! +@end enumerate + +@quotation Note +Maintainers are happy to give commit access to people who have been +contributing for some time and have a track record---don't be shy and +don't underestimate your work! + +However, note that the project is working towards a more automated patch +review and merging system, which, as a consequence, may lead us to have +fewer people with commit access to the main repository. Stay tuned! +@end quotation + +When you get commit access, please make sure to follow the policy below (discussions of the policy can take place on @email{guix-devel@@gnu.org}).