[libvirt] [PATCH] libvirt: virsh: Kill all uses of __FUNCTION__ in error messages
Noella Ashu
ashu.noella207 at gmail.com
Fri Apr 3 10:04:37 UTC 2015
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 at 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.
> 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
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150403/f27b3f65/attachment-0001.htm>
More information about the libvir-list
mailing list