Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
In the patch, _length_ is searched in the class and its base class. But what
happens with a third level?
class my_array3(my_array2):
pass
--
___
Python tracker rep...@bugs.python.org
New submission from Zhiping Deng kofreesty...@gmail.com:
http://docs.python.org/c-api/init.html?highlight=py_addpendingcall#Py_AddPendingCall
void Py_AddPendingCall(int (*func)(void *, void *arg))
which should be
void Py_AddPendingCall(int (*func)(void *), void *arg)
--
assignee:
Roundup Robot devnull@devnull added the comment:
New changeset fc2def15ab11 by Ezio Melotti in branch '2.7':
#11865: fix typo in init.rst.
http://hg.python.org/cpython/rev/fc2def15ab11
New changeset 6e090d78857c by Ezio Melotti in branch '3.1':
#11865: fix typo in init.rst.
Ezio Melotti ezio.melo...@gmail.com added the comment:
Fixed, thanks for the report!
--
assignee: docs@python - ezio.melotti
nosy: +ezio.melotti
resolution: - fixed
stage: - committed/rejected
status: open - closed
versions: +Python 3.1, Python 3.2, Python 3.3
Changes by Christoph Gohlke cgoh...@uci.edu:
--
nosy: +cgohlke
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11835
___
___
Python-bugs-list
kaifeng cafe...@gmail.com added the comment:
Found a minor defect of Python 3.2 / 3.3: line 1676 of xml/etree/ElementTree.py
was:
del self.target, self._parser # get rid of circular references
should be:
del self.target, self._target, self.parser, self._parser # get rid of
circular
STINNER Victor victor.stin...@haypocalc.com added the comment:
It looks like pthread_mutex_lock() and pthread_mutex_unlock() are not reentrant
on some platforms (in some implementations of the pthread API).
Antoine: if I understand correctly your patch, if we have a pending signal, all
next
Antoine Pitrou pit...@free.fr added the comment:
Antoine: if I understand correctly your patch, if we have a pending
signal, all next signals will be simply ignored.
I don't think so, please re-read.
--
___
Python tracker rep...@bugs.python.org
STINNER Victor victor.stin...@haypocalc.com added the comment:
pthread_sigmask() can be used to avoid reentrant call,
but it has no effect on the second case: signal_handler()
called twice at the same time in two different threads.
Same problem if we use sa_mask field of sigaction() (e.g.
STINNER Victor victor.stin...@haypocalc.com added the comment:
I don't think so, please re-read.
Oh, I thought that Py_AddPendingCall() was used to store the pending signal.
But no, it asks Python main loop to check which signal has been trigerred, and
we can only do it once for all signals.
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
title: test_signals() of test_threadsignals failure on Mac OS X -
signal_handler() is not reentrant: deadlock in Py_AddPendingCall()
___
Python tracker rep...@bugs.python.org
STINNER Victor victor.stin...@haypocalc.com added the comment:
Le lundi 18 avril 2011 à 10:40 +, STINNER Victor a écrit :
Attached patch should fix this issue:
- signal_handler() and PyErr_SetInterrupt() only call Py_AddPendingCall() on
the first signal (if is_tripped is zero): it
Changes by STINNER Victor victor.stin...@haypocalc.com:
Added file: http://bugs.python.org/file21703/signal_versionadded.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11768
___
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
title: test_signals() of test_threadsignals failure on Mac OS X -
signal_handler() is not reentrant: deadlock in Py_AddPendingCall()
___
Python tracker rep...@bugs.python.org
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
This file has a SyntaxError!
A comma is certainly missing at the end of line 47
--
nosy: +amaury.forgeotdarc
resolution: - invalid
status: open - closed
___
Python tracker
New submission from Peter Saveliev svinota.savel...@gmail.com:
The _newname() function has no locking.
It is called from the new thread constructor. Such constructor is executed
within parent thread. So, when several threads create new threads
simultaneously, there can be race condition,
Antoine Pitrou pit...@free.fr added the comment:
Well, since it is a bugfix, you should apply it to all versions.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11768
___
Tomasz Melcer li...@o2.pl added the comment:
If __str__ is meant to be a user-understandable way of expressing an URL, the
best way to do that is to show it the same way user sees it in usually. This is
now done by .geturl() call.
This way I wouldn't have to remember to serialize it in cases
Changes by Ernst Sjöstrand ern...@gmail.com:
--
nosy: +Ernst.Sjöstrand
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5115
___
___
Python-bugs-list
Changes by Giampaolo Rodola' g.rod...@gmail.com:
--
versions: +Python 3.3 -Python 3.0
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5115
___
___
Senthil Kumaran sent...@uthcode.com added the comment:
Tomasz,
I think, you misunderstood the purpose of urlparse library. urlparse is for
parsing the URL and into its components. The url could be a mailto:, svn+ssh or
http,https. The requirement for parsing comes when you are designing
Changes by R. David Murray rdmur...@bitdance.com:
--
versions: +Python 2.7
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11867
___
___
Changes by R. David Murray rdmur...@bitdance.com:
--
priority: - normal
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11828
___
___
Meador Inge mead...@gmail.com added the comment:
But what happens with a third level?
That doesn't work. I have a test case for that, but the test case has
a typo that caused it to pass. I will fix the test case and the code.
Thanks.
--
___
Roundup Robot devnull@devnull added the comment:
New changeset 51a551acb60d by R David Murray in branch '3.2':
#11492: rewrite header folding algorithm. Less code, more passing tests.
http://hg.python.org/cpython/rev/51a551acb60d
New changeset fcd20a565b95 by R David Murray in branch
Changes by R. David Murray rdmur...@bitdance.com:
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11492
Roundup Robot devnull@devnull added the comment:
New changeset 319f7af9ee5e by Victor Stinner in branch '3.1':
Issue #11768: The signal handler of the signal module only calls
http://hg.python.org/cpython/rev/319f7af9ee5e
New changeset 28ab8c6ad8f9 by Victor Stinner in branch '3.2':
(Merge 3.1)
Roundup Robot devnull@devnull added the comment:
New changeset 7d355c2ff3d4 by Victor Stinner in branch '2.7':
(Merge 3.1) Issue #11768: The signal handler of the signal module only calls
http://hg.python.org/cpython/rev/7d355c2ff3d4
New changeset ebd080593351 by Victor Stinner in branch '2.7':
R. David Murray rdmur...@bitdance.com added the comment:
This is fixed in 3.2/3.3 by the fix for issue 11492. The suggested fix for 2.7
is more radical than I'm comfortable with for a point release. I'm open to
argument on that, but in the meantime I'm closing the issue with 11492 as the
STINNER Victor victor.stin...@haypocalc.com added the comment:
The deadlock is fixed. Reopen the issue if you still see test_threadsignals
hang on a buildbot.
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
R. David Murray rdmur...@bitdance.com added the comment:
As of the fix for issue 11492, the email package only uses continuation_ws when
folding RFC2047 encoded words. So I consider this issue fixed. (I have, by
the way, come around to the view that we should never be introducing or
R. David Murray rdmur...@bitdance.com added the comment:
This now works correctly in 3.2/3.3 (see issue 11492). Note that the
whitespace compression is too deeply embeded in the 2.7 email package for there
to be any way to fix it there.
--
resolution: - duplicate
stage: test needed
Torsten Becker torsten.bec...@gmail.com added the comment:
I pushed my changes to a hg repository, they are in the two branches
startswith-slices-issue11828-2.7 and startswith-slices-issue11828-3.1.
--
hgrepos: +21
___
Python tracker
Changes by Torsten Becker torsten.bec...@gmail.com:
Added file: http://bugs.python.org/file21706/2b48fd451c85.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11828
___
Changes by Torsten Becker torsten.bec...@gmail.com:
--
hgrepos: +22
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11828
___
___
Python-bugs-list
Changes by Torsten Becker torsten.bec...@gmail.com:
Removed file: http://bugs.python.org/file21706/2b48fd451c85.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11828
___
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8407
___
___
STINNER Victor victor.stin...@haypocalc.com added the comment:
sigprocmask or (better) pthread_sigmask is required to fix #11859 bug.
---
Python has a test for broken pthread_sigmask. Extract of configure.in:
AC_MSG_CHECKING(if PTHREAD_SCOPE_SYSTEM is supported)
New submission from Carl Meyer c...@dirtcircle.com:
The opening paragraph of the lifecycle of a patch devguide page contains a
confusing parenthetical aside implying that an svn-like workflow would mean
never *saving* anything to your working copy and using hg diff to generate a
patch. This
Éric Araujo mer...@netwok.org added the comment:
Sure, +1 on using the submodule name in the module or currentmodule directive.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11785
___
Charles-Francois Natali neolo...@free.fr added the comment:
kaifeng cafe...@gmail.com added the comment:
I added 'malloc_trim' to the test code and rerun the test with Python 2.5 /
3.2 on CentOS 5.3. The problem still exists.
Well, malloc_trim can fail, but how did you add it ? Did you
Éric Araujo mer...@netwok.org added the comment:
The “remote hg repo” is supposed to be the URI of a clone (not of a changeset)
of cpython (not distutils2), so the repos you added don’t work. Can you remove
them and add a patch file instead? Thanks.
The first changeset looks good; I don’t
Changes by Łukasz Langa luk...@langa.pl:
--
assignee: - lukasz.langa
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11858
___
___
Python-bugs-list
Łukasz Langa luk...@langa.pl added the comment:
Thanks for the catch. I'll add some tests and push it.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11858
___
Amaury Forgeot d'Arc amaur...@gmail.com added the comment:
It's the very first usage of PyModuleDef::m_free.
Martin, do you agree with the path?
--
nosy: +amaury.forgeotdarc, loewis
___
Python tracker rep...@bugs.python.org
Andreas Stührk andy-pyt...@hammerhartes.de added the comment:
How about applying the workaround patch to Python 3.2? An unprecise error
message is way better than a segfault.
--
___
Python tracker rep...@bugs.python.org
Éric Araujo mer...@netwok.org added the comment:
Thanks for the patch. Follow the “review” link above to find my review.
+1 on making the smallest possible change. The paths inside the
Filelist/Manifest objects can continue to use os.sep, we only care about
write_file for this bug. Oh, and
Éric Araujo mer...@netwok.org added the comment:
Could you please at least describe the test that you mentioned (link
doesn't work).
I still have a local copy \o/. Yannick’s patch looks like this:
def test_skip_build(self):
pkg_pth, dist = self.create_dist()
# With the
Roundup Robot devnull@devnull added the comment:
New changeset 2f3fda9d3906 by R David Murray in branch 'default':
#11731: simplify/enhance parser/generator API by introducing policy objects.
http://hg.python.org/cpython/rev/2f3fda9d3906
--
nosy: +python-dev
Ned Deily n...@acm.org added the comment:
Thanks for the improvement. Applied in changeset cc43ed7af5f2.
--
nosy: +ned.deily
resolution: - fixed
stage: - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
New submission from Ned Deily n...@acm.org:
As far as I can see, the developer's guide does not mention at all the new
Rietveld code review tool integration with the Python bug tracker. The guide
should describe how the tool is expected to be used both by core developers
reviewing submitted
New submission from STINNER Victor victor.stin...@haypocalc.com:
test_3_join_in_forked_from_thread() of test_threading failed on x86 Ubuntu
Shared 3.x buildbot:
---
[201/354] test_threading
[41179 refs]
[40407 refs]
[40407 refs]
[40407 refs]
Timeout (1:00:00)!
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
nosy: +ezio.melotti
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11869
___
___
STINNER Victor victor.stin...@haypocalc.com added the comment:
Hum, it does still fail with a timeout of 1 hour:
---
[ 41/354] test_mmap
Timeout (1:00:00)!
Thread 0x7fff70439ca0:
File
Changes by Brett Cannon br...@python.org:
--
nosy: +brett.cannon
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11863
___
___
Python-bugs-list
Sijin Joseph sijinjos...@gmail.com added the comment:
From reading http://support.microsoft.com/kb/899147 it does look like the .dll
extension needs to have the Read Execute permission in order to have code
from the .dll be executed. It's odd that there isn't documentation that's
easily
Martin v. Löwis mar...@v.loewis.de added the comment:
Jeroen: I'm tempted to close this issue with no change. The code, as it stands,
models what Microsoft does in it's CRT (see crt/src/stat.c:__tdtoxmode). There
is also a straight-forward motivation to this: this is the list of extensions
STINNER Victor victor.stin...@haypocalc.com added the comment:
A timeout of 60 minutes looks to be fine. Today I debuged a thread+signal issue
which gave me an headache, so I prefer to close this issue: only add create
thread per file, and not a thread per function.
--
resolution: -
New submission from STINNER Victor victor.stin...@haypocalc.com:
While trying to reproduce issue #11870 using gdb -args ./python
Lib/test/regrtest.py -F -v --timeout=600 test_threading, I had the following
error on Linux:
--
test_default_timeout
Ned Deily n...@acm.org added the comment:
Note that the various HFS variants, the default file system types on OS X, do
not support sparse files at all. So any tests of large files require at some
point that the system writes out all the data in the file.
Ned Deily n...@acm.org added the comment:
That said, on my iMac (2.4 Ghz Intel Core 2 Duo):
$ time ./python -m test -v -u largefile test_mmap
[...]
--
Ran 24 tests in 87.673s
OK
1 test OK.
real1m27.875s
user0m0.186s
STINNER Victor victor.stin...@haypocalc.com added the comment:
We can use the following function to check if the filesystem does support
sparse files:
def support_sparse_file(directory):
import tempfile
with tempfile.NamedTemporaryFile(dir=directory) as tmpfile:
# Create a
Alexander Boyd a...@opengroove.org added the comment:
This is not fixed. The accepted fix doesn't take NotImplemented into account,
with the result that comparing two mutually-incomparable objects whose ordering
operations were generated with total_ordering causes a stack overflow instead
of
Jesús Cea Avión j...@jcea.es added the comment:
Torsten, could you possibly publish branches too for 3.2 and default (3.3)?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11828
___
Raymond Hettinger raymond.hettin...@gmail.com added the comment:
I'm not sure that we really care about handling NotImplemented (practicality
beats purity). At some point, if someone writing a class wants complete
control over the rich comparison methods, then they're going to have to write
Alexander Boyd a...@opengroove.org added the comment:
But it seems pointless to force someone to implement all of the rich comparison
methods when they may want to do something as simple as this:
class Foo:
...
def __lt__(self, other):
if not isinstance(other, Foo):
Changes by Brian Curtin br...@python.org:
--
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5162
___
___
Python-bugs-list
Raymond Hettinger raymond.hettin...@gmail.com added the comment:
It may seem pointless, but it takes less than a minute to do it and it would be
both faster and clearer to do it manually. There's a limit to how much
implicit code generation can or should be done automatically.
Also, I'm
Alexander Boyd a...@opengroove.org added the comment:
Ok. I did write that against Python 2, so I wasn't aware of __eq__ and __ne__.
I'll keep that in mind.
I am curious, however, as to how this could break existing code. It seems like
code that relies on a stack overflow is already broken as
Raymond Hettinger raymond.hettin...@gmail.com added the comment:
I am curious, however, as to how this could break existing code.
It seems like code that relies on a stack overflow is already
broken as it is.
Probably so. I worry about changes in semantics but it might be harmless.
STINNER Victor victor.stin...@haypocalc.com added the comment:
Timing on the x86 Tiger buildbot: time ./python.exe -m test -v -u largefile
test_mmap gives approx 5 minutes. The buildbot is a Mac mini, Intel Core Duo
1.8 GHz, 2 GB of memory, Mac OS X 10.4.10, HD: 232 GB (182 GB available) WDC
Ned Deily n...@acm.org added the comment:
Maybe that buildbot machine is just overloaded. If I understand the buildbot
waterfall output correctly, it looks like there are often 4 simultaneous test
runs happening on that machine, one each for 2.7, 3.1, 3.2 and 3.x.
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +gps
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11870
___
___
Python-bugs-list mailing list
kaifeng cafe...@gmail.com added the comment:
I applied your patch to Python 3.2, also I added a function call to
'malloc_trim' via ctypes, as you can see in issue11849_test2.py.
In fact I have a daemon written in Python 2.5, parsing an XML of size 10+ MB
every 5 minutes, after 16+ hours
Brian Cain brian.c...@gmail.com added the comment:
Please don't close the issue.
Joining aside, the basic point (But when size = 7279, the data submitted
reaches 64k, so the writting thread blocks on the write syscall.) is not
clear from the docs, right?
IMO, it would be nice if I could ask my
Changes by Terry J. Reedy tjre...@udel.edu:
Removed file: http://bugs.python.org/file21709/unnamed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8426
___
Terry J. Reedy tjre...@udel.edu added the comment:
Please do not send html responses, as they result in a spurious 'unnamed' file
being attached.
Please do suggest a specific change.
Should this be changed to a doc issue?
--
___
Python tracker
Brian Curtin br...@python.org added the comment:
Here's a patch replacing Modules/_multiprocessing/win32_functions.c and
PC/_subprocess.c with a common PC/_windows.c. There's not much to the patch
despite its size -- it just shuffles around the C code and does a few renames
in the appropriate
Changes by Brian Curtin br...@python.org:
--
keywords: +needs review
stage: - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11750
___
Changes by Santoso Wijaya santoso.wij...@gmail.com:
--
nosy: +santa4nt
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11750
___
___
80 matches
Mail list logo