Roundup Robot added the comment:
New changeset 8777e59a99bd by Berker Peksag in branch '3.5':
Issue #24109: Include Tools/scripts/diff.py instead of duplicating it in
difflib documentation.
https://hg.python.org/cpython/rev/8777e59a99bd
New changeset aa2517e9f9ce by Berker Peksag in branch
Berker Peksag added the comment:
Thanks Keith.
--
nosy: +berker.peksag
resolution: - fixed
stage: commit review - resolved
status: open - closed
versions: +Python 3.6
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24109
Keith Gray added the comment:
I took a look at Tools/scripts/diff.py and it looks like it got converted to
use argparse 9 months ago. I think I should be able to just include that in the
difflib documentation directly.
My next question is which branches need to have this changed? Do I just
Keith Gray added the comment:
Here is the patch. I have tested it against tip. Let me know if you need
anything else.
--
keywords: +patch
Added file: http://bugs.python.org/file39268/issue-24109.patch
___
Python tracker rep...@bugs.python.org
R. David Murray added the comment:
Since the Tools script was only changed in 3.5 (I was looking at the wrong
branch), I think the docs should only be changed for 3.5.
--
stage: - commit review
versions: -Python 2.7, Python 3.2, Python 3.3, Python 3.4, Python 3.5
Changes by R. David Murray rdmur...@bitdance.com:
--
versions: +Python 3.5 -Python 3.6
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24109
___
___
R. David Murray added the comment:
It wouldn't be a bad idea. Of course, you'd also need to update
Tools/script/diff.py. Tests should be added as well, since we have started
adding tests for things in Tools. The tests should be written first, to make
sure that the behavior of 'diff' does
New submission from Keith Gray:
The documentation for optparse states it has been deprecated in favor of
argparse since 2.7. However, the library documentation for difflib still uses
optparse in the example.
https://docs.python.org/2/library/difflib.html#a-command-line-interface-to-difflib