* [PATCH] CI: Fix fallout from adding elfutils-dev to the build container
@ 2023-12-05 22:56 Andrew Cooper
2023-12-05 22:59 ` Stefano Stabellini
0 siblings, 1 reply; 2+ messages in thread
From: Andrew Cooper @ 2023-12-05 22:56 UTC (permalink / raw)
To: Xen-devel
Cc: Andrew Cooper, Anthony PERARD, Stefano Stabellini, Michal Orzel,
Doug Goldstein, Roger Pau Monné
Commit 948e03303138 ("automation/alpine: add elfutils-dev") had an unintended
consequence of causing Qemu to gain a runtime dependency on libdw.so
The {adl,zen3p}-pci-hvm-x86-64-gcc-debug tests, which are the only two tests
that run the built Qemu, started failing with:
Error loading shared library libdw.so.1: No such file or directory (needed by /usr/local/lib/xen/bin/qemu-system-i386)
Error relocating /usr/local/lib/xen/bin/qemu-system-i386: dwfl_begin: symbol not found
Update the test container with libelf to cope.
While editing the runtime dependency list, fix up two other problems causing
bloat. texinfo isn't a runtime dependency, and we should be using xz itself,
not it's development libraries.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Anthony PERARD <anthony.perard@citrix.com>
CC: Stefano Stabellini <sstabellini@kernel.org>
CC: Michal Orzel <michal.orzel@amd.com>
CC: Doug Goldstein <cardoe@cardoe.com>
CC: Roger Pau Monné <roger.pau@citrix.com>
I've already pushed the x86 container as part of confirming the fix.
---
automation/tests-artifacts/alpine/3.18-arm64v8.dockerfile | 3 +--
automation/tests-artifacts/alpine/3.18.dockerfile | 4 ++--
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/automation/tests-artifacts/alpine/3.18-arm64v8.dockerfile b/automation/tests-artifacts/alpine/3.18-arm64v8.dockerfile
index 94f69621f40e..0e5ae7f2b4d8 100644
--- a/automation/tests-artifacts/alpine/3.18-arm64v8.dockerfile
+++ b/automation/tests-artifacts/alpine/3.18-arm64v8.dockerfile
@@ -21,10 +21,9 @@ RUN \
apk add python3 && \
apk add zlib && \
apk add ncurses && \
- apk add texinfo && \
apk add yajl && \
apk add libaio && \
- apk add xz-dev && \
+ apk add xz && \
apk add util-linux && \
apk add argp-standalone && \
apk add libfdt && \
diff --git a/automation/tests-artifacts/alpine/3.18.dockerfile b/automation/tests-artifacts/alpine/3.18.dockerfile
index f1b4a8b7a191..9cde6c9ad4da 100644
--- a/automation/tests-artifacts/alpine/3.18.dockerfile
+++ b/automation/tests-artifacts/alpine/3.18.dockerfile
@@ -22,10 +22,9 @@ RUN \
apk add python3 && \
apk add zlib && \
apk add ncurses && \
- apk add texinfo && \
apk add yajl && \
apk add libaio && \
- apk add xz-dev && \
+ apk add xz && \
apk add util-linux && \
apk add argp-standalone && \
apk add libfdt && \
@@ -34,6 +33,7 @@ RUN \
apk add curl && \
apk add udev && \
apk add pciutils && \
+ apk add libelf && \
\
# Xen
cd / && \
base-commit: ff1178062094837d55ef342070e58316c43a54c9
prerequisite-patch-id: 477e3af5692ee0daa13d795fdf78384be604fd66
prerequisite-patch-id: 60d13b1c04d8a808a42d20b3432270cfd87a47fc
prerequisite-patch-id: 457b56a295e75d2d9f837b44cd483812ca66cd85
--
2.30.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] CI: Fix fallout from adding elfutils-dev to the build container
2023-12-05 22:56 [PATCH] CI: Fix fallout from adding elfutils-dev to the build container Andrew Cooper
@ 2023-12-05 22:59 ` Stefano Stabellini
0 siblings, 0 replies; 2+ messages in thread
From: Stefano Stabellini @ 2023-12-05 22:59 UTC (permalink / raw)
To: Andrew Cooper
Cc: Xen-devel, Anthony PERARD, Stefano Stabellini, Michal Orzel,
Doug Goldstein, Roger Pau Monné
[-- Attachment #1: Type: text/plain, Size: 3051 bytes --]
On Tue, 5 Dec 2023, Andrew Cooper wrote:
> Commit 948e03303138 ("automation/alpine: add elfutils-dev") had an unintended
> consequence of causing Qemu to gain a runtime dependency on libdw.so
>
> The {adl,zen3p}-pci-hvm-x86-64-gcc-debug tests, which are the only two tests
> that run the built Qemu, started failing with:
>
> Error loading shared library libdw.so.1: No such file or directory (needed by /usr/local/lib/xen/bin/qemu-system-i386)
> Error relocating /usr/local/lib/xen/bin/qemu-system-i386: dwfl_begin: symbol not found
>
> Update the test container with libelf to cope.
>
> While editing the runtime dependency list, fix up two other problems causing
> bloat. texinfo isn't a runtime dependency, and we should be using xz itself,
> not it's development libraries.
>
> Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
Reviewed-by: Stefano Stabellini <sstabellini@kernel.org>
> ---
> CC: Anthony PERARD <anthony.perard@citrix.com>
> CC: Stefano Stabellini <sstabellini@kernel.org>
> CC: Michal Orzel <michal.orzel@amd.com>
> CC: Doug Goldstein <cardoe@cardoe.com>
> CC: Roger Pau Monné <roger.pau@citrix.com>
>
> I've already pushed the x86 container as part of confirming the fix.
> ---
> automation/tests-artifacts/alpine/3.18-arm64v8.dockerfile | 3 +--
> automation/tests-artifacts/alpine/3.18.dockerfile | 4 ++--
> 2 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/automation/tests-artifacts/alpine/3.18-arm64v8.dockerfile b/automation/tests-artifacts/alpine/3.18-arm64v8.dockerfile
> index 94f69621f40e..0e5ae7f2b4d8 100644
> --- a/automation/tests-artifacts/alpine/3.18-arm64v8.dockerfile
> +++ b/automation/tests-artifacts/alpine/3.18-arm64v8.dockerfile
> @@ -21,10 +21,9 @@ RUN \
> apk add python3 && \
> apk add zlib && \
> apk add ncurses && \
> - apk add texinfo && \
> apk add yajl && \
> apk add libaio && \
> - apk add xz-dev && \
> + apk add xz && \
> apk add util-linux && \
> apk add argp-standalone && \
> apk add libfdt && \
> diff --git a/automation/tests-artifacts/alpine/3.18.dockerfile b/automation/tests-artifacts/alpine/3.18.dockerfile
> index f1b4a8b7a191..9cde6c9ad4da 100644
> --- a/automation/tests-artifacts/alpine/3.18.dockerfile
> +++ b/automation/tests-artifacts/alpine/3.18.dockerfile
> @@ -22,10 +22,9 @@ RUN \
> apk add python3 && \
> apk add zlib && \
> apk add ncurses && \
> - apk add texinfo && \
> apk add yajl && \
> apk add libaio && \
> - apk add xz-dev && \
> + apk add xz && \
> apk add util-linux && \
> apk add argp-standalone && \
> apk add libfdt && \
> @@ -34,6 +33,7 @@ RUN \
> apk add curl && \
> apk add udev && \
> apk add pciutils && \
> + apk add libelf && \
> \
> # Xen
> cd / && \
>
> base-commit: ff1178062094837d55ef342070e58316c43a54c9
> prerequisite-patch-id: 477e3af5692ee0daa13d795fdf78384be604fd66
> prerequisite-patch-id: 60d13b1c04d8a808a42d20b3432270cfd87a47fc
> prerequisite-patch-id: 457b56a295e75d2d9f837b44cd483812ca66cd85
> --
> 2.30.2
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-12-05 23:06 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-05 22:56 [PATCH] CI: Fix fallout from adding elfutils-dev to the build container Andrew Cooper
2023-12-05 22:59 ` Stefano Stabellini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.