Re: [PATCH libosinfo] oses: Add RHEL Server for AArch64 (rhelsa2.0)

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

 



Hey,

On Wed, Dec 03, 2014 at 09:24:50AM -0500, Cole Robinson wrote:
> Long term this will likely just merge with RHEL proper, but until then
> it's slated to be its own product.
> 
> Signed-off-by: Cole Robinson <crobinso@xxxxxxxxxx>
> ---
>  data/oses/rhel.xml.in                              | 51 ++++++++++++++++++++++
>  data/schemas/libosinfo.rng                         |  1 +
>  ...ELSA-1.5-20141017.1-Server-aarch64-dvd1.iso.txt | 29 ++++++++++++
>  3 files changed, 81 insertions(+)
>  create mode 100644 test/isodata/rhel/rhelsa2.0/RHELSA-1.5-20141017.1-Server-aarch64-dvd1.iso.txt
> 
> diff --git a/data/oses/rhel.xml.in b/data/oses/rhel.xml.in
> index 791d470..b1e70e0 100644
> --- a/data/oses/rhel.xml.in
> +++ b/data/oses/rhel.xml.in
> @@ -1171,6 +1171,9 @@
>      </installer>
>    </os>
>  
> +
> +  <!-- RHEL atomic -->
> +
>    <os id="http://redhat.com/rhel-atomic/7.0";>
>      <short-id>rhel-atomic-7.0</short-id>
>      <_name>Red Hat Enterprise Linux Atomic Host 7.0</_name>
> @@ -1212,4 +1215,52 @@
>        </recommended>
>      </resources>
>    </os>
> +
> +
> +  <!-- RHEL Server for Aarch64 (RHELSA) -->
> +
> +  <os id="http://redhat.com/rhelsa/2.0";>
> +    <short-id>rhelsa2.0</short-id>
> +    <_name>Red Hat Enterprise Linux Server for AArch64 2.0</_name>
> +    <version>2.0</version>

Just a quick comment, version is 2.0 here but the volume id in the test
case is:
Volume id: RHELSA-1.5 Server.aarch64
so version is inconsistent, maybe the volume-id regexp needs to be more
specific?

Christophe

Attachment: pgpQa0aGUT2Q9.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