classification
Title: argparse: Issue 15906 patch; positional with nargs='*' and string default
Type: behavior Stage:
Components: Library (Lib) Versions: Python 3.4, Python 2.7
process
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: bethard, chris.jerdonek, paul.j3
Priority: normal Keywords:

Created on 2013-02-20 04:58 by paul.j3, last changed 2013-02-20 05:04 by chris.jerdonek.

Messages (1)
msg182469 - (view) Author: paul j3 (paul.j3) * Date: 2013-02-20 04:58
The production argparse applies the type conversion to a string default whether it is needed or not.  With the 12776 and 15906 patch, that conversion is postponed, so that it is applied only once.  However, for a positional argument with nargs='*', that conversion is never applied.

For example, with:
add_argument('foo', type=FileType('r'), default='anyfile', nargs='*')

the development version, with parse_args([]) produces
Namespace(foo='anyfile')

The previous code tested this default, raising an error if there was an IOError.  But even if it successfully opened the file, the namespace
 would get the string value, not the opened file.

With nargs = '?', the result is an IOError, or an opened file.

It is evident from the code (but not the documentation) that the best
default for the '*' positional is a list of appropriate type of objects.
In the case of FileTypes, about the only choices, without opening a
file, are: [] and [sys.stdin].
History
Date User Action Args
2013-02-20 05:04:05chris.jerdoneksetnosy: + bethard, chris.jerdonek
2013-02-20 04:58:17paul.j3create