Author eric.araujo
Recipients docs@python, eli.bendersky, eric.araujo, ezio.melotti, georg.brandl, terry.reedy
Date 2010-11-18.16:46:51
SpamBayes Score 1.57952e-10
Marked as misclassified No
Message-id <1290098812.81.0.263401865969.issue9312@psf.upfronthosting.co.za>
In-reply-to
Content
Patch tweaked and committed in r86521.  Gory details follow.

The patch applied with only two fuzzy hunks and one rejected hunk.

I did a careful review of "" → ``""`` changes and edited a number of them.  I’m confident in my choices but am of course open to comments on the checkin.

My remark about not using :program:`name --option` was wrong.  It is actually supported, see for instance in shutil.rst “Unix command :program:`cp -p`”.  I apologize for causing you unnecessary work.  I changed that back before committing.

I ran “make clean html” in the docs (no warnings) and looked at the HTML output for each change.  Modules that wrongly used :option: are fixed; conversely, timeit and unittest now have a good-looking, indexed list of options, the program/cmdoption combo you used is terrific!

Actually, I found that so cool that I added an option list for “unittest discover” (was a table) and “compileall” (was a plain paragraph, missing two options).  Consider that an experiment; I’m willing to revert them if people prefer the old form.

I’ll rewrap too long or too short lines in a following commit.  I’ll also backport relevant parts to 3.1 and 2.7 with the approval of the release manager.

Thank you Eli for this contribution.  Thanks Terry and Georg for the help.
History
Date User Action Args
2010-11-18 16:46:53eric.araujosetrecipients: + eric.araujo, georg.brandl, terry.reedy, ezio.melotti, eli.bendersky, docs@python
2010-11-18 16:46:52eric.araujosetmessageid: <1290098812.81.0.263401865969.issue9312@psf.upfronthosting.co.za>
2010-11-18 16:46:51eric.araujolinkissue9312 messages
2010-11-18 16:46:51eric.araujocreate