diff mbox series

testsuite: Remove .exe suffix in prune_gcc_output

Message ID 20210908084956.1444294-1-christophe.lyon@foss.st.com
State New
Headers show
Series testsuite: Remove .exe suffix in prune_gcc_output | expand

Commit Message

Christophe Lyon Sept. 8, 2021, 8:49 a.m. UTC
When running the testsuite under Windows, we noticed failures in
testcase which attempt to match compiler error messages containing the
name of the executable.

For instance, gcc.dg/analyzer/signal-4a.c tries to match 'cc1:' which
obviously fails when the executable is called cc1.exe.

This patch removes the .exe suffix from various toolchain executables
to avoid this problem.

2021-09-08  Christophe Lyon  <christophe.lyon@foss.st.com>
	    Torbjörn SVENSSON <torbjorn.svensson@st.com>

	gcc/testsuite/
	* lib/prune.exp (prune_gcc_output): Remove .exe suffix from
	toolchain executables names.
---
 gcc/testsuite/lib/prune.exp | 3 +++
 1 file changed, 3 insertions(+)

Comments

Christophe Lyon Sept. 13, 2021, 8:31 a.m. UTC | #1
ping?


On 08/09/2021 10:49, Christophe Lyon wrote:
> When running the testsuite under Windows, we noticed failures in
> testcase which attempt to match compiler error messages containing the
> name of the executable.
>
> For instance, gcc.dg/analyzer/signal-4a.c tries to match 'cc1:' which
> obviously fails when the executable is called cc1.exe.
>
> This patch removes the .exe suffix from various toolchain executables
> to avoid this problem.
>
> 2021-09-08  Christophe Lyon  <christophe.lyon@foss.st.com>
> 	    Torbjörn SVENSSON <torbjorn.svensson@st.com>
>
> 	gcc/testsuite/
> 	* lib/prune.exp (prune_gcc_output): Remove .exe suffix from
> 	toolchain executables names.
> ---
>   gcc/testsuite/lib/prune.exp | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/gcc/testsuite/lib/prune.exp b/gcc/testsuite/lib/prune.exp
> index 91f165bec38..fac212ecf60 100644
> --- a/gcc/testsuite/lib/prune.exp
> +++ b/gcc/testsuite/lib/prune.exp
> @@ -37,6 +37,9 @@ proc prune_gcc_output { text } {
>       # Handle any freeform regexps.
>       set text [handle-dg-regexps $text]
>   
> +    # Remove Windows .exe suffix
> +    regsub -all "(as|cc1|cc1plus|collect2|f951|ld|lto-wrapper)\.exe?:" $text {\1:} text
> +
>       regsub -all "(^|\n)(\[^\n\]*: \[iI\]|I)n ((static member |lambda )?function|member|method|(copy )?constructor|destructor|instantiation|substitution|program|subroutine|block-data)\[^\n\]*" $text "" text
>       regsub -all "(^|\n)\[^\n\]*(: )?At (top level|global scope):\[^\n\]*" $text "" text
>       regsub -all "(^|\n)\[^\n\]*:   (recursively )?required \[^\n\]*" $text "" text
Jeff Law Sept. 19, 2021, 2:46 p.m. UTC | #2
On 9/8/2021 2:49 AM, Christophe Lyon via Gcc-patches wrote:
> When running the testsuite under Windows, we noticed failures in
> testcase which attempt to match compiler error messages containing the
> name of the executable.
>
> For instance, gcc.dg/analyzer/signal-4a.c tries to match 'cc1:' which
> obviously fails when the executable is called cc1.exe.
>
> This patch removes the .exe suffix from various toolchain executables
> to avoid this problem.
>
> 2021-09-08  Christophe Lyon  <christophe.lyon@foss.st.com>
> 	    Torbjörn SVENSSON <torbjorn.svensson@st.com>
>
> 	gcc/testsuite/
> 	* lib/prune.exp (prune_gcc_output): Remove .exe suffix from
> 	toolchain executables names.
OK
jeff
diff mbox series

Patch

diff --git a/gcc/testsuite/lib/prune.exp b/gcc/testsuite/lib/prune.exp
index 91f165bec38..fac212ecf60 100644
--- a/gcc/testsuite/lib/prune.exp
+++ b/gcc/testsuite/lib/prune.exp
@@ -37,6 +37,9 @@  proc prune_gcc_output { text } {
     # Handle any freeform regexps.
     set text [handle-dg-regexps $text]
 
+    # Remove Windows .exe suffix
+    regsub -all "(as|cc1|cc1plus|collect2|f951|ld|lto-wrapper)\.exe?:" $text {\1:} text
+
     regsub -all "(^|\n)(\[^\n\]*: \[iI\]|I)n ((static member |lambda )?function|member|method|(copy )?constructor|destructor|instantiation|substitution|program|subroutine|block-data)\[^\n\]*" $text "" text
     regsub -all "(^|\n)\[^\n\]*(: )?At (top level|global scope):\[^\n\]*" $text "" text
     regsub -all "(^|\n)\[^\n\]*:   (recursively )?required \[^\n\]*" $text "" text