[virt-tools-list] [PATCH virt-viewer 3/3] Write vm name to config file as comment

Jonathon Jongsma jjongsma at redhat.com
Wed Jun 18 21:05:29 UTC 2014


ping (all 3 patches)?

On Thu, Jun 12, 2014 at 4:07 PM, Jonathon Jongsma <jjongsma at redhat.com> 
wrote:
> ---
>  src/virt-viewer-app.c | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)
> 
> diff --git a/src/virt-viewer-app.c b/src/virt-viewer-app.c
> index a986cba..482a4ee 100644
> --- a/src/virt-viewer-app.c
> +++ b/src/virt-viewer-app.c
> @@ -242,6 +242,20 @@ virt_viewer_app_save_config(VirtViewerApp *self)
>          g_warning("failed to create config directory");
>      g_free(dir);
>  
> +    if (priv->uuid && priv->guest_name) {
> +        // if there's no comment for this uuid settings group, add a 
> comment
> +        // with the vm name so user can make sense of it later.
> +        gchar* comment = g_key_file_get_comment(priv->config, 
> priv->uuid, NULL, &error);
> +        if (error) {
> +            g_warning("Unable to get comment from key file: %s", 
> error->message);
> +            g_clear_error(&error);
> +        } else {
> +            if (!comment || strlen(comment) == 0)
> +                g_key_file_set_comment(priv->config, priv->uuid, 
> NULL, priv->guest_name, NULL);
> +        }
> +        g_free(comment);
> +    }
> +
>      if ((data = g_key_file_to_data(priv->config, NULL, &error)) == 
> NULL ||
>          !g_file_set_contents(priv->config_file, data, -1, &error)) {
>          g_warning("Couldn't save configuration: %s", error->message);
> -- 
> 1.9.3
> 
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list




More information about the virt-tools-list mailing list