[PATCH v2 4/6] Makefile: Do not install the container testroot if not running tests

[Sam James]

Adhemerval Zanella <adhemerval.zanella@linaro.org> writes:

> Every subdirectory tests target depends on
> $(objpfx)testroot.pristine/install.stamp, whose recipe performs a full
> DESTDIR installation (about 12s and 154MB on x86_64).  The testroot is
> only consumed by the container tests, which do not run when
> run-built-tests is no, so skip it entirely in that case.

I'd noticed that and didn't debug it, my bad..

>
> This saves about 10s on x86_64 [1] and 30s on aarch64 [2].
>
> [1] Ryzen 5900x, 12c/24t, gcc 16.1.1, binutils 2.26, and GNU make 4.3
> [2] N1, 80c, gcc 15.1.1, binutils 2.25, GNU make 4.3

Reviewed-by: Sam James <sam@gentoo.org>

> ---
>  Makefile | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/Makefile b/Makefile
> index a6e5317de7b..79dbe0f40d3 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -792,8 +792,13 @@ else
>  LINKS_DSO_PROGRAM = links-dso-program
>  endif
>  
> +# The testroot is only used by the container tests, which are not run
> +# when run-built-tests is no; skip the installation entirely in that
> +# case.
> +ifeq ($(run-built-tests),yes)
>  $(tests-container) $(addsuffix /tests,$(subdirs)) : \
>  		$(objpfx)testroot.pristine/install.stamp
> +endif
>  $(objpfx)testroot.pristine/install.stamp :
>  	test -d $(objpfx)testroot.pristine || \
>  	  mkdir $(objpfx)testroot.pristine
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 418 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/libc-alpha/attachments/20260703/eb361f92/attachment.sig>