Cherry-pick upstream fixes for TEST-74-AUX-UTILS

diff --git a/debian/patches/series b/debian/patches/series
index 661f0c480a264cfdceb00182173bca921dfae870..13d01a56bcd8882535c4afc227f44ba1c728afe2 100644 (file)
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,4 +1,7 @@
 p11kit-switch-to-dlopen.patch
+test-re-enable-skipped-systemd-firstboot-locale-messages-.patch
+test-cover-some-interactive-error-paths-in-firstboot.patch
+test-handle-Debian-s-etc-default-locale-in-testsuite-74.f.patch
 debian/Use-Debian-specific-config-files.patch
 debian/Bring-tmpfiles.d-tmp.conf-in-line-with-Debian-defaul.patch
 debian/Make-run-lock-tmpfs-an-API-fs.patch

diff --git a/debian/patches/test-cover-some-interactive-error-paths-in-firstboot.patch b/debian/patches/test-cover-some-interactive-error-paths-in-firstboot.patch
new file mode 100644 (file)
index 0000000..a1d7ce8
--- /dev/null
+++ b/debian/patches/test-cover-some-interactive-error-paths-in-firstboot.patch
@@ -0,0 +1,61 @@
+From: Frantisek Sumsal <frantisek@sumsal.cz>
+Date: Sun, 6 Nov 2022 22:40:33 +0100
+Subject: test: cover some interactive/error paths in firstboot
+
+(cherry picked from commit dbd8dbdfc1d440c44714edcfa513b697f39f4fed)
+---
+ test/units/testsuite-74.firstboot.sh | 30 +++++++++++++++++++++++++++++-
+ 1 file changed, 29 insertions(+), 1 deletion(-)
+
+diff --git a/test/units/testsuite-74.firstboot.sh b/test/units/testsuite-74.firstboot.sh
+index f17b35e..fdea34b 100755
+--- a/test/units/testsuite-74.firstboot.sh
++++ b/test/units/testsuite-74.firstboot.sh
+@@ -9,7 +9,7 @@ if ! command -v systemd-firstboot >/dev/null; then
+ fi
+ 
+ at_exit() {
+-    if [[ -v ROOT && -n "$ROOT" ]]; then
++    if [[ -n "${ROOT:-}" ]]; then
+         ls -lR "$ROOT"
+         rm -fr "$ROOT"
+     fi
+@@ -133,6 +133,30 @@ diff <(awk -F: '/^root/ { print $2; }' /etc/shadow) <(awk -F: '/^root/ { print $
+ [[ -e /etc/vconsole.conf ]] && diff /etc/vconsole.conf "$ROOT/etc/vconsole.conf"
+ [[ -e /etc/localtime ]] && diff <(readlink /etc/localtime) <(readlink "$ROOT/etc/localtime")
+ 
++# --prompt-* options
++rm -fr "$ROOT"
++mkdir -p "$ROOT/bin"
++touch "$ROOT/bin/fooshell" "$ROOT/bin/barshell"
++# We can do only limited testing here, since it's all an interactive stuff,
++# so --prompt and --prompt-root-password are skipped on purpose
++echo -ne "\nfoo\nbar\n" | systemd-firstboot --root="$ROOT" --prompt-locale
++grep -q "LANG=foo" "$ROOT/etc/locale.conf"
++grep -q "LC_MESSAGES=bar" "$ROOT/etc/locale.conf"
++echo -ne "\nfoo\n" | systemd-firstboot --root="$ROOT" --prompt-keymap
++grep -q "KEYMAP=foo" "$ROOT/etc/vconsole.conf"
++echo -ne "\nEurope/Berlin\n" | systemd-firstboot --root="$ROOT" --prompt-timezone
++readlink "$ROOT/etc/localtime" | grep -q "Europe/Berlin$"
++echo -ne "\nfoobar\n" | systemd-firstboot --root="$ROOT" --prompt-hostname
++grep -q "foobar" "$ROOT/etc/hostname"
++echo -ne "\n/bin/fooshell\n" | systemd-firstboot --root="$ROOT" --prompt-root-shell
++grep -q "^root:.*:0:0:.*:/bin/fooshell$" "$ROOT/etc/passwd"
++# Existing files should not get overwritten
++echo -ne "\n/bin/barshell\n" | systemd-firstboot --root="$ROOT" --prompt-root-shell
++grep -q "^root:.*:0:0:.*:/bin/fooshell$" "$ROOT/etc/passwd"
++# Now without the welcome screen but with force
++echo -ne "/bin/barshell\n" | systemd-firstboot --root="$ROOT" --force --prompt-root-shell --welcome=no
++grep -q "^root:.*:0:0:.*:/bin/barshell$" "$ROOT/etc/passwd"
++
+ # Assorted tests
+ rm -fr "$ROOT"
+ mkdir "$ROOT"
+@@ -142,3 +166,7 @@ grep -E "[a-z0-9]{32}" "$ROOT/etc/machine-id"
+ 
+ systemd-firstboot --root="$ROOT" --delete-root-password
+ diff <(echo) <(awk -F: '/^root/ { print $2; }' "$ROOT/etc/shadow")
++
++(! systemd-firstboot --root="$ROOT" --root-shell=/bin/nonexistentshell)
++(! systemd-firstboot --root="$ROOT" --machine-id=invalidmachineid)
++(! systemd-firstboot --root="$ROOT" --timezone=Foo/Bar)

diff --git a/debian/patches/test-handle-Debian-s-etc-default-locale-in-testsuite-74.f.patch b/debian/patches/test-handle-Debian-s-etc-default-locale-in-testsuite-74.f.patch
new file mode 100644 (file)
index 0000000..c4a285e
--- /dev/null
+++ b/debian/patches/test-handle-Debian-s-etc-default-locale-in-testsuite-74.f.patch
@@ -0,0 +1,104 @@
+From: Nick Rosbrook <nick.rosbrook@canonical.com>
+Date: Tue, 22 Nov 2022 12:50:33 -0500
+Subject: test: handle Debian's /etc/default/locale in
+ testsuite-74.firstboot.sh
+
+This handles a Debian-specific quirk where /etc/default/locale is used
+instead of /etc/locale.conf. There is currently special handling for
+this in testsuite-73.sh, so the quirk should be handled here too for
+consistency.
+
+(cherry picked from commit bb59fdc1e3a7119f3680d309147020fce9bf67b5)
+---
+ test/units/testsuite-74.firstboot.sh | 34 ++++++++++++++++++++--------------
+ 1 file changed, 20 insertions(+), 14 deletions(-)
+
+diff --git a/test/units/testsuite-74.firstboot.sh b/test/units/testsuite-74.firstboot.sh
+index fdea34b..36e265e 100755
+--- a/test/units/testsuite-74.firstboot.sh
++++ b/test/units/testsuite-74.firstboot.sh
+@@ -24,6 +24,12 @@ ROOT_HASHED_PASSWORD1='$6$foobarsalt$YbwdaATX6IsFxvWbY3QcZj2gB31R/LFRFrjlFrJtTTq
+ # shellcheck disable=SC2016
+ ROOT_HASHED_PASSWORD2='$6$foobarsalt$q.P2932zYMLbKnjFwIxPI8y3iuxeuJ2BgE372LcZMMnj3Gcg/9mJg2LPKUl.ha0TG/.fRNNnRQcLfzM0SNot3.'
+ 
++# Debian and Ubuntu use /etc/default/locale instead of /etc/locale.conf. Make
++# sure we use the appropriate path for locale configuration.
++LOCALE_PATH="/etc/locale.conf"
++[ -e "$LOCALE_PATH" ] || LOCALE_PATH="/etc/default/locale"
++[ -e "$LOCALE_PATH" ] || systemd-firstboot --locale=C.UTF-8
++
+ # Create a minimal root so we don't modify the testbed
+ ROOT=test-root
+ mkdir -p "$ROOT/bin"
+@@ -31,14 +37,14 @@ mkdir -p "$ROOT/bin"
+ touch "$ROOT/bin/fooshell" "$ROOT/bin/barshell"
+ 
+ systemd-firstboot --root="$ROOT" --locale=foo
+-grep -q "LANG=foo" "$ROOT/etc/locale.conf"
+-rm -fv "$ROOT/etc/locale.conf"
++grep -q "LANG=foo" "$ROOT$LOCALE_PATH"
++rm -fv "$ROOT$LOCALE_PATH"
+ systemd-firstboot --root="$ROOT" --locale-messages=foo
+-grep -q "LC_MESSAGES=foo" "$ROOT/etc/locale.conf"
+-rm -fv "$ROOT/etc/locale.conf"
++grep -q "LC_MESSAGES=foo" "$ROOT$LOCALE_PATH"
++rm -fv "$ROOT$LOCALE_PATH"
+ systemd-firstboot --root="$ROOT" --locale=foo --locale-messages=bar
+-grep -q "LANG=foo" "$ROOT/etc/locale.conf"
+-grep -q "LC_MESSAGES=bar" "$ROOT/etc/locale.conf"
++grep -q "LANG=foo" "$ROOT$LOCALE_PATH"
++grep -q "LC_MESSAGES=bar" "$ROOT$LOCALE_PATH"
+ 
+ systemd-firstboot --root="$ROOT" --keymap=foo
+ grep -q "KEYMAP=foo" "$ROOT/etc/vconsole.conf"
+@@ -82,8 +88,8 @@ systemd-firstboot --root="$ROOT" \
+                   --root-password-hashed="$ROOT_HASHED_PASSWORD2" \
+                   --root-shell=/bin/barshell \
+                   --kernel-command-line="hello.world=0"
+-grep -q "LANG=foo" "$ROOT/etc/locale.conf"
+-grep -q "LC_MESSAGES=bar" "$ROOT/etc/locale.conf"
++grep -q "LANG=foo" "$ROOT$LOCALE_PATH"
++grep -q "LC_MESSAGES=bar" "$ROOT$LOCALE_PATH"
+ grep -q "KEYMAP=foo" "$ROOT/etc/vconsole.conf"
+ readlink "$ROOT/etc/localtime" | grep -q "Europe/Berlin$"
+ grep -q "foobar" "$ROOT/etc/hostname"
+@@ -103,8 +109,8 @@ systemd-firstboot --root="$ROOT" --force \
+                   --root-password-hashed="$ROOT_HASHED_PASSWORD2" \
+                   --root-shell=/bin/barshell \
+                   --kernel-command-line="hello.world=0"
+-grep -q "LANG=locale-overwrite" "$ROOT/etc/locale.conf"
+-grep -q "LC_MESSAGES=messages-overwrite" "$ROOT/etc/locale.conf"
++grep -q "LANG=locale-overwrite" "$ROOT$LOCALE_PATH"
++grep -q "LC_MESSAGES=messages-overwrite" "$ROOT$LOCALE_PATH"
+ grep -q "KEYMAP=keymap-overwrite" "$ROOT/etc/vconsole.conf"
+ readlink "$ROOT/etc/localtime" | grep -q "/CET$"
+ grep -q "hostname-overwrite" "$ROOT/etc/hostname"
+@@ -118,7 +124,7 @@ rm -fr "$ROOT"
+ mkdir "$ROOT"
+ # Copy everything at once (--copy)
+ systemd-firstboot --root="$ROOT" --copy
+-diff /etc/locale.conf "$ROOT/etc/locale.conf"
++diff $LOCALE_PATH "$ROOT$LOCALE_PATH"
+ diff <(awk -F: '/^root/ { print $7; }' /etc/passwd) <(awk -F: '/^root/ { print $7; }' "$ROOT/etc/passwd")
+ diff <(awk -F: '/^root/ { print $2; }' /etc/shadow) <(awk -F: '/^root/ { print $2; }' "$ROOT/etc/shadow")
+ [[ -e /etc/vconsole.conf ]] && diff /etc/vconsole.conf "$ROOT/etc/vconsole.conf"
+@@ -127,7 +133,7 @@ rm -fr "$ROOT"
+ mkdir "$ROOT"
+ # Copy everything at once, but now by using separate switches
+ systemd-firstboot --root="$ROOT" --copy-locale --copy-keymap --copy-timezone --copy-root-password --copy-root-shell
+-diff /etc/locale.conf "$ROOT/etc/locale.conf"
++diff $LOCALE_PATH "$ROOT$LOCALE_PATH"
+ diff <(awk -F: '/^root/ { print $7; }' /etc/passwd) <(awk -F: '/^root/ { print $7; }' "$ROOT/etc/passwd")
+ diff <(awk -F: '/^root/ { print $2; }' /etc/shadow) <(awk -F: '/^root/ { print $2; }' "$ROOT/etc/shadow")
+ [[ -e /etc/vconsole.conf ]] && diff /etc/vconsole.conf "$ROOT/etc/vconsole.conf"
+@@ -140,8 +146,8 @@ touch "$ROOT/bin/fooshell" "$ROOT/bin/barshell"
+ # We can do only limited testing here, since it's all an interactive stuff,
+ # so --prompt and --prompt-root-password are skipped on purpose
+ echo -ne "\nfoo\nbar\n" | systemd-firstboot --root="$ROOT" --prompt-locale
+-grep -q "LANG=foo" "$ROOT/etc/locale.conf"
+-grep -q "LC_MESSAGES=bar" "$ROOT/etc/locale.conf"
++grep -q "LANG=foo" "$ROOT$LOCALE_PATH"
++grep -q "LC_MESSAGES=bar" "$ROOT$LOCALE_PATH"
+ echo -ne "\nfoo\n" | systemd-firstboot --root="$ROOT" --prompt-keymap
+ grep -q "KEYMAP=foo" "$ROOT/etc/vconsole.conf"
+ echo -ne "\nEurope/Berlin\n" | systemd-firstboot --root="$ROOT" --prompt-timezone

diff --git a/debian/patches/test-re-enable-skipped-systemd-firstboot-locale-messages-.patch b/debian/patches/test-re-enable-skipped-systemd-firstboot-locale-messages-.patch
new file mode 100644 (file)
index 0000000..f852722
--- /dev/null
+++ b/debian/patches/test-re-enable-skipped-systemd-firstboot-locale-messages-.patch
@@ -0,0 +1,29 @@
+From: Frantisek Sumsal <frantisek@sumsal.cz>
+Date: Fri, 4 Nov 2022 09:44:34 +0100
+Subject: test: re-enable skipped systemd-firstboot --locale-messages= test
+
+Since the original issue should be resolved by #25253.
+
+(cherry picked from commit 59377dbef288bff2e9df1254f9cbccd3c7d726a5)
+---
+ test/units/testsuite-74.firstboot.sh | 7 +++----
+ 1 file changed, 3 insertions(+), 4 deletions(-)
+
+diff --git a/test/units/testsuite-74.firstboot.sh b/test/units/testsuite-74.firstboot.sh
+index 02f9f5c..f17b35e 100755
+--- a/test/units/testsuite-74.firstboot.sh
++++ b/test/units/testsuite-74.firstboot.sh
+@@ -33,10 +33,9 @@ touch "$ROOT/bin/fooshell" "$ROOT/bin/barshell"
+ systemd-firstboot --root="$ROOT" --locale=foo
+ grep -q "LANG=foo" "$ROOT/etc/locale.conf"
+ rm -fv "$ROOT/etc/locale.conf"
+-# FIXME: https://github.com/systemd/systemd/issues/25249
+-#systemd-firstboot --root="$ROOT" --locale-messages=foo
+-#grep -q "LC_MESSAGES=foo" "$ROOT/etc/locale.conf"
+-#rm -fv "$ROOT/etc/locale.conf"
++systemd-firstboot --root="$ROOT" --locale-messages=foo
++grep -q "LC_MESSAGES=foo" "$ROOT/etc/locale.conf"
++rm -fv "$ROOT/etc/locale.conf"
+ systemd-firstboot --root="$ROOT" --locale=foo --locale-messages=bar
+ grep -q "LANG=foo" "$ROOT/etc/locale.conf"
+ grep -q "LC_MESSAGES=bar" "$ROOT/etc/locale.conf"