Hello +Michal,I will make the corrections and re send the patch. Sorry for the errors.Thanks,NoellaOn Fri, Apr 3, 2015 at 10:19 AM, Michal Privoznik <mprivozn redhat com> 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.
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:105: trailing whitespace.
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:118: trailing whitespace.
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:233: trailing whitespace.
/home/zippy/work/libvirt/libvirt.git/.git/rebase-apply/patch:309: trailing whitespace.
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:
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.