Thanks Michal,
I will look at another introductory bug to work on. Any suggestions will
help.
Thanks,
Noella
On Fri, Apr 3, 2015 at 1:13 PM, Michal Privoznik mpriv...@redhat.com
wrote:
On 03.04.2015 12:33, Noella Ashu wrote:
The error output of snapshot-revert should be more friendly. There
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
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
On 03.04.2015 12:33, 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
/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
(-)
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
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