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 <pianne...@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 <pianne...@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)
