[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] docker: Fix git-archive for submodules
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH] docker: Fix git-archive for submodules |
Date: |
Mon, 6 Jun 2016 15:43:33 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.0 |
On 03/06/2016 10:42, Fam Zheng wrote:
> If dtc or pixman submodule is initialized, "git archive" in the subshell
> gets a relative filename against $(SRC_PATH), while we have already
> cd'ed into the submodule directory.
>
> Prefix it with $(CURDIR), which is the full path to the working
> directory.
>
> Signed-off-by: Fam Zheng <address@hidden>
> ---
> tests/docker/Makefile.include | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/docker/Makefile.include b/tests/docker/Makefile.include
> index 2fd2ca3..3788cd8 100644
> --- a/tests/docker/Makefile.include
> +++ b/tests/docker/Makefile.include
> @@ -29,7 +29,7 @@ make-archive-maybe = $(if $(wildcard $1/*), \
>
> CUR_TIME := $(shell date +%Y-%m-%d-%H.%M.%S.$$$$)
> # Makes the definition constant after the first expansion
> -DOCKER_SRC_COPY = $(eval DOCKER_SRC_COPY :=
> docker-src.$(CUR_TIME))$(DOCKER_SRC_COPY)
> +DOCKER_SRC_COPY = $(eval DOCKER_SRC_COPY :=
> $(CURDIR)/docker-src.$(CUR_TIME))$(DOCKER_SRC_COPY)
>
> $(DOCKER_SRC_COPY):
> @mkdir $@
>
Good, another possibility is to use "> $2" instead of "-o $2" in
git-archive. I was going to send a patch now for this possibility, you
can choose the one you prefer.
Thanks,
Paolo