Fedora arm64 packages for gdb testing

Fedora arm64 packages for gdb testing

[Luis Machado]

Hi,

I noticed the Fedora arm64 worker for GDB is not running the tests properly. This is due to a missing libexpat.

There might be other packages missing, but for now libexpat should make the gdb tests work fine.

It would also be nice to change ulimit -c to unlimited for GDB tests, and to change the core file pattern to something like "core.%p", as opposed to using systemd-coredump.

We might need more packages for a complete GDB testsuite run, but since we're not doing that yet, the above should be enough.
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

Re: Fedora arm64 packages for gdb testing

[Mark Wielaard]

Hi Luis,

On Fri, Jul 15, 2022 at 01:05:15PM +0100, Luis Machado wrote:
> I noticed the Fedora arm64 worker for GDB is not running the tests
> properly. This is due to a missing libexpat.
>
> There might be other packages missing, but for now libexpat should
> make the g> db tests work fine.

I installed expat-devel on the machine and this does indeed seem to
have turned the tests using the gdbserver target green.

https://sourceware.org/pipermail/gdb-testers/2022q3/171821.html

I noticed libexpat-devel isn't installed on other builders. Is it only
necessary on arm? Or should we make sure it is always installed?

Thanks,

Mark

Re: Fedora arm64 packages for gdb testing

[Luis Machado]

On 7/17/22 17:39, Mark Wielaard wrote:
> Hi Luis,
>
> On Fri, Jul 15, 2022 at 01:05:15PM +0100, Luis Machado wrote:
>> I noticed the Fedora arm64 worker for GDB is not running the tests
>> properly. This is due to a missing libexpat.
>>
>> There might be other packages missing, but for now libexpat should
>> make the g> db tests work fine.
>
> I installed expat-devel on the machine and this does indeed seem to
> have turned the tests using the gdbserver target green.
>
> https://sourceware.org/pipermail/gdb-testers/2022q3/171821.html
>
> I noticed libexpat-devel isn't installed on other builders. Is it only
> necessary on arm? Or should we make sure it is always installed?

It may be the case that other targets don't rely as much on XML descriptions as the Arm port does.

I'd recommend always installing libexpat on the builders to get the best outcome.
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

Re: Fedora arm64 packages for gdb testing

[Mark Wielaard]

[-- Attachment #1: Type: text/plain, Size: 918 bytes --]

Hi Luis,

On Mon, Jul 18, 2022 at 09:20:23AM +0100, Luis Machado wrote:
> > I noticed libexpat-devel isn't installed on other builders. Is it only
> > necessary on arm? Or should we make sure it is always installed?
>
> It may be the case that other targets don't rely as much on XML
> descriptions as the Arm port does.
>
> I'd recommend always installing libexpat on the builders to get the
> best outcome.

OK I checked and/or installed for:

fedora-s390x expat-devel-2.4.7-1.fc36.s390x
fedora-ppc64le expat-devel-2.4.7-1.fc36.ppc64le
centos-x86_64 expat-devel-2.1.0-14.el7_9.x86_64
debian-i386 libexpat1-dev-2.2.10-2+deb11u3
debian-armhf libexpat1-dev-2.2.6-2+deb10u4
debian-arm64 libexpat1-dev-2.2.6-2+deb10u4

And the fedora and debian containers now got it added.
opensuse containers already had libexpat-devel installed.
(note how each distro calls the package by a slightly differnt name, sigh)

Cheers,

Mark

[-- Attachment #2: 0001-Add-libexpat-dev-expat-devel-to-fedora-and-debian-co.patch --]
[-- Type: text/x-diff, Size: 3363 bytes --]

From 97e90a71c93e7810b517fb9ed5006a3d958840bf Mon Sep 17 00:00:00 2001
From: Mark Wielaard <mark@klomp.org>
Date: Mon, 18 Jul 2022 23:43:17 +0200
Subject: [PATCH] Add libexpat-dev/expat-devel to fedora and debian containers

opensuse containers already had libexpat-devel installed
---
 builder/containers/Containerfile-debian-stable  | 1 +
 builder/containers/Containerfile-debian-testing | 1 +
 builder/containers/Containerfile-fedora-latest  | 1 +
 builder/containers/Containerfile-fedora-rawhide | 1 +
 4 files changed, 4 insertions(+)

diff --git a/builder/containers/Containerfile-debian-stable b/builder/containers/Containerfile-debian-stable
index c1ca117..d7c1973 100644
--- a/builder/containers/Containerfile-debian-stable
+++ b/builder/containers/Containerfile-debian-stable
@@ -9,6 +9,7 @@ RUN apt-get update && \
       coreutils make git autotools-dev autoconf dejagnu automake gettext bison flex \
       gawk m4 pkg-config gettext autopoint \
       bzip2 libbz2-dev xz-utils liblzma-dev gzip zlib1g-dev zstd libzstd-dev \
+      libexpat-dev \
       findutils file tar curl libarchive-tools libarchive-dev libcurl4-gnutls-dev \
       libmicrohttpd-dev libsqlite3-dev libelf-dev libdw-dev elfutils \
       libdebuginfod-dev \
diff --git a/builder/containers/Containerfile-debian-testing b/builder/containers/Containerfile-debian-testing
index b72b7fe..57c1e63 100644
--- a/builder/containers/Containerfile-debian-testing
+++ b/builder/containers/Containerfile-debian-testing
@@ -9,6 +9,7 @@ RUN apt-get update && \
       coreutils make git autotools-dev autoconf dejagnu automake gettext bison flex \
       gawk m4 pkg-config gettext autopoint \
       bzip2 libbz2-dev xz-utils liblzma-dev gzip zlib1g-dev zstd libzstd-dev \
+      libexpat-dev \
       findutils file tar curl libarchive-tools libarchive-dev libcurl4-gnutls-dev \
       libmicrohttpd-dev libsqlite3-dev libelf-dev libdw-dev elfutils \
       libdebuginfod-dev \
diff --git a/builder/containers/Containerfile-fedora-latest b/builder/containers/Containerfile-fedora-latest
index e8e4ad4..d662195 100644
--- a/builder/containers/Containerfile-fedora-latest
+++ b/builder/containers/Containerfile-fedora-latest
@@ -7,6 +7,7 @@ RUN dnf upgrade -y && \
         patch util-linux diffutils iproute bsdtar cpio procps \
         coreutils make git autoconf dejagnu automake gettext-devel bison flex \
         bzip2 bzip2-devel xz xz-devel gzip zlib-devel zstd libzstd-devel \
+        expat-devel \
         findutils file tar curl libarchive-devel libcurl-devel \
         libmicrohttpd-devel sqlite-devel elfutils elfutils-devel ncurses-devel \
         texinfo \
diff --git a/builder/containers/Containerfile-fedora-rawhide b/builder/containers/Containerfile-fedora-rawhide
index 1405276..543828a 100644
--- a/builder/containers/Containerfile-fedora-rawhide
+++ b/builder/containers/Containerfile-fedora-rawhide
@@ -7,6 +7,7 @@ RUN dnf upgrade -y && \
         patch util-linux diffutils iproute bsdtar cpio procps \
         coreutils make git autoconf dejagnu automake gettext-devel bison flex \
         bzip2 bzip2-devel xz xz-devel gzip zlib-devel zstd libzstd-devel \
+        expat-devel \
         findutils file tar curl libarchive-devel libcurl-devel \
         libmicrohttpd-devel sqlite-devel elfutils elfutils-devel ncurses-devel \
         texinfo \
-- 
2.30.2

Re: Fedora arm64 packages for gdb testing

[Luis Machado]

On 7/18/22 22:49, Mark Wielaard wrote:
> Hi Luis,
>
> On Mon, Jul 18, 2022 at 09:20:23AM +0100, Luis Machado wrote:
>>> I noticed libexpat-devel isn't installed on other builders. Is it only
>>> necessary on arm? Or should we make sure it is always installed?
>>
>> It may be the case that other targets don't rely as much on XML
>> descriptions as the Arm port does.
>>
>> I'd recommend always installing libexpat on the builders to get the
>> best outcome.
>
> OK I checked and/or installed for:
>
> fedora-s390x expat-devel-2.4.7-1.fc36.s390x
> fedora-ppc64le expat-devel-2.4.7-1.fc36.ppc64le
> centos-x86_64 expat-devel-2.1.0-14.el7_9.x86_64
> debian-i386 libexpat1-dev-2.2.10-2+deb11u3
> debian-armhf libexpat1-dev-2.2.6-2+deb10u4
> debian-arm64 libexpat1-dev-2.2.6-2+deb10u4
>
> And the fedora and debian containers now got it added.
> opensuse containers already had libexpat-devel installed.
> (note how each distro calls the package by a slightly differnt name, sigh)
>
> Cheers,
>
> Mark

Thanks Mark. Yeah, unfortunately there is some naming variations.
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.