[PATCH] libphobos, testsuite: Add prune clauses for two Darwin cases.

[PATCH] libphobos, testsuite: Add prune clauses for two Darwin cases.

[Iain Sandoe]

Depending on the permutation of CPU, OS version and shared/non-
shared library inclusion, we get can get two warnings from the
external tools (ld64, dsymutil) which are not actually GCC issues
but relate to the external tools.  These are already pruned in
the main testsuite, this adds them to the library.

tested on x86_64,i686-darwin17 where the problem shows up.
OK for master / backports?
thanks
Iain

Signed-off-by: Iain Sandoe <iain@sandoe.co.uk>

libphobos/ChangeLog:

	* testsuite/lib/libphobos.exp: Prune warnings from external
	tool bugs.
---
 libphobos/testsuite/lib/libphobos.exp | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/libphobos/testsuite/lib/libphobos.exp b/libphobos/testsuite/lib/libphobos.exp
index 3be2092b12e..2af430a0e45 100644
--- a/libphobos/testsuite/lib/libphobos.exp
+++ b/libphobos/testsuite/lib/libphobos.exp
@@ -90,6 +90,13 @@ proc libphobos-dg-test { prog do_what extra_tool_flags } {
 }
 
 proc libphobos-dg-prune { system text } {
+
+    # Ignore harmless warnings from Xcode.
+    regsub -all "(^|\n)\[^\n\]*ld: warning: could not create compact unwind for\[^\n\]*" $text "" text
+
+    # Ignore dsymutil warning (tool bug is actually linker)
+    regsub -all "(^|\n)\[^\n\]*could not find object file symbol for symbol\[^\n\]*" $text "" text
+
     return $text
 }
 
-- 
2.24.3 (Apple Git-128)

Re: [PATCH] libphobos, testsuite: Add prune clauses for two Darwin cases.

[Iain Buclaw]

Excerpts from Iain Sandoe's message of November 19, 2021 4:59 pm:
> Depending on the permutation of CPU, OS version and shared/non-
> shared library inclusion, we get can get two warnings from the
> external tools (ld64, dsymutil) which are not actually GCC issues
> but relate to the external tools.  These are already pruned in
> the main testsuite, this adds them to the library.
> 
> tested on x86_64,i686-darwin17 where the problem shows up.
> OK for master / backports?
> thanks
> Iain
> 

OK from me.
Iain.


> Signed-off-by: Iain Sandoe <iain@sandoe.co.uk>
> 
> libphobos/ChangeLog:
> 
> 	* testsuite/lib/libphobos.exp: Prune warnings from external
> 	tool bugs.
> ---
>  libphobos/testsuite/lib/libphobos.exp | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/libphobos/testsuite/lib/libphobos.exp b/libphobos/testsuite/lib/libphobos.exp
> index 3be2092b12e..2af430a0e45 100644
> --- a/libphobos/testsuite/lib/libphobos.exp
> +++ b/libphobos/testsuite/lib/libphobos.exp
> @@ -90,6 +90,13 @@ proc libphobos-dg-test { prog do_what extra_tool_flags } {
>  }
>  
>  proc libphobos-dg-prune { system text } {
> +
> +    # Ignore harmless warnings from Xcode.
> +    regsub -all "(^|\n)\[^\n\]*ld: warning: could not create compact unwind for\[^\n\]*" $text "" text
> +
> +    # Ignore dsymutil warning (tool bug is actually linker)
> +    regsub -all "(^|\n)\[^\n\]*could not find object file symbol for symbol\[^\n\]*" $text "" text
> +
>      return $text
>  }
>  
> -- 
> 2.24.3 (Apple Git-128)
> 
>