Re: [v3 3/5] fedora,script: Install qemu-guest-agent

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

 



On Thu, Nov 19, 2015 at 09:12:33AM -0500, Zeeshan Ali (Khattak) wrote:
> On Mon, Nov 16, 2015 at 3:57 AM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> > On Fri, Nov 13, 2015 at 01:18:53PM -0500, Zeeshan Ali (Khattak) wrote:
> >> For desktop profile.
> >> ---
> >>  .../fedoraproject.org/fedora-kickstart-desktop.xml.in              | 7 +++++++
> >>  1 file changed, 7 insertions(+)
> >>
> >> diff --git a/data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in b/data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in
> >> index be7a965..8ad2f05 100644
> >> --- a/data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in
> >> +++ b/data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in
> >> @@ -156,6 +156,13 @@ url --mirrorlist=https://mirrors.fedoraproject.org/metalink?repo=fedora-install-
> >>  xorg-x11-drv-qxl
> >>  spice-vdagent
> >>
> >> +<xsl:choose>
> >> +  <xsl:when test="os/version &gt; 15">
> >> +# Also Qemu agent
> >> +qemu-guest-agent
> >
> > I don't think the comment is needed.
> 
> IMO it's as much needed as the comment above:
> 
> "# QXL driver and SPICE vdagent"

I know there is a not really useful comment above it, but I don't think
that's a good reason for the addition of a new one.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Fedora Users]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux