[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] SMBIOS (Set of 10 patches)
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] SMBIOS (Set of 10 patches) |
Date: |
Mon, 10 Mar 2014 13:14:28 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0 |
On 03/10/2014 12:17 PM, Gabriel L. Somlo wrote:
>> but instead create 1 top-level thread with all patches under the cover
>> letter:
>>
>> 0/10
>> |- 1/10
>> |- 2/10
>> ...
>> |- 10/10
>
> OK, so I screwed up, now what ? Should I re-send the 10 patches
> in-reply-to the existing cover letter, or leave it be for now to
> avoid further spamming the list ?
Don't stress too hard - we were all new once :)
Personally, I'd wait a day or two for any review comments, then send a
v2 properly threaded with those comments addressed.
You might also find it helpful to read
http://wiki.qemu.org/Contribute/SubmitAPatch (and if we are missing
something there, let us know how to improvement - newcomers are the BEST
sources of information on how well a page designed to help newcomers
actually does its job).
>
>> 'qemu send-email -10 --annotate --cover-letter' defaults to the proper
>> threading, along with letting you modify your cover letter in your
>> editor before actually sending.
>
> That sounds awesome, but there's no "send-email" script under my
> qemu/scripts folder (using latest git).
Upstream git.git has included send-email for years; but many distros
ship it as a separate package because of the additional dependencies it
drags in. On Fedora systems, it is part of the 'git-email' package.
Hmm - something to add to the SubmitAPatch wiki page - we recommend
send-email without saying where to get it on common distros :)
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature