Guido van Rossum added the comment:
I'm trying to let go of the AIX hang. Here's a brain dump of what I've figured
out so far.
* There were a lot of red herrings in the early discussion. This hang doesn't
seem to have anything to do with nonblocking connect() or sockets, nor even
signals.
*
Roundup Robot added the comment:
New changeset 97ad9af5d5e7 by Serhiy Storchaka in branch 'default':
Issue #12866: Fix bias() for 24-bit. Add more tests.
http://hg.python.org/cpython/rev/97ad9af5d5e7
--
___
Python tracker rep...@bugs.python.org
Serhiy Storchaka added the comment:
LGTM (except unrelated empty line at the end of Modules/_sre.c).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17087
___
STINNER Victor added the comment:
You must update unit tests in test_faulthandler.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19306
___
___
Charles-François Natali added the comment:
David Edelsohn added the comment:
AIX has an equivalent to strace (called truss). I have recorded all AIX
system calls and signals for test_process_interactive, which hangs, following
all children created by fock. The uncompressed file is 82MB or
Guido van Rossum added the comment:
Apparently, the stdout pipe was closed by the parent process
Could it be that selecting for *read* on the *write* end of a pipe is
always ready? In _UnixWritePipeTransport there's a read handler that
immediately closes the pipe as soon as it called. I
Charles-François Natali added the comment:
Guido van Rossum added the comment:
Apparently, the stdout pipe was closed by the parent process
Could it be that selecting for *read* on the *write* end of a pipe is
always ready?
That's exactly what I was thinking when I read the code below:
New submission from David Coles:
Tools/gdb/libpython.py is currently Python 3 incompatible. Unfortunately recent
versions of gdb (such as the one provided in Ubuntu 13.10) may be linked
against Python 3 rather than Python 2, breaking debugging support.
Most of the issues appear to be trivial
Changes by Ned Deily n...@acm.org:
--
nosy: +dmalcolm
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19308
___
___
Python-bugs-list mailing list
New submission from Charles-François Natali:
See https://groups.google.com/forum/#!topic/python-tulip/9T2_tGWe0Sc
The attached patch just changes waitpid(0) to waitpid(-1), and comes with a
trivial test.
--
components: Library (Lib)
files: asyncio_process_group.diff
keywords: easy,
Serhiy Storchaka added the comment:
Here is a patch. Now audioop functions no more accept str, but accept
bytes-like objects instead.
--
keywords: +patch
stage: needs patch - patch review
title: Deprecate accepting strings as arguments in audioop functions - audioop
functions
Nick Coghlan added the comment:
Looks good to me: +1 :)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19307
___
___
Python-bugs-list mailing
Roundup Robot added the comment:
New changeset 29764a7bd6ba by Serhiy Storchaka in branch 'default':
Issue #17087: Improved the repr for regular expression match objects.
http://hg.python.org/cpython/rev/29764a7bd6ba
--
nosy: +python-dev
___
Python
Serhiy Storchaka added the comment:
Thanks all participants for the discussion.
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17087
Nick Coghlan added the comment:
The patch needs to be rebased on top of the issue 19307 patch, but I like this
approach.
I say go ahead and commit it whenever you're ready :)
--
___
Python tracker rep...@bugs.python.org
New submission from Charles-François Natali:
The current SIGCHILD handler has two bugs:
- it reschedules itself if waitpid() returns 0: so if this ever happens, it
will enter a busy-loop until all children have exited
- it doesn't reschedule itself if waitpid() succeeds in reaping a child:
Stefan Krah added the comment:
Not sure if I'll ever want to support this...
I think it can be closed as wont-fix. We had a poll on python-dev
about --without-threads a while ago, and the only systems that needed
it were older OpenBSD systems and a Fujitsu supercomputer with the
Fujitsu
Ned Deily added the comment:
Still intermittently failing on 3.4.0a4 when running the entire test suite.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18650
___
Stefan Krah added the comment:
It looks like it happened again:
http://buildbot.python.org/all/builders/x86%20Gentoo%20Non-Debug%203.x/builds/5223/steps/test/logs/stdio
==
ERROR: test_default_timeout
New submission from Martin Matusiak:
The offending section:
http://docs.python.org/devguide/faq.html#how-do-i-find-which-changeset-introduced-a-bug-or-regression
I think this could be improved a bit. The key point is that hg bisect
--bad/good is a command relative to the checked out changeset.
Georg Brandl added the comment:
Yep (should there be a policy about this somewhere)?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15663
___
New submission from Martin Matusiak:
- The purpose of this document is to outline how the latter three steps of the
process works.
work
--
components: Devguide
files: typo_compiler.diff
keywords: patch
messages: 200567
nosy: ezio.melotti, numerodix
priority: normal
severity: normal
New submission from Antoine Pitrou:
Given the huge leakage in test_ast, it looks like a compiler issue.
results for 68a7bc8bb663 on branch default
test_grammar leaked [3, 3, 3] references, sum=9
test_opcodes leaked [16, 16, 16] references, sum=48
New submission from Martin Matusiak:
- Querying data from the node structs can be done with the following macros
(which are all defined in Include/token.h
They are actually in Include/node.h, which is logical, because that is where
node is defined.
--
components: Devguide
files:
New submission from Martin Matusiak:
Location:
http://docs.python.org/devguide/compiler.html#parse-trees
- To tie all of this example, consider the rule for ‘while’:
Probably meant to be: To tie all of this together with an example, ...
- The node representing this will have TYPE(node) ==
Serhiy Storchaka added the comment:
Patch is rebased to tip.
--
Added file: http://bugs.python.org/file32245/skip_tests_3.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18702
___
Changes by Christian Heimes li...@cheimes.de:
--
nosy: +christian.heimes
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19313
___
___
Changes by Barry A. Warsaw ba...@python.org:
--
nosy: +barry
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19307
___
___
Python-bugs-list mailing
New submission from Martin Matusiak:
- All code relating to the arena is in either Include/pyarena.h or
Python/pyarena.c .
I propose: All code relating to the arena is either in Include/pyarena.h or in
Python/pyarena.c .
--
components: Devguide
files: wording_compiler.diff
keywords:
Martin Matusiak added the comment:
- This needs to only be called in strategic areas where the compiler exits.
I propose: This only needs to be called in strategic areas where the compiler
exits.
--
Added file: http://bugs.python.org/file32247/wording_compiler2.diff
Changes by Nick Coghlan ncogh...@gmail.com:
--
assignee: benjamin.peterson - ncoghlan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19313
___
___
Nick Coghlan added the comment:
Just spotted the bug in http://hg.python.org/cpython/rev/b4a325275fb0
The Py_XINCREF(name); call should have been removed, as it's the counterpart
to the removed Py_CLEAR(u-u_qualname); call
--
nosy: +ncoghlan
___
Roundup Robot added the comment:
New changeset 84a8b797c5c5 by Nick Coghlan in branch 'default':
Close #19313: remove no longer needed Py_XINCREF
http://hg.python.org/cpython/rev/84a8b797c5c5
--
nosy: +python-dev
resolution: - fixed
stage: - committed/rejected
status: open - closed
Martin Matusiak added the comment:
- The functions called to generate AST nodes from the parse tree all have the
name ast_for_xx where xx is what the grammar rule that the function handles
(alias_for_import_name is the exception to this).
I'm not sure if this ought to be where xx is the
Nick Coghlan added the comment:
Before:
$ ./python -m test -R 3:3 test_ast
[1/1] test_ast
beginning 6 repetitions
123456
..
test_ast leaked [6885, 6885, 6885] references, sum=20655
test_ast leaked [4888, 4890, 4890] memory blocks, sum=14668
1 test failed:
test_ast
After:
$ ./python -m
Martin Matusiak added the comment:
- Function and macros for creating and using asdl_seq * types as found in
Python/asdl.c and Include/asdl.h:
I propose: The following are functions and macros for creating and using
asdl_seq * types as found in Python/asdl.c and Include/asdl.h:
--
Martin Matusiak added the comment:
- As for handling the line number on which a statement is defined, is handled
by compiler_visit_stmt() and thus is not a worry.
I don't understand the final clause here. What is not a worry and why would it
be a worry?
The grammar is awkward as well.
Martin Matusiak added the comment:
- But you will also need to change the ‘compiler’ package. The key files to do
that are Lib/compiler/pyassem.py and Lib/compiler/pycodegen.py .
compiler was removed in 2.6 or 2.7 iirc. I think it's safe to remove these
two sentences.
--
Added file:
Matthias Klose added the comment:
should go into 2.7 as well.
--
nosy: +doko
versions: +Python 2.7
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19308
___
David Edelsohn added the comment:
To test this theory, it should be sufficient to comment out
self._loop.add_reader(self._fileno, self._read_ready)
When I comment out this line, test_subprocess_interactive succeeds on AIX.
--
___
Python tracker
Martin Matusiak added the comment:
- If you wish to make changes that affect the output of bytecode without having
to update the magic number each time (while testing your changes) you can just
delete your old .py(c|o) files! Even though you will end up changing the magic
number if you change
Martin Matusiak added the comment:
- marshaling
marshalling
--
Added file: http://bugs.python.org/file32251/wording_typo.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19316
___
Elazar Gershuni added the comment:
Looks good to me.
Is it possible to add it to 2.7? I think it won't break any PEP8-following code
(e.g. not testing for type equality/identity)
--
___
Python tracker rep...@bugs.python.org
Martin Matusiak added the comment:
- import.c
- Home of the magic number (named MAGIC) for bytecode versioning
Probably out of date. I cannot find MAGIC being defined in this file.
--
___
Python tracker rep...@bugs.python.org
Martin Matusiak added the comment:
- Lib/
- compiler/
- pyassem.py
- One of the files that must be modified if Include/opcode.h is changed.
- pycodegen.py
- One of the files that must be modified if Include/opcode.h is changed.
More mentions of the compiler package.
--
Added file:
Roundup Robot added the comment:
New changeset 44ac81e6d584 by Serhiy Storchaka in branch '2.7':
Issue #16038: CVE-2013-1752: ftplib: Limit amount of data read by
http://hg.python.org/cpython/rev/44ac81e6d584
New changeset 38db4d0726bd by Serhiy Storchaka in branch '3.3':
Issue #16038:
Changes by Serhiy Storchaka storch...@gmail.com:
--
versions: -Python 2.7, Python 3.1, Python 3.3, Python 3.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16038
___
David Edelsohn added the comment:
For completeness, the highlights of the new truss trace output after the
echo.py change and only tracing the main process to avoid confusion from the
interleaved output:
test_subprocess_interactive (test.test_asyncio.test_events.PollEventLoopTests)
...
Vajrasky Kok added the comment:
Added patch to add timezone support for sqlite3 datetime adapter.
--
keywords: +patch
nosy: +vajrasky
Added file:
http://bugs.python.org/file32253/add_timezone_support_for_sqlite3_datetime_adapter.patch
___
Python
Serhiy Storchaka added the comment:
Here is a port of changeset 8a6def3add5b for 2.7. However getreply() is not
tested yet.
--
Added file: http://bugs.python.org/file32254/smtplib_maxline-2.7.patch
___
Python tracker rep...@bugs.python.org
Arfrever Frehtes Taifersar Arahesis added the comment:
(3.1 branch is open to security fixes.)
--
versions: +Python 3.1
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16038
___
New submission from Maciej Bliziński:
On Solaris, when you want to link shared libraries from custom directories, you
most often don't modify the system search path, but instead set RPATH in your
binaries. For example, OpenCSW packages Python into /opt/csw, and sets Python
executable's RPATH
Changes by Maciej Bliziński maciej.blizin...@gmail.com:
Removed file:
http://bugs.python.org/file32255/find_library_looks_into_rpath.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19317
___
Changes by Maciej Bliziński maciej.blizin...@gmail.com:
Added file: http://bugs.python.org/file32256/find_library_looks_into_rpath.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19317
___
Changes by Maciej Bliziński maciej.blizin...@gmail.com:
Removed file:
http://bugs.python.org/file32256/find_library_looks_into_rpath.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19317
___
Changes by Maciej Bliziński maciej.blizin...@gmail.com:
Added file: http://bugs.python.org/file32257/find_library_looks_into_rpath.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19317
___
Sunny K added the comment:
This issue is present in 3.4 too.
Added patch for 3.4.
--
keywords: +patch
nosy: +sunfinite
versions: +Python 3.4
Added file: http://bugs.python.org/file32258/pydoc.patch
___
Python tracker rep...@bugs.python.org
Guido van Rossum added the comment:
I added an LGTM plus a pointer to
http://code.google.com/p/tulip/issues/detail?id=68
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19309
___
Guido van Rossum added the comment:
Ah, sorry, I didn't see this before reviewing your other change to the same
code. Your 2nd bullet is http://code.google.com/p/tulip/issues/detail?id=68
--
___
Python tracker rep...@bugs.python.org
Changes by Guido van Rossum gu...@python.org:
--
assignee: - gvanrossum
resolution: - wont fix
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19295
___
Guido van Rossum added the comment:
Could it be that selecting for *read* on the *write* end of a pipe is
always ready?
That's exactly what I was thinking when I read the code below: that's
definitely a possibility on AIX.
David confirmed that it is the _read_ready() that closes the pipe
Changes by Maciej Bliziński maciej.blizin...@gmail.com:
Removed file:
http://bugs.python.org/file32257/find_library_looks_into_rpath.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19317
___
Changes by Maciej Bliziński maciej.blizin...@gmail.com:
Added file: http://bugs.python.org/file32259/find_library_looks_into_rpath.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19317
___
Charles-François Natali added the comment:
I guess we'll have to write platform-dependent code and make this an
optional feature. (Essentially, on platforms like AIX, for a
write-pipe, connection_lost() won't be called unless you try to write
some more bytes to it.)
I do believe that so
Richard Oudkerk added the comment:
I guess we'll have to write platform-dependent code and make this an
optional feature. (Essentially, on platforms like AIX, for a
write-pipe, connection_lost() won't be called unless you try to write
some more bytes to it.)
If we are not capturing
Martin Matusiak added the comment:
I have been thinking about a fix for this. A straightforward fix would be to
add a kwarg readrc=True to the constructor of Pdb that will default to reading
the rc files as it does now, and allows disabling this default.
The implication is that all tests in
Christian Heimes added the comment:
I think it's more likely that my patch is triggering an existing bug. The
locking code for the SSL module and OpenSSL doesn't release locks on fork. I
have attached an experimental patch that unlocks all locks in the client.
Please try if it resolves the
Giampaolo Rodola' added the comment:
You are right. I will try to provide patches for other Python versions
later next week.
On Sun, Oct 20, 2013 at 5:08 PM, Arfrever Frehtes Taifersar Arahesis
rep...@bugs.python.org wrote:
Arfrever Frehtes Taifersar Arahesis added the comment:
(3.1
New submission from James Lu:
break 2 would break out of one loop then break out of another.
break
break
would just break once and not execute the second break.
break 2 when there are only 1 thing to break would raise raise a SyntaxError:
Can only break 1 time, need to break 2 times.
New submission from Christoph Baumgartner:
The documentation about ctypes.sizeof:
Returns the size in bytes of a ctypes type or instance memory buffer. Does the
same as the C sizeof() function.
'sizeof' is an operator. I would drop the parentheses as well.
--
assignee: docs@python
Changes by Martin Matusiak numero...@gmail.com:
--
nosy: +numerodix
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8083
___
___
Python-bugs-list
Changes by Martin Matusiak numero...@gmail.com:
--
nosy: +numerodix
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7757
___
___
Python-bugs-list
Changes by Martin Matusiak numero...@gmail.com:
--
nosy: +numerodix
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19318
___
___
Python-bugs-list
Changes by Martin Matusiak numero...@gmail.com:
--
nosy: +numerodix
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19319
___
___
Python-bugs-list
Changes by Martin Matusiak numero...@gmail.com:
--
nosy: +numerodix
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17951
___
___
Python-bugs-list
James Lu added the comment:
You would have to do this:
for i in range(1,10):
broke = True
for x in range(2,5):
break
else:
broke = False
if broke:
break
to break twice, and you can't break only once!
--
___
Roundup Robot added the comment:
New changeset 4cdb9f04494b by Charles-François Natali in branch 'default':
Issue #19309: asyncio: make waitpid() wait for all child processes, not only
http://hg.python.org/cpython/rev/4cdb9f04494b
--
nosy: +python-dev
Charles-François Natali added the comment:
Committed! (The retry loop is addressed in another issue).
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
Serhiy Storchaka added the comment:
Rebase patch to tip.
--
Added file: http://bugs.python.org/file32262/cOrderedDict-2.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16991
___
Martin Matusiak added the comment:
I see one potential problem with this, namely that refactoring code that
contains break n statements would become more error prone whenever the depth
of the code block gets modified. So if you had something like:
for i in range(10):
for j in range(10):
James Lu added the comment:
Big example:
pygame, event proccessing loop running. the user clicks Quit, you do
break 2.
psuedocode:
while True:
for event in pygame.events.get():
if event.type==pygame.QUIT:
break 2
james
On Sun, Oct 20, 2013 at 2:36 PM, Martin Matusiak
James Lu added the comment:
Every new feature takes on new challenges and harder ways to debug.
But what about using that very confusing code that I showed that only let's you
break one amount, that would be harder to debug!
--
___
Python tracker
Mark Lawrence added the comment:
I suggest that there is a zero chance of this being actioned unless the OP
wants to do all of the work and attach all of the required patches to this
issue.
--
nosy: +BreamoreBoy
___
Python tracker
Arfrever Frehtes Taifersar Arahesis added the comment:
'continue' also should support this feature.
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19318
___
James Lu added the comment:
Oh, yes,yes,yes!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19318
___
___
Python-bugs-list mailing list
Guido van Rossum added the comment:
No, there's a use case for reading after the child exited, if there is a
grandchild still writing.
--Guido van Rossum (sent from Android phone)
On Oct 20, 2013 10:37 AM, Richard Oudkerk rep...@bugs.python.org wrote:
Richard Oudkerk added the comment:
I
Antoine Pitrou added the comment:
Language ideas such as this are best discussed on the python-ideas mailing-list:
https://mail.python.org/mailman/listinfo/python-ideas
(note that IMO this particular one has very little chance to succeed :-))
--
nosy: +pitrou
resolution: - postponed
Antoine Pitrou added the comment:
Do you want to provide a patch? It's hard for someone with a py2-linked gcc to
make sure that the libpython pluging works properly.
--
nosy: +pitrou
priority: normal - high
___
Python tracker rep...@bugs.python.org
Antoine Pitrou added the comment:
I don't think we want the devguide to become a Mercurial manual.
Also, it is easy to get help for a Mercurial command: just type hg help
bisect.
--
nosy: +pitrou
resolution: - wont fix
status: open - pending
___
David Coles added the comment:
Sure thing. I've got a patch attached to
https://bugs.launchpad.net/ubuntu/+source/gdb/+bug/1241668 but want to make
sure that it doesn't break things on the py2-linked version.
--
___
Python tracker
Antoine Pitrou added the comment:
Attached patch makes the get() function asynchronous.
--
Added file: http://bugs.python.org/file32263/tornado_bench2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19236
Antoine Pitrou added the comment:
Can you please try the attached updated patch?
--
keywords: +patch
stage: - patch review
type: - behavior
Added file: http://bugs.python.org/file32264/gdb_py3.patch
___
Python tracker rep...@bugs.python.org
Serhiy Storchaka added the comment:
LGTM.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18958
___
___
Python-bugs-list mailing list
Serhiy Storchaka added the comment:
LGTM. But perhaps it's worth to test json.load() with binary file. I don't see
much sense in testing json.loads() with numbers.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19307
Sunny K added the comment:
Added patch for 3.4.
The patch demarcates the output by adding a {...} around the dictionary
portion. Please let me know if this is the right format or if not required at
all. It is a simple change.
--
nosy: +sunfinite
Added file:
Changes by Antoine Pitrou pit...@free.fr:
--
resolution: - out of date
stage: - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18913
___
Antoine Pitrou added the comment:
Unless it's a security issue, it should probably only go into 3.4.
--
nosy: +pitrou
versions: -Python 2.7, Python 3.3
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18527
Antoine Pitrou added the comment:
Unrelated, but transp._process_exited will be called from the main thread even
if the loop is running from another thread. Shouldn't call_soon_threadsafe be
used here?
--
nosy: +pitrou
___
Python tracker
Antoine Pitrou added the comment:
Sorry, scratch that. add_signal_handler() won't even accept being called from a
non-main thread, according to the comments.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19310
Antoine Pitrou added the comment:
Since the alpha appears to have been delayed, I think it would be good to
commit.
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19297
___
1 - 100 of 186 matches
Mail list logo