Changes by Serhiy Storchaka storch...@gmail.com:
--
assignee: - serhiy.storchaka
components: +Library (Lib), Tests -Extension Modules
nosy: +serhiy.storchaka
versions: +Python 2.7 -Python 3.5
___
Python tracker rep...@bugs.python.org
New submission from Roundup Robot:
New changeset 73dad296c483 by Serhiy Storchaka in branch '2.7':
Issue 19276: Fix tests for wave files on big-endian platforms.
http://hg.python.org/cpython/rev/73dad296c483
New changeset 236884b40a2d by Serhiy Storchaka in branch '3.3':
Issue 19276: Fix tests
New submission from Guillaume Lebourgeois:
After the fetch of a webpage with a wrongly declared encoding, the use of
codecs module for a conversion crashes.
The issue is reproducible this way :
content = b+1911\' rel=\'stylesheet\' type=\'text/css\' /\nlink
rel=alternate
Changes by Serhiy Storchaka storch...@gmail.com:
--
assignee: - serhiy.storchaka
components: +Unicode
nosy: +ezio.melotti, serhiy.storchaka
stage: - needs patch
versions: +Python 3.4
___
Python tracker rep...@bugs.python.org
Changes by STINNER Victor victor.stin...@gmail.com:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19279
___
___
Python-bugs-list
Changes by Antoine Pitrou pit...@free.fr:
--
priority: normal - critical
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19275
___
___
Roundup Robot added the comment:
New changeset 58aaf7542efe by Christian Heimes in branch 'default':
Issue #19275: Fix test_site on AMD64 Snow Leopard
http://hg.python.org/cpython/rev/58aaf7542efe
--
nosy: +python-dev
___
Python tracker
Christian Heimes added the comment:
Let's hope that did the trick.
--
assignee: - christian.heimes
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19275
___
Nick Coghlan added the comment:
I added tests to the patch and tweaked a few details. Once the local test run
gives it a clean bill of health, I'll commit it so it will be included in 3.4a4
this weekend and you can check it works with Blender.
--
assignee: - ncoghlan
Esa Peuha added the comment:
Just by looking at the code, there still seems to be a problem: test_site.py
was changed in 0b6052f2a8ee to test that the collections module isn't imported,
but due to a bug it (re)tests the re module instead (re_mods needs to be
changed to collection_mods).
Roundup Robot added the comment:
New changeset 9cd88b39ef62 by Nick Coghlan in branch 'default':
Issue #16129: Add `Py_SetStandardStreamEncoding`
http://hg.python.org/cpython/rev/9cd88b39ef62
--
nosy: +python-dev
___
Python tracker
Nick Coghlan added the comment:
Putting this to pending for the moment - please try it out with Blender once
3.4a4 is released, and then we'll close it.
--
resolution: - fixed
stage: needs patch - committed/rejected
status: open - pending
___
Roundup Robot added the comment:
New changeset 537e13ca7683 by Nick Coghlan in branch 'default':
Issue #16129: Move Py_SetStandardStreamEncoding declaration
http://hg.python.org/cpython/rev/537e13ca7683
--
status: pending - open
___
Python tracker
Bastien Montagne added the comment:
Thanks Nick, we’ll do it for sure. :)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16129
___
___
Roundup Robot added the comment:
New changeset 22247b7d17fa by Nick Coghlan in branch 'default':
Close #19266: contextlib.ignore - contextlib.suppress
http://hg.python.org/cpython/rev/22247b7d17fa
--
nosy: +python-dev
resolution: - fixed
stage: needs patch - committed/rejected
status:
David Edelsohn added the comment:
The patch does not appear to fix the tests for wave on big-endian PPC64 Linux.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19276
___
New submission from R. David Murray:
In issue 18891, Stephen Turnbull wondered if adding a datatype to represent
mime types would be worthwhile.
I think it would be. A mimetype is a pair (maintype/subtype), and while one
may test the subparts independently in logic, the representation and
R. David Murray added the comment:
Thanks everyone for the reviews. I've opened up a new issue for Stephen's
mimetype object suggestion (issue 19280).
I'm still planning to add some examples to the docs, so I'm going to leave this
open until I do that, but I wanted to get the code itself
Serhiy Storchaka added the comment:
It fixed a bug in wave tests, but not a bug in the wave module. I'm working on
this.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19276
___
Richard Oudkerk added the comment:
I can reproduce the problem on the Non-Debug Gentoo buildbot using only
os.fork() and os.kill(pid, signal.SIGTERM). See
http://hg.python.org/cpython/file/9853d3a20849/Lib/test/_test_multiprocessing.py#l339
To investigate further I think strace and/or
Changes by Ethan Furman et...@stoneleaf.us:
--
keywords: +patch
Added file: http://bugs.python.org/file32155/issue19272.stoneleaf.01.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19272
Matthew Barnett added the comment:
The bytestring literal isn't valid. It starts with b and later on has an
unescaped followed by more characters.
Also, the usual way to decode by using the .decode method.
I get this:
content = b+1911\' rel=\'stylesheet\' type=\'text/css\' /\nlink
Guillaume Lebourgeois added the comment:
My fault, bad paste. Should have written :
content = b'+1911\' rel=\'stylesheet\' type=\'text/css\' /\nlink
rel=alternate type=application/rss+xml'
codecs.utf_7_decode(content, replace, True)
Traceback (most recent call last):
File stdin, line 1,
Guillaume Lebourgeois added the comment:
Also, the usual way to decode by using the .decode method.
The original bug happened using requests library, so I have no leverage on the
used method for decoding.
But if you used the replace mode with your methodology, you would have raised
the same
Nick Coghlan added the comment:
Indeed, 'utf-7' and the 'replace' error handler don't get along in this case.
--
nosy: +ncoghlan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19279
___
Nick Coghlan added the comment:
That is, I can locally reproduce the behaviour Guillaume describes on the
latest tip build.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19279
___
Roundup Robot added the comment:
New changeset 51480f6428a5 by Nick Coghlan in branch 'default':
Skip #16129 test until I debug cross-platform issues
http://hg.python.org/cpython/rev/51480f6428a5
--
___
Python tracker rep...@bugs.python.org
Nick Coghlan added the comment:
Buildbots weren't happy with the test case. I've done two extra commits (one to
turn off the diff limit, then a second one to skip it entirely), so will
hopefully be able to debug it and keep this in for the alpha.
(Larry - FYI, I'll have this test fixed or the
Nick Coghlan added the comment:
According to
http://buildbot.python.org/all/builders/AMD64%20Ubuntu%20LTS%203.x/builds/2716/steps/test/logs/stdio,
output in the subprocess was fine, but for some reason sys.stdout.encoding and
sys.stderr.encoding were None in the test itself o.O
--
Nick Coghlan added the comment:
Antoine pointed out this also needs a :versionadded: tag in the docs.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16129
___
Nick Coghlan added the comment:
So reusing the run command from the buildbot still passed on my system.
I thought using sys.stdout.encoding (etc) would reliably get me the default IO
encoding, but it appears not :(
--
___
Python tracker
R. David Murray added the comment:
I fixed the out of space last night. (Someday I'll get around to figuring out
which test it is that is leaving a bunch of data around when it fails, but I
haven't yet).
I've installed strace and gdb on the bots, please send me your public key and
I'll set
Nick Coghlan added the comment:
http://buildbot.python.org/all/builders/AMD64%20FreeBSD%2010.0%203.x/builds/579/steps/test/logs/stdio
shows the same symptoms
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16129
Serhiy Storchaka added the comment:
Here is a patch for 3.3+.
Other versions are affected too. They don't raise SystemError, but produce
illegal unicode string on wide build.
E.g. in Python 2.7:
'a+/,+IKw-b'.decode('utf-7', 'replace')
u'a\ufffd\U003f20acb'
\U003f20ac is illegal code.
As
Changes by STINNER Victor victor.stin...@gmail.com:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18191
___
___
Python-bugs-list
Changes by STINNER Victor victor.stin...@gmail.com:
--
nosy: +haypo
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1644818
___
___
Python-bugs-list
STINNER Victor added the comment:
You don't want to mention the return value (-1) on error in the doc?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16129
___
Stefan Krah added the comment:
Unfortunately the tests pass when I run them manually. On my FreeBSD
bot thsi may be relevant:
o LC_* is C.
o The filesystem encoding is ascii.
--
nosy: +skrah
___
Python tracker rep...@bugs.python.org
Richard Oudkerk added the comment:
I fixed the out of space last night. (Someday I'll get around to figuring
out which test it is that is leaving a bunch of data around when it fails,
but I haven't yet).
It looks like on the Debug Gentoo buildbot configure and clean are failing.
Georg Brandl added the comment:
Thanks for the suggestion; the parameters have already been added in Python
3.3, see http://docs.python.org/3/library/zlib.html#zlib.compressobj.
--
nosy: +georg.brandl
resolution: - out of date
status: open - closed
Georg Brandl added the comment:
Actually, compressobj() has always supported up to five arguments (named level,
method, wbits, memlevel, strategy); I'll document this for 2.7 for issue 19277.
The new thing in 3.3 are support for keyword arguments and the zdict parameter.
--
Roundup Robot added the comment:
New changeset c54c8e71b79a by Georg Brandl in branch '2.7':
Closes #19277: document all parameters of compressobj().
http://hg.python.org/cpython/rev/c54c8e71b79a
--
nosy: +python-dev
resolution: - fixed
stage: - committed/rejected
status: open -
Changes by R. David Murray rdmur...@bitdance.com:
--
Removed message: http://bugs.python.org/msg200151
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19277
___
R. David Murray added the comment:
Hmm. Looks like the name of the temp files used by the failing test (which is
in bsddb, by the way) have changed, so my delete command didn't delete them.
So now /tmp is definitely clean, and the next build on that bot should work
(crosses fingers).
R. David Murray added the comment:
Hmm. Looks like the name of the temp files used by the failing test (which is
in bsddb, by the way) have changed, so my delete command didn't delete them.
So now /tmp is definitely clean, and the next build on that bot should work
(crosses fingers).
Changes by Brett Cannon br...@python.org:
--
assignee: - brett.cannon
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18416
___
___
Changes by Serhiy Storchaka storch...@gmail.com:
--
stage: needs patch - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19276
___
___
Serhiy Storchaka added the comment:
What are array.array('l').itemsize and array.array('i').itemsize on PPC64 Linux?
I suppose that this patch should fix the wave module on 64-bit platforms.
--
keywords: +patch
stage: - needs patch
Added file:
Stefan Krah added the comment:
I've looked at this, and the question is: Do we really need a capsule
API? All relevant symbols in _decimal start with Py* and shouldn't
need to be static.
--
___
Python tracker rep...@bugs.python.org
David Edelsohn added the comment:
import array
print array.array('l').itemsize
8
print array.array('i').itemsize
4
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19276
___
Guido van Rossum added the comment:
Another day, another patch.
I'd rather like to commit this (and then iterate as needed), it makes my
workflow for porting to Windows a little easier.
Larry???
--
Added file: http://bugs.python.org/file32158/asyncio8.patch
Larry Hastings added the comment:
Does it break anything? (Besides, possibly, itself?)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19262
___
Serhiy Storchaka added the comment:
Well. Here is a patch. I have changed repr() a little. repr() now contains
match type qualified name (_sre.SRE_Match). groups now equals
len(m.groups()). span representation now contains a comma (as repr(m.span())).
Raymond, Ezio, is it good to you?
Antoine Pitrou added the comment:
For the record, why would this patch fix the issue? test_wave works fine under
other 64-bit platforms.
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19276
Serhiy Storchaka added the comment:
Thank you David. I suspected this.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19276
___
___
Guido van Rossum added the comment:
It doesn't break anything AFAICT. (It's a pure addition except for one new
extension module on Windows).
However I just discovered that apparently regrtest doesn't automatically
run tests in a subdirectory of the test package. I'm guessing I'll need to
add
Roundup Robot added the comment:
New changeset 9d2605f24a86 by Serhiy Storchaka in branch '2.7':
Issue #19276: Fixed the wave module on 64-bit big-endian platforms.
http://hg.python.org/cpython/rev/9d2605f24a86
New changeset a1a4a527c699 by Serhiy Storchaka in branch '3.3':
Issue #19276: Fixed
Larry Hastings added the comment:
If its breakage is restricted to itself (and its tests) then you have my
blessing to check it in. (Sorry, can't help you with the test expertise.)
--
___
Python tracker rep...@bugs.python.org
Serhiy Storchaka added the comment:
Perhaps other 64-bit platforms are little-endian?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19276
___
Adam Polkosnik added the comment:
I've got bitten by a different variation of this bug.
In my case the issue can be summarized by:
zipfile.BadZipfile: File name in directory Windows\TEMP\test.tmp and header
C:\Windows\TEMP\test.tmp differ.
Attached is a patch for Python27/lib/zipfile.py. I
Changes by Guido van Rossum gu...@python.org:
--
keywords: +needs review -patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19262
___
___
Guido van Rossum added the comment:
OK, no more giant patches. It's checked in. I've also solved the regrtest
issues by adding some code in __init__.py and __main__.py.
I expect to iterate a bit over the next few days.
--
___
Python tracker
Richard Oudkerk added the comment:
I finally have a gdb backtrace of a stuck child (started using os.fork() not
multiprocessing):
#1 0xb76194da in ?? () from /lib/libc.so.6
#2 0xb6d59755 in ?? ()
from
Marc-Andre Lemburg added the comment:
On 17.10.2013 21:01, Stefan Krah wrote:
Stefan Krah added the comment:
I've looked at this, and the question is: Do we really need a capsule
API? All relevant symbols in _decimal start with Py* and shouldn't
need to be static.
You need a capsule in
Barry A. Warsaw added the comment:
On Aug 15, 2013, at 08:25 PM, Antoine Pitrou wrote:
Barry, do you still want to keep this issue open?
I don't necessarily need to. We've patched the Ubuntu version to be safe, so
I guess we'll just carry that delta along until 3.4.
--
Roundup Robot added the comment:
New changeset 510bf951210f by Ned Deily in branch 'default':
Issue #19275: Fix test_site failure on OS X due to typo.
http://hg.python.org/cpython/rev/510bf951210f
--
___
Python tracker rep...@bugs.python.org
Ethan Furman added the comment:
Where do we stand with this issue?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17576
___
___
Python-bugs-list
Ned Deily added the comment:
Thanks for the observation, Esa. The typo is now fixed and the test is no
longer failing on OS X.
--
nosy: +ned.deily
resolution: - fixed
stage: - committed/rejected
status: open - closed
___
Python tracker
Serhiy Storchaka added the comment:
Fixed. Thank you David for your report.
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19276
Serhiy Storchaka added the comment:
Thank you David. Fixed in issue19276.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18919
___
___
Guido van Rossum added the comment:
Now asyncio/tulip has landed in the 3.4 stdlib, asyncore will be effectively
obsolete starting 3.4 (even if we don't mark it so). Its presence is required
for backwards compatibility, but that doesn't mean we should encourage people
to keep using it by
Richard Oudkerk added the comment:
Actually, according to strace the call which blocks is
futex(0xb7839454, FUTEX_WAIT_PRIVATE, 1, NULL
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19227
Nick Coghlan added the comment:
On 18 Oct 2013 02:41, STINNER Victor rep...@bugs.python.org wrote:
STINNER Victor added the comment:
You don't want to mention the return value (-1) on error in the doc?
I guess it should explicitly say any non-zero return value indicates an
error rather
Alexander Boyd added the comment:
Yep, that fixes it.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19213
___
___
Python-bugs-list mailing list
Roundup Robot added the comment:
New changeset 30f33e6a04c1 by Ned Deily in branch 'default':
Issue #19262: Install asyncio and test_asyncio directories.
http://hg.python.org/cpython/rev/30f33e6a04c1
--
nosy: +python-dev
___
Python tracker
R. David Murray added the comment:
Based on Marc-Andre recommendation at the end of issue 11678, I plan to commit
this as a bug fix, and add arch to 2.7 while I'm at it. Unless there are
objections.
--
nosy: +anikom15, eric.araujo, georg.brandl, pitrou, python-dev, terry.reedy
Larry Hastings added the comment:
Attached is a patch against current trunk merging Argument Clinic. I already
got the go-ahead to merge (took a poll in python-dev, got lots of +1s), but I
still wanted to post it here to let it soak a little, even only if it's just a
day. Unless something
101 - 177 of 177 matches
Mail list logo
