Re: [libvirt] [PATCH 00/13] Move generic virsh data to a separate module vsh

2015-08-04 Thread Martin Kletzander
[getting back to this after *loong* time, sorry] On Fri, Jul 17, 2015 at 02:30:20PM +0200, Erik Skultety wrote: Actually, I found out this is easier to review as a one patch with various diff options used for various parts of the patch. Some questions and suggestions below. Why vshClientHooks

Re: [libvirt] [PATCH 00/13] Move generic virsh data to a separate module vsh

2015-07-17 Thread Erik Skultety
> Actually, I found out this is easier to review as a one patch with > various diff options used for various parts of the patch. Some > questions and suggestions below. > > Why vshClientHooks and vshCmdGrp aren't in the vshControl struct? If > we move the client helpers into a library (I think t

Re: [libvirt] [PATCH 00/13] Move generic virsh data to a separate module vsh

2015-07-13 Thread Martin Kletzander
On Mon, Jun 29, 2015 at 05:37:34PM +0200, Erik Skultety wrote: The idea behind this is that in order to introduce virt-admin client (and later some commands/APIs), there are lots of methods in virsh that can be easily reused by other potential clients like command and command argument passing or

Re: [libvirt] [PATCH 00/13] Move generic virsh data to a separate module vsh

2015-07-01 Thread Erik Skultety
>>> >> >> This is rather big change. Do you perhaps have a public repo with all >> the patches applied? >> > > I've also noticed 03/13 is missing. > > Michal > I do have it in my inbox (I always CC myself when sending patches), I've also sent another copy earlier today... still nothing. I have

Re: [libvirt] [PATCH 00/13] Move generic virsh data to a separate module vsh

2015-07-01 Thread Erik Skultety
> > This is rather big change. Do you perhaps have a public repo with all > the patches applied? > > Michal I forked libvirt mirror repo on github and pushed as my local branch virt-shell https://github.com/eskultety/libvirt/tree/virt-shell. Erik -- libvir-list mailing list libvir-list@redhat.

Re: [libvirt] [PATCH 00/13] Move generic virsh data to a separate module vsh

2015-06-30 Thread Michal Privoznik
On 30.06.2015 16:12, Michal Privoznik wrote: > On 29.06.2015 17:37, Erik Skultety wrote: >> The idea behind this is that in order to introduce virt-admin client (and >> later >> some commands/APIs), there are lots of methods in virsh that can be easily >> reused by other potential clients like com

Re: [libvirt] [PATCH 00/13] Move generic virsh data to a separate module vsh

2015-06-30 Thread Michal Privoznik
On 29.06.2015 17:37, Erik Skultety wrote: > The idea behind this is that in order to introduce virt-admin client (and > later > some commands/APIs), there are lots of methods in virsh that can be easily > reused by other potential clients like command and command argument passing or > error report

[libvirt] [PATCH 00/13] Move generic virsh data to a separate module vsh

2015-06-29 Thread Erik Skultety
The idea behind this is that in order to introduce virt-admin client (and later some commands/APIs), there are lots of methods in virsh that can be easily reused by other potential clients like command and command argument passing or error reporting. !!! IMPORTANT !!! These patches cannot be compi