diff options
| author | Alex Bennée <alex.bennee@linaro.org> | 2019-09-03 10:33:39 +0100 |
|---|---|---|
| committer | Alex Bennée <alex.bennee@linaro.org> | 2019-09-10 09:38:33 +0100 |
| commit | 8480517d4cd33627f7edd3a18829ddd11550a0f3 (patch) | |
| tree | 4f16f01b87027151ab191c609fd2c15566022f9d /tests/docker | |
| parent | 4030289a44bc5f3b4f728d080bda1678c171fb6e (diff) | |
configure: clean-up container cross compile detect
The introduction of podman support inadvertently broke configure's
detect of the container support as the configure probe didn't specify
an engine type. To fix this in docker.py:
- only (re)set USE_ENGINE if --engine is specified
- enhance the output so docker is no longer just yes
In the configure script we can at least start cleaning up the
detecting and naming of variables. To avoid too much churn the
conversion of the various make DOCKER_foo variables has been left for
future clean-ups.
Fixes: 9459f754134b
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Tested-by: David Hildenbrand <david@redhat.com>
Diffstat (limited to 'tests/docker')
| -rwxr-xr-x | tests/docker/docker.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/tests/docker/docker.py b/tests/docker/docker.py index 4bba29e104..e23209f71e 100755 --- a/tests/docker/docker.py +++ b/tests/docker/docker.py @@ -536,9 +536,9 @@ class ProbeCommand(SubCommand): try: docker = Docker() if docker._command[0] == "docker": - print("yes") + print("docker") elif docker._command[0] == "sudo": - print("sudo") + print("sudo docker") elif docker._command[0] == "podman": print("podman") except Exception: @@ -651,7 +651,8 @@ def main(): cmd.args(subp) subp.set_defaults(cmdobj=cmd) args, argv = parser.parse_known_args() - USE_ENGINE = args.engine + if args.engine: + USE_ENGINE = args.engine return args.cmdobj.run(args, argv) |