[libvirt] [PATCH] admin: Fix default uri config option name s/admin_uri_default/uri_default

Martin Kletzander mkletzan at redhat.com
Wed Jul 27 14:25:43 UTC 2016


On Wed, Jul 27, 2016 at 02:22:52PM +0200, Erik Skultety wrote:
>The original name 'admin_uri_default' was introduced to our code by commit
>dbecb87f. However, at that time we already had a separate config file for
>admin library but the commit mentioned above didn't properly adjust the
>config's option name. The result is that when we're loading the config, we
>check a non-existent config option (there's not much to do with the URIs
>anyway, since we only allow local connection). Additionally, virt-admin's man
>page documents, that the default URI can be altered by setting
>admin_uri_default option. So the fix proposed by this patch leaves the
>libvirt-admin.conf as is and adjusts the naming in the code as well as in the
>virt-admin's man page.
>
>Signed-off-by: Erik Skultety <eskultet at redhat.com>
>---
> src/libvirt-admin.c  | 2 +-
> tools/virt-admin.pod | 7 ++++---
> 2 files changed, 5 insertions(+), 4 deletions(-)
>

ACK
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160727/5c6b5906/attachment-0001.sig>


More information about the libvir-list mailing list