qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: On patch series version tags, and also cover letters (was: [PATCH v2


From: Kshitij Suri
Subject: Re: On patch series version tags, and also cover letters (was: [PATCH v2 2/2] Added parameter to take screenshot with screendump as PNG)
Date: Tue, 29 Mar 2022 15:10:11 +0530
User-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.7.0


On 29/03/22 2:31 pm, Daniel P. Berrangé wrote:
On Tue, Mar 29, 2022 at 09:59:55AM +0200, Markus Armbruster wrote:
Kshitij Suri <kshitij.suri@nutanix.com> writes:

On 29/03/22 12:12 pm, Markus Armbruster wrote:
If I count correctly, this is the fifth posting tagged "v2".  Don't do
that, please, as it's quite confusing.

Thank you for your review and I apologise for that since I am fairly
new to upstreaming. As per what I read version updates should be done
only when there are significant design changes to the patch which
didn't happen in the v2 version. Will update it to v3 and send the
patch.
We all make mistakes :)

The purpose of the version tag in the subject is to help humans with
keeping track of patch submissions.  Increment it for every submission.

If you need to resend a submission completely unchanged for some reason,
you may want to keep the tag and add "RESEND".

A cover letter (git format-patch --cover-letter) lets you write an
introduction to the whole series.  Simple series may not need an
introduction, but complex ones do.  I always use one except when the
"series" is a single patch.

Keeping a change log in the cover letter helps people who already
reviewed previous iterations.
FYI, using the 'git-publish' tool instead of 'git send-email' or
'git format-patch' helps you do all these things. It automatically
sets & increments the version number, it prompts for a cover letter
and remembers what you wrote next time.

With regards,
Daniel
Oh didn't know about it, will try to use git-publish from now on. Thank you for your guidance for support throughout the issue!


Regards,
Kshitij Suri



reply via email to

[Prev in Thread] Current Thread [Next in Thread]