Re: [Qemu-devel] [RFC PATCH 5/8] docker: Restrict the 'travis' job to the Travis image

[Alex Bennée]

Philippe Mathieu-Daudé <address@hidden> writes:

> We can still run any test in Travis.


I don't know about this one. The travis job isn't a fill in for real
travis, but a way to work through the matrix. I don't think it needs to
be restricted to any particular image.

>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
>  tests/docker/Makefile.include | 10 ++++++++--
>  1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> index 91d9665517..7d9d568eee 100644
> --- a/tests/docker/Makefile.include
> +++ b/tests/docker/Makefile.include
> @@ -99,11 +99,17 @@ docker-image-tricore-cross: docker-image-debian9
>
>  # Expand all the pre-requistes for each docker image and test combination
>  $(foreach i,$(DOCKER_IMAGES) $(DOCKER_DEPRECATED_IMAGES), \
> -     $(foreach t,$(DOCKER_TESTS) $(DOCKER_TOOLS), \
> +     $(foreach t,$(DOCKER_TESTS), \
>               $(eval .PHONY: address@hidden) \
>               $(eval address@hidden: docker-image-$i address@hidden) \
> +             $(eval docker-test: address@hidden) \
>       ) \
> -     $(foreach t,$(DOCKER_TESTS), \
> +)
> +# we only run Travis tests on the Travis image
> +$(foreach i,travis, \
> +     $(foreach t,$(DOCKER_TOOLS), \
> +             $(eval .PHONY: address@hidden) \
> +             $(eval address@hidden: docker-image-$i address@hidden) \
>               $(eval docker-test: address@hidden) \
>       ) \
>  )


--
Alex Bennée