[PATCH v2] Mark mtrace tests UNSUPPORTED if bug-ga2.mtrace or tst-leaks2.mtrace are missing

[PATCH v2] Mark mtrace tests UNSUPPORTED if bug-ga2.mtrace or tst-leaks2.mtrace are missing

[Stefan Liebler]

Starting with commit 29fddfc7dfd6444fa61a256e9a0d0127545e1f2e, the
tests posix/bug-ga2 and resolv/tst-leaks2 are test-container tests.

If test-container.c returns with EXIT_UNSUPPORTED, the tests with
mtrace() are not executed and the mtrace files do not exist.

Therefore the "mtrace-analysis-part" of those tests are marked
UNSUPPORTED if the mtrace files are missing.

Reported-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
---
 posix/Makefile  | 4 +++-
 resolv/Makefile | 4 +++-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/posix/Makefile b/posix/Makefile
index fa2d0675cd..a69a38f795 100644
--- a/posix/Makefile
+++ b/posix/Makefile
@@ -358,7 +358,9 @@ $(objpfx)tst-getconf.out: tst-getconf.sh $(objpfx)getconf
 	$(evaluate-test)
 
 $(objpfx)bug-ga2-mem.out: $(objpfx)bug-ga2.out
-	$(common-objpfx)malloc/mtrace $(objpfx)bug-ga2.mtrace > $@; \
+	test -r $(objpfx)bug-ga2.mtrace \
+	|| ( echo "bug-ga2.mtrace does not exist" > $@; exit 77; ) \
+	&& $(common-objpfx)malloc/mtrace $(objpfx)bug-ga2.mtrace > $@; \
 	$(evaluate-test)
 
 bug-ga2-ENV = MALLOC_TRACE=$(objpfx)bug-ga2.mtrace
diff --git a/resolv/Makefile b/resolv/Makefile
index 462c111e13..1d7ffd868e 100644
--- a/resolv/Makefile
+++ b/resolv/Makefile
@@ -158,7 +158,9 @@ $(objpfx)mtrace-tst-leaks.out: $(objpfx)tst-leaks.out
 
 tst-leaks2-ENV = MALLOC_TRACE=$(objpfx)tst-leaks2.mtrace
 $(objpfx)mtrace-tst-leaks2.out: $(objpfx)tst-leaks2.out
-	$(common-objpfx)malloc/mtrace $(objpfx)tst-leaks2.mtrace > $@; \
+	test -r $(objpfx)tst-leaks2.mtrace \
+	|| ( echo "tst-leaks2.mtrace does not exist" > $@; exit 77; ) \
+	&& $(common-objpfx)malloc/mtrace $(objpfx)tst-leaks2.mtrace > $@; \
 	$(evaluate-test)
 
 tst-resolv-res_ninit-ENV = MALLOC_TRACE=$(objpfx)tst-resolv-res_ninit.mtrace
-- 
2.23.0

Re: [PATCH v2] Mark mtrace tests UNSUPPORTED if bug-ga2.mtrace or tst-leaks2.mtrace are missing

[Andreas Schwab]

On Nov 24 2020, Stefan Liebler via Libc-alpha wrote:

> diff --git a/posix/Makefile b/posix/Makefile
> index fa2d0675cd..a69a38f795 100644
> --- a/posix/Makefile
> +++ b/posix/Makefile
> @@ -358,7 +358,9 @@ $(objpfx)tst-getconf.out: tst-getconf.sh $(objpfx)getconf
>  	$(evaluate-test)
>  
>  $(objpfx)bug-ga2-mem.out: $(objpfx)bug-ga2.out
> -	$(common-objpfx)malloc/mtrace $(objpfx)bug-ga2.mtrace > $@; \
> +	test -r $(objpfx)bug-ga2.mtrace \
> +	|| ( echo "bug-ga2.mtrace does not exist" > $@; exit 77; ) \
> +	&& $(common-objpfx)malloc/mtrace $(objpfx)bug-ga2.mtrace > $@; \

You can avoid the duplication of the redirection by adding a group
around the command list.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510  2552 DF73 E780 A9DA AEC1
"And now for something completely different."

Re: [PATCH v2] Mark mtrace tests UNSUPPORTED if bug-ga2.mtrace or tst-leaks2.mtrace are missing

[Stefan Liebler]

On 11/24/20 11:47 AM, Andreas Schwab wrote:
> On Nov 24 2020, Stefan Liebler via Libc-alpha wrote:
> 
>> diff --git a/posix/Makefile b/posix/Makefile
>> index fa2d0675cd..a69a38f795 100644
>> --- a/posix/Makefile
>> +++ b/posix/Makefile
>> @@ -358,7 +358,9 @@ $(objpfx)tst-getconf.out: tst-getconf.sh $(objpfx)getconf
>>  	$(evaluate-test)
>>  
>>  $(objpfx)bug-ga2-mem.out: $(objpfx)bug-ga2.out
>> -	$(common-objpfx)malloc/mtrace $(objpfx)bug-ga2.mtrace > $@; \
>> +	test -r $(objpfx)bug-ga2.mtrace \
>> +	|| ( echo "bug-ga2.mtrace does not exist" > $@; exit 77; ) \
>> +	&& $(common-objpfx)malloc/mtrace $(objpfx)bug-ga2.mtrace > $@; \
> 
> You can avoid the duplication of the redirection by adding a group
> around the command list.
> 
> Andreas.
> 

okay. See in v3:
"[PATCH v3] Mark mtrace tests UNSUPPORTED if bug-ga2.mtrace or
tst-leaks2.mtrace are missing"
https://sourceware.org/pipermail/libc-alpha/2020-November/119953.html

Bye,
Stefan