Re: [PATCH] virt-admin: get rid of LIBVIRT_DEFAULT_ADMIN_URI env var

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/04/16 15:00, Ján Tomko wrote:
> There is a LIBVIRT_ADMIN_DEFAULT_URI environment variable
> which is honored by virAdmConnectOpen and documented
> in the virt-admin man page.
> 
> LIBVIRT_DEFAULT_ADMIN_URI is undocumented and this is its
> only occurrence.
> ---
>  tools/virt-admin.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/tools/virt-admin.c b/tools/virt-admin.c
> index edb8690..f0a49a3 100644
> --- a/tools/virt-admin.c
> +++ b/tools/virt-admin.c
> @@ -670,7 +670,6 @@ main(int argc, char **argv)
>  {
>      vshControl _ctl, *ctl = &_ctl;
>      vshAdmControl virtAdminCtl;
> -    const char *defaultConn;
>      bool ret = true;
>  
>      memset(ctl, 0, sizeof(vshControl));
> @@ -719,9 +718,6 @@ main(int argc, char **argv)
>  
>      virFileActivateDirOverride(argv[0]);
>  
> -    if ((defaultConn = virGetEnvBlockSUID("LIBVIRT_DEFAULT_ADMIN_URI")))
> -        ctl->connname = vshStrdup(ctl, defaultConn);
> -
>      if (!vshInit(ctl, cmdGroups, NULL))
>          exit(EXIT_FAILURE);
>  
> 

ACK

Erik

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]