Martin v. Löwis mar...@v.loewis.de added the comment:
The one thing that IMO needs to be decided before this can be accept is
the version compatibility: what Python versions must this code stay
compatible with? That decision then needs to be implemented.
Apart from this (and the additional
Retro vinet...@gmail.com added the comment:
I noticed this by installing the Python 2.5.4 and Python 2.6.1 and
Python 3.0.1 binaries. Please fix these issues if you can.
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5390
Martin v. Löwis mar...@v.loewis.de added the comment:
I did not backport the fix to 2.5.4, so that release doesn't have it
fixed. Since the 2.5 branch is now closed, no changes will be made to it.
For 2.6.1 and 3.0.1, it works fine for me. See the attached screenshot.
Added file:
Tarek Ziadé ziade.ta...@gmail.com added the comment:
done in r70049 and r70051
--
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5378
___
Tarek Ziadé ziade.ta...@gmail.com added the comment:
Done in r70049 and r70051.
Thanks for everyone's help !
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1533164
___
Changes by Tarek Ziadé ziade.ta...@gmail.com:
--
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1533164
___
___
Changes by Tarek Ziadé ziade.ta...@gmail.com:
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1533164
___
___
Python-bugs-list mailing list
Unsubscribe:
Georg Brandl ge...@python.org added the comment:
I can't see at the moment how the unicode_decode_call_errorhandler call
can be made without startinpos being previously set to some value.
Antoine, maybe you can verify?
--
assignee: - pitrou
nosy: +georg.brandl, pitrou
Martin v. Löwis mar...@v.loewis.de added the comment:
See comment 1 in
https://bugzilla.redhat.com/show_bug.cgi?id=236535
It might be that this patch still doesn't solve the problem, namely if
there are non-library .py files in the distribution. They get compiled
by brp-python-bytecompile, but
Martin v. Löwis mar...@v.loewis.de added the comment:
Tarek, I think you committed too quickly. Please consider my comments.
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1533164
___
Martin v. Löwis mar...@v.loewis.de added the comment:
simplejson maintains Python 2.4+ compatibility, but json maintains 2.6+.
I could produce another patch that manually removes these few remaining
nits if it's necessary.
I don't quite understand this: isn't json/decoder.py and
Antoine Pitrou pit...@free.fr added the comment:
Hmm, I know nothing about UTF7...
Anyway, looking at the code, the utf7Error code path can be called from
the following places (trunk line numbers):
- line 1595, and startinpos was set three lines before
- a bunch of places in the if (inShift) {
Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp added the comment:
Fixed in r70052(trunk), r70053(py3k).
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5386
New submission from Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp:
On Python3000, mmap.read_byte returns str not bytes, and mmap.write_byte
accepts str. Is this intended behavior?
import mmap
m = mmap.mmap(-1, 10)
type(m.read_byte())
class 'str'
m.write_byte(a)
m.write_byte(ba)
Maybe
Martin v. Löwis mar...@v.loewis.de added the comment:
Indeed, I think it should use the b code, instead of the c code.
Please discuss this on python-dev, though.
It might not be ok to backport this to 3.0, since it may break existing
code.
--
nosy: +loewis
Martin v. Löwis mar...@v.loewis.de added the comment:
Furthermore, all other uses of the c code might need to be reconsidered.
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5391
___
New submission from Gabriel Genellina gagsl-...@yahoo.com.ar:
Set sys.setrecursionlimit to 50 or lower. Then, the second time the
recursion limit is reached, the interpreter crashes with a stack
overflow.
This happens both with released 3.0.1 and the py3k branch, on Windows.
At least on my PC,
Tarek Ziadé ziade.ta...@gmail.com added the comment:
I've read the thread while working in the patch,
and comment #28 explains that brp-python-bytecompile is called over
the rpms and requires pyo files within the file list.
plus:
http://fedoraproject.org/wiki/Packaging/Python#Including_pyos
Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp added the comment:
Fixed in r70056(trunk) and r70057(py3k).
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1733986
Changes by Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp:
--
versions: -Python 3.0
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5391
___
___
Changes by Benjamin Peterson benja...@python.org:
--
nosy: +loewis, pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5392
___
___
Georg Brandl ge...@python.org added the comment:
I can reproduce that (with the same limits) on Linux here.
It doesn't happen with 2.6.
(Although it is technically not a crash but a controlled abort().)
--
nosy: +georg.brandl
___
Python tracker
STINNER Victor victor.stin...@haypocalc.com added the comment:
loewis Furthermore, all other uses of the c code might
loewis need to be reconsidered.
$ grep 'BuildValue.*c' */*c
Modules/_cursesmodule.c:return Py_BuildValue(c, rtn);
Modules/mmapmodule.c: return Py_BuildValue(c,
STINNER Victor victor.stin...@haypocalc.com added the comment:
@tjreedy: Do you expect conversion to small int if __int__() result
fits in a small int?
class A:
def __int__(self):
return 1L
x=int(A())
print repr(x), type(x)
Result with Python 2.5.1: 1L type 'long'
Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp added the comment:
I think more *bytes* cleanup is needed for mmap module documentation
implementation. (and other modules?) Especially mmap.find() and its friends.
import mmap
m = mmap.mmap(-1, 10)
m[:] = b0123456789
m.find(b'2')
2
m.find('2')
Mark Dickinson dicki...@gmail.com added the comment:
The behaviour doesn't contradict the documentation, as far as I can
tell,
so I agree with Terry that this is not a bug.
If we want the result from the built-in int function to have type int
whenever possible (that is, whenever the result
Changes by Mark Dickinson dicki...@gmail.com:
--
type: behavior - feature request
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5377
___
___
John Levon movem...@users.sourceforge.net added the comment:
Surely readline is irrelevant anyway. The Python spec guarantees
behaviour, and that guarantee is currently broken.
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1975
New submission from Michael Newman michael.b.new...@gmail.com:
cmath.cos and cmath.cosh have nResult typo in their help text.
Python 3.0.1 (r301:69561, Feb 13 2009, 20:04:18) [MSC v.1500 32 bit
(Intel)] on win32
Type help, copyright, credits or license for more information.
import cmath
Antoine Pitrou pit...@free.fr added the comment:
This is probably due to the recursion overflow recovery code in py3k,
which has a hard-wired constant of 50 somewhere :-)
(is setting the recursion limit so low a requirement for your
application? or were you just experimenting with it? as Georg
Mark Dickinson dicki...@gmail.com added the comment:
Fixed in the trunk in r70062. Thanks for the report!
--
nosy: +marketdickinson
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5393
Guido van Rossum gu...@python.org added the comment:
Hm, I'm not sure why Adam brought up readline. The behavior is
certainly guaranteed (I put that guarantee in myself long ago :-) and it
should be fixed. I have no opinion about the proposed patch, since I
cannot test this and have long lost
Mark Dickinson dicki...@gmail.com added the comment:
Good catch! Added defined(SIZEOF_WCHAR) to the testcapi code as well,
and removed the change to PC/pyconfig.h, since we don't need it any
more...
Added file:
http://bugs.python.org/file13210/unicode_fromwidechar_surrogate-7.patch
Changes by Benjamin Peterson benja...@python.org:
--
priority: critical - normal
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2848
___
___
New submission from Akira Kitada akit...@gmail.com:
As written in its README, Distutils must remain compatible with 2.3,
but it isn't. Attached patch fixes this.
--
assignee: tarek
components: Distutils
files: python23_compat.diff
keywords: patch
messages: 82920
nosy: akitada, tarek
Benjamin Peterson benja...@python.org added the comment:
In the io-c branch, read1() is now a member of BufferedIOBase.
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4996
___
Benjamin Peterson benja...@python.org added the comment:
This has been cured in the io-c branch.
--
nosy: +benjamin.peterson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4263
___
Benjamin Peterson benja...@python.org added the comment:
I just fixed the last failing test_io.
(I'm listing as dependencies issues we can close after the branch is
merged.)
--
dependencies: +BufferedWriter non-blocking overage, io.TextIOWrapper calls
buffer.read1()
Changes by Benjamin Peterson benja...@python.org:
--
dependencies: +possible deadlock in IO library (Lib/io.py)
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4565
___
Benjamin Peterson benja...@python.org added the comment:
Yes, this is solved in the io-c branch. Antoine, do you think we should
switch _pyio to use the RLock?
--
nosy: +benjamin.peterson
___
Python tracker rep...@bugs.python.org
Benjamin Peterson benja...@python.org added the comment:
I'm going to close this as a duplicate of #4996 (same fix), which will
be fixed when we merge the io-c branch.
--
nosy: +benjamin.peterson
resolution: - duplicate
status: open - closed
___
Benjamin Peterson benja...@python.org added the comment:
Does io in C help with this?
--
nosy: +benjamin.peterson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1407
___
Benjamin Peterson benja...@python.org added the comment:
I'm no ssl expert either, but the patch looks fine to me.
--
nosy: +benjamin.peterson
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4967
Changes by Benjamin Peterson benja...@python.org:
--
dependencies: +Duplicate UTF-16 BOM if a file is open in append mode, utf-16
BOM is not skipped after seek(0)
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4565
Benjamin Peterson benja...@python.org added the comment:
These StringIO bugs should be dealt with:
#5264
#5265
#5266
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4565
___
Akira Kitada akit...@gmail.com added the comment:
3rd party C modules are put in site-packages,
so just having importer of 64-bit python look at lib64-dynload is not
enough for solving this.
To work around this problem, I did some hacks on my local Python to look
at lib and lib64. It worked,
Changes by Akira Kitada akit...@gmail.com:
--
assignee: - tarek
components: +Windows
nosy: +tarek
type: - compile error
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2698
___
Changes by Akira Kitada akit...@gmail.com:
--
assignee: - tarek
components: +Windows
nosy: +tarek
type: - compile error
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2941
___
Changes by Akira Kitada akit...@gmail.com:
--
assignee: - tarek
components: +Windows
nosy: +tarek
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2200
___
Changes by Akira Kitada akit...@gmail.com:
Removed file: http://bugs.python.org/file12954/issue4010.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4010
___
Akira Kitada akit...@gmail.com added the comment:
Updated issue4010 to honor os.environ['CPPFLAGS'].
Added file: http://bugs.python.org/file13212/issue4010.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4010
Tarek Ziadé ziade.ta...@gmail.com added the comment:
right, this code is not covered by tests. I'll apply your patch, and add
a test case for that.
--
priority: - high
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5394
Changes by Tarek Ziadé ziade.ta...@gmail.com:
--
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2385
___
___
Python-bugs-list
Antoine Pitrou pit...@free.fr added the comment:
Closing, as I don't think there's any point trying to micro-optimize the
Python version.
--
resolution: later - wont fix
status: open - closed
___
Python tracker rep...@bugs.python.org
Antoine Pitrou pit...@free.fr added the comment:
I don't know. The RLock is a lot slower than the normal non-recursive
variation, on the other hand I'm not sure we care about performance of
the Python version that much. Opinions welcome.
___
Python tracker
Changes by Benjamin Peterson benja...@python.org:
--
assignee: - janssen
nosy: +janssen
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5238
___
Benjamin Peterson benja...@python.org added the comment:
2009/2/28 Antoine Pitrou rep...@bugs.python.org:
Antoine Pitrou pit...@free.fr added the comment:
I don't know. The RLock is a lot slower than the normal non-recursive
variation, on the other hand I'm not sure we care about
Martin v. Löwis mar...@v.loewis.de added the comment:
I've read the thread while working in the patch,
and comment #28 explains that brp-python-bytecompile is called over
the rpms and requires pyo files within the file list.
Sure, I'm not objecting. I claim that the patch might be
New submission from Eduardo Aguiar agu...@users.sourceforge.net:
At arraymodule.c (line 1258):
nread = fread(item + (Py_SIZE(self) - n) * itemsize,
itemsize, n, fp);
if (nread (size_t)n) {
Py_SIZE(self) -= (n -
New submission from Eduardo Aguiar agu...@users.sourceforge.net:
At posixmodule.c (line 6306)
static PyObject *
posix_read(PyObject *self, PyObject *args)
{
int fd, size, n;
PyObject *buffer;
if (!PyArg_ParseTuple(args, ii:read, fd, size))
return NULL;
Alexandre Vassalotti alexan...@peadrop.com added the comment:
The motivation behind the restriction was to prevent the tp_doc member
to get truncated. Since tp_doc is a C string, it cannot contains embeded
null bytes.
However, it seems tp_doc is not used at all for heap types. So, I don't
mind
Mike Meyer m...@users.sourceforge.net added the comment:
The args attribute gets created by __init__ and nothing in the class
removes it. I don't think it's unreasonable for the class to expect the
attribute to not vanish on it. Possibly it should be spelled __args (or
declared private :-), but
Changes by Alexandre Vassalotti alexan...@peadrop.com:
Added file:
http://bugs.python.org/file13214/remove_null_restriction_on_classdoc.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5373
___
Changes by Alexandre Vassalotti alexan...@peadrop.com:
Removed file:
http://bugs.python.org/file13213/remove_null_restriction_on_classdoc.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5373
___
Benjamin Peterson benja...@python.org added the comment:
Ok, will leave as is.
--
versions: -Python 3.0, Python 3.1
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4967
___
Robin Dunn ro...@alldunn.com added the comment:
Thanks for the code. I've verified your findings and I've also
converted nested to an extension module and built it with distutils and
was still able to make it correctly load the themed common controls when
imported from Python, however I had to
Georg Brandl ge...@python.org added the comment:
Looks good. (Alternately, you could also set tp_doc to NULL in the
embedded \0 case.)
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5373
___
Hirokazu Yamamoto ocean-c...@m2.ccsnet.ne.jp added the comment:
Patch attached. read_byte and write_byte use integer as byte, and other
bytes related cleanup.
--
keywords: +patch
Added file: http://bugs.python.org/file13215/py3k_mmap_and_bytes.patch
Akira Kitada akit...@gmail.com added the comment:
There are two options for that in bdist_rpm.
--python path to Python interpreter to hard-code in the .spec
file (default: python)
--fix-python hard-code the exact path to the current Python
Changes by Akira Kitada akit...@gmail.com:
--
assignee: - tarek
components: +Windows
nosy: +tarek
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4214
___
Cherniavsky Beni c...@users.sf.net added the comment:
Perhaps it should not be an error at all? The default should probably
stay strict to the spec, but IMHO the module should provide an optional
lenient parsing mode that also accepts single quotes.
Why support single quotes and not any other
Matthew Barnett pyt...@mrabarnett.plus.com added the comment:
issue2636-features-5.diff includes:
Bugfixes
Added \G anchor (from Perl).
\G is the anchor at the start of a search, so re.search(r'\G(\w)') is
the same as re.match(r'(\w)').
re.findall normally performs a series of searches, each
Raymond Hettinger rhettin...@users.sourceforge.net added the comment:
+1 on Steven's request for a better error message.
+1 on Beni's request for looser input requirements for better
interoperability with Python's repr. OTOH, I've never found it hard to
write: s.replace(', '').
--
Mads Kiilerich m...@kiilerich.com added the comment:
Martin,
What is the goal of bdist_rpm? I haven't seen that stated explicitly
anywhere, but I assume the goal is to make a fair attempt to easily
create usable RPMs for some software already using distutil,
acknowledging that it might not work
Akira Kitada akit...@gmail.com added the comment:
'ignore' was introduced in Python 2.6 but distutils has to keep Python
2.3 compatible.
See: http://bugs.python.org/issue5052
So I guess you have to wait some more years before
dropping distutils.dir_util and distutils.file_util.
--
Changes by Akira Kitada akit...@gmail.com:
--
type: - feature request
versions: +Python 2.6, Python 2.7, Python 3.0, Python 3.1 -Python 2.4
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1355826
Martin v. Löwis mar...@v.loewis.de added the comment:
What is the goal of bdist_rpm? I haven't seen that stated explicitly
anywhere, but I assume the goal is to make a fair attempt to easily
create usable RPMs for some software already using distutil,
acknowledging that it might not work in
77 matches
Mail list logo