Re: [PATCHv3 1/2] VMware: Support more than 2 driver backends

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

 



On Fri, Sep 27, 2013 at 06:03:03PM +0800, Daniel Veillard wrote:
> On Tue, Sep 24, 2013 at 11:24:30AM -0500, Doug Goldstein wrote:
> > Currently the VMware version check code only supports two types of
> > VMware backends, Workstation and Player. But in the near future we will
> > have an additional one so we need to support more. Additionally, we
> > discover and cache the path to the vmrun binary so we should use that
> > path when using the cooresponding binary from the VMware VIX SDK.
> > ---
> > 
> > change from v2:
> > * No change
> > 
> > change from v1:
> > * Added default case so we don't potentially pass NULL to virCommand
> 
>   Since it adds a new product support with very little changes,
> looks (IMHO) safe and Matthias looks fine with it, I'm okay to
> have those two patches pushed for 1.1.3.
>   So please make the few changes suggested by Matthias, and push
> preferably today as I would like to make rc2 over the week-end,

  Okay, i pushed before rc2, taking Matthias comments into account,

Daniel

-- 
Daniel Veillard      | Open Source and Standards, Red Hat
veillard@xxxxxxxxxx  | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
http://veillard.com/ | virtualization library  http://libvirt.org/

--
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]