[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 02/24] docker: add --include-files argument t
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH v4 02/24] docker: add --include-files argument to 'build' command |
Date: |
Wed, 31 May 2017 17:31:17 +0100 |
User-agent: |
mu4e 0.9.19; emacs 25.2.50.2 |
Philippe Mathieu-Daudé <address@hidden> writes:
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> tests/docker/Makefile.include | 3 +++
> tests/docker/docker.py | 11 ++++++++---
> 2 files changed, 11 insertions(+), 3 deletions(-)
>
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> index 03eda37bf4..fe1a9a53ff 100644
> --- a/tests/docker/Makefile.include
> +++ b/tests/docker/Makefile.include
> @@ -51,6 +51,7 @@ docker-image-%: $(DOCKER_FILES_DIR)/%.docker
> $(SRC_PATH)/tests/docker/docker.py build qemu:$* $< \
> $(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","$*")
>
> @@ -107,6 +108,8 @@ docker:
> @echo ' NOUSER Define to disable adding current user
> to containers passwd.'
> @echo ' NOCACHE=1 Ignore cache when build images.'
> @echo ' EXECUTABLE=<path> Include executable in image.'
> + @echo ' EXTRA_FILES="<path> [... <path>]"'
> + @echo ' Include extra files in image.'
>
> # This rule if for directly running against an arbitrary docker target.
> # It is called by the expanded docker targets (e.g. make
> diff --git a/tests/docker/docker.py b/tests/docker/docker.py
> index 6ddc6e4c2a..5401e58fce 100755
> --- a/tests/docker/docker.py
> +++ b/tests/docker/docker.py
> @@ -237,6 +237,10 @@ class BuildCommand(SubCommand):
> help="""Specify a binary that will be copied to
> the
> container together with all its dependent
> libraries""")
> + parser.add_argument("--extra-files", "-f", nargs='*',
> + help="""Specify files that will be copied in the
> + Docker image, fulfilling the ADD directive from
> the
> + Dockerfile""")
> parser.add_argument("--add-current-user", "-u", dest="user",
> action="store_true",
> help="Add the current user to image's passwd")
> @@ -270,10 +274,11 @@ class BuildCommand(SubCommand):
> print "%s exited with code %d" % (docker_pre, rc)
> return 1
>
> - # Do we include a extra binary?
> + # Include files used by ADD directives found within the
> Dockerfile.
Minor nit, as we are not actually parsing the Dockerfile here maybe it
should be re-worded to something like:
"Copy any extra files into the Docker context. These can be included by the
use
of the ADD directive in the Dockerfile."?
Anyway:
Reviewed-by: Alex Bennée <address@hidden>
> if args.include_executable:
> - _copy_binary_with_libs(args.include_executable,
> - docker_dir)
> + _copy_binary_with_libs(args.include_executable, docker_dir)
> + for filename in args.extra_files or []:
> + _copy_with_mkdir(filename, docker_dir)
>
> argv += ["--build-arg=" + k.lower() + "=" + v
> for k, v in os.environ.iteritems()
--
Alex Bennée
- [Qemu-devel] [PATCH v4 00/24] docker/shippable: cross-build mipsel and powerpc targets, Philippe Mathieu-Daudé, 2017/05/27
- [Qemu-devel] [PATCH v4 01/24] docker: let _copy_with_mkdir() sub_path argument be optional, Philippe Mathieu-Daudé, 2017/05/27
- [Qemu-devel] [PATCH v4 02/24] docker: add --include-files argument to 'build' command, Philippe Mathieu-Daudé, 2017/05/27
- Re: [Qemu-devel] [PATCH v4 02/24] docker: add --include-files argument to 'build' command,
Alex Bennée <=
- [Qemu-devel] [PATCH v4 03/24] docker: rebuild image if 'extra files' checksum does not match, Philippe Mathieu-Daudé, 2017/05/27
- [Qemu-devel] [PATCH v4 04/24] docker: add 'apt-fake' script which generate fake debian packages, Philippe Mathieu-Daudé, 2017/05/27
- [Qemu-devel] [PATCH v4 05/24] docker: install ca-certificates package in base image, Philippe Mathieu-Daudé, 2017/05/27
- [Qemu-devel] [PATCH v4 06/24] docker: create deb-src entry and setup Emdebian in the same layer, Philippe Mathieu-Daudé, 2017/05/27
- [Qemu-devel] [PATCH v4 08/24] docker: use eatmydata, install common build packages in base image, Philippe Mathieu-Daudé, 2017/05/27
- [Qemu-devel] [PATCH v4 10/24] docker: use eatmydata in debian arm64 image, Philippe Mathieu-Daudé, 2017/05/27
- [Qemu-devel] [PATCH v4 09/24] docker: use eatmydata in debian armhf image, Philippe Mathieu-Daudé, 2017/05/27
- [Qemu-devel] [PATCH v4 07/24] docker: use better regex to generate deb-src entries, Philippe Mathieu-Daudé, 2017/05/27
- [Qemu-devel] [PATCH v4 11/24] docker: add extra libs to armhf target to extend codebase coverage, Philippe Mathieu-Daudé, 2017/05/27