<div dir="ltr">Hello +Michal,<div><br></div><div>I have sent a new patch to the mailing list with the whitespaces removed. This should work fine now. Please just let me know if you have any other issues.</div><div><br></div><div>Thanks,</div><div>Noella</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 3, 2015 at 11:04 AM, Noella Ashu <span dir="ltr"><<a href="mailto:ashu.noella207@gmail.com" target="_blank">ashu.noella207@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello +Michal,<div><br></div><div>I will make the corrections and re send the patch. Sorry for the errors.</div><div><br></div><div>Thanks,</div><div>Noella</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 3, 2015 at 10:19 AM, Michal Privoznik <span dir="ltr"><<a href="mailto:mprivozn@redhat.com" target="_blank">mprivozn@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 01.04.2015 14:46, Noella Ashu wrote:<br>
> The error output of snapshot-revert should be more friendly. There is no<br>
> need to show up virDomainRevertToSnapshot to user. virError already includes<br>
>  __FUNCTION__ information in a separate member of the struct, so repeating<br>
> it in the message is redundant and leads to situations where higher level<br>
> code ends up reporting the lower level name We correctly converted the<br>
>  error output making it more succinct and user-friendly.<br>
><br>
> Resolves: <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1086726" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1086726</a><br>
> ---<br>
>  src/libvirt-domain-snapshot.c |  30 +++----<br>
>  src/libvirt-domain.c          | 201 ++++++++++++++++++------------------------<br>
>  2 files changed, 96 insertions(+), 135 deletions(-)<br>
<br>
</span>I'm having some difficulties applying this patch:<br>
<br>
Applying: libvirt: virsh: Kill all uses of __FUNCTION__ in error messages<br>
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:91: trailing whitespace.<br>
        virReportInvalidArg(stream, "%s",<br>
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:105: trailing whitespace.<br>
        virReportInvalidArg(flags, "%s",<br>
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:118: trailing whitespace.<br>
        virReportInvalidArg(flags, "%s",<br>
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:233: trailing whitespace.<br>
        virReportInvalidArg(conn, "%s",<br>
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:309: trailing whitespace.<br>
        virReportInvalidArg(nkeycodes, "%s",<br>
error: patch failed: src/libvirt-domain.c:11284<br>
error: src/libvirt-domain.c: patch does not apply<br>
Patch failed at 0001 libvirt: virsh: Kill all uses of __FUNCTION__ in error messages<br>
The copy of the patch that failed is found in:<br>
   /home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch<br>
When you have resolved this problem, run "git am --continue".<br>
If you prefer to skip this patch, run "git am --skip" instead.<br>
To restore the original branch and stop patching, run "git am --abort".<br>
<br>
<br>
Can you please rebase and resend? Oh, and don't forget to run 'make syntax-check check' before sending a patch.<br>
<span><font color="#888888"><br>
Michal<br>
</font></span></blockquote></div><br></div>
</div></div></blockquote></div><br></div>