On Sat, Apr 30, 2016 at 10:41:59AM -0000, stef...@apache.org wrote: > Work towards Python 3 compatibility. Since we already require Python 2.7+, > we can use the new syntax without further limiting our Python 2 support. > > Fix leftovers to complete the replacement of print statements with function > calls.
You didn't actually change the code to use the print function. The syntax just happens to work on Python 2.x because the () group the expression (like parens in math) but it's still a single expression provide to the print keyword. If you actually want to use the print function then you need to add: [[[ from __future__ import print_function ]]] to the top of the Python files. That would also let you use print() for the writes to specific files. > Modified: subversion/trunk/contrib/server-side/fsfsverify.py > URL: > http://svn.apache.org/viewvc/subversion/trunk/contrib/server-side/fsfsverify.py?rev=1741742&r1=1741741&r2=1741742&view=diff > ============================================================================== > --- subversion/trunk/contrib/server-side/fsfsverify.py (original) > +++ subversion/trunk/contrib/server-side/fsfsverify.py Sat Apr 30 10:41:59 > 2016 > @@ -1136,7 +1136,7 @@ if __name__ == '__main__': > (options, args) = parser.parse_args() > > if len(args) != 1: > - print >>sys.stderr, "Please specify exactly one rev file." > + sys.stderr.write("Please specify exactly one rev file.\n") For example, this would change to: [[[ from __future__ import print_function print('Please specify exactly one rev file.', file=sys.stderr) ]]] This also maintains the behavior of an automatic newline. Cheers, -- James GPG Key: 4096R/331BA3DB 2011-12-05 James McCoy <james...@jamessan.com>
