On Thu, Mar 10, 2016 at 02:58:52PM +0100, Ján Tomko wrote:
On Thu, Mar 10, 2016 at 05:54:07AM +0100, Martin Kletzander wrote:
> It does not have a suffix ByName because there are no other means of
> looking up the server and since the name is known, this should be the
> preferred one.
>
> Signed-off-by: Martin Kletzander <mkletzan(a)redhat.com>
> ---
> daemon/admin_server.c | 10 ++++++++++
> daemon/admin_server.h | 5 +++++
> include/libvirt/libvirt-admin.h | 4 ++++
> src/admin/admin_protocol.x | 16 +++++++++++++++-
> src/admin_protocol-structs | 8 ++++++++
> src/libvirt-admin.c | 36 ++++++++++++++++++++++++++++++++++++
> src/libvirt_admin_private.syms | 2 ++
> src/libvirt_admin_public.syms | 1 +
> 8 files changed, 81 insertions(+), 1 deletion(-)
>
ACK from me, maybe Erik wants to see this too.
I pushed the series except this patch, thanks for the reviews, I'll let
Erik express himself about this one.