Update old Python 2 code in Docs/tools/extensions/patchlevel.py

[DimitrisJim]

BPO	29349
Nosy	@db3l, @ned-deily, @vadmium, @DimitrisJim
PRs	[Do Not Merge] Convert Misc/NEWS so that it is managed by towncrier #552
Files	patchlevel.patch patchlevel_print.patch patchlevel_with.patch

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2017-01-30.18:08:11.154>
created_at = <Date 2017-01-23.09:41:21.129>
labels = ['type-bug', '3.7', 'docs']
title = 'Update old Python 2 code in Docs/tools/extensions/patchlevel.py'
updated_at = <Date 2017-03-31.16:36:27.119>
user = 'https://github.com/DimitrisJim'

bugs.python.org fields:

activity = <Date 2017-03-31.16:36:27.119>
actor = 'dstufft'
assignee = 'docs@python'
closed = True
closed_date = <Date 2017-01-30.18:08:11.154>
closer = 'ned.deily'
components = ['Documentation']
creation = <Date 2017-01-23.09:41:21.129>
creator = 'Jim Fasarakis-Hilliard'
dependencies = []
files = ['46387', '46405', '46406']
hgrepos = []
issue_num = 29349
keywords = ['patch']
message_count = 14.0
messages = ['286065', '286136', '286178', '286438', '286440', '286445', '286450', '286451', '286452', '286454', '286455', '286458', '286461', '286496']
nosy_count = 6.0
nosy_names = ['db3l', 'ned.deily', 'docs@python', 'python-dev', 'martin.panter', 'Jim Fasarakis-Hilliard']
pr_nums = ['552']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue29349'
versions = ['Python 3.5', 'Python 3.6', 'Python 3.7']

[DimitrisJim]

There's a hidden Python 2 print call in the script that only gets reached after you move the Doc/ folder outside the main CPython directory and run make html.

Additionally, an obsolete way of assuring a file gets closed is used (changed to use with statement).

P.S: Tagged this with the Documentation component, maybe build is more applicable but I wasn't sure.

[vadmium]

Is it okay to only fix this in 3.5+? 3.4 only gets security fixes now.

Either way, the “with” statement changes is not a bug fix and should only go into 3.7.

[DimitrisJim]

I'm breaking these to separate files to make it easier to apply. I also noticed that other files in [Doc/tools/extensions/](https://github.com/python/cpython/blob/main/Doc/tools/extensions) use old constructs so I'm not sure about the with.

I'm guessing that either it should be changed in other files too or, since it's working fine, leave it as is.

[python-dev]

New changeset ecdc864884a9 by Martin Panter in branch '3.5':
Issue bpo-29349: Fix Python 2 syntax in documentation build code
https://hg.python.org/cpython/rev/ecdc864884a9

New changeset cdcb33f37bf3 by Martin Panter in branch '3.6':
Issues bpo-29349: Merge Py 2 fix 3.5
https://hg.python.org/cpython/rev/cdcb33f37bf3

New changeset db8b917bbfdd by Martin Panter in branch 'default':
Issues bpo-29349: Merge Py 2 fix 3.6
https://hg.python.org/cpython/rev/db8b917bbfdd

New changeset ca7d2af9920e by Martin Panter in branch 'default':
Issues bpo-29349: Add NEWS for 3.7; use “with” statement
https://hg.python.org/cpython/rev/ca7d2af9920e

[vadmium]

I think the general rule is to clean up code if you are doing something else in nearby code, but don’t go out of your way with unnecessary cleanups to arbitrary code. Otherwise it adds too much noise to the repository history, review process, risks adding bugs, etc, for little gain.

Anyway, thanks for the patch!

[ned-deily]

These changes have broken some buildbots, for example, the OS X dmg buildbots which are still using Python 2 for sphinx builds of the docs. They should be version agnostic as much as possible.

http://buildbot.python.org/all/builders/bolen-dmg-3.x/builds/59

[vadmium]

Thanks Ned. Do you know what version of Python Sphinx uses (which runs patchlevel.py)?

According to bpo-28039, David set up Python 2.7 so that “make touch” would work. But the log also uses a python2.5 command, and apparently Python 2.3 also installed on that buildbot.

If we can rely on 2.6+, we just need to add “from __future__ import print_function”. Otherwise, we may need something like

if sys.stderr is not None:
    sys.stderr.write(...)

and also do something about that “with” statement.

[ned-deily]

Dunno for sure, perhaps David can answer that. But the Sphinx docs imply that Sphinx 1.4.6 requires at least 2.6 and I don't test the installer build with anything less than 2.7.

[vadmium]

According to <https://bugs.python.org/issue17861#msg217417\>, Ned says 2.6+ is already needed to build the Python 3.5 documentation, so maybe Sphinx uses that.