Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Pavel Hrdina
On Fri, Jun 24, 2016 at 04:11:52PM +0100, Daniel P. Berrange wrote: > On Fri, Jun 24, 2016 at 05:04:30PM +0200, Michal Privoznik wrote: > > On 24.06.2016 16:06, Daniel P. Berrange wrote: > > > On Fri, Jun 24, 2016 at 03:59:38PM +0200, Michal Privoznik wrote: > > >> On 24.06.2016 15:33, Daniel P.

Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Andrea Bolognani
On Fri, 2016-06-24 at 17:04 +0200, Michal Privoznik wrote: > On 24.06.2016 16:06, Daniel P. Berrange wrote: > > On Fri, Jun 24, 2016 at 03:59:38PM +0200, Michal Privoznik wrote: > > > On 24.06.2016 15:33, Daniel P. Berrange wrote: > > > > On Fri, Jun 24, 2016 at 03:12:23PM +0200, Michal Privoznik

Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Daniel P. Berrange
On Fri, Jun 24, 2016 at 05:04:30PM +0200, Michal Privoznik wrote: > On 24.06.2016 16:06, Daniel P. Berrange wrote: > > On Fri, Jun 24, 2016 at 03:59:38PM +0200, Michal Privoznik wrote: > >> On 24.06.2016 15:33, Daniel P. Berrange wrote: > >>> On Fri, Jun 24, 2016 at 03:12:23PM +0200, Michal

Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Michal Privoznik
On 24.06.2016 16:06, Daniel P. Berrange wrote: > On Fri, Jun 24, 2016 at 03:59:38PM +0200, Michal Privoznik wrote: >> On 24.06.2016 15:33, Daniel P. Berrange wrote: >>> On Fri, Jun 24, 2016 at 03:12:23PM +0200, Michal Privoznik wrote: Currently, the daemon requires libvirt-admin.so because

Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Erik Skultety
On 24/06/16 16:00, Erik Skultety wrote: > On 24/06/16 15:12, Michal Privoznik wrote: >> Currently, the daemon requires libvirt-admin.so because the >> functions encoding/decoding RPC messages for admin APIs live >> there. But this makes it very hard to split admin API into its >> own separate

Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Erik Skultety
On 24/06/16 16:06, Daniel P. Berrange wrote: > On Fri, Jun 24, 2016 at 03:59:38PM +0200, Michal Privoznik wrote: >> On 24.06.2016 15:33, Daniel P. Berrange wrote: >>> On Fri, Jun 24, 2016 at 03:12:23PM +0200, Michal Privoznik wrote: Currently, the daemon requires libvirt-admin.so because the

Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Daniel P. Berrange
On Fri, Jun 24, 2016 at 03:59:38PM +0200, Michal Privoznik wrote: > On 24.06.2016 15:33, Daniel P. Berrange wrote: > > On Fri, Jun 24, 2016 at 03:12:23PM +0200, Michal Privoznik wrote: > >> Currently, the daemon requires libvirt-admin.so because the > >> functions encoding/decoding RPC messages

Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Erik Skultety
On 24/06/16 15:12, Michal Privoznik wrote: > Currently, the daemon requires libvirt-admin.so because the > functions encoding/decoding RPC messages for admin APIs live > there. But this makes it very hard to split admin API into its > own separate package: if libvirt-admin.so is going to live in a

Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Michal Privoznik
On 24.06.2016 15:33, Daniel P. Berrange wrote: > On Fri, Jun 24, 2016 at 03:12:23PM +0200, Michal Privoznik wrote: >> Currently, the daemon requires libvirt-admin.so because the >> functions encoding/decoding RPC messages for admin APIs live >> there. But this makes it very hard to split admin API

Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Andrea Bolognani
On Fri, 2016-06-24 at 15:12 +0200, Michal Privoznik wrote: > Currently, the daemon requires libvirt-admin.so because the > functions encoding/decoding RPC messages for admin APIs live > there. But this makes it very hard to split admin API into its > own separate package: if libvirt-admin.so is

Re: [libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Daniel P. Berrange
On Fri, Jun 24, 2016 at 03:12:23PM +0200, Michal Privoznik wrote: > Currently, the daemon requires libvirt-admin.so because the > functions encoding/decoding RPC messages for admin APIs live > there. But this makes it very hard to split admin API into its > own separate package: if

[libvirt] [PATCH] daemon: Drop dependency on libvirt-admin.so

2016-06-24 Thread Michal Privoznik
Currently, the daemon requires libvirt-admin.so because the functions encoding/decoding RPC messages for admin APIs live there. But this makes it very hard to split admin API into its own separate package: if libvirt-admin.so is going to live in a separate package than the daemon, either both