Re: [PATCH libosinfo] test-isodetect: add Debian's arm flavours

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

 



On Tue, Jul 11, 2017 at 09:58:26PM +0200, Guido Günther wrote:
> to avoid "Unknown arch" debug messges
> ---
>  tests/test-isodetect.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/test-isodetect.c b/tests/test-isodetect.c
> index c72b8fc..6a38963 100644
> --- a/tests/test-isodetect.c
> +++ b/tests/test-isodetect.c
> @@ -128,8 +128,12 @@ static struct ISOInfo *load_iso(GFile *file, const gchar *shortid, const gchar *
>          arch = "ia64";
>      else if (strstr(name, "s390x"))
>          arch = "s390x";
> -    else if (strstr(name, "aarch64"))
> +    else if (strstr(name, "aarch64") ||
> +             strstr(name, "arm64"))
>          arch = "aarch64";

This change looks good.

> +    else if (strstr(name, "armhf") ||
> +             strstr(name, "armv7l"))
> +        arch = "armhf";

However for this one I'd expect arch = "armv7l" as this is what is in
the RNG schema? My understanding of
https://bugzilla.redhat.com/show_bug.cgi?id=719609#c13 is that armv7l is
a soft floating point arch, armv7hl would be hard floating point. And
armhf is hard floating point as well. So I don't think we can consider
the 2 are the same here. You probably need to add a new ARM arch to the
schema file, and use that for armhf.

Christophe

>      else {
>          g_debug("Unknown arch in %s", name);
>          arch = "i386";
> -- 
> 2.13.2
> 
> _______________________________________________
> Libosinfo mailing list
> Libosinfo@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libosinfo

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