All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] tests/docker: fix ccache directory variable expansion
@ 2026-09-15  7:24 Anders Roxell
  2026-09-16 16:53 ` Alex Bennée
  0 siblings, 1 reply; 2+ messages in thread
From: Anders Roxell @ 2026-09-15  7:24 UTC (permalink / raw)
  To: alex.bennee, pierrick.bouvier; +Cc: qemu-devel, Anders Roxell

DOCKER_QEMU_CCACHE_DIR is missing $() around DOCKER_QEMU_CACHE_DIR, so
$HOME/.cache/qemu is never created.  podman then fails to mount it:

  Error: statfs /home/user/.cache/qemu: no such file or directory

It only works if the directory already exists.  Add the missing $().

Fixes: 8a1975e4d44b ("tests/docker: expose $HOME/.cache/qemu as docker volume")
Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
---
 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 0adddb6a5c62..e506add8aa8b 100644
--- a/tests/docker/Makefile.include
+++ b/tests/docker/Makefile.include
@@ -199,7 +199,7 @@ docker-help: docker
 # Where QEMU caches build artefacts
 DOCKER_QEMU_CACHE_DIR := $$HOME/.cache/qemu
 # Use a global constant ccache directory to speed up repetitive builds
-DOCKER_QEMU_CCACHE_DIR := DOCKER_QEMU_CACHE_DIR/docker-ccache
+DOCKER_QEMU_CCACHE_DIR := $(DOCKER_QEMU_CACHE_DIR)/docker-ccache
 
 # This rule if for directly running against an arbitrary docker target.
 # It is called by the expanded docker targets (e.g. make
-- 
2.53.0



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] tests/docker: fix ccache directory variable expansion
  2026-09-15  7:24 [PATCH] tests/docker: fix ccache directory variable expansion Anders Roxell
@ 2026-09-16 16:53 ` Alex Bennée
  0 siblings, 0 replies; 2+ messages in thread
From: Alex Bennée @ 2026-09-16 16:53 UTC (permalink / raw)
  To: Anders Roxell; +Cc: pierrick.bouvier, qemu-devel

Anders Roxell <anders.roxell@linaro.org> writes:

> DOCKER_QEMU_CCACHE_DIR is missing $() around DOCKER_QEMU_CACHE_DIR, so
> $HOME/.cache/qemu is never created.  podman then fails to mount it:
>
>   Error: statfs /home/user/.cache/qemu: no such file or directory
>
> It only works if the directory already exists.  Add the missing $().
>
> Fixes: 8a1975e4d44b ("tests/docker: expose $HOME/.cache/qemu as docker volume")
> Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
> ---
>  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 0adddb6a5c62..e506add8aa8b 100644
> --- a/tests/docker/Makefile.include
> +++ b/tests/docker/Makefile.include
> @@ -199,7 +199,7 @@ docker-help: docker
>  # Where QEMU caches build artefacts
>  DOCKER_QEMU_CACHE_DIR := $$HOME/.cache/qemu
>  # Use a global constant ccache directory to speed up repetitive builds
> -DOCKER_QEMU_CCACHE_DIR := DOCKER_QEMU_CACHE_DIR/docker-ccache
> +DOCKER_QEMU_CCACHE_DIR := $(DOCKER_QEMU_CACHE_DIR)/docker-ccache
>  
>  # This rule if for directly running against an arbitrary docker target.
>  # It is called by the expanded docker targets (e.g. make

Queued to testing/next, thanks.

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-09-16 16:54 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-15  7:24 [PATCH] tests/docker: fix ccache directory variable expansion Anders Roxell
2026-09-16 16:53 ` Alex Bennée

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.