From: Akira Yokosawa <akiyks@gmail.com>
To: "Paul E. McKenney" <paulmck@kernel.org>
Cc: perfbook@vger.kernel.org, Akira Yokosawa <akiyks@gmail.com>
Subject: [PATCH -perfbook 0/6] Update Dockerfiles
Date: Thu, 15 Jun 2023 18:48:00 +0900 [thread overview]
Message-ID: <feeecfa4-ed23-bede-5487-49129030431f@gmail.com> (raw)
Hi Paul,
Commit a80a57b21a5e ("fixsvgfonts.sh: Convert sans-serif into 'DejaVu Sans'")
assumed that having "DejaVu Sans Mono" is sufficent for "DejaVu Sans".
It turns out that docker/Dockerfile.fedora doesn't cover "DejaVu Sans".
Fedora's dejavu-sans-mono-fonts package doesn't contain "DejaVu Sans Mono"!
There is a meta package of the name dejavu-fonts-all which covers them
both as well as dejavu-serif-fonts.
It also turns out that Answer to #9 in FAQ-BUILD.txt saying:
As for Ubuntu and Fedora, packages listed in #5 should cover
all the font families needed.
is wrong on the Fedora front. DejaVu and Liberation fonts need
font packages dejavu-fonts-all and liberation-fonts.
Furthermore, Fedora 38 (released this April) has a regression of
Inkscape where font markup is corrupted in SVG --> PDF conversion.
So Deckerfile.fedora should stay with Fedora 37 for the moment.
Quick search of upstream Inkscape's issue tracker didn't hit the
font markup regression. I'll open an issue there unless somebody
else beats me to it.
So, this patch set consists of the following
Patch 1/6 adds check of "DejaVu Sans" as nice-to-have font.
Patch 2/6 updates FAQ-BUILD.txt and Dockerfile.fedora for nice-to-have
font packages.
Patch 3/6 updates Dockerfile.fedora to stay with Fedora 37, using an ARG
variable so that building from fedora:38 can be possible.
It also adds popller-utils to check the font markup info in PDF properties.
Patch 4/6 updates Dockerfile (from ubuntu) so that the default base
image is the "latest" (jammy at the moment). There is no reason to stick
with focal (20.04).
Patch 5/6 adds poppler-utils to Dockerfile (from ubuntu) to detect
Inkscape regression early in case it actually sneaks in.
(Note: Ubuntu 23.04 is safe at the moment.)
Patch 6/6 changes the default uid:gid pair of container images built
from Dockerfiles to 0:0, which is for rootless mode docker/podman.
Repository at dockerhub for prebuilt images is changed to
akiyks/perfbook-build, where the tags "latest" and "fedora" have
uid:gid pair of 0:0.
Thanks, Akira
--
Akira Yokosawa (6):
Makefile: Add 'DejaVu Sans' to nice-to-have fonts
FAQ-BUILD: Update nice-to-have fonts for SVG figures
docker/Dockerfile.fedora: Stay with Fedora 37 for the moment
docker/Dockerfile: Use 'latest' as the default tag
docker/Dockerfile: Add poppler-utils package
Dockerfile: Make uid:gid = 0:0 the default
FAQ-BUILD.txt | 37 ++++++++++++++++++++-----------------
Makefile | 15 ++++++++++++---
docker/Dockerfile | 9 +++++----
docker/Dockerfile.fedora | 16 +++++++++++-----
4 files changed, 48 insertions(+), 29 deletions(-)
base-commit: a80a57b21a5e8ea3dfa90471126920a1131682f7
--
2.25.1
next reply other threads:[~2023-06-15 9:48 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-15 9:48 Akira Yokosawa [this message]
2023-06-15 9:49 ` [PATCH -perfbook 1/6] Makefile: Add 'DejaVu Sans' to nice-to-have fonts Akira Yokosawa
2023-06-15 9:51 ` [PATCH -perfbook 2/6] FAQ-BUILD: Update nice-to-have fonts for SVG figures Akira Yokosawa
2023-06-15 9:52 ` [PATCH -perfbook 3/6] docker/Dockerfile.fedora: Stay with Fedora 37 for the moment Akira Yokosawa
2023-06-15 9:53 ` [PATCH -perfbook 4/6] docker/Dockerfile: Use 'latest' as the default tag Akira Yokosawa
2023-06-15 9:54 ` [PATCH -perfbook 5/6] docker/Dockerfile: Add poppler-utils package Akira Yokosawa
2023-06-15 9:56 ` [PATCH -perfbook 6/6] Dockerfile: Make uid:gid = 0:0 the default Akira Yokosawa
2023-06-15 10:43 ` [PATCH -perfbook 0/6] Update Dockerfiles Akira Yokosawa
2023-06-16 6:07 ` Leonardo Brás
2023-06-16 11:05 ` Akira Yokosawa
2023-09-22 23:52 ` Akira Yokosawa
2023-09-27 2:49 ` Leonardo Brás
2023-06-15 16:49 ` Paul E. McKenney
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=feeecfa4-ed23-bede-5487-49129030431f@gmail.com \
--to=akiyks@gmail.com \
--cc=paulmck@kernel.org \
--cc=perfbook@vger.kernel.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 read-only IMAP folder(s) and NNTP newsgroup(s).