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 |
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.
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 --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 }
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(-)