diff options
author | Stefan Weil <sw@weilnetz.de> | 2022-11-17 17:25:20 +0000 |
---|---|---|
committer | Alex Bennée <alex.bennee@linaro.org> | 2022-11-22 09:48:52 +0000 |
commit | 47fdc8fb82fc8dd182b4923a69cefadc87419e0d (patch) | |
tree | 32be29592f27b27a524b42bdce19cf6f08b327f3 /configure | |
parent | 6d71357a3b651ec9db126e4862b77e13165427f5 (diff) | |
download | qemu-47fdc8fb82fc8dd182b4923a69cefadc87419e0d.zip qemu-47fdc8fb82fc8dd182b4923a69cefadc87419e0d.tar.gz qemu-47fdc8fb82fc8dd182b4923a69cefadc87419e0d.tar.bz2 |
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 <sw@weilnetz.de>
Message-Id: <20221030083510.310584-1-sw@weilnetz.de>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Message-Id: <20221117172532.538149-2-alex.bennee@linaro.org>
Diffstat (limited to 'configure')
-rwxr-xr-x | configure | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -1780,7 +1780,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 ;; |