From: Thomas Petazzoni via buildroot <buildroot@buildroot.org>
To: Fiona Klute via buildroot <buildroot@buildroot.org>
Cc: Fiona Klute <fiona.klute@gmx.de>,
Christian Stewart <christian@aperture.us>
Subject: Re: [Buildroot] [PATCH 1/2] package/tini: restore use as docker-init
Date: Thu, 10 Oct 2024 23:40:41 +0200 [thread overview]
Message-ID: <20241010234041.0aade127@windsurf> (raw)
In-Reply-To: <20241010132912.3061663-1-fiona.klute@gmx.de>
On Thu, 10 Oct 2024 15:29:11 +0200
Fiona Klute via buildroot <buildroot@buildroot.org> wrote:
> From: "Fiona Klute (WIWA)" <fiona.klute@gmx.de>
>
> Commit 24cac9c4e6a36fd14b263a8eecf7ae7d58ab5a59 "system: add tini as
> init system" made BR2_PACKAGE_TINI depend on BR2_INIT_TINI. This is
> incorrect: tini is not only needed when building a container image
> using Buildroot, but also to run Docker containers on Buildroot
> systems using another another init system, if Docker is configured to
> provide an init process for the container (e.g. with "docker run
> --init"). The tini package already provides the "docker-init" symlink
> required for that.
>
> Example error without tini installed:
>
> $ docker run --rm -ti --init alpine
> docker: Error response from daemon: exec: "docker-init": executable file not found in $PATH.
>
> Signed-off-by: Fiona Klute (WIWA) <fiona.klute@gmx.de>
> ---
> package/tini/Config.in | 1 -
> 1 file changed, 1 deletion(-)
Thanks, both applied. On PATCH 2/2, I wasn't 100% sure, but it's simple
enough, clarifies this dependency that isn't entirely obvious, and can
always be reworked/reverted if other people think it wasn't the right
solution.
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
prev parent reply other threads:[~2024-10-10 21:40 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-10 13:29 [Buildroot] [PATCH 1/2] package/tini: restore use as docker-init Fiona Klute via buildroot
2024-10-10 13:29 ` [Buildroot] [PATCH 2/2] package/docker-engine: add optional dependency on tini Fiona Klute via buildroot
2024-10-10 18:25 ` [Buildroot] [PATCH 1/2] package/tini: restore use as docker-init Christian Stewart via buildroot
2024-10-10 21:40 ` Thomas Petazzoni via buildroot [this message]
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=20241010234041.0aade127@windsurf \
--to=buildroot@buildroot.org \
--cc=christian@aperture.us \
--cc=fiona.klute@gmx.de \
--cc=thomas.petazzoni@bootlin.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 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.