[PATCH] Harmonize headers between both dg-extract-results scripts

[PATCH] Harmonize headers between both dg-extract-results scripts

[Paul Iannetta]

Hi,

This is a small patch so that both dg-extract-results.py and
dg-extract-results.sh share the same header.  In particular, it fixes
the fact that the regexp r'^Test Run By (\S+) on (.*)$' was never
matched in the python file.

Thanks,
--
Paul
Kalray


Author: Paul Iannetta <piannetta@kalrayinc.com>
Date:   Thu Sep 14 15:43:58 2023 +0200

    Harmonize headers between both dg-extract-results scripts

    The header of the python version looked like:
    Target is ...
    Host   is ...
    The header of the bash version looked like:
    Test run by ... on ...
    Target is ...

    After this change both headers look like:
    Test run by ... on ...
    Target is ...
    Host   is ...

    The order of the tests is not the same but since dg-cmp-results.sh it
    does not matter much.

    contrib/ChangeLog:

    2023-09-14  Paul Iannetta  <piannetta@kalrayinc.com>

            * dg-extract-results.py: Print the "Test run" line.
            * dg-extract-results.sh: Print the "Host" line.

diff --git a/contrib/dg-extract-results.py b/contrib/dg-extract-results.py
index 30aa68771d4..34da1808c5f 100644
--- a/contrib/dg-extract-results.py
+++ b/contrib/dg-extract-results.py
@@ -113,7 +113,7 @@ class Prog:
         # Whether to create .sum rather than .log output.
         self.do_sum = True
         # Regexps used while parsing.
-        self.test_run_re = re.compile (r'^Test Run By (\S+) on (.*)$')
+        self.test_run_re = re.compile (r'^Test run by (\S+) on (.*)$')
         self.tool_re = re.compile (r'^\t\t=== (.*) tests ===$')
         self.result_re = re.compile (r'^(PASS|XPASS|FAIL|XFAIL|UNRESOLVED'
                                      r'|WARNING|ERROR|UNSUPPORTED|UNTESTED'
diff --git a/contrib/dg-extract-results.sh b/contrib/dg-extract-results.sh
index ff6c50d029c..57f6fe0e997 100755
--- a/contrib/dg-extract-results.sh
+++ b/contrib/dg-extract-results.sh
@@ -271,7 +271,7 @@ cat $SUM_FILES \
 
 # Write the begining of the combined summary file.
 
-head -n 2 $FIRST_SUM
+head -n 3 $FIRST_SUM
 echo
 echo "		=== $TOOL tests ==="
 echo

Re: [PATCH] Harmonize headers between both dg-extract-results scripts

[Paul Iannetta]

On Thu, Sep 14, 2023 at 04:24:33PM +0200, Paul Iannetta wrote:
> Hi,
> 
> This is a small patch so that both dg-extract-results.py and
> dg-extract-results.sh share the same header.  In particular, it fixes
> the fact that the regexp r'^Test Run By (\S+) on (.*)$' was never
> matched in the python file.

By the way, the bash script dg-extract-results.sh checks whether
python is available by invoking python.  However, it seems that the
policy on newer machines is to not provide python as a symlink (at
least on Ubuntu 22.04 and above; and RHEL 8).  Therefore, we might
want to also check against python3 so that the bash script does not
fail to find python even though it is available.

Thanks,
Paul


> Author: Paul Iannetta <piannetta@kalrayinc.com>
> Date:   Thu Sep 14 15:43:58 2023 +0200
> 
>     Harmonize headers between both dg-extract-results scripts
> 
>     The header of the python version looked like:
>     Target is ...
>     Host   is ...
>     The header of the bash version looked like:
>     Test run by ... on ...
>     Target is ...
> 
>     After this change both headers look like:
>     Test run by ... on ...
>     Target is ...
>     Host   is ...
> 
>     The order of the tests is not the same but since dg-cmp-results.sh it
>     does not matter much.
> 
>     contrib/ChangeLog:
> 
>     2023-09-14  Paul Iannetta  <piannetta@kalrayinc.com>
> 
>             * dg-extract-results.py: Print the "Test run" line.
>             * dg-extract-results.sh: Print the "Host" line.
> 
> diff --git a/contrib/dg-extract-results.py b/contrib/dg-extract-results.py
> index 30aa68771d4..34da1808c5f 100644
> --- a/contrib/dg-extract-results.py
> +++ b/contrib/dg-extract-results.py
> @@ -113,7 +113,7 @@ class Prog:
>          # Whether to create .sum rather than .log output.
>          self.do_sum = True
>          # Regexps used while parsing.
> -        self.test_run_re = re.compile (r'^Test Run By (\S+) on (.*)$')
> +        self.test_run_re = re.compile (r'^Test run by (\S+) on (.*)$')
>          self.tool_re = re.compile (r'^\t\t=== (.*) tests ===$')
>          self.result_re = re.compile (r'^(PASS|XPASS|FAIL|XFAIL|UNRESOLVED'
>                                       r'|WARNING|ERROR|UNSUPPORTED|UNTESTED'
> diff --git a/contrib/dg-extract-results.sh b/contrib/dg-extract-results.sh
> index ff6c50d029c..57f6fe0e997 100755
> --- a/contrib/dg-extract-results.sh
> +++ b/contrib/dg-extract-results.sh
> @@ -271,7 +271,7 @@ cat $SUM_FILES \
>  
>  # Write the begining of the combined summary file.
>  
> -head -n 2 $FIRST_SUM
> +head -n 3 $FIRST_SUM
>  echo
>  echo "		=== $TOOL tests ==="
>  echo

[PING] [PATCH] Harmonize headers between both dg-extract-results scripts

[Paul Iannetta]

On Mon, Sep 18, 2023 at 08:39:34AM +0200, Paul Iannetta wrote:
> On Thu, Sep 14, 2023 at 04:24:33PM +0200, Paul Iannetta wrote:
> > Hi,
> > 
> > This is a small patch so that both dg-extract-results.py and
> > dg-extract-results.sh share the same header.  In particular, it fixes
> > the fact that the regexp r'^Test Run By (\S+) on (.*)$' was never
> > matched in the python file.
> 
> By the way, the bash script dg-extract-results.sh checks whether
> python is available by invoking python.  However, it seems that the
> policy on newer machines is to not provide python as a symlink (at
> least on Ubuntu 22.04 and above; and RHEL 8).  Therefore, we might
> want to also check against python3 so that the bash script does not
> fail to find python even though it is available.
> 
> Thanks,
> Paul
> 
> 
> > Author: Paul Iannetta <piannetta@kalrayinc.com>
> > Date:   Thu Sep 14 15:43:58 2023 +0200
> > 
> >     Harmonize headers between both dg-extract-results scripts
> > 
> >     The header of the python version looked like:
> >     Target is ...
> >     Host   is ...
> >     The header of the bash version looked like:
> >     Test run by ... on ...
> >     Target is ...
> > 
> >     After this change both headers look like:
> >     Test run by ... on ...
> >     Target is ...
> >     Host   is ...
> > 
> >     The order of the tests is not the same but since dg-cmp-results.sh it
> >     does not matter much.
> > 
> >     contrib/ChangeLog:
> > 
> >     2023-09-14  Paul Iannetta  <piannetta@kalrayinc.com>
> > 
> >             * dg-extract-results.py: Print the "Test run" line.
> >             * dg-extract-results.sh: Print the "Host" line.
> > 
> > diff --git a/contrib/dg-extract-results.py b/contrib/dg-extract-results.py
> > index 30aa68771d4..34da1808c5f 100644
> > --- a/contrib/dg-extract-results.py
> > +++ b/contrib/dg-extract-results.py
> > @@ -113,7 +113,7 @@ class Prog:
> >          # Whether to create .sum rather than .log output.
> >          self.do_sum = True
> >          # Regexps used while parsing.
> > -        self.test_run_re = re.compile (r'^Test Run By (\S+) on (.*)$')
> > +        self.test_run_re = re.compile (r'^Test run by (\S+) on (.*)$')
> >          self.tool_re = re.compile (r'^\t\t=== (.*) tests ===$')
> >          self.result_re = re.compile (r'^(PASS|XPASS|FAIL|XFAIL|UNRESOLVED'
> >                                       r'|WARNING|ERROR|UNSUPPORTED|UNTESTED'
> > diff --git a/contrib/dg-extract-results.sh b/contrib/dg-extract-results.sh
> > index ff6c50d029c..57f6fe0e997 100755
> > --- a/contrib/dg-extract-results.sh
> > +++ b/contrib/dg-extract-results.sh
> > @@ -271,7 +271,7 @@ cat $SUM_FILES \
> >  
> >  # Write the begining of the combined summary file.
> >  
> > -head -n 2 $FIRST_SUM
> > +head -n 3 $FIRST_SUM
> >  echo
> >  echo "		=== $TOOL tests ==="
> >  echo

Re: [PING] [PATCH] Harmonize headers between both dg-extract-results scripts

[Jeff Law]

On 9/25/23 03:55, Paul Iannetta wrote:
> On Mon, Sep 18, 2023 at 08:39:34AM +0200, Paul Iannetta wrote:
>> On Thu, Sep 14, 2023 at 04:24:33PM +0200, Paul Iannetta wrote:
>>> Hi,
>>>
>>> This is a small patch so that both dg-extract-results.py and
>>> dg-extract-results.sh share the same header.  In particular, it fixes
>>> the fact that the regexp r'^Test Run By (\S+) on (.*)$' was never
>>> matched in the python file.
>>
>> By the way, the bash script dg-extract-results.sh checks whether
>> python is available by invoking python.  However, it seems that the
>> policy on newer machines is to not provide python as a symlink (at
>> least on Ubuntu 22.04 and above; and RHEL 8).  Therefore, we might
>> want to also check against python3 so that the bash script does not
>> fail to find python even though it is available.
>>
>> Thanks,
>> Paul
>>
>>
>>> Author: Paul Iannetta <piannetta@kalrayinc.com>
>>> Date:   Thu Sep 14 15:43:58 2023 +0200
>>>
>>>      Harmonize headers between both dg-extract-results scripts
>>>
>>>      The header of the python version looked like:
>>>      Target is ...
>>>      Host   is ...
>>>      The header of the bash version looked like:
>>>      Test run by ... on ...
>>>      Target is ...
>>>
>>>      After this change both headers look like:
>>>      Test run by ... on ...
>>>      Target is ...
>>>      Host   is ...
>>>
>>>      The order of the tests is not the same but since dg-cmp-results.sh it
>>>      does not matter much.
>>>
>>>      contrib/ChangeLog:
>>>
>>>      2023-09-14  Paul Iannetta  <piannetta@kalrayinc.com>
>>>
>>>              * dg-extract-results.py: Print the "Test run" line.
>>>              * dg-extract-results.sh: Print the "Host" line.
OK
jeff

Re: [PING] [PATCH] Harmonize headers between both dg-extract-results scripts

[Paul Iannetta]

On Tue, Sep 26, 2023 at 08:29:11AM -0600, Jeff Law wrote:
> 
> 
> On 9/25/23 03:55, Paul Iannetta wrote:
> > On Mon, Sep 18, 2023 at 08:39:34AM +0200, Paul Iannetta wrote:
> > > On Thu, Sep 14, 2023 at 04:24:33PM +0200, Paul Iannetta wrote:
> > > > Hi,
> > > > 
> > > > This is a small patch so that both dg-extract-results.py and
> > > > dg-extract-results.sh share the same header.  In particular, it fixes
> > > > the fact that the regexp r'^Test Run By (\S+) on (.*)$' was never
> > > > matched in the python file.
> > > 
> > > By the way, the bash script dg-extract-results.sh checks whether
> > > python is available by invoking python.  However, it seems that the
> > > policy on newer machines is to not provide python as a symlink (at
> > > least on Ubuntu 22.04 and above; and RHEL 8).  Therefore, we might
> > > want to also check against python3 so that the bash script does not
> > > fail to find python even though it is available.
> > > 
> > > Thanks,
> > > Paul
> > > 
> > > 
> > > > Author: Paul Iannetta <piannetta@kalrayinc.com>
> > > > Date:   Thu Sep 14 15:43:58 2023 +0200
> > > > 
> > > >      Harmonize headers between both dg-extract-results scripts
> > > > 
> > > >      The header of the python version looked like:
> > > >      Target is ...
> > > >      Host   is ...
> > > >      The header of the bash version looked like:
> > > >      Test run by ... on ...
> > > >      Target is ...
> > > > 
> > > >      After this change both headers look like:
> > > >      Test run by ... on ...
> > > >      Target is ...
> > > >      Host   is ...
> > > > 
> > > >      The order of the tests is not the same but since dg-cmp-results.sh it
> > > >      does not matter much.
> > > > 
> > > >      contrib/ChangeLog:
> > > > 
> > > >      2023-09-14  Paul Iannetta  <piannetta@kalrayinc.com>
> > > > 
> > > >              * dg-extract-results.py: Print the "Test run" line.
> > > >              * dg-extract-results.sh: Print the "Host" line.
> OK
> jeff
> 
Thanks,
May I ask to you commit it to trunk on my behalf?

Paul

(Kalray has a corporate copyright assignment with the FSF, so there
should be no licensing problem)

Re: [PING] [PATCH] Harmonize headers between both dg-extract-results scripts

[Jeff Law]

On 9/29/23 02:19, Paul Iannetta wrote:
> On Tue, Sep 26, 2023 at 08:29:11AM -0600, Jeff Law wrote:
>>
>>
>> On 9/25/23 03:55, Paul Iannetta wrote:
>>> On Mon, Sep 18, 2023 at 08:39:34AM +0200, Paul Iannetta wrote:
>>>> On Thu, Sep 14, 2023 at 04:24:33PM +0200, Paul Iannetta wrote:
>>>>> Hi,
>>>>>
>>>>> This is a small patch so that both dg-extract-results.py and
>>>>> dg-extract-results.sh share the same header.  In particular, it fixes
>>>>> the fact that the regexp r'^Test Run By (\S+) on (.*)$' was never
>>>>> matched in the python file.
>>>>
>>>> By the way, the bash script dg-extract-results.sh checks whether
>>>> python is available by invoking python.  However, it seems that the
>>>> policy on newer machines is to not provide python as a symlink (at
>>>> least on Ubuntu 22.04 and above; and RHEL 8).  Therefore, we might
>>>> want to also check against python3 so that the bash script does not
>>>> fail to find python even though it is available.
>>>>
>>>> Thanks,
>>>> Paul
>>>>
>>>>
>>>>> Author: Paul Iannetta <piannetta@kalrayinc.com>
>>>>> Date:   Thu Sep 14 15:43:58 2023 +0200
>>>>>
>>>>>       Harmonize headers between both dg-extract-results scripts
>>>>>
>>>>>       The header of the python version looked like:
>>>>>       Target is ...
>>>>>       Host   is ...
>>>>>       The header of the bash version looked like:
>>>>>       Test run by ... on ...
>>>>>       Target is ...
>>>>>
>>>>>       After this change both headers look like:
>>>>>       Test run by ... on ...
>>>>>       Target is ...
>>>>>       Host   is ...
>>>>>
>>>>>       The order of the tests is not the same but since dg-cmp-results.sh it
>>>>>       does not matter much.
>>>>>
>>>>>       contrib/ChangeLog:
>>>>>
>>>>>       2023-09-14  Paul Iannetta  <piannetta@kalrayinc.com>
>>>>>
>>>>>               * dg-extract-results.py: Print the "Test run" line.
>>>>>               * dg-extract-results.sh: Print the "Host" line.
>> OK
>> jeff
>>
> Thanks,
> May I ask to you commit it to trunk on my behalf?
Done.  Thanks for letting me know you didn't have write access.

Jeff

Re: [PING] [PATCH] Harmonize headers between both dg-extract-results scripts

[Thomas Schwinge]

Hi!

On 2023-09-29T08:52:24-0600, Jeff Law <jeffreyalaw@gmail.com> wrote:
> On 9/29/23 02:19, Paul Iannetta wrote:
>> On Tue, Sep 26, 2023 at 08:29:11AM -0600, Jeff Law wrote:
>>> On 9/25/23 03:55, Paul Iannetta wrote:
>>>> On Mon, Sep 18, 2023 at 08:39:34AM +0200, Paul Iannetta wrote:
>>>>> On Thu, Sep 14, 2023 at 04:24:33PM +0200, Paul Iannetta wrote:
>>>>>> This is a small patch so that both dg-extract-results.py and
>>>>>> dg-extract-results.sh share the same header.  In particular, it fixes
>>>>>> the fact that the regexp r'^Test Run By (\S+) on (.*)$' was never
>>>>>> matched in the python file.

Thanks for looking into this!

I confirm that 'Test run by' lines now appear in
'contrib/dg-extract-results.py'-filtered '*.sum' files.

However, a number of upper-case 'Test Run By' instances do remain in GCC
sources:

    $ git grep --cached 'Test Run By'
    contrib/dg-extract-results.py:        # and in which the second is the associated 'Test Run By' line.
    contrib/dglib.pm:               } elsif(/^(Test Run By \S+ on|runtest completed at) ... (.{3}) (\d{1,2}) (\d{1,2}):(\d{1,2}):(\d{1,2}) (\d{4})/) {
    gcc/testsuite/ada/acats/run_all.sh:display "Test Run By $USER on `date`"
    gotools/Makefile.am:    @echo "Test Run By $${USER} on `date`" > gotools.head
    gotools/Makefile.in:@NATIVE_TRUE@       @echo "Test Run By $${USER} on `date`" > gotools.head
    libgo/Makefile.am:      @echo "Test Run By $${USER} on `date`" > libgo.head
    libgo/Makefile.in:      @echo "Test Run By $${USER} on `date`" > libgo.head

..., and I observe the following "regression" via the upper-case instance
in 'gcc/testsuite/ada/acats/run_all.sh':

    --- [...]/gcc/testsuite/ada/acats/acats.sum
    +++ [...]/gcc/testsuite/ada/acats/acats.sum
    @@ -1,4 +1,3 @@
    -Test Run By tschwing on Sun Sep 17 22:02:20 CEST 2023
                    === acats configuration ===
    [...]

(The upper-case 'echo's in gotools ('[...]/gotools/gotools.sum') and
libgo ('[...]/[...]/libgo/libgo.sum') evidently are not affected by your
change; presumably this testing doesn't use
'contrib/dg-extract-results.py'.)

Is this (case variants) maybe something that has changed in DejaGnu at
some point in time?  (I have not checked.)

I suggest that we adapt all remaining upper-case instances in GCC,
similar to your change.  And/or, as applicable, recognize both variants
(or ignore case distinctions generally)?


>>>>> By the way, the bash script dg-extract-results.sh checks whether
>>>>> python is available by invoking python.  However, it seems that the
>>>>> policy on newer machines is to not provide python as a symlink (at
>>>>> least on Ubuntu 22.04 and above; and RHEL 8).  Therefore, we might
>>>>> want to also check against python3 so that the bash script does not
>>>>> fail to find python even though it is available.

See <https://gcc.gnu.org/PR109668> "'python' vs. 'python3'".


>> May I ask to you commit it to trunk on my behalf?

> Done.  Thanks for letting me know you didn't have write access.

Given Paul's (and colleagues'?) ongoing work on GCC (Kalray KVX back end,
complex numbers support), is it maybe now time to enable Git write access
for him (them?)?

<https://gcc.gnu.org/gitwrite.html>, "write after approval".


>>>>>> Author: Paul Iannetta <piannetta@kalrayinc.com>
>>>>>> Date:   Thu Sep 14 15:43:58 2023 +0200
>>>>>>
>>>>>>       Harmonize headers between both dg-extract-results scripts
>>>>>>
>>>>>>       The header of the python version looked like:
>>>>>>       Target is ...
>>>>>>       Host   is ...
>>>>>>       The header of the bash version looked like:
>>>>>>       Test run by ... on ...
>>>>>>       Target is ...
>>>>>>
>>>>>>       After this change both headers look like:
>>>>>>       Test run by ... on ...
>>>>>>       Target is ...
>>>>>>       Host   is ...
>>>>>>
>>>>>>       The order of the tests is not the same but since dg-cmp-results.sh it
>>>>>>       does not matter much.
>>>>>>
>>>>>>       contrib/ChangeLog:
>>>>>>
>>>>>>       2023-09-14  Paul Iannetta  <piannetta@kalrayinc.com>
>>>>>>
>>>>>>               * dg-extract-results.py: Print the "Test run" line.
>>>>>>               * dg-extract-results.sh: Print the "Host" line.

>>>>>> diff --git a/contrib/dg-extract-results.py b/contrib/dg-extract-results.py
>>>>>> index 30aa68771d4..34da1808c5f 100644
>>>>>> --- a/contrib/dg-extract-results.py
>>>>>> +++ b/contrib/dg-extract-results.py
>>>>>> @@ -113,7 +113,7 @@ class Prog:
>>>>>>          # Whether to create .sum rather than .log output.
>>>>>>          self.do_sum = True
>>>>>>          # Regexps used while parsing.
>>>>>> -        self.test_run_re = re.compile (r'^Test Run By (\S+) on (.*)$')
>>>>>> +        self.test_run_re = re.compile (r'^Test run by (\S+) on (.*)$')
>>>>>>          self.tool_re = re.compile (r'^\t\t=== (.*) tests ===$')
>>>>>>          self.result_re = re.compile (r'^(PASS|XPASS|FAIL|XFAIL|UNRESOLVED'
>>>>>>                                       r'|WARNING|ERROR|UNSUPPORTED|UNTESTED'
>>>>>> diff --git a/contrib/dg-extract-results.sh b/contrib/dg-extract-results.sh
>>>>>> index ff6c50d029c..57f6fe0e997 100755
>>>>>> --- a/contrib/dg-extract-results.sh
>>>>>> +++ b/contrib/dg-extract-results.sh
>>>>>> @@ -271,7 +271,7 @@ cat $SUM_FILES \
>>>>>>
>>>>>>  # Write the begining of the combined summary file.
>>>>>>
>>>>>> -head -n 2 $FIRST_SUM
>>>>>> +head -n 3 $FIRST_SUM
>>>>>>  echo
>>>>>>  echo "          === $TOOL tests ==="
>>>>>>  echo
-----------------
Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955

Re: [PING] [PATCH] Harmonize headers between both dg-extract-results scripts

[Jeff Law]

On 10/18/23 03:35, Thomas Schwinge wrote:
> 
> Is this (case variants) maybe something that has changed in DejaGnu at
> some point in time?  (I have not checked.)
No idea :-)

> 
> I suggest that we adapt all remaining upper-case instances in GCC,
> similar to your change.  And/or, as applicable, recognize both variants
> (or ignore case distinctions generally)?
Yea, we should try to get this commonized.  Probably wise to recognize 
both variants as well -- especially if there are instances of these 
strings which aren't under GCC's contorl.


> 
> Given Paul's (and colleagues'?) ongoing work on GCC (Kalray KVX back end,
> complex numbers support), is it maybe now time to enable Git write access
> for him (them?)?
> 
> <https://gcc.gnu.org/gitwrite.html>, "write after approval".
Sure.  I'd sponsor them.

jeff

[PATCH] Ignore case of header line in dg-extract-results.py

[Paul Iannetta]

[-- Attachment #1: Type: text/plain, Size: 1219 bytes --]

On Thu, Oct 19, 2023 at 10:48:17AM -0600, Jeff Law wrote:
> On 10/18/23 03:35, Thomas Schwinge wrote:
> > 
> > Is this (case variants) maybe something that has changed in DejaGnu at
> > some point in time?  (I have not checked.)
> No idea :-)
>
Yes, it changed around 2016.

> > I suggest that we adapt all remaining upper-case instances in GCC,
> > similar to your change.  And/or, as applicable, recognize both variants
> > (or ignore case distinctions generally)?
> Yea, we should try to get this commonized.  Probably wise to recognize both
> variants as well -- especially if there are instances of these strings which
> aren't under GCC's contorl.
> 
In retrospect, I also think that we the regex should be case
insensitive, that will allow to be compatible with older releases of
dejagnu and will incur less changes in GCC. (cf. attached patch)

> > Given Paul's (and colleagues'?) ongoing work on GCC (Kalray KVX back end,
> > complex numbers support), is it maybe now time to enable Git write access
> > for him (them?)?
> > 
> > <https://gcc.gnu.org/gitwrite.html>, "write after approval".
> Sure.  I'd sponsor them.

Thanks. May I request an account on sourceware.org, mentioning you as
our sponsor?

Paul



[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-dg-extract-results.py-Ignore-case-in-header-line.patch --]
[-- Type: text/x-diff, Size: 1400 bytes --]

From ce418afa1d3098603e26e1fd2ee262a8ab72e5ab Mon Sep 17 00:00:00 2001
From: Paul Iannetta <piannetta@kalrayinc.com>
Date: Tue, 24 Oct 2023 09:48:42 +0200
Subject: [PATCH] dg-extract-results.py: Ignore case in header line

DejaGNU changed its header line from "Test Run By" to "Test run by"
around 2016.  This patch makes it so that both alternatives are
correcly detected.

contrib/ChangeLog:

2023-10-24  Paul Iannetta  <piannetta@kalrayinc.com>

	* dg-extract-results.py: Make the test_run regex case
	  insensitive.
---
 contrib/dg-extract-results.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/contrib/dg-extract-results.py b/contrib/dg-extract-results.py
index 0bc65d30eaf..e92e8756ccc 100644
--- a/contrib/dg-extract-results.py
+++ b/contrib/dg-extract-results.py
@@ -113,7 +113,8 @@ class Prog:
         # Whether to create .sum rather than .log output.
         self.do_sum = True
         # Regexps used while parsing.
-        self.test_run_re = re.compile (r'^Test run by (\S+) on (.*)$')
+        self.test_run_re = re.compile (r'^Test run by (\S+) on (.*)$',
+                                       re.IGNORECASE)
         self.tool_re = re.compile (r'^\t\t=== (.*) tests ===$')
         self.result_re = re.compile (r'^(PASS|XPASS|FAIL|XFAIL|UNRESOLVED'
                                      r'|WARNING|ERROR|UNSUPPORTED|UNTESTED'
-- 
2.35.1.500.gb896f729e2

Re: [PATCH] Ignore case of header line in dg-extract-results.py

[Alexandre Oliva]

On Oct 24, 2023, Paul Iannetta <piannetta@kalrayinc.com> wrote:

> 	* dg-extract-results.py: Make the test_run regex case
> 	  insensitive.

It looks reasonable to me, but I'm not sure this is a change I'm
entitled to approve.  Thanks!

-- 
Alexandre Oliva, happy hacker            https://FSFLA.org/blogs/lxo/
   Free Software Activist                   GNU Toolchain Engineer
More tolerance and less prejudice are key for inclusion and diversity
Excluding neuro-others for not behaving ""normal"" is *not* inclusive

Re: [PATCH] Ignore case of header line in dg-extract-results.py

[Jeff Law]

On 10/25/23 22:09, Alexandre Oliva wrote:
> On Oct 24, 2023, Paul Iannetta <piannetta@kalrayinc.com> wrote:
> 
>> 	* dg-extract-results.py: Make the test_run regex case
>> 	  insensitive.
> 
> It looks reasonable to me, but I'm not sure this is a change I'm
> entitled to approve.  Thanks!
Even so, I trust your judgment, so OK for the trunk :-)

jeff