Roundup Robot added the comment:
New changeset 8c60bb098eff by R David Murray in branch '3.2':
#5088: document behavior of optparse defaults with 'append' action.
http://hg.python.org/cpython/rev/8c60bb098eff
New changeset 0c2bdd2c2032 by R David Murray in branch 'default':
Merge #5088:
Louis Deflandre added the comment:
Hello,
Tell me if the issue is too old to deserve comments anymore. But I would like
to challenge the conclusion made in this issue.
The message msg9944 stated Think of the default as the initial list but it is
inconsistent with the proper meaning of
R. David Murray added the comment:
It may or may not be too old to deserve comments, but it is too old to do
anything about it. This can't change for backward compatibility reasons, and
since optparse is deprecated in favor of argparse it isn't getting any new
features. We still need to
Changes by Eli Bendersky eli...@gmail.com:
--
nosy: -eli.bendersky
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5088
___
___
Python-bugs-list
Eli Bendersky eli...@gmail.com added the comment:
I fuzzily recall there was somewhere a decision to use the American spelling of
some words, like s/behaviour/behavior/ appearing in this patch. Is this right,
or was it decided that it doesn't matter?
--
nosy: +eli.bendersky
status:
Eli Bendersky eli...@gmail.com added the comment:
Éric, also re your previous message, I personally would prefer seeing contrary
to what one can think removed altogether. IMHO it's too chatty for an official
document and doesn't really add new information over the sentence it follows.
Raymond Hettinger rhettin...@users.sourceforge.net added the comment:
Note, the :attr:`~Option.dest` variable is a list which includes default
values if any are defined. Options on the command-line are appended to
this list. Accordingly, the list may contain both the default value and
the
Changes by Éric Araujo mer...@netwok.org:
--
status: open - pending
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5088
___
___
Python-bugs-list
Changes by Éric Araujo mer...@netwok.org:
--
assignee: d...@python - eric.araujo
status: pending - open
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5088
___
Sandro Tosi sandro.t...@gmail.com added the comment:
Hi Éric,
thanks a lot for your review. Your comments are just fine, so I'm attaching a
new patch that contains them.
Yes, it's really nice to see one's work being accepted, and I do recognize I
have a lot to learn about python procedures
Éric Araujo mer...@netwok.org added the comment:
Forgot this one:
`appended`
I don’t remember the default reST role being used in the Python docs; I don’t
even remember what it is. Your example makes me suspect emphasis, so using
*appended* would do the same thing and be explicit.
Sandro Tosi sandro.t...@gmail.com added the comment:
On Mon, Oct 4, 2010 at 20:18, Éric Araujo rep...@bugs.python.org wrote:
Éric Araujo mer...@netwok.org added the comment:
Forgot this one:
`appended`
I don’t remember the default reST role being used in the Python docs; I don’t
even
Éric Araujo mer...@netwok.org added the comment:
I think I looked in other part of the optparse.rst file how it's done
and used that
Alright, let’s leave it alone then.
contrary to what one can think
Still not a native speaker, but wouldn’t “may think” be more suited here?
ah ok, it might me
Changes by Éric Araujo mer...@netwok.org:
--
resolution: - accepted
status: open - pending
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5088
___
Sandro Tosi sandro.t...@gmail.com added the comment:
Hello,
attached a patch to add documentation about action=append and a default value.
Regards,
Sandro
--
keywords: +patch
nosy: +sandro.tosi
Added file: http://bugs.python.org/file19087/issue5088-py3k.patch
Éric Araujo mer...@netwok.org added the comment:
Thanks for the patch. Some remarks:
:attr:`~Option.dest` variable, that already contains the default value
I would have used “which” here, but I’m not a native speaker.
not replaced (contrary to what one can think)
I’d have used a comma, not
Andrew McNabb amcn...@mcnabbs.org added the comment:
I think that optparse is doing the right thing here. I think that your code
example should be changed to:
import optparse
parser = optparse.OptionParser()
parser.add_option(-o, --option, action = append)
options, args =
R. David Murray rdmur...@bitdance.com added the comment:
I agree with Andrew, but I think the optparse documentation should make it
clear that this is what happens with a default value for an 'append' action.
It is *implied* by the fact that append says it appends to a list, and default
says
New submission from Fons Dijkstra f...@xs4all.nl:
Although it is possible to specify a default value for an append action,
either directly or through the set_default() function, it is not handled
correctly when this default is overruled by the user. Take for example
the following code:
19 matches
Mail list logo