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 <eskul...@redhat.com> --- src/libvirt-admin.c | 2 +- tools/virt-admin.pod | 7 ++++--- 2 files changed, 5 insertions(+), 4 deletions(-)
ACK
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
