Kirill Elagin added the comment:
Ah, I’m so dumb. Of course the tests work as there are multiple addresses but
still just one field.
Here is the test for multiple fields.
--
Added file: http://bugs.python.org/file39263/multiple_fields_test.patch
Serhiy Storchaka added the comment:
Proposed patch fixes three related issues: issue24094, issue24095 and
issue24105.
--
assignee: - serhiy.storchaka
keywords: +patch
stage: - patch review
versions: +Python 3.5
Added file:
New submission from Bob Stein:
`%b%42` produces a ValueError exception instead of outputting '101010'
Details here: http://stackoverflow.com/a/29997703/673991
--
messages: 242388
nosy: BobStein
priority: normal
severity: normal
status: open
title: %b does not work, as a binary output
Bob Stein added the comment:
Ah, you're right, my mistake.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24112
___
___
Python-bugs-list mailing
New submission from Antony Lee:
Since PEP445, the suppressions should target _PyObject_{Free,Realloc} instead
of PyObject_{Free,Realloc}.
--
messages: 242382
nosy: Antony.Lee
priority: normal
severity: normal
status: open
title: Valgrind suppression file should be updated
versions:
paul added the comment:
@Serhiy:
Not all of my bugs are in the same module.
Sure, I will group them by module in the future.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24105
___
Steven D'Aprano added the comment:
This is not a bug, since %b is not supported in Python 2, only in Python 3:
https://docs.python.org/2/library/stdtypes.html#string-formatting-operations
No new features will be added to 2.7, so if you need %b you can use Python 3,
or in Python 2.7 you can
Petr Viktorin added the comment:
Not true (on 3.3 2.7).
import sys
import x
Traceback (most recent call last):
File stdin, line 1, in module
File /tmp/x.py, line 1, in module
import y
File /tmp/y.py, line 1, in module
1/0
ZeroDivisionError: division by zero
sys.modules['x']
Walter Dörwald added the comment:
The patch does indeed fix the segmentation fault. However the exception message
looks confusing:
TypeError: don't know how to handle UnicodeEncodeError in error callback
--
___
Python tracker
R. David Murray added the comment:
Ah, but that is intentional. Those fields can only appear once per message,
per the RFC. The new email API will raise an error if you attempt to add them
more than once. Perhaps we should raise an error instead of ignoring the
duplicates, given that we
Serhiy Storchaka added the comment:
Py_TYPE() is necessary when the argument is not of type PyObject* (e.g.
PyUnicodeObject*).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24102
___
Keith Gray added the comment:
I took a look at Tools/scripts/diff.py and it looks like it got converted to
use argparse 9 months ago. I think I should be able to just include that in the
difflib documentation directly.
My next question is which branches need to have this changed? Do I just
Keith Gray added the comment:
Here is the patch. I have tested it against tip. Let me know if you need
anything else.
--
keywords: +patch
Added file: http://bugs.python.org/file39268/issue-24109.patch
___
Python tracker rep...@bugs.python.org
R. David Murray added the comment:
Ah, I *thought* there was an issue for that, but I didn't find it when I
searched. So this is just a doc issue to fix the docs to reflect current
reality.
--
___
Python tracker rep...@bugs.python.org
Serhiy Storchaka added the comment:
Here is a patch that makes error message consistent with type checking.
--
Added file:
http://bugs.python.org/file39266/codecs_error_handlers_issubclass_2.patch
___
Python tracker rep...@bugs.python.org
Changes by Serhiy Storchaka storch...@gmail.com:
--
keywords: +patch
Added file:
http://bugs.python.org/file39265/codecs_error_handlers_issubclass.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24102
Serhiy Storchaka added the comment:
Here is simpler reproducer:
import codecs
class X(str):
__class__ = UnicodeEncodeError
codecs.ignore_errors(X())
The problem is that PyObject_IsInstance() is fooled by custom __class__, but
then builtin error handlers handle error object as having
Walter Dörwald added the comment:
Looks much better. However shouldn't:
exc-ob_type-tp_name
be:
Py_TYPE(exc)-tp_name
(although there are still many spots in the source that still use
ob_type-tp_name)
--
___
Python tracker
New submission from Michael Smith:
In its __init__ method, shlex.shlex sets self.debug = 0. An `if self.debug:`
statement follows shortly thereafter and without allowing the user to change
self.debug.
The code inside the if statement is unreachable. Users should either be
permitted to set
Michael Smith added the comment:
Hat tip abarnert on StackOverflow for digging in.
http://stackoverflow.com/questions/29996208/putting-shlex-in-debug-mode
This code was introduced in https://hg.python.org/cpython/rev/81a121d21340
--
___
Python
Kirill Elagin added the comment:
Oh, I see now.
It is a good idea to raise an error either in `send_message` or at the moment
when a second `To`/`Cc`/`Bcc` header is added to the message.
--
resolution: - not a bug
status: open - closed
___
Python
New submission from Kees Bos:
In certain corner cases, the ctypes.util can raise an error for a uninitialized
variable 'path' when with sunos5 the clre program exists, but fails to return
valid output lines.
(Also in 2.7.10rc0)
--
components: ctypes
files: ctypes.util-path.patch
Nick Coghlan added the comment:
The main reason I suggest using the postfix parenthetical syntax is to make
it clear that we're exposing behavioural feature flags for a single
underlying type. A prefix syntax would make them look like distinct types,
which would be misleading in a different way.
R. David Murray added the comment:
Since the Tools script was only changed in 3.5 (I was looking at the wrong
branch), I think the docs should only be changed for 3.5.
--
stage: - commit review
versions: -Python 2.7, Python 3.2, Python 3.3, Python 3.4, Python 3.5
Eric Snow added the comment:
Either the docs are out-of-date or they are really poorly worded. Most likely
it's the former, but I'm taking a look.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24081
Changes by R. David Murray rdmur...@bitdance.com:
--
stage: - commit review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24081
___
___
Changes by R. David Murray rdmur...@bitdance.com:
--
versions: +Python 3.5 -Python 3.6
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24109
___
___
New submission from Serhiy Storchaka:
PyObject_IsInstance() and PyObject_IsSubclass() cat return 0, 1, or -1. But
some code use if (PyObject_IsInstance(...)) or if
(!PyObject_IsInstance(...)). This should be fixed.
--
assignee: serhiy.storchaka
components: Extension Modules,
R. David Murray added the comment:
OK, I can't reproduce it either, neither in python3 nor python2. Brett, is
this left over from a long time ago?
Heh. I just tried another experiment and got an interesting result:
rdmurray@pydev:~/python/p34cat temp1.py
import temp2
foo = 1
R. David Murray added the comment:
Note that calling reload on temp1.temp2 will fail with an error that temp2 is
not in sys.modules. So maybe the caveat needs rewording rather than deletion.
--
___
Python tracker rep...@bugs.python.org
R. David Murray added the comment:
Added some review comments.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23088
___
___
Python-bugs-list
Eric Snow added the comment:
I'm pretty sure this is the culprit:
changeset: 32882:331e60d8ce6da19b168849418776fea0940787ec
branch: legacy-trunk
user:Tim Peters tim.pet...@gmail.com
date:Mon Aug 02 03:52:12 2004 +
summary: PyImport_ExecCodeModuleEx(): remove
R. David Murray added the comment:
Thanks, Petr. (And Eric.)
--
resolution: - fixed
stage: commit review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24081
R. David Murray added the comment:
Thanks Christophe and Merlijn.
--
nosy: +r.david.murray
resolution: - fixed
stage: - resolved
status: open - closed
versions: +Python 2.7, Python 3.5
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset d356e68de236 by Raymond Hettinger in branch '2.7':
Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq.
https://hg.python.org/cpython/rev/d356e68de236
--
___
Python tracker
Roundup Robot added the comment:
New changeset d356e68de236 by Raymond Hettinger in branch '2.7':
Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq.
https://hg.python.org/cpython/rev/d356e68de236
--
___
Python tracker
Roundup Robot added the comment:
New changeset d356e68de236 by Raymond Hettinger in branch '2.7':
Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq.
https://hg.python.org/cpython/rev/d356e68de236
--
___
Python tracker
Roundup Robot added the comment:
New changeset 3cdeafd18e61 by R David Murray in branch '3.4':
#24081: Remove obsolete caveat from import docs.
https://hg.python.org/cpython/rev/3cdeafd18e61
New changeset d57e0c6d292d by R David Murray in branch 'default':
Merge: #24081: Remove obsolete caveat
Roundup Robot added the comment:
New changeset c5c65ef84a77 by R David Murray in branch '3.4':
#24108: Update fnmatch.translate example to show correct output.
https://hg.python.org/cpython/rev/c5c65ef84a77
New changeset cc6aed8ecb0d by R David Murray in branch 'default':
Merge: #24108: Update
R. David Murray added the comment:
Although I like the look of the repr Terry proposes better, I agree with Nick:
it would imply that the types were distinct, which they are not.
--
nosy: +r.david.murray
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset 813854f49f9d by Raymond Hettinger in branch '3.4':
Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq.
https://hg.python.org/cpython/rev/813854f49f9d
--
nosy: +python-dev
___
Python
Roundup Robot added the comment:
New changeset 813854f49f9d by Raymond Hettinger in branch '3.4':
Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq.
https://hg.python.org/cpython/rev/813854f49f9d
--
nosy: +python-dev
___
Python
Roundup Robot added the comment:
New changeset 813854f49f9d by Raymond Hettinger in branch '3.4':
Issues #24099, #24100, and #24101: Fix free-after-use bug in heapq.
https://hg.python.org/cpython/rev/813854f49f9d
--
nosy: +python-dev
___
Python
Serhiy Storchaka added the comment:
It would be nice to add tests based on provided demo scripts.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24100
___
Eric Snow added the comment:
I've verified that the documentation is correct under Python 2.3. The behavior
changed under Python 2.4 (and the docs were not updated). I expect that the
change in behavior is an unintended consequence of a change in the import
system for 2.4. There were 7 in
Eric Snow added the comment:
patch LGTM for the 3 branches.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24081
___
___
Python-bugs-list
Serhiy Storchaka added the comment:
An example is Enum.
--
nosy: +serhiy.storchaka
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23572
___
___
Roundup Robot added the comment:
New changeset 5ce760e2cc59 by Vinay Sajip in branch '2.7':
Issue #24060: Made logging.Formatter documentation a little clearer.
https://hg.python.org/cpython/rev/5ce760e2cc59
New changeset 88c141233d1e by Vinay Sajip in branch '3.4':
Issue #24060: Made
Eric Snow added the comment:
As I mentioned, I'm pretty sure that the failing venv test is due to the
bundled pip. Here's the test output:
test test_venv failed -- Traceback (most recent call last):
File /home/esnow/projects/cpython/Lib/test/test_venv.py, line 356, in
test_with_pip
Wolfgang Maier added the comment:
uploading an alternate, possibly slightly clearer version of the patch
--
Added file: http://bugs.python.org/file39269/statistics._sum.v2.patch
___
Python tracker rep...@bugs.python.org
Serhiy Storchaka added the comment:
Here is a patch that fixes issue24091, issue24092, issue24093, and several
other similar bugs.
--
assignee: - serhiy.storchaka
keywords: +patch
stage: - patch review
versions: +Python 3.5
Added file:
R. David Murray added the comment:
I added some review comments.
Something I don't understand: if settimeout sets NONBLOCK, and CLOEXEC is
always set now, why does s=socket.socket() yield something whose type is
SOCK_STREAM, while settimeout causes type to change? I don't understand the
Eric Snow added the comment:
Looks like setuptool's pkg_resources is directly importing
importlib._bootstrap. I've filed a bug:
https://bitbucket.org/pypa/setuptools/issue/378. In the meantime, what are our
options for getting that test passing?
--
Changes by Eric Snow ericsnowcurren...@gmail.com:
Removed file: http://bugs.python.org/file39271/path-based-importlib.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23911
___
Eric Snow added the comment:
Here's the correct patch.
--
Added file: http://bugs.python.org/file39272/path-based-importlib.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23911
___
Thomas Wouters added the comment:
The latest patch looks good to me. No need to do the additional AST refactoring
if it's going to make PEP 492's implementor's life harder (but I do read
Guido's comment as a reason to check this in sooner rather than later :) So,
unless anyone objects I'll
Nick Coghlan added the comment:
As a compatibility hack for setuptools versions with the issue, I'd suggest
making importlib._bootstrap.setup alias SourceFileLoader back into
importlib._bootstrap, with an explanation and link to back to this issue in
a comment.
--
Nick Coghlan added the comment:
I vaguely recall discussing that change, and my recollection is that it was
deliberate in order to evict improperly configured modules - we didn't
count it as a significant compatibility break because it solely affected an
error handling path.
I guess we never
Eric Snow added the comment:
Gah. I had tried exactly that but did it in the wrong spot. Here's an updated
patch which fixes the test.
--
Added file: http://bugs.python.org/file39273/path-based-importlib.diff
___
Python tracker
Raymond Hettinger added the comment:
Go ahead and add tests if you like. I don't see the point.
The fix was to simply backport code that was already in Py3.5.
and the code exercised by the exploit scripts is no longer there.
--
___
Python tracker
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24045
___
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14376
___
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24052
___
Steve Dower added the comment:
Added a patch that:
* adds the applocal option to pyvenv.cfg
* updates the docs to describe how pyvenv.cfg (home and applocal) affect
sys.path
* puts a default pyvenv.cfg in the ZIP distribution
* adds significantly more buffer overflow protection to getpathp.c
*
R. David Murray added the comment:
The behavior of s.type has changed in 3.5: it now behaves like the proposed new
method behaves. Looking at the source I can't figure out why this is so. It
doesn't seem as though it should be, but this needs to be sorted out before we
can proceed.
R. David Murray added the comment:
Wait, I take it back. I must have run the test incorrectly.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue21327
___
Serhiy Storchaka added the comment:
The patch for issue24091 fixes this issue.
--
resolution: - duplicate
stage: - resolved
status: open - closed
superseder: - Use after free in Element.extend (1)
___
Python tracker rep...@bugs.python.org
Serhiy Storchaka added the comment:
The patch for issue24091 fixes this issue.
--
resolution: - duplicate
stage: needs patch - resolved
status: open - closed
superseder: - Use after free in Element.extend (1)
___
Python tracker
Eric Snow added the comment:
Here's an updated patch with _pathy.py changed to _bootstrap_external.py
(and similar changes with freezing). The patch does not include fixing the
venv test (i.e. the bundled pip).
Also, I'll be adding a note to NEWS.
--
Added file:
Changes by Eric Snow ericsnowcurren...@gmail.com:
Removed file: http://bugs.python.org/file39273/path-based-importlib.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23911
___
Changes by Eric Snow ericsnowcurren...@gmail.com:
Removed file: http://bugs.python.org/file39274/path-based-importlib.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23911
___
Raymond Hettinger added the comment:
I should add that the crasher scripts share common features and could perhaps
be built into a unified test tool. Also, some care should be taken to make the
tool mostly independent of non-guaranteed del behavior or CPython specific
ref-counting logic to
Roundup Robot added the comment:
New changeset 02e3bf65b2f8 by Eric Snow in branch 'default':
Issue #23911: Move path-based bootstrap code to a separate frozen module.
https://hg.python.org/cpython/rev/02e3bf65b2f8
--
nosy: +python-dev
___
Python
Changes by Eric Snow ericsnowcurren...@gmail.com:
--
resolution: - fixed
stage: patch review - resolved
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23911
___
George Jenkins added the comment:
Heh, just noticed this issue passed its 10 year anniversary!
If someone has time to review my patch, that would be much appreciated. Thanks!
--
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset 0e8c9cf1fede by Benjamin Peterson in branch '3.3':
just sort the items tuple directly (closes #24094)
https://hg.python.org/cpython/rev/0e8c9cf1fede
New changeset f9027b10b3c4 by Benjamin Peterson in branch '3.4':
merge 3.3 (#24094)
paul j3 added the comment:
It's the spaces and/or brackets in the metavar, along with a usage line that is
long enough to wrap, that is raising this error.
It's a known problem.
http://bugs.python.org/issue11874
--
nosy: +paul.j3
___
Python
Changes by Eric Snow ericsnowcurren...@gmail.com:
Added file: http://bugs.python.org/file39274/path-based-importlib.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23911
___
Changes by Eric Snow ericsnowcurren...@gmail.com:
Added file: http://bugs.python.org/file39275/path-based-importlib.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue23911
___
Guido van Rossum added the comment:
Thanks for the review Thomas! And yes, that's what I meant. :-)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2292
___
Guido van Rossum added the comment:
Yeah, looks good -- Łuke, can you commit this?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue24018
___
___
Benjamin Peterson added the comment:
It certainly would be nice to have documentation.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2292
___
82 matches
Mail list logo