Author ned.deily
Recipients Todd.Rovito, asvetlov, docs@python, eric.araujo, georg.brandl, ned.deily, terry.reedy, zach.ware
Date 2013-02-18.07:04:23
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1361171064.28.0.872634240334.issue16893@psf.upfronthosting.co.za>
In-reply-to
Content
There are a few problems with the proposed patch (v2).  I commented on those in Rietveld.  But, beyond that, I'm not convinced that the generated help.txt is an improvement over the original help.txt.  While it is formatted more consistently (a good thing), it is significantly longer (547 lines vs 369).  Some info is lost in the translation, for one, the menu div bars (---).  Some of the markup looks odd to me as plain text, for example:

    Upon startup with the ``-s`` option, IDLE will execute the file
    referenced by the environment variables ``IDLESTARTUP`` or
    ``PYTHONSTARTUP``. IDLE first checks for ``IDLESTARTUP``; if
    ``IDLESTARTUP`` is present the file referenced is run.  If 
    ``IDLESTARTUP`` is not present, IDLE checks for ``PYTHONSTARTUP``.

Also, I noticed that the deprecation notice, about running without a subprocess, near the end of the help text seems to be missing from the .rst.
History
Date User Action Args
2013-02-18 07:04:24ned.deilysetrecipients: + ned.deily, georg.brandl, terry.reedy, eric.araujo, asvetlov, docs@python, Todd.Rovito, zach.ware
2013-02-18 07:04:24ned.deilysetmessageid: <1361171064.28.0.872634240334.issue16893@psf.upfronthosting.co.za>
2013-02-18 07:04:24ned.deilylinkissue16893 messages
2013-02-18 07:04:23ned.deilycreate