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

Message ID 20211119155914.9269-1-iain@sandoe.co.uk
State Committed
Commit 532fae74c3b1197023a765272892e12e123f0a56
Headers
Series libphobos, testsuite: Add prune clauses for two Darwin cases. |

Commit Message

Iain Sandoe Nov. 19, 2021, 3:59 p.m. UTC
  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(+)
  

Comments

Iain Buclaw Nov. 19, 2021, 7:18 p.m. UTC | #1
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)
> 
>
  

Patch

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
 }