diff mbox series

[v2] testsuite: Pattern does not match when using --specs=nano.specs

Message ID 20240208174431.833656-1-torbjorn.svensson@foss.st.com
State New
Headers show
Series [v2] testsuite: Pattern does not match when using --specs=nano.specs | expand

Commit Message

Torbjörn SVENSSON Feb. 8, 2024, 5:44 p.m. UTC
Changes since v1:
- Replaced .* with [^\r\n]* to avoid matching newline.


Ok for trunk and releases/gcc-13?

--

When running the testsuite for newlib nano, the --specs=nano.specs
option is used.  This option prepends cpp_unique_options with
"-isystem =/include/newlib-nano" so that the newlib nano header files
override the newlib standard ones.  As the -isystem option is prepended,
the -quiet option is no longer the first option to cc1.  Adjust the test
accordingly.

Patch has been verified on Windows and Linux.

gcc/testsuite/ChangeLog:

	* gcc.misc-tests/options.exp: Allow other options before the
	-quite option for cc1.

Signed-off-by: Torbjörn SVENSSON <torbjorn.svensson@foss.st.com>
---
 gcc/testsuite/gcc.misc-tests/options.exp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Mike Stump Feb. 9, 2024, 12:07 a.m. UTC | #1
On Feb 8, 2024, at 9:44 AM, Torbjörn SVENSSON <torbjorn.svensson@foss.st.com> wrote:
> 
> Changes since v1:
> - Replaced .* with [^\r\n]* to avoid matching newline.
> 
> Ok for trunk and releases/gcc-13?

Ok.
Torbjörn SVENSSON Feb. 9, 2024, 8:47 a.m. UTC | #2
On 2024-02-09 01:07, Mike Stump wrote:
> On Feb 8, 2024, at 9:44 AM, Torbjörn SVENSSON <torbjorn.svensson@foss.st.com> wrote:
>>
>> Changes since v1:
>> - Replaced .* with [^\r\n]* to avoid matching newline.
>>
>> Ok for trunk and releases/gcc-13?
> 
> Ok.

Pushed as 1175d1b35ce7bf8ee7c9b37b334370f01eb95335 and 
810b0b3f75c454da3f6b5722870716796d2d7a83.
diff mbox series

Patch

diff --git a/gcc/testsuite/gcc.misc-tests/options.exp b/gcc/testsuite/gcc.misc-tests/options.exp
index ec026ecf77d..6e6e40c183d 100644
--- a/gcc/testsuite/gcc.misc-tests/options.exp
+++ b/gcc/testsuite/gcc.misc-tests/options.exp
@@ -57,7 +57,7 @@  proc check_for_all_options {language gcc_options compiler_pattern as_pattern ld_
 	remote_file build delete $dumpfile
     }   
 
-    if {![regexp -- "/${compiler}(\\.exe)? -quiet.*$compiler_pattern" $gcc_output]} {
+    if {![regexp -- "/${compiler}(\\.exe)? \[^\r\n\]*-quiet.*$compiler_pattern" $gcc_output]} {
 	fail "$test (compiler options)"
 	return
     }