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

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

 



On Fri, Nov 13, 2015 at 11:41 AM, Daniel P. Berrange
<berrange@xxxxxxxxxx> wrote:
> On Fri, Nov 13, 2015 at 11:39:49AM -0500, Zeeshan Ali (Khattak) wrote:
>> On Fri, Nov 13, 2015 at 11:19 AM, Daniel P. Berrange
>> <berrange@xxxxxxxxxx> wrote:
>> > On Fri, Nov 13, 2015 at 11:16:15AM -0500, Zeeshan Ali (Khattak) wrote:
>> >> For desktop profile.
>> >> ---
>> >>  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 be7a965..24a4d6b 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,9 @@ url --mirrorlist=https://mirrors.fedoraproject.org/metalink?repo=fedora-install-
>> >>  xorg-x11-drv-qxl
>> >>  spice-vdagent
>> >>
>> >> +# Also Qemu agent
>> >> +qemu-guest-agent
>> >
>> > This surely needs to be made conditional based on some version
>> > of Fedora, likewise for the other changes in this series, as the
>> > one kicksart file is shared across many versions of the OS at
>> > least some of which I expect pre-date qemu-guest-agent.
>>
>> It actually seems to predate Fedora itself. :)
>
> Unfortunately not - qemu-guest-agent is a sub-RPM of 'qemu' so
> the changelog you're seeing is the changelog for qemu as a whole.
>
> I added it to Fedora in this dist-git commit
>
>   commit f057293300c43e749ee543178ab95f0e285949cd
>   Author: Daniel P. Berrange <berrange@xxxxxxxxxx>
>   Date:   Wed Oct 5 16:00:36 2011 +0100
>
>     Fix packaging of the QEMU guest agent

Ah ok, so I'll modify this patch. As for others:

* RHEL 6.* had this package and this is the first RHEL release for
which we list installer scripts for, so no issue there AFAIK.

* Debian seems to have added the package in debian squeeze (6) and
desktop installer script is only listed from wheezy (7) and onwards,
so no need for conditionals there either.


-- 
Regards,

Zeeshan Ali (Khattak)
________________________________________
Befriend GNOME: http://www.gnome.org/friends/

_______________________________________________
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