[virt-tools-list] [PATCH virt-viewer 5/7] Add VirtViewerSession:file property

[Christophe Fergeau]

On Tue, Nov 27, 2012 at 02:01:14PM +0100, Marc-André Lureau wrote:
> On Tue, Nov 27, 2012 at 10:45 AM, Christophe Fergeau <cfergeau at redhat.com>wrote:
> 
> > A bit confused by the commit log, use of the file to define the connection
> > parameters is done in the next commit. The 'Also' is a clear hint that this
> > commit should be split, and it would have been nice to explain why this
> > mime type is needed in the log.
> >
> 
> Maybe I should stop using "also" in my commit messages!
> 
> I don't think it should be split, if the session has a file, it means we
> need to save to recent with that file and its correct mime.

Yup, you're probably right here, I take my comment back ;)

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/virt-tools-list/attachments/20121127/786e07e7/attachment.sig>