New submission from Graham Dumpleton graham.dumple...@gmail.com:
This is a followup bug report to fix wrong implementation of
_PyGILState_Reinit() introduced by http://bugs.python.org/issue10517.
I don't have a standalone test case yet. Problem occurs under mod_wsgi with
Python 2.7.2 and thus
Graham Dumpleton graham.dumple...@gmail.com added the comment:
Whoops. Missed the error. The fatal error that occurs is:
Fatal Python error: Couldn't create autoTLSkey mapping
--
___
Python tracker rep...@bugs.python.org
Changes by Charles-François Natali neolo...@free.fr:
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13156
___
___
Python-bugs-list
Charles-François Natali neolo...@free.fr added the comment:
Note that this doesn't apply to default: the problem is that 2.7 and 3.2 don't
use native TLS, and with the ad-hoc TLS implementation, a NULL value isn't
supported:
/* Internal helper.
* If the current thread has a mapping for
Graham Dumpleton graham.dumple...@gmail.com added the comment:
The PyGILState_Ensure() function is only for when working with the main
interpreter. These external threads are not calling into the main interpreter.
Because they are external threads, calling PyGILState_Ensure() and then
Charles-François Natali neolo...@free.fr added the comment:
So, the documentation you quote is only to do with the main
interpreter and is not how things work for sub interpreters.
You're right, my bad.
However, it would probably be better to destroy/reset the autoTLSkey
even if the current
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13156
___
___
Charles-François Natali neolo...@free.fr added the comment:
So the solution would be simply to backout this patch on 2.7 and 3.2.
Actually, I just checked, and the native TLS implementation is present in 3.2,
so this problem shouldn't show up: did you test it with 3.2?
AFAICT, this should
Graham Dumpleton graham.dumple...@gmail.com added the comment:
True. Doesn't appear to be an issue with Python 3.2.2. Only Python 2.7.2.
I was not aware that the TLS mechanism was changed in Python 3.X so assumed
would also affect it.
So, looks like the change shouldn't have been applied to
Sébastien Sablé sa...@users.sourceforge.net added the comment:
I got this problem again this morning while compiling pywin32.
This problem is not specific to me, anyone using Visual Studio 2010 to compile
Python is experiencing the same issue:
Antoine Pitrou pit...@free.fr added the comment:
How many moons before Python 2.7.3 though?
If you convince python-dev that's it's a critical bug (is it?) I suppose it
could happen soon.
--
___
Python tracker rep...@bugs.python.org
New submission from STINNER Victor victor.stin...@haypocalc.com:
It is no more possible to build Python outside its source directory. Try using:
cd python sources directory
mkdir release
cd release
../configure
make
Attached patch should fix this issue.
--
files: build.patch
keywords:
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
components: +Build
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13157
___
___
New submission from Sebastien Binet bi...@cern.ch:
hi there,
it seems tarfile in python 3.2.2 (as installed in archlinux, but I don't see
any additional patch applied on top of the vanilla sources:
http://projects.archlinux.org/svntogit/packages.git/tree/trunk/PKGBUILD?h=packages/python
) has
David Schnur dnsch...@gmail.com added the comment:
I haven't commented since opening this issue, but I've been following along.
Regarding Marc-Andre's latest comment, I think whether to embed a manifest or
not is a separate issue. The current behavior is to embed a manifest, and so
it
Changes by higery shoulderhig...@gmail.com:
Added file: http://bugs.python.org/file23387/adb2cb19ca9b.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8668
___
Sébastien Sablé sa...@users.sourceforge.net added the comment:
I had the same issue today. The patch solved the problem.
Thanks
--
nosy: +sable
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6807
Changes by Brian Curtin br...@python.org:
--
components: +Windows
nosy: +brian.curtin
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6807
___
___
Éric Araujo mer...@netwok.org added the comment:
Yes, you did. There are a few minor edits I’ll make to the patch, but
otherwise this is good to commit.
--
assignee: tarek - eric.araujo
versions: +3rd party
___
Python tracker rep...@bugs.python.org
Éric Araujo mer...@netwok.org added the comment:
#9878 should also help with start-up time.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13150
___
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13150
___
___
Éric Araujo mer...@netwok.org added the comment:
Actually, #9878 should supersede this bug: it proposes to generate a C module
to avoid parsing Makefile and pyconfig.h, and your patch proposes to generate a
Python module with the same goal.
--
___
Éric Araujo mer...@netwok.org added the comment:
Thanks for the updated patch. There are things that should be removed (for
example functions like get_develop_method, given that we only support
packaging-based projects), and there are a few things to clean up in the tests.
What’s more
Martin v. Löwis mar...@v.loewis.de added the comment:
It would be interesting to see how often the bad case triggers, i.e. that a
write-only stringio sees any of the other methods invoked at all.
As a special case, you may consider that .truncate(0) doesn't really need to
realize the buffer
Martin v. Löwis mar...@v.loewis.de added the comment:
If you don't use git-style diffs, Rietveld will much better accommodate patches
that don't apply to tip cleanly. Unfortunately, hg git-style diffs don't
indicate the base revision, so Rietveld guesses that the base line is tip, and
then
Antoine Pitrou pit...@free.fr added the comment:
Actually, #9878 should supersede this bug: it proposes to generate a C
module to avoid parsing Makefile and pyconfig.h, and your patch
proposes to generate a Python module with the same goal.
Well, #9878 doesn't have a patch, but perhaps
New submission from Nadeem Vawda nadeem.va...@gmail.com:
As mentioned in issue 6715, the buffer growth strategy used by _io.FileIO
has quadratic running time if each reallocation is O(n). The code in
question is new_buffersize() from Modules/_io/fileio.c:
if (currentsize SMALLCHUNK) {
Antoine Pitrou pit...@free.fr added the comment:
Yes, these are things I've been wondering about. The use-case for an
append-only StringIO is obviously overlapping with the use-case for using
''.join(). However, the implementation I'm proposing is better than ''.join()
when writing very small
Martin v. Löwis mar...@v.loewis.de added the comment:
Modules/_lzmamodule.c:364: Py_BEGIN_ALLOW_THREADS
It seems that the Windows version at least is not thread-safe. If so, you
would need an LZMA lock when releasing the GIL.
Does the class need to be thread-safe, though?
As a matter of
Éric Araujo mer...@netwok.org added the comment:
On line 118, replacing 'install' with 'install_dist' should fix it.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13151
___
Martin v. Löwis mar...@v.loewis.de added the comment:
Correct. I copied the algorithm from _io.FileIO, under the assumption
that there was a reason for not using a simpler O(n log n) doubling
strategy. Do you know of any reason for this? Or is it safe to ignore it?
I don't know, but I'd say
New submission from Éric Araujo mer...@netwok.org:
When we first added actions to pysetup, the install command was renamed to
install_dist to avoid a name conflict between the install command and the
install action. Later, the run action was introduced and now serves to
namespace all
Changes by Éric Araujo mer...@netwok.org:
--
nosy: +tarek
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13160
___
___
Python-bugs-list mailing
Changes by Éric Araujo mer...@netwok.org:
--
dependencies: +Rename install_dist to install
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13151
___
Nadeem Vawda nadeem.va...@gmail.com added the comment:
To elaborate on the policy: giving bogus data in cases of race
conditions is ok; crashing the interpreter or corrupting memory
is not. If bogus data is given, it would be useful if the bogosity
can be specified (e.g. when multiple
Éric Araujo mer...@netwok.org added the comment:
I suggest that it do so in this instance also, using *script* (in
bold-faced italic) as the entry title.
What do you think about :file:`{script}`? file+{} is what Sphinx uses for
filenames with replaceable parts, which map to the HTML var
New submission from Eli Bendersky eli...@gmail.com:
Report from Joao Carneiro in the docs@ maillist:
I would like to report that I found a mistake in the document of help(set).
In the document all the __i*__ methods are described exactly like the same
methods
Éric Araujo mer...@netwok.org added the comment:
I made a few edits and committed to distutils and packaging. Then I ported the
packaging patch to distutils2, which supports Python 2.4-2.7, and got this
failure:
FAIL: test_glob_to_re (__main__.ManifestTestCase)
Charles-François Natali neolo...@free.fr added the comment:
Here is a patch for import.c.
Looks good to me.
This new patch also fixes importlib.
path_tmp = path + '.tmp'
with _io.FileIO(path_tmp, 'wb') as file:
file.write(data)
Ezio Melotti ezio.melo...@gmail.com added the comment:
The italics parts are easier to recognize when they are within regular text
(e.g. :file:`path/with/python{XY}/file`). If the whole text is in italic
people might not notice the difference.
--
Antoine Pitrou pit...@free.fr added the comment:
This new patch also fixes importlib.
path_tmp = path + '.tmp'
with _io.FileIO(path_tmp, 'wb') as file:
file.write(data)
_os.rename(path_tmp, path)
I don't know
Changes by Thomas Holmes tho...@devminded.com:
Added file: http://bugs.python.org/file23388/7db732ac6796.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12344
___
Éric Araujo mer...@netwok.org added the comment:
Looks good, will commit.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3902
___
___
Éric Araujo mer...@netwok.org added the comment:
Great. Do you want to update the docs (packaging/setupscript and maybe
packaging/commandref too) or shall I do it?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12393
Nadeem Vawda nadeem.va...@gmail.com added the comment:
I've attached a patch that makes the suggested change to FileIO, and also
to _bz2.BZ2Compressor/Decompressor (which currently have the same issue).
--
keywords: +patch
Added file: http://bugs.python.org/file23389/buffer-growth.diff
Changes by Nadeem Vawda nadeem.va...@gmail.com:
--
stage: needs patch - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13159
___
___
Changes by STINNER Victor victor.stin...@haypocalc.com:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13159
___
___
Changes by Thomas Holmes tho...@devminded.com:
Added file: http://bugs.python.org/file23390/1bb1132840e6.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12344
___
Thomas Holmes tho...@devminded.com added the comment:
Still working with Éric to determine the proper behavior of
get_reinitialized_commands. Latest patch is known incorrect but a step closer
to our destination.
--
___
Python tracker
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 8bbfb24d4824 by Victor Stinner in branch 'default':
Issue #12367: Add a test on error attribute of select.error
http://hg.python.org/cpython/rev/8bbfb24d4824
--
nosy: +python-dev
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset ee4fe16d9b48 by Charles-François Natali in branch '2.7':
Issue #13156: revert changeset f6feed6ec3f9, which was only relevant for native
http://hg.python.org/cpython/rev/ee4fe16d9b48
--
nosy: +python-dev
STINNER Victor victor.stin...@haypocalc.com added the comment:
The issue has been fixed by the merge of the PEP 3151: I now get a
BrokenPipeError with errno=32 and winerror=232.
--
status: open - closed
___
Python tracker rep...@bugs.python.org
Terry J. Reedy tjre...@udel.edu added the comment:
Since #9878 proposes an *alternate* solution to *part* of the sysconfig
problem, I disagree with 'supersede'. A Python solution would be more useful
for other implementations if enough of the sysconfig info is not CPython
specific.
A CPython
Charles-François Natali neolo...@free.fr added the comment:
Here's a patch for 3.2 and default which calls
PyThread_set_key_value() only if there was an auto thread state
previously associated (while the current code works with pthread TLS,
there are other implementations which may behave
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 1811c4e5527f by Victor Stinner in branch 'default':
Issue #13157: Fix building Python outside its source tree
http://hg.python.org/cpython/rev/1811c4e5527f
--
nosy: +python-dev
Terry J. Reedy tjre...@udel.edu added the comment:
Since *script* is a file -- that is all variable -- that seems appropriate.
Bold italic tends to be more notably different from bold italic than the normal
pair. In any case, italic is the doc standard for function parameter names. It
seems
STINNER Victor victor.stin...@haypocalc.com added the comment:
Without the patch:
python3.2 -m timeit 'x=é+x*1' 'x[1:]'
10 loops, best of 3: 2.18 usec per loop
python3.3 -m timeit 'x=é+x*1'
STINNER Victor victor.stin...@haypocalc.com added the comment:
With find_max_char4.patch:
python3.3 -m timeit 'x=é+x*1' 'x[1:]'
10 loops, best of 3: 1.96 usec per loop
--
___
Python tracker rep...@bugs.python.org
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 3313ce92cef7 by Charles-François Natali in branch '2.7':
Issue #13156: Add an entry in Misc/NEWS.
http://hg.python.org/cpython/rev/3313ce92cef7
--
___
Python tracker
Changes by Mike Hoy mho...@gmail.com:
--
nosy: +mikehoy
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13154
___
___
Python-bugs-list mailing list
Charles-François Natali neolo...@free.fr added the comment:
Or perhaps append the PID to the name of the temp file ?
(easier done in Python than in C :-))
I don't really like appending PIDs to generate file names:
- if you have multiple processes at the same time, they'll all write
their own
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment:
Great. Do you want to update the docs (packaging/setupscript and maybe
packaging/commandref too) or shall I do it?
Feel free to do it, if that's OK with you. I've got a couple of other pythonv
issues which are taking my time at the
STINNER Victor victor.stin...@haypocalc.com added the comment:
find_max_char5.patch:
- don't use adjusted ~mask+1: precompute the right max_char
- rename findwidth.h to find_max_char.h
- add some #undef
--
Added file: http://bugs.python.org/file23392/find_max_char5.patch
Mike Hoy mho...@gmail.com added the comment:
:~/peps$ find . -name *.txt | xargs grep canterbury
./pep-3152.txt:Author: Gregory Ewing greg.ew...@canterbury.ac.nz
./pep-0380.txt:Author: Gregory Ewing greg.ew...@canterbury.ac.nz
./pep-0284.txt:Greg Ewing greg.ew...@canterbury.ac.nz
Roumen Petrov bugtr...@roumenpetrov.info added the comment:
Which is platform with broken VPATH support and/or make ?
--
nosy: +rpetrov
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13157
STINNER Victor victor.stin...@haypocalc.com added the comment:
with atomic_create(final path, 'b') as f:
See issues #8604 and #8828.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13146
STINNER Victor victor.stin...@haypocalc.com added the comment:
Which is platform with broken VPATH support and/or make ?
Well, my commit is maybe useless...
I tried to have two builds at the same time:
- debug in Python source code (e.g. ~/python)
- release in a subdirectory in the Python
Roundup Robot devn...@psf.upfronthosting.co.za added the comment:
New changeset 9c7d3207fc15 by Antoine Pitrou in branch 'default':
Issue #13155: Optimize finding the optimal character width of an unicode string
http://hg.python.org/cpython/rev/9c7d3207fc15
--
nosy: +python-dev
Antoine Pitrou pit...@free.fr added the comment:
find_max_char5.patch:
- don't use adjusted ~mask+1: precompute the right max_char
- rename findwidth.h to find_max_char.h
- add some #undef
Thank you, I've committed this version.
--
resolution: - fixed
stage: - committed/rejected
Vinay Sajip vinay_sa...@yahoo.co.uk added the comment:
On line 118, replacing 'install' with 'install_dist' should fix it.
Sadly, it just defers the problem:
vinay@eta-natty:~/projects/dory$ pysetup3 run bdist_wininst
running bdist_wininst
running build
running build_py
running build_scripts
Roumen Petrov bugtr...@roumenpetrov.info added the comment:
I wonder why you are not stopped by:
configure: error: source directory already configured; run make distclean
there first.
(for sure you remove some files but not all)
Applied patch is save but I cannot understand what actually is
New submission from Paul Moore p.f.mo...@gmail.com:
I am trying to create a pysetup package that contains a precompiled binary
extension, which can be installed without compiling, by using the resource file
feature of setup.cfg. This may be incorrect usage of pysetup, but at a minimum,
a
New submission from Ram Rachum r...@rachum.com:
Look here:
http://hg.python.org/cpython/file/3313ce92cef7/Lib/smtplib.py#l279
`port` and `host` are confused.
I saw this is fixed on 3.2; possibly it should be fixed in the next 2.7 micro
release too.
--
components: Library (Lib)
Antoine Pitrou pit...@free.fr added the comment:
The patch looks good to me, thanks.
--
versions: +Python 2.7, Python 3.2
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13159
___
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +georg.brandl
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13154
___
___
Python-bugs-list
Senthil Kumaran sent...@uthcode.com added the comment:
Fixed in 25ff1adf5f30
--
nosy: +orsenthil
resolution: - fixed
stage: - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13154
Changes by John O'Connor tehj...@gmail.com:
--
nosy: +jcon
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13150
___
___
Python-bugs-list mailing
Antoine Pitrou pit...@free.fr added the comment:
Please double check the removal of _PyUnicode_XStrip.
I think it's ok. It is private, undocumented, and the only references
returned by Google Code Search are in Python itself.
I'll review the patch later, thank you.
--
title: Move
Changes by Nadeem Vawda nadeem.va...@gmail.com:
Added file: http://bugs.python.org/file23393/bdf0afbbbd80.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6715
___
Changes by Nadeem Vawda nadeem.va...@gmail.com:
Removed file: http://bugs.python.org/file23168/591277fe6b4a.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6715
___
Nadeem Vawda nadeem.va...@gmail.com added the comment:
Patch update time! Noteworthy changes:
* Windows build support, thanks to Amaury
* Thread safety for LZMACompressor and LZMADecompressor, by means of
per-instance locks
* O(n) buffer growth strategy
I've tried running the tests on Windows,
81 matches
Mail list logo