* [PATCH] Harmonize headers between both dg-extract-results scripts @ 2023-09-14 14:24 Paul Iannetta 2023-09-18 6:39 ` Paul Iannetta 0 siblings, 1 reply; 11+ messages in thread From: Paul Iannetta @ 2023-09-14 14:24 UTC (permalink / raw) To: gcc-patches; +Cc: rdsandiford 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 ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Harmonize headers between both dg-extract-results scripts 2023-09-14 14:24 [PATCH] Harmonize headers between both dg-extract-results scripts Paul Iannetta @ 2023-09-18 6:39 ` Paul Iannetta 2023-09-25 9:55 ` [PING] " Paul Iannetta 0 siblings, 1 reply; 11+ messages in thread From: Paul Iannetta @ 2023-09-18 6:39 UTC (permalink / raw) To: gcc-patches; +Cc: rdsandiford 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 ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PING] [PATCH] Harmonize headers between both dg-extract-results scripts 2023-09-18 6:39 ` Paul Iannetta @ 2023-09-25 9:55 ` Paul Iannetta 2023-09-26 14:29 ` Jeff Law 0 siblings, 1 reply; 11+ messages in thread From: Paul Iannetta @ 2023-09-25 9:55 UTC (permalink / raw) To: gcc-patches; +Cc: rdsandiford, oliva 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 ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PING] [PATCH] Harmonize headers between both dg-extract-results scripts 2023-09-25 9:55 ` [PING] " Paul Iannetta @ 2023-09-26 14:29 ` Jeff Law 2023-09-29 8:19 ` Paul Iannetta 0 siblings, 1 reply; 11+ messages in thread From: Jeff Law @ 2023-09-26 14:29 UTC (permalink / raw) To: Paul Iannetta, gcc-patches; +Cc: rdsandiford, oliva 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 ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PING] [PATCH] Harmonize headers between both dg-extract-results scripts 2023-09-26 14:29 ` Jeff Law @ 2023-09-29 8:19 ` Paul Iannetta 2023-09-29 14:52 ` Jeff Law 0 siblings, 1 reply; 11+ messages in thread From: Paul Iannetta @ 2023-09-29 8:19 UTC (permalink / raw) To: Jeff Law; +Cc: gcc-patches, rdsandiford, oliva 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) ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PING] [PATCH] Harmonize headers between both dg-extract-results scripts 2023-09-29 8:19 ` Paul Iannetta @ 2023-09-29 14:52 ` Jeff Law 2023-10-18 9:35 ` Thomas Schwinge 0 siblings, 1 reply; 11+ messages in thread From: Jeff Law @ 2023-09-29 14:52 UTC (permalink / raw) To: Paul Iannetta; +Cc: gcc-patches, rdsandiford, oliva 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 ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PING] [PATCH] Harmonize headers between both dg-extract-results scripts 2023-09-29 14:52 ` Jeff Law @ 2023-10-18 9:35 ` Thomas Schwinge 2023-10-19 16:48 ` Jeff Law 0 siblings, 1 reply; 11+ messages in thread From: Thomas Schwinge @ 2023-10-18 9:35 UTC (permalink / raw) To: Jeff Law, Paul Iannetta; +Cc: gcc-patches, rdsandiford, oliva 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 ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PING] [PATCH] Harmonize headers between both dg-extract-results scripts 2023-10-18 9:35 ` Thomas Schwinge @ 2023-10-19 16:48 ` Jeff Law 2023-10-24 8:04 ` [PATCH] Ignore case of header line in dg-extract-results.py Paul Iannetta 0 siblings, 1 reply; 11+ messages in thread From: Jeff Law @ 2023-10-19 16:48 UTC (permalink / raw) To: Thomas Schwinge, Paul Iannetta; +Cc: gcc-patches, rdsandiford, oliva 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 ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH] Ignore case of header line in dg-extract-results.py 2023-10-19 16:48 ` Jeff Law @ 2023-10-24 8:04 ` Paul Iannetta 2023-10-26 4:09 ` Alexandre Oliva 0 siblings, 1 reply; 11+ messages in thread From: Paul Iannetta @ 2023-10-24 8:04 UTC (permalink / raw) To: Jeff Law; +Cc: Thomas Schwinge, gcc-patches, rdsandiford, oliva [-- 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 ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Ignore case of header line in dg-extract-results.py 2023-10-24 8:04 ` [PATCH] Ignore case of header line in dg-extract-results.py Paul Iannetta @ 2023-10-26 4:09 ` Alexandre Oliva 2023-10-26 4:13 ` Jeff Law 0 siblings, 1 reply; 11+ messages in thread From: Alexandre Oliva @ 2023-10-26 4:09 UTC (permalink / raw) To: Paul Iannetta; +Cc: Jeff Law, Thomas Schwinge, gcc-patches, rdsandiford 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 ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] Ignore case of header line in dg-extract-results.py 2023-10-26 4:09 ` Alexandre Oliva @ 2023-10-26 4:13 ` Jeff Law 0 siblings, 0 replies; 11+ messages in thread From: Jeff Law @ 2023-10-26 4:13 UTC (permalink / raw) To: Alexandre Oliva, Paul Iannetta; +Cc: Thomas Schwinge, gcc-patches, rdsandiford 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 ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2023-10-26 4:13 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-09-14 14:24 [PATCH] Harmonize headers between both dg-extract-results scripts Paul Iannetta 2023-09-18 6:39 ` Paul Iannetta 2023-09-25 9:55 ` [PING] " Paul Iannetta 2023-09-26 14:29 ` Jeff Law 2023-09-29 8:19 ` Paul Iannetta 2023-09-29 14:52 ` Jeff Law 2023-10-18 9:35 ` Thomas Schwinge 2023-10-19 16:48 ` Jeff Law 2023-10-24 8:04 ` [PATCH] Ignore case of header line in dg-extract-results.py Paul Iannetta 2023-10-26 4:09 ` Alexandre Oliva 2023-10-26 4:13 ` Jeff Law
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).