Re: [PATCH] Run docker probe only if docker or podman are available
Stefan Weil writes: > The docker probe uses "sudo -n" which can cause an e-mail with a security > warning > each time when configure is run. Therefore run docker probe only if either > docker > or podman are available. > > That avoids the problematic "sudo -n" on build environments which have neither > docker nor podman installed. Queued to for-7.2/misc-fixes, thanks. -- Alex Bennée
Re: [PATCH] Run docker probe only if docker or podman are available
Am 30.10.22 um 09:35 schrieb Stefan Weil: The docker probe uses "sudo -n" which can cause an e-mail with a security warning each time when configure is run. Therefore run docker probe only if either docker or podman are available. That avoids the problematic "sudo -n" on build environments which have neither docker nor podman installed. Fixes: c4575b59155e2e00 ("configure: store container engine in config-host.mak") Signed-off-by: Stefan Weil --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 81561be7c1..3af99282b7 100755 --- a/configure +++ b/configure @@ -1779,7 +1779,7 @@ fi # functions to probe cross compilers container="no" -if test $use_containers = "yes"; then +if test $use_containers = "yes" && (has "docker" || has "podman"); then case $($python "$source_path"/tests/docker/docker.py probe) in *docker) container=docker ;; podman) container=podman ;; Can this patch be applied by qemu-trivial? For me those security e-mails are a bug and should be at least avoided as far as possible in the new QEMU release. Thanks, Stefan
[PATCH] Run docker probe only if docker or podman are available
The docker probe uses "sudo -n" which can cause an e-mail with a security warning each time when configure is run. Therefore run docker probe only if either docker or podman are available. That avoids the problematic "sudo -n" on build environments which have neither docker nor podman installed. Fixes: c4575b59155e2e00 ("configure: store container engine in config-host.mak") Signed-off-by: Stefan Weil --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 81561be7c1..3af99282b7 100755 --- a/configure +++ b/configure @@ -1779,7 +1779,7 @@ fi # functions to probe cross compilers container="no" -if test $use_containers = "yes"; then +if test $use_containers = "yes" && (has "docker" || has "podman"); then case $($python "$source_path"/tests/docker/docker.py probe) in *docker) container=docker ;; podman) container=podman ;; -- 2.30.2