Hello +Michal,
I will make the corrections and re send the patch. Sorry for the errors.
Thanks,
Noella
On Fri, Apr 3, 2015 at 10:19 AM, Michal Privoznik <mprivozn@xxxxxxxxxx> wrote:
On 01.04.2015 14:46, Noella Ashu wrote:
> The error output of snapshot-revert should be more friendly. There is no
> need to show up virDomainRevertToSnapshot to user. virError already includes
> __FUNCTION__ information in a separate member of the struct, so repeating
> it in the message is redundant and leads to situations where higher level
> code ends up reporting the lower level name We correctly converted the
> error output making it more succinct and user-friendly.
>
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1086726
> ---
> src/libvirt-domain-snapshot.c | 30 +++----
> src/libvirt-domain.c | 201 ++++++++++++++++++------------------------
> 2 files changed, 96 insertions(+), 135 deletions(-)
I'm having some difficulties applying this patch:
Applying: libvirt: virsh: Kill all uses of __FUNCTION__ in error messages
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:91: trailing whitespace.
virReportInvalidArg(stream, "%s",
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:105: trailing whitespace.
virReportInvalidArg(flags, "%s",
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:118: trailing whitespace.
virReportInvalidArg(flags, "%s",
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:233: trailing whitespace.
virReportInvalidArg(conn, "%s",
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:309: trailing whitespace.
virReportInvalidArg(nkeycodes, "%s",
error: patch failed: src/libvirt-domain.c:11284
error: src/libvirt-domain.c: patch does not apply
Patch failed at 0001 libvirt: virsh: Kill all uses of __FUNCTION__ in error messages
The copy of the patch that failed is found in:
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Can you please rebase and resend? Oh, and don't forget to run 'make syntax-check check' before sending a patch.
Michal
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list