[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 02/24] docker: add --include-files argument t
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v3 02/24] docker: add --include-files argument to 'build' command |
Date: |
Wed, 24 May 2017 13:21:23 +0800 |
User-agent: |
Mutt/1.8.0 (2017-02-23) |
On Sun, 05/21 00:29, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> tests/docker/Makefile.include | 5 ++++-
> tests/docker/docker.py | 12 +++++++++---
> 2 files changed, 13 insertions(+), 4 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..68cca25f89 100755
> --- a/tests/docker/docker.py
> +++ b/tests/docker/docker.py
> @@ -237,6 +237,11 @@ 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 sent to the
> + Docker daemon. The daemon will copy those files
> into
> + the built image. The ADD directive of the
> Dockerfile
> + specify where a file is placed into the image""")
Or more precisely, "The daemon will copy these files into the docker build
directory, which can be copied to the built image with ADD directives in
Dockerfile"?
> parser.add_argument("--add-current-user", "-u", dest="user",
> action="store_true",
> help="Add the current user to image's passwd")
> @@ -270,10 +275,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.
> 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()
> --
> 2.11.0
>
[Qemu-devel] [PATCH v3 04/24] docker: add 'apt-fake' script which generate fake debian packages, Philippe Mathieu-Daudé, 2017/05/20
[Qemu-devel] [PATCH v3 05/24] docker: install ca-certificates package in base image, Philippe Mathieu-Daudé, 2017/05/20
[Qemu-devel] [PATCH v3 06/24] docker: create deb-src entry and setup Emdebian in the same layer, Philippe Mathieu-Daudé, 2017/05/20
[Qemu-devel] [PATCH v3 10/24] docker: use eatmydata in debian arm64 image, Philippe Mathieu-Daudé, 2017/05/20