Issue1937
Created on 2012-06-29.12:48:38 by malte.vesper, last changed 2012-06-29.20:24:24 by pjenvey.
|
| msg7276 (view) |
Author: malte vesper (malte.vesper) |
Date: 2012-06-29.12:48:38 |
|
posixpath.join fails if the first parameter is none.
pylint uses this, suggested fix:
In the function arround line 58 extend:
path = a
to
path = a or ''
|
| msg7282 (view) |
Author: Frank Wierzbicki (fwierzbicki) |
Date: 2012-06-29.17:39:38 |
|
Maybe I'm not understanding this one but:
Python 2.7.2+ (default, Oct 4 2011, 20:06:09)
[GCC 4.6.1] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import posixpath
>>> posixpath.join(None, "foo")
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/usr/lib/python2.7/posixpath.py", line 68, in join
elif path == '' or path.endswith('/'):
AttributeError: 'NoneType' object has no attribute 'endswith'
>>>
|
| msg7286 (view) |
Author: Philip Jenvey (pjenvey) |
Date: 2012-06-29.20:24:19 |
|
We match the CPython behavior as Frank says. If you wanted this changed you'd have to take it up with CPython
However, None is not a valid path. join's docs states that it will "Join one or more path components intelligently."
None of the path functions allow None values. I don't see this as a valid request
|
|
| Date |
User |
Action |
Args |
| 2012-06-29 20:24:24 | pjenvey | set | resolution: fixed -> invalid |
| 2012-06-29 20:24:19 | pjenvey | set | status: open -> closed
resolution: fixed
messages:
+ msg7286
nosy:
+ pjenvey |
| 2012-06-29 17:39:38 | fwierzbicki | set | nosy:
+ fwierzbicki
messages:
+ msg7282 |
| 2012-06-29 12:48:38 | malte.vesper | create | |
|
