[libvirt] [PATCH] Sheepdog: Auto Adding volume and correct refresh volume problem. Resent file

Eric Blake eblake at redhat.com
Mon Jan 27 14:23:09 UTC 2014

On 01/27/2014 04:11 AM, joel SIMOES wrote:
> From: Joel SIMOES <joel.simoes at laposte.net>

Your title says 'Re: ...', but there is no In-reply-to heading that
states what you were replying to.  Furthermore, including the text
'Resent file' in the subject line looks awkward when used as a git
commit (look at 'git shortlog -30' for an idea of good subject lines,
and note that none of them mention 'Resent file' in the text).

If this is a second version of the patch, it is better to use 'git
send-email --subject-prefix=PATCHv2' to make it clear that it is a
resend, rather than munging the subject line by hand outside of the []

> ---

Also, your commit message body is still empty.  Remember, the subject
line should be brief (the WHAT), the commit message body before the ---
should not be empty (the WHY), and the space after the --- is useful for
reviewer's notes (HOW this differs from v1).

> +++ b/src/storage/storage_backend_sheepdog.c
> @@ -86,7 +86,8 @@ virStorageBackendSheepdogParseNodeInfo(virStoragePoolDefPtr pool,
>          pool->available = pool->capacity - pool->allocation;
>          return 0;
> -    } while ((p = next));
> +    }
> +    while ((p = next));

This hunk is still spurious.  I pointed it out last time, and it is
rather annoying to review patches where items pointed out previously
aren't addressed.

Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140127/d00d06bb/attachment-0001.sig>

More information about the libvir-list mailing list