[v3,gdb/testsuite] updated analyze-racy-logs.py to python3

Message ID 20211001204317.37170-1-blarsen@redhat.com
State Superseded
Headers show
Series
  • [v3,gdb/testsuite] updated analyze-racy-logs.py to python3
Related show

Commit Message

Stafford Horne via Gdb-patches Oct. 1, 2021, 8:43 p.m.
Since python 2 is no longer supported on most distributions, update the
script to run under python while while still being runnable under
python2.
---
 gdb/testsuite/analyze-racy-logs.py | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

-- 
2.27.0

Comments

Stafford Horne via Gdb-patches Oct. 1, 2021, 8:53 p.m. | #1
Change updated -> update in the subject as well.

The patch is OK with that changed.

Simon

On 2021-10-01 16:43, Bruno Larsen via Gdb-patches wrote:
> Since python 2 is no longer supported on most distributions, update the

> script to run under python while while still being runnable under

> python2.

> ---

>  gdb/testsuite/analyze-racy-logs.py | 14 +++++++-------

>  1 file changed, 7 insertions(+), 7 deletions(-)

> 

> diff --git a/gdb/testsuite/analyze-racy-logs.py

> b/gdb/testsuite/analyze-racy-logs.py

> index 604a272831c..4724e1bdbfc 100755

> --- a/gdb/testsuite/analyze-racy-logs.py

> +++ b/gdb/testsuite/analyze-racy-logs.py

> @@ -1,4 +1,4 @@

> -#!/usr/bin/env python

> +#!/usr/bin/env python3

> 

>  # Copyright (C) 2016-2021 Free Software Foundation, Inc.

>  #

> @@ -145,7 +145,7 @@ def identify_racy_tests():

>      # to be ignored.  For example, tests both in the PASS and KFAIL

>      # states should not be considered racy.

>      ignored_tests = set()

> -    for s1, s2 in ignore_relations.iteritems():

> +    for s1, s2 in ignore_relations.items():

>          try:

>              ignored_tests |= all_tests[s1] & all_tests[s2]

>          except:

> @@ -159,16 +159,16 @@ def identify_racy_tests():

>      racy_tests = racy_tests - ignored_tests

> 

>      # Print the header.

> -    print "\t\t=== gdb racy tests ===\n"

> +    print("\t\t=== gdb racy tests ===\n")

> 

>      # Print each test.

>      for line in sorted(racy_tests):

> -        print line

> +        print(line)

> 

>      # Print the summary.

> -    print "\n"

> -    print "\t\t=== gdb Summary ===\n"

> -    print "# of racy tests:\t\t%d" % len(racy_tests)

> +    print("\n")

> +    print("\t\t=== gdb Summary ===\n")

> +    print("# of racy tests:\t\t%d" % len(racy_tests))

> 

> 

>  if __name__ == "__main__":

Patch

diff --git a/gdb/testsuite/analyze-racy-logs.py b/gdb/testsuite/analyze-racy-logs.py
index 604a272831c..4724e1bdbfc 100755
--- a/gdb/testsuite/analyze-racy-logs.py
+++ b/gdb/testsuite/analyze-racy-logs.py
@@ -1,4 +1,4 @@ 
-#!/usr/bin/env python
+#!/usr/bin/env python3
 
 # Copyright (C) 2016-2021 Free Software Foundation, Inc.
 #
@@ -145,7 +145,7 @@  def identify_racy_tests():
     # to be ignored.  For example, tests both in the PASS and KFAIL
     # states should not be considered racy.
     ignored_tests = set()
-    for s1, s2 in ignore_relations.iteritems():
+    for s1, s2 in ignore_relations.items():
         try:
             ignored_tests |= all_tests[s1] & all_tests[s2]
         except:
@@ -159,16 +159,16 @@  def identify_racy_tests():
     racy_tests = racy_tests - ignored_tests
 
     # Print the header.
-    print "\t\t=== gdb racy tests ===\n"
+    print("\t\t=== gdb racy tests ===\n")
 
     # Print each test.
     for line in sorted(racy_tests):
-        print line
+        print(line)
 
     # Print the summary.
-    print "\n"
-    print "\t\t=== gdb Summary ===\n"
-    print "# of racy tests:\t\t%d" % len(racy_tests)
+    print("\n")
+    print("\t\t=== gdb Summary ===\n")
+    print("# of racy tests:\t\t%d" % len(racy_tests))
 
 
 if __name__ == "__main__":