[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 19/20] docker: add special rule for deboostra
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v3 19/20] docker: add special rule for deboostrapped images |
Date: |
Mon, 2 Jul 2018 17:32:47 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
On 07/02/2018 11:30 AM, Alex Bennée wrote:
> We might as well have a custom rule for this. For one thing the
> dependencies are different. As the primary dependency for
> docker-image-% could never be docker-image-debian-bootstrap we can
> drop that test in the main rule as well.
>
> Signed-off-by: Alex Bennée <address@hidden>
>
> ---
> v3:
> - changed rule to docker-binfmt-image-debian-%:
> - leave executable in the main rule
> - rm stray whitespace fix
> ---
> tests/docker/Makefile.include | 23 ++++++++++++++++++++---
> 1 file changed, 20 insertions(+), 3 deletions(-)
>
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> index 8641f5da2c..fdc87b8a57 100644
> --- a/tests/docker/Makefile.include
> +++ b/tests/docker/Makefile.include
> @@ -49,9 +49,6 @@ docker-image-%: $(DOCKER_FILES_DIR)/%.docker
> "CHECK", "$*")
> else
> docker-image-%: $(DOCKER_FILES_DIR)/%.docker
> - @if test "$@" = docker-image-debian-bootstrap -a -z "$(EXECUTABLE)";
> then \
> - echo WARNING: EXECUTABLE is not set, debootstrap may fail. 2>&1
> ; \
> - fi
> $(call quiet-command,\
> $(DOCKER_SCRIPT) build qemu:$* $< \
> $(if $V,,--quiet) $(if $(NOCACHE),--no-cache) \
> @@ -59,6 +56,26 @@ docker-image-%: $(DOCKER_FILES_DIR)/%.docker
> $(if $(EXTRA_FILES),--extra-files $(EXTRA_FILES))\
> $(if $(EXECUTABLE),--include-executable=$(EXECUTABLE)),\
> "BUILD","$*")
> +
> +# Special rule for debootstraped binfmt linux-user images
> +docker-binfmt-image-debian-%: $(DOCKER_FILES_DIR)/debian-bootstrap.docker
> + @if test -z "$(EXECUTABLE)"; then \
> + echo WARNING: EXECUTABLE is not set, debootstrap may fail. 2>&1
> ; \
> + fi
> + @if test -z "$(DEB_ARCH)"; then
> \
> + echo WARNING: DEB_ARCH is not set, debootstrap may fail. 2>&1 ;
> \
> + fi
> + @if test -z "$(DEB_TYPE)"; then
> \
> + echo WARNING: DEB_TYPE is not set, debootstrap may fail. 2>&1 ;
> \
> + fi
> + $(call quiet-command,
> \
Here I had to add:
+ DEB_ARCH=$(DEB_ARCH) \
+ DEB_TYPE=$(DEB_TYPE) \
> + $(DOCKER_SCRIPT) build qemu:debian-$* $< \
> + $(if $V,,--quiet) $(if $(NOCACHE),--no-cache) \
> + $(if $(NOUSER),,--add-current-user) \
> + $(if $(EXTRA_FILES),--extra-files $(EXTRA_FILES))\
> + $(if $(EXECUTABLE),--include-executable=$(EXECUTABLE)), \
> + "BUILD","binfmt debian-$* (debootstrapped)")
> +
> endif
>
> # Enforce dependencies for composite images
>
- [Qemu-devel] [PATCH v3 14/20] docker: Clean the MXE base image, (continued)
- [Qemu-devel] [PATCH v3 14/20] docker: Clean the MXE base image, Alex Bennée, 2018/07/02
- [Qemu-devel] [PATCH v3 08/20] build-system: add coverage-report target, Alex Bennée, 2018/07/02
- [Qemu-devel] [PATCH v3 11/20] docker: filter out linux-user builds for mingw, Alex Bennée, 2018/07/02
- [Qemu-devel] [PATCH v3 20/20] docker: add linux-user powered cross builder for QEMU, Alex Bennée, 2018/07/02
- [Qemu-devel] [PATCH v3 16/20] docker: drop QEMU build-dep from bootstrap, Alex Bennée, 2018/07/02
- [Qemu-devel] [PATCH v3 12/20] docker: ubuntu: Update the package list before installing new ones, Alex Bennée, 2018/07/02
- [Qemu-devel] [PATCH v3 19/20] docker: add special rule for deboostrapped images, Alex Bennée, 2018/07/02
- Re: [Qemu-devel] [PATCH v3 19/20] docker: add special rule for deboostrapped images,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH v3 17/20] docker: debian-bootstrap.pre allow customising of variant/url, Alex Bennée, 2018/07/02
- [Qemu-devel] [PATCH v3 15/20] docker: Do not run tests in 'intermediate' images, Alex Bennée, 2018/07/02
- [Qemu-devel] [PATCH v3 18/20] docker: add special handling for FROM:debian-%-user targets, Alex Bennée, 2018/07/02
- Re: [Qemu-devel] [PATCH v3 00/20] Travis, Code Coverage and Cross Build updates, no-reply, 2018/07/02