Re: [PATCH v2] Code related to vm_choose_dialog moved into separate files

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

 



Hi,

On Thu, Oct 09, 2014 at 08:26:01AM -0400, Pavel Grunt wrote:
> > I believe this can be squashed into the ovirt commit as this hasn't
> > been
> > committed yet? If this makes sense to you, I'll do the change myself
> > before pushing.
> > 
> It makes sense.
> 
> > You also need to update po/POTFILES.in when you add new files with
> > translations. cd po/; intltool-update -m; will tell you about the
> > missing files. I've fixed that locally so no need to send another
> > iteration of the patches.
> > 
> > Regarding the commit logs, I'd tend to use
> > "Show VM chooser dialog when oVirt VM name is missing"
> > and
> > "Show VM chooser dialog when starting virt-viewer with no arg" (even
> > though it's a bit long)
> > When looking at the git short log (git log --oneline), it's more
> > explicit what these commits are about this way.
> > Do you mind if I change them?
> > 
> I don't mind, it sounds better.

Ok, cool, I've pushed the series now.

Christophe

Attachment: pgpI0gDvGo23Q.pgp
Description: PGP signature

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list

[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux