On Thu, Jun 12, 2014 at 04:07:04PM -0500, Jonathon Jongsma 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);
> + }
Why are you trying to preserve existing comments? In case the user added
some of his own to the VM? This means if the user change the name of the
VM without changing its UUId the comment won't get updated
Christophe
Attachment:
pgpZbUTnmHszw.pgp
Description: PGP signature
_______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
