From nobody Tue Feb 10 01:19:12 2026 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of redhat.com designates 207.211.31.81 as permitted sender) client-ip=207.211.31.81; envelope-from=libvir-list-bounces@redhat.com; helo=us-smtp-delivery-1.mimecast.com; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 207.211.31.81 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1586190058; cv=none; d=zohomail.com; s=zohoarc; b=lIDmtZnuJTR6EPZFBM5E/YaTRbqtJL+KQWpZUPuzoG1gzvZh1JUk9bRebJ5VNKqMlkZ449kHWxaYzyQW/bxa8bO5GjKuQHl3uA46qHvfF8J9nTLQhNYllJ9lvWhnQRIKlxBO+ilflt3KDDF9t2+tu3SO4QXQz+yMItJZOqKNG+0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1586190058; h=Content-Type:Content-Transfer-Encoding:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=UdFziM+ST8ms98nwcPbrEpHcNc6pOt7B27KgtPs6mqk=; b=UCHmLN1BZMFMD+Y3tOKcLgjaCMfcTQSBK6WiSYonu1wrX0398cUjbC0+UWs7eq2UcvfMhBK6dj9l5L2iTIHnpQaYaTQ4HnweULJD+UgdNN92e3+YM6yDOzzRo5JTKDc1u878nCSleoXZwU8wooe2QiC0nyOGqsMYx6RfUvNktNE= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of redhat.com designates 207.211.31.81 as permitted sender) smtp.mailfrom=libvir-list-bounces@redhat.com; dmarc=pass header.from= (p=none dis=none) header.from= Return-Path: Received: from us-smtp-delivery-1.mimecast.com (us-smtp-2.mimecast.com [207.211.31.81]) by mx.zohomail.com with SMTPS id 158619005809666.74707696495818; Mon, 6 Apr 2020 09:20:58 -0700 (PDT) Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-407-wDH8A1R6Pp2rIorpzwBpNg-1; Mon, 06 Apr 2020 12:20:49 -0400 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id C70A1DB61; Mon, 6 Apr 2020 16:20:43 +0000 (UTC) Received: from colo-mx.corp.redhat.com (colo-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.21]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 97C8B6EF97; Mon, 6 Apr 2020 16:20:43 +0000 (UTC) Received: from lists01.pubmisc.prod.ext.phx2.redhat.com (lists01.pubmisc.prod.ext.phx2.redhat.com [10.5.19.33]) by colo-mx.corp.redhat.com (Postfix) with ESMTP id 3EDD393A76; Mon, 6 Apr 2020 16:20:43 +0000 (UTC) Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) by lists01.pubmisc.prod.ext.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id 036GKTnQ027933 for ; Mon, 6 Apr 2020 12:20:29 -0400 Received: by smtp.corp.redhat.com (Postfix) id 8EE435417F; Mon, 6 Apr 2020 16:20:29 +0000 (UTC) Received: from kinshicho.usersys.redhat.com (unknown [10.40.196.71]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 80C2AB19D6 for ; Mon, 6 Apr 2020 16:20:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1586190057; h=from:from:sender:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:list-id:list-help: list-unsubscribe:list-subscribe:list-post; bh=UdFziM+ST8ms98nwcPbrEpHcNc6pOt7B27KgtPs6mqk=; b=MngZKz6GzRd4FnoMJCI4+qVVCUV9MCKweR+LEOiNIImWeUQQ8ZxknRgII0x/WxA9n6CIuK Bv1WVh692IUXwZbbtMgSzjkGez276QJw0jDD3EycnFAaymzp/8UdwracgMw0F3HryXyOov IcHJQ2z0gh1KyMaoLC//6pik3qBgND8= X-MC-Unique: wDH8A1R6Pp2rIorpzwBpNg-1 From: Andrea Bolognani To: libvir-list@redhat.com Subject: [libvirt PATCH 06/10] docs: Add committer-guidelines.rst Date: Mon, 6 Apr 2020 18:20:06 +0200 Message-Id: <20200406162010.332813-7-abologna@redhat.com> In-Reply-To: <20200406162010.332813-1-abologna@redhat.com> References: <20200406162010.332813-1-abologna@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-loop: libvir-list@redhat.com X-BeenThere: libvir-list@redhat.com X-Mailman-Version: 2.1.12 Precedence: junk List-Id: Development discussions about the libvirt library & tools List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Transfer-Encoding: quoted-printable X-ZohoMail-DKIM: pass (identity @redhat.com) Content-Type: text/plain; charset="utf-8" While it's good to have these rules written down for reference, they apply exclusively to committers, who by definition are familiar with the project and probably work on it daily, so there's no need to have them front and center when a separate page will do. Signed-off-by: Andrea Bolognani --- docs/committer-guidelines.rst | 33 +++++++++++++++++++++++++++++++++ docs/hacking.rst | 33 --------------------------------- 2 files changed, 33 insertions(+), 33 deletions(-) create mode 100644 docs/committer-guidelines.rst diff --git a/docs/committer-guidelines.rst b/docs/committer-guidelines.rst new file mode 100644 index 0000000000..3be08e6fed --- /dev/null +++ b/docs/committer-guidelines.rst @@ -0,0 +1,33 @@ +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D +Committer guidelines +=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D + +The AUTHORS files indicates the list of people with commit access +right who can actually merge the patches. + +The general rule for committing a patch is to make sure it has +been reviewed properly in the mailing-list first, usually if a +couple of people gave an ACK or +1 to a patch and nobody raised an +objection on the list it should be good to go. If the patch +touches a part of the code where you're not the main maintainer, +or where you do not have a very clear idea of how things work, +it's better to wait for a more authoritative feedback though. +Before committing, please also rebuild locally, run 'make check +syntax-check', and make sure you don't raise errors. + +An exception to 'review and approval on the list first' is fixing +failures to build: + +- if a recently committed patch breaks compilation on a platform + or for a given driver, then it's fine to commit a minimal fix + directly without getting the review feedback first +- if make check or make syntax-check breaks, if there is an + obvious fix, it's fine to commit immediately. The patch should + still be sent to the list (or tell what the fix was if + trivial), and 'make check syntax-check' should pass too, before + committing anything +- fixes for documentation and code comments can be managed in the + same way, but still make sure they get reviewed if non-trivial. +- (ir)regular pulls from other repositories or automated updates, + such as the keycodemap submodule updates, pulling in new + translations or updating the container images for the CI system diff --git a/docs/hacking.rst b/docs/hacking.rst index 35c246bf26..39e303c103 100644 --- a/docs/hacking.rst +++ b/docs/hacking.rst @@ -321,36 +321,3 @@ There is more on this subject, including lots of links= to background reading on the subject, on `Richard Jones' guide to working with open source projects `__. - -Libvirt committer guidelines -=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D - -The AUTHORS files indicates the list of people with commit access -right who can actually merge the patches. - -The general rule for committing a patch is to make sure it has -been reviewed properly in the mailing-list first, usually if a -couple of people gave an ACK or +1 to a patch and nobody raised an -objection on the list it should be good to go. If the patch -touches a part of the code where you're not the main maintainer, -or where you do not have a very clear idea of how things work, -it's better to wait for a more authoritative feedback though. -Before committing, please also rebuild locally, run 'make check -syntax-check', and make sure you don't raise errors. - -An exception to 'review and approval on the list first' is fixing -failures to build: - -- if a recently committed patch breaks compilation on a platform - or for a given driver, then it's fine to commit a minimal fix - directly without getting the review feedback first -- if make check or make syntax-check breaks, if there is an - obvious fix, it's fine to commit immediately. The patch should - still be sent to the list (or tell what the fix was if - trivial), and 'make check syntax-check' should pass too, before - committing anything -- fixes for documentation and code comments can be managed in the - same way, but still make sure they get reviewed if non-trivial. -- (ir)regular pulls from other repositories or automated updates, - such as the keycodemap submodule updates, pulling in new - translations or updating the container images for the CI system --=20 2.25.1