From: "Yann E. MORIN" <yann.morin.1998@free.fr>
To: James Hilliard <james.hilliard1@gmail.com>
Cc: Norbert Lange <nolange79@gmail.com>,
Sen Hastings <sen@phobosdpl.com>,
buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 1/1] package/systemd: select kdb
Date: Tue, 1 Aug 2023 22:01:54 +0200 [thread overview]
Message-ID: <20230801200154.GE421096@scaer> (raw)
In-Reply-To: <20230731214559.99648-1-james.hilliard1@gmail.com>
James, All,
On 2023-07-31 15:45 -0600, James Hilliard spake thusly:
> We need to select BR2_PACKAGE_KBD when building systemd with vconsole.
>
> Fixes:
> systemd-vconsole-setup[129]: /usr/bin/loadkeys failed with exit status 1.
>
> Signed-off-by: James Hilliard <james.hilliard1@gmail.com>
> ---
> package/systemd/Config.in | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/package/systemd/Config.in b/package/systemd/Config.in
> index bea717a456..f3e561e661 100644
> --- a/package/systemd/Config.in
> +++ b/package/systemd/Config.in
> @@ -563,6 +563,7 @@ config BR2_PACKAGE_SYSTEMD_UTMP
> config BR2_PACKAGE_SYSTEMD_VCONSOLE
> bool "enable vconsole tool"
> default y
> + select BR2_PACKAGE_KBD
This is trivial to explain, indeed, but what is not trivial to explan
but most interesting, and as I did ask in my preview mail, is to explain
why:
- in systemd 252, loadkeys was already needed and called from
systemd-vconsole-setup (check the code, it was already there in 252
and most probably even long before that...)
- with systemd 252, our test cases were not failing, even though
loadkeys was needed
- with systemd 254, the same test case now fails.
This we need to understand.
That way, we know if our test was broken, or if the new dependency
should be backported to previous maintenance branches...
(and now I see that Thomas had the same questions).
> help
> systemd-vconsole-setup is an early boot service that
> configures the virtual console font and console keymap.
> --
> 2.34.1
>
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
next prev parent reply other threads:[~2023-08-01 20:02 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-07-31 21:45 [Buildroot] [PATCH 1/1] package/systemd: select kdb James Hilliard
2023-07-31 21:57 ` Thomas Petazzoni via buildroot
2023-07-31 22:07 ` James Hilliard
2023-07-31 22:18 ` Thomas Petazzoni via buildroot
2023-08-01 20:01 ` Yann E. MORIN [this message]
2023-08-01 21:09 ` James Hilliard
2023-08-02 14:11 ` Yann E. MORIN
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=20230801200154.GE421096@scaer \
--to=yann.morin.1998@free.fr \
--cc=buildroot@buildroot.org \
--cc=james.hilliard1@gmail.com \
--cc=nolange79@gmail.com \
--cc=sen@phobosdpl.com \
/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