From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Xen-devel <xen-devel@lists.xenproject.org>
Cc: Andrew Cooper <andrew.cooper3@citrix.com>,
Ian Jackson <iwj@xenproject.org>, Wei Liu <wl@xen.org>,
Doug Goldstein <cardoe@cardoe.com>
Subject: [PATCH 2/2] CI: Add zstd/etc libraries to build environments
Date: Fri, 26 Mar 2021 13:45:58 +0000 [thread overview]
Message-ID: <20210326134558.7751-3-andrew.cooper3@citrix.com> (raw)
In-Reply-To: <20210326134558.7751-1-andrew.cooper3@citrix.com>
zstd compression support is new in 4.15, but isn't currently tested anywhere
in CI.
bzip2/zlib was missing in some cases, so add those as well.
Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
---
CC: Ian Jackson <iwj@xenproject.org>
CC: Wei Liu <wl@xen.org>
CC: Doug Goldstein <cardoe@cardoe.com>
For 4.15. Add build time testing for a currently untested feature. This
should have been part of the series which included 8169f82049efb but I don't
think it really qualifies for a Fixes: tag.
RFC for two reasons. First, this needs extending to other containers when
I've finished chasing package names, and the CI runs haven't completed yet.
---
.cirrus.yml | 2 +-
automation/build/alpine/3.12.dockerfile | 2 ++
automation/build/ubuntu/focal.dockerfile | 2 ++
3 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/.cirrus.yml b/.cirrus.yml
index fdb1c9c94d..05e72b92db 100644
--- a/.cirrus.yml
+++ b/.cirrus.yml
@@ -4,7 +4,7 @@ freebsd_template: &FREEBSD_TEMPLATE
APPEND_LIB: /usr/local/lib
APPEND_INCLUDES: /usr/local/include
- install_script: pkg install -y seabios gmake
+ install_script: pkg install -y seabios gmake zlib bzip2 zstd
pkgconf python libiconv bison perl5
yajl lzo2 pixman argp-standalone
libxml2 glib git
diff --git a/automation/build/alpine/3.12.dockerfile b/automation/build/alpine/3.12.dockerfile
index 8631d221db..8d61f3860b 100644
--- a/automation/build/alpine/3.12.dockerfile
+++ b/automation/build/alpine/3.12.dockerfile
@@ -38,6 +38,8 @@ RUN \
apk add xz-dev && \
apk add yajl-dev && \
apk add zlib-dev && \
+ apk add bzip2-dev && \
+ apk add zstd-dev && \
\
# qemu build deps
apk add bison && \
diff --git a/automation/build/ubuntu/focal.dockerfile b/automation/build/ubuntu/focal.dockerfile
index 6c4565745b..0558519a97 100644
--- a/automation/build/ubuntu/focal.dockerfile
+++ b/automation/build/ubuntu/focal.dockerfile
@@ -30,6 +30,8 @@ RUN apt-get update && \
bin86 \
bcc \
liblzma-dev \
+ libzstd-dev \
+ libbz2-dev \
# libc6-dev-i386 for Xen < 4.15
libc6-dev-i386 \
libnl-3-dev \
--
2.11.0
next prev parent reply other threads:[~2021-03-26 13:46 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-26 13:45 [PATCH for-4.15 0/2] CI adjustments Andrew Cooper
2021-03-26 13:45 ` [PATCH 1/2] tools: Drop gettext as a build dependency Andrew Cooper
2021-03-26 14:11 ` Ian Jackson
2021-03-26 15:27 ` Andrew Cooper
2021-03-29 9:01 ` Roger Pau Monné
2021-03-26 13:45 ` Andrew Cooper [this message]
2021-03-26 14:10 ` [PATCH 2/2] CI: Add zstd/etc libraries to build environments Ian Jackson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210326134558.7751-3-andrew.cooper3@citrix.com \
--to=andrew.cooper3@citrix.com \
--cc=cardoe@cardoe.com \
--cc=iwj@xenproject.org \
--cc=wl@xen.org \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).