Re: [PATCH] fedora: Install spice-webdavd dependency

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

 



On Fri, Aug 26, 2016 at 07:23:19PM +0300, Visarion Alexandru wrote:
> From: Visarion Alexandru <viorel.visarion@xxxxxxxxx>
> 
> GNOME-Boxes is developing a folder sharing feature which will
> need spice-webdavd installed in the guest for it to work.
> 
> Let's make the folder sharing feature available right after
> express install in Fedora, by automaticaly installing spice-webdavd,
> if available.
> ---
>  data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in | 3 +++
>  1 file changed, 3 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 bd10fba..d79034e 100644
> --- a/data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in
> +++ b/data/install-script/fedoraproject.org/fedora-kickstart-desktop.xml.in
> @@ -157,6 +157,9 @@ xorg-x11-drv-qxl
>  spice-vdagent
>  
>  <xsl:choose>
> +  <xsl:when test="os/version &gt; 19">
> +spice-webdavd
> +  </xsl:when>

Only the first matching condition of a <xsl:choose> is run. So by adding
this you've now prevented qemu-guest-agent being installed. Just use an
standalone <xsl:if>

>    <xsl:when test="os/version &gt; 15">
>  qemu-guest-agent
>    </xsl:when>

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

_______________________________________________
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