Re: [PATCH] Added support for qemu upto version 2.3.0

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

 



On Wed, Jul 08, 2015 at 06:46:44PM +0530, T A Mahadevan wrote:
> ---
>  data/hypervisors/qemu.xml.in | 107 +++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 107 insertions(+)
> 
> diff --git a/data/hypervisors/qemu.xml.in b/data/hypervisors/qemu.xml.in
> index 03590a3..8fa2e0b 100644
> --- a/data/hypervisors/qemu.xml.in
> +++ b/data/hypervisors/qemu.xml.in
> @@ -253,4 +253,111 @@
>      <upgrades id="http://qemu.org/1.6.1"/>
>    </platform>
>  
> +  <platform id="http://qemu.org/1.7.1";>
> +    <short-id>qemu-1.7.1</short-id>
> +    <_name>QEMU 1.7.1</_name>
> +    <_vendor>qemu.org</_vendor>
> +    <version>1.7.1</version>
> +    <release-date>2014-03-03</release-date>
> +    <upgrades id="http://qemu.org/1.7.0"/>
> +  </platform>
> +
> +  <platform id="http://qemu.org/1.7.2";>
> +    <short-id>qemu-1.7.2</short-id>
> +    <_name>QEMU 1.7.2</_name>
> +    <_vendor>qemu.org</_vendor>
> +    <version>1.7.2</version>
> +    <release-date>2014-07-21</release-date>
> +    <upgrades id="http://qemu.org/1.7.1"/>
> +  </platform>
> +
> +  <platform id="http://qemu.org/2.0.0";>
> +    <short-id>qemu-2.0.0</short-id>
> +    <_name>QEMU 2.0.0</_name>
> +    <_vendor>qemu.org</_vendor>
> +    <version>2.0.0</version>
> +    <release-date>2014-04-17</release-date>
> +    <upgrades id="http://qemu.org/1.7.2"/>
> +  </platform>
> +
> +  <platform id="http://qemu.org/2.0.1";>
> +    <short-id>qemu-2.0.1</short-id>
> +    <_name>QEMU 2.0.1</_name>
> +    <_vendor>qemu.org</_vendor>
> +    <version>2.0.1</version>
> +    <release-date>2014-08-15</release-date>
> +    <upgrades id="http://qemu.org/2.0.0"/>
> +  </platform>
> +
> +  <platform id="http://qemu.org/2.0.2";>
> +    <short-id>qemu-2.0.2</short-id>
> +    <_name>QEMU 2.0.2</_name>
> +    <_vendor>qemu.org</_vendor>
> +    <version>2.0.2</version>
> +    <release-date>2014-08-18</release-date>
> +    <upgrades id="http://qemu.org/2.0.1"/>
> +  </platform>
> +
> +  <platform id="http://qemu.org/2.1.0";>
> +    <short-id>qemu-2.1.0</short-id>
> +    <_name>QEMU 2.1.0</_name>
> +    <_vendor>qemu.org</_vendor>
> +    <version>2.1.0</version>
> +    <release-date>2014-08-01</release-date>
> +    <upgrades id="http://qemu.org/2.0.2"/>
> +  </platform>
> +
> +  <platform id="http://qemu.org/2.1.1";>
> +    <short-id>qemu-2.1.1</short-id>
> +    <_name>QEMU 2.1.1</_name>
> +    <_vendor>qemu.org</_vendor>
> +    <version>2.1.1</version>
> +    <release-date>2014-09-10</release-date>
> +    <upgrades id="http://qemu.org/2.1.0"/>
> +  </platform>
> +
> +  <platform id="http://qemu.org/2.1.2";>
> +    <short-id>qemu-2.1.2</short-id>
> +    <_name>QEMU 2.1.2</_name>
> +    <_vendor>qemu.org</_vendor>
> +    <version>2.1.2</version>
> +    <release-date>2014-09-25</release-date>
> +    <upgrades id="http://qemu.org/2.1.1"/>
> +  </platform>
> +
> +  <platform id="http://qemu.org/2.1.3";>
> +    <short-id>qemu-2.1.3</short-id>
> +    <_name>QEMU 2.1.3</_name>
> +    <_vendor>qemu.org</_vendor>
> +    <version>2.1.3</version>
> +    <release-date>2015-01-22</release-date>
> +    <upgrades id="http://qemu.org/2.1.2"/>
> +  </platform>
> +
> +  <platform id="http://qemu.org/2.2.0";>
> +    <short-id>qemu-2.2.0</short-id>
> +    <_name>QEMU 2.2.0</_name>
> +    <_vendor>qemu.org</_vendor>
> +    <version>2.2.0</version>
> +    <release-date>2014-12-09</release-date>
> +    <upgrades id="http://qemu.org/2.1.3"/>
> +  </platform>

This patch looks good to me, one minor comment would be that it's a bit
weird to have qemu 2.2.0 upgrade qemu 2.1.3 when 2.1.3 was released
after 2.2.0, but since this makes logical sense when looking at the
version numbers, I'd say it's fine.

Christophe

Attachment: pgpH8xJnfNvoq.pgp
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