[libvirt] [PATCH] Revert "admin: Rename virAdmConnect to virAdmDaemon"

[Martin Kletzander]

On Mon, Dec 14, 2015 at 09:31:33AM +0100, Erik Skultety wrote:
>On 10/12/15 15:54, Daniel P. Berrange wrote:
>> On Thu, Dec 10, 2015 at 03:46:45PM +0100, Erik Skultety wrote:
>>> Commmit df8192aa introduced admin related rename and some minor
>>> (caused by automated approach, aka sed) and some more severe isues along with
>>> it. First reason to revert is the inconsistency with libvirt library.
>>
>> We have release 1.3.0 with this header file, so IMHO we can't just
>> rename the public APIs now.
>>
>Yes, I understand that Daniel and I know our policy, but what I thought
>was that since it's still disabled and cannot be used properly (besides
>editing the code and rebuilding upstream and still wouldn't do much),
>there might be an exception to this...obviously, it's not, so at least
>we could revert the client internal changes, I guess.
>

Yes, moreover the header file is not distributed yet, I took the
precautions so that we can discuss API-incompatible changes for now and
finally come to a conclusion about some of these details and move on.

Thinking about it over and over again, I'm probably OK with being
consistent with the libvirt library even though I don't like it as
much.  One other thing to solve both naming problems, but it would add
APIs that are not needed now, is to have virAdmConnectGetDaemon which
would return virAdmDaemonPtr.  But that might be getting too
complicated.

I would say that if nobody chimes in with another opinion (for, let's
say, around a week) the safest way will be sticking with the naming
that's consistent with libvirt library, despite the fact that I,
personally, am not in favor of that.

>Erik
>
>--
>libvir-list mailing list
>libvir-list at redhat.com
>https://www.redhat.com/mailman/listinfo/libvir-list
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20151215/37f22bf2/attachment-0001.sig>