This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author zach.ware
Recipients Todd.Rovito, asvetlov, docs@python, eric.araujo, georg.brandl, ned.deily, terry.reedy, zach.ware
Date 2013-02-20.19:50:51
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1361389853.37.0.750180450086.issue16893@psf.upfronthosting.co.za>
In-reply-to
Content
Here's a new version of the patch, with the fixes that Ned pointed out.

I also tried to address concerns about lost information; menu divisions have been added to Doc/library/idle.rst, along with the blurb about running without a subprocess being deprecated.  Also, every instance of :kbd:`C-x` has been expanded to :kbd:`Control-x`, as that's how help.txt has commands written.  A rather unrelated change that I snuck in while I was editing idle.rst was to move the index markers for Class browser and Path browser to be above those entries rather than below.

The generated help.txt is significantly longer than the old version but I don't think that's all bad.  Most of the extra lines are new whitespace or things that had been a single line being broken up into two.  I personally thought the old help.txt was rather too dense in some places, though I might agree that the generated version is a bit sparse in others.

The paragraph about environment variables does have a rather unfortunate number of backticks, but I don't think it's unreadable.  It's also information that wasn't present in the original help.txt.

Thank you for the review, Ned :)
History
Date User Action Args
2013-02-20 19:50:54zach.waresetrecipients: + zach.ware, georg.brandl, terry.reedy, ned.deily, eric.araujo, asvetlov, docs@python, Todd.Rovito
2013-02-20 19:50:53zach.waresetmessageid: <1361389853.37.0.750180450086.issue16893@psf.upfronthosting.co.za>
2013-02-20 19:50:53zach.warelinkissue16893 messages
2013-02-20 19:50:53zach.warecreate