From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7A165D6204E for ; Tue, 19 Nov 2024 09:13:13 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tDKI0-0003Sv-So; Tue, 19 Nov 2024 04:12:53 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tDKHv-0003S4-95 for qemu-devel@nongnu.org; Tue, 19 Nov 2024 04:12:47 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tDKHr-00080E-Nt for qemu-devel@nongnu.org; Tue, 19 Nov 2024 04:12:47 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732007562; h=from:from:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=axxdCSVMVXPIgO9acZUtuxAKuCPl+sQtmavnQTTcXaQ=; b=UgPnqecTizX9jIjenXNcrivuRp4mp92sghET1xwKnM+vu3EZVmWGwajzXuE7PDyeaMn/aF ZLv8RqkVgBCnYwqse+yMgvD598CzINAq49v1Kp3Ab9lq1H84fUuYD2bg9G+atFrbfrC7G9 Lnn4/67dSbEXoofnlgBpwGCV/j8bATI= Received: from mx-prod-mc-03.mail-002.prod.us-west-2.aws.redhat.com (ec2-54-186-198-63.us-west-2.compute.amazonaws.com [54.186.198.63]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-32-37ONWuuyO_aEgJF3o4xcqg-1; Tue, 19 Nov 2024 04:12:38 -0500 X-MC-Unique: 37ONWuuyO_aEgJF3o4xcqg-1 X-Mimecast-MFC-AGG-ID: 37ONWuuyO_aEgJF3o4xcqg Received: from mx-prod-int-04.mail-002.prod.us-west-2.aws.redhat.com (mx-prod-int-04.mail-002.prod.us-west-2.aws.redhat.com [10.30.177.40]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mx-prod-mc-03.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id E39DB19560B8; Tue, 19 Nov 2024 09:12:35 +0000 (UTC) Received: from redhat.com (unknown [10.42.28.110]) by mx-prod-int-04.mail-002.prod.us-west-2.aws.redhat.com (Postfix) with ESMTPS id 21DE11956054; Tue, 19 Nov 2024 09:12:27 +0000 (UTC) Date: Tue, 19 Nov 2024 09:12:24 +0000 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= To: Marcin Juszkiewicz Cc: Pierrick Bouvier , qemu-devel@nongnu.org, Philippe =?utf-8?Q?Mathieu-Daud=C3=A9?= , Richard Henderson , Andrew Melnychenko , Jason Wang , Thomas Huth , alex.bennee@linaro.org, Vladimir Sementsov-Ogievskiy , Fabiano Rosas , Kevin Wolf , Markus Armbruster , Eric Blake , qemu-arm@nongnu.org, Yuri Benditovich , manos.pitsidianakis@linaro.org, qemu-block@nongnu.org, Michael Roth , Konstantin Kostiuk , Paolo Bonzini , Peter Xu , gustavo.romero@linaro.org, Peter Maydell Subject: Re: [PATCH 2/7] docs/devel: add git-publish for patch submitting Message-ID: References: <20241118172357.475281-1-pierrick.bouvier@linaro.org> <20241118172357.475281-3-pierrick.bouvier@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/2.2.12 (2023-09-09) X-Scanned-By: MIMEDefang 3.0 on 10.30.177.40 Received-SPF: pass client-ip=170.10.133.124; envelope-from=berrange@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -21 X-Spam_score: -2.2 X-Spam_bar: -- X-Spam_report: (-2.2 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.142, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_VALIDITY_CERTIFIED_BLOCKED=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org On Tue, Nov 19, 2024 at 09:41:40AM +0100, Marcin Juszkiewicz wrote: > W dniu 18.11.2024 o 18:23, Pierrick Bouvier pisze: > > Signed-off-by: Pierrick Bouvier > > --- > > docs/devel/submitting-a-patch.rst | 14 ++++++++++++++ > > 1 file changed, 14 insertions(+) > > > > diff --git a/docs/devel/submitting-a-patch.rst b/docs/devel/submitting-a-patch.rst > > index 349c32ee3a9..953682f20cb 100644 > > --- a/docs/devel/submitting-a-patch.rst > > +++ b/docs/devel/submitting-a-patch.rst > > @@ -237,6 +237,20 @@ attachments can be used as a last resort on a first-time submission. > > .. _if_you_cannot_send_patch_emails: > > +Use git-publish > > +~~~~~~~~~~~~~~~ > > + > > +If you already configured git send-email, you can simply use `git-publish > > +`__ to send series. > > + > > +:: > > + > > + $ git checkout master -b my-feature > > + $ # work on new commits, add your 'Signed-off-by' lines to each > > + $ git publish > > + $ ... more work, rebase on master, ... > > + $ git publish # will send a v2 > > You recommend 'b4 shazam' in 3/7 patch so why not here? Both 'b4' and > 'git-publish' seem to do same stuff - handle patch series and send them > upstream. git-publish is what we already recommended to people both elsewhere in this file, and more prominently in the README.rst file, so maintaining that practice is correct. > b4 allows to keep To/Cc emails inside of cover letter which makes it easy to > not miss anyone needed. git-publish automatically CC's people by correlating the files touched in the commits against our MAINTAINERS file, so it "does the right thing" in the majority of cases. We recommend it because it makes it much harder for novice users to mess up patch series submission. With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|