Chris Rebert added the comment:
Revised patch yet again to instead speak of non-object, non-array values and
JSON null, boolean, number, or string values.
Re: Petri, the patch already mentions the specific parameters one can use to
get stricter behavior, albeit not in that particular short
Changes by Chris Rebert pyb...@rebertia.com:
--
nosy: +cvrebert
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13212
___
___
Python-bugs-list
Roundup Robot added the comment:
New changeset e587426d719f by Ned Deily in branch 'default':
Issue #15037: Use correct path to system terminfo database.
http://hg.python.org/cpython/rev/e587426d719f
--
___
Python tracker rep...@bugs.python.org
Eric Snow added the comment:
So this boils down to set_data() not having a mode parameter, right? Alas,
the obvious approach, adding it, breaks backward compatibility. The
alternative is to use source_from_cache() in set_data() to get the source path,
and then get the mode there. Of
Changes by Eric Snow ericsnowcurren...@gmail.com:
--
stage: - needs patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2051
___
___
Nick Coghlan added the comment:
I have a patch that works, and that I think may point the way to a replacement
API for set_data in 3.4. Just running the full test suite before I check it in.
The test you posted definitely saved me a lot of time.
My version bypasses the cache, though. This
New submission from Nemeskey Dávid:
The C expat library provides XML_StopParser() method that allows the parsing to
be stopped from the handler functions. It would be nice to have this option in
Python as well, maybe by adding StopParser() method to the XMLParser class.
--
components:
Nick Coghlan added the comment:
Also, my patch precisely recreates 3.2 behaviour, so it will mean #6074 may
also affect 3.3. That isn't a regression, hence not a release blocker, but
would still be good to get fixed for rc1.
--
___
Python tracker
Nick Coghlan added the comment:
Adding 3.3 to affected versions, since I'm about to check in a fix for #2051
that will reintroduce this problem.
--
nosy: +eric.snow
versions: +Python 3.3
___
Python tracker rep...@bugs.python.org
Changes by Chris Rebert pyb...@rebertia.com:
--
nosy: +cvrebert
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15136
___
___
Python-bugs-list
Changes by Chris Rebert pyb...@rebertia.com:
--
nosy: +cvrebert
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14965
___
___
Python-bugs-list
Roundup Robot added the comment:
New changeset 3a831a0a29c4 by Nick Coghlan in branch 'default':
Close #2501: Permission bits are once again correctly copied from the source
file to the cached bytecode file. Test by Eric Snow.
http://hg.python.org/cpython/rev/3a831a0a29c4
--
nosy:
Changes by Nick Coghlan ncogh...@gmail.com:
--
Removed message: http://bugs.python.org/msg168983
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2501
___
Aaron Galea added the comment:
Its a distutils issue not py2exe. Not sure where I should post about this
problem.
The change I applied was in msvc9compiler.py in distutils. Once this change has
been applied py2exe was rebuild to generate the correct runw.exe with the
correct settings in the
Nick Coghlan added the comment:
(oops, transposed the digits in the checkin message)
New changeset 3a831a0a29c4 by Nick Coghlan in branch 'default':
Close #2051: Permission bits are once again correctly copied from the source
file to the cached bytecode file. Test by Eric Snow.
Roundup Robot added the comment:
New changeset cfb11045fc8a by Nick Coghlan in branch 'default':
Close #2051: Oops, transposed the digits in the issue number in the previous
commit
http://hg.python.org/cpython/rev/cfb11045fc8a
--
nosy: +python-dev
resolution: - fixed
stage: needs
Mark Dickinson added the comment:
I think *both* proposals are sensible. Fraction already has .from_decimal
(using Decimal), so .to_decimal (also using Decimal) is sensible.
Well, there's a difference: conversion from Decimal to Fraction is
well-defined, with a unique, unambiguous result
Анхбаяр Лхагвадорж added the comment:
Ding.
--
nosy: +Анхбаяр.Лхагвадорж
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10076
___
___
James added the comment:
It turns out I don't really understand how frame objects work. My patch can
crash python if you do this:
class A:
... def f(*args):
... args = 1
... print(super())
...
A().f()
python: Objects/typeobject.c:6516: super_init: Assertion
New submission from Stefan Holek:
With virtualenv I can do
$ virtualenv .
but with pyvenv I get
$pyvenv .
Error: Directory exists: /Users/stefan/sandbox/foo
Please allow pyvenv to apply to existing directories.
--
components: None
messages: 168990
nosy: stefanholek
Changes by Ned Deily n...@acm.org:
--
nosy: +vinay.sajip
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15776
___
___
Python-bugs-list mailing list
Charles-François Natali added the comment:
I wouldn't rely on O_APPEND too much:
- it won't work on NFS, and probably other non-local filesystems
- it doesn't actually guarantee atomicity, because even though the the file
offset and the write is done with locking, there is still the possibility
Changes by Ralf Schmitt python-b...@systemexit.de:
--
nosy: +schmir
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15723
___
___
Python-bugs-list
Mike Hoy added the comment:
Lambda Changes patch.
--
Added file:
http://bugs.python.org/file26982/issue11776-first-easy-part-lambda-.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11776
New submission from Ross Lagerwall:
results for fa745ed89b7a on branch default
test_capi leaked [2, 2, 2] references, sum=6
Command line was: ['./python', '-m', 'test.regrtest', '-uall', '-R',
'3:3:/home/antoine/cpython/refleaks/reflogEr4Oyp',
Charles-François Natali added the comment:
This doesn't look thread-related:
load: 0.50 cmd: python 10630 [runnable] 0.27u 13.07s 46% 10976k
load: 0.87 cmd: python 10630 [runnable] 0.27u 94.24s 98% 10976k
See the huge system time?
This probably means that the kernel is struggling with path
Amaury Forgeot d'Arc added the comment:
The failing assertion is the assert(PyTuple_Check(obj)) added by your patch.
At this point, obj is not the arguments tuple, but the first entry in
f-f_localsplus. Maybe this block should be moved a bit earlier?
--
nosy: +amaury.forgeotdarc
Charles-François Natali added the comment:
synchronized has the merit of reusing Java's denomination, which this
decorator intends to mimic, see
http://docs.oracle.com/javase/tutorial/essential/concurrency/syncmeth.html
Basically, synchronize works with the object/class (implicit) reentrant
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
nosy: +ezio.melotti
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15753
___
___
James added the comment:
Sorry, I wasn't very clear. super() currently works by assuming that self is
the first entry in f_localsplus, which is defeated, for example, by doing:
class A:
... def f(self):
... del self
... super()
...
A().f()
Traceback (most recent
Andrew Svetlov added the comment:
Does it mean implicit implying --upgrade option if venv dir is '.'?
--
nosy: +asvetlov
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15776
___
Roundup Robot added the comment:
New changeset 081507b4ae40 by Ross Lagerwall in branch '3.2':
Issue 15777: Fix a refleak in _posixsubprocess.
http://hg.python.org/cpython/rev/081507b4ae40
--
nosy: +python-dev
___
Python tracker
Ronald Oussoren added the comment:
plutil(1) supports writing json format.
That written, the opensource parts of CoreFoundation on opensource.apple.com
don't support reading or writing json files.
I'm therefore -1 w.r.t. adding support for json formatted plist files, support
for json can
moijes12 added the comment:
r.david.murray: I'd love to work on this and that is why I'm following this and
634412. But I'm just getting started with contributing to Python and I've
contributed to only to 15442 and thats not even accepted yet.
--
Andrew Svetlov added the comment:
Changing exception type is not backward compatible and should be documented at
least if that change is really required.
Personally I slightly prefer to leave SystemError untouched
--
nosy: +asvetlov
___
Python
Changes by Andrew Svetlov andrew.svet...@gmail.com:
--
nosy: +asvetlov
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15751
___
___
Michal Božoň added the comment:
this is still unfixed in Python 2.x
--
nosy: +mykhal
versions: +Python 2.6, Python 2.7
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10606
___
Vinay Sajip added the comment:
Running
pyvenv --clear .
should work, but doesn't because of the way the venv directory is initialised -
a shutil.rmtree() call is used. This can cause problems on Windows (current
directory is regarded as open and so cannot be deleted) and also on Posix,
Ronald Oussoren added the comment:
What code duplication do you mean?
The code in the switch statement at line 71 looks similar for the various case,
but it isn't: there is variation in the types uses in the casts (and
particularly in the cast used to read the input value).
--
Changes by Ronald Oussoren ronaldousso...@mac.com:
Removed file: http://bugs.python.org/file26972/smime.p7s
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13370
___
Ronald Oussoren added the comment:
Attached version of the patch without the change to the powerpc specific code.
I will commit this version later today (after another testing round).
--
Added file: http://bugs.python.org/file26984/issue_13370-2.txt
Trent Nelson added the comment:
Link to my freebsd-fs e-mail:
http://lists.freebsd.org/pipermail/freebsd-fs/2012-August/014964.html
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15748
Nick Coghlan added the comment:
I'm not sure it makes sense to call this new API PyGILState_EnsureEx. My
concern is that the behaviour is quite different in the presence of an existing
thread state:
Ensure:
- if a thread state exists, use that interpreter
- otherwise, use the default
Andrew Svetlov added the comment:
I don't like current --clear behavior, it's really useless because now venv
just deletes everything from virtual environment.
You lose not only virtual env but files from your project also.
virtualenv cleans only env/Lib directory, that's much better.
I think
Antoine Pitrou added the comment:
New API:
- if a thread state exists, and the interpreter doesn't match the
requested one, fail with an error
- otherwise, use the requested interpreter
That's not what I'm proposing. What I'm proposing is that the new API
uses a per-interpreter TLS key (so
Vinay Sajip added the comment:
Venvs should be regarded as throwaway; there is really no reason to add other
files (e.g. project files) to venvs.
Two common patterns are:
1. Use a single place for all venvs (virtualenvwrapper does this)
2. Use a venv in a subdirectory of a project directory
Trent Nelson added the comment:
Hacked the test to print out info each loop:
test_getcwd_long_pathnames (test.test_posix.PosixTester) ... [0/0]
getcwd-test-directory-0123456789abcdef-01234567890abcdef
[1/57] getcwd-test-directory-0123456789abcdef-01234567890abcdef
[2/114]
Nick Coghlan added the comment:
And the current autoTLSkey could move into the interpreter state object? I
like it - that's a lot more flexible than the approach I was thinking of.
--
title: Add PyGILState_SwitchInterpreter - Support subinterpreters in the GIL
state API
Graham Dumpleton added the comment:
It is past my bed time and not thinking straight, but I believe Antoine is
aligned with what I had in mind, as need multiple thread states per OS thread
where each is associated with separate interpreter.
My main reason for allowing NULL to EnsureEX rather
Andrew Svetlov added the comment:
Well, I see. Agree with your patch then, it is correct.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15776
___
Roundup Robot added the comment:
New changeset cbd4fbb90e95 by Martin v. Löwis in branch 'default':
Issue #15511: Drop explicit dependency on pythonxy.lib from amd64 configuration.
http://hg.python.org/cpython/rev/cbd4fbb90e95
--
nosy: +python-dev
Martin v. Löwis added the comment:
I now recall that I worked around this issue, by just copying python33.lib in
the place where it's expected.
The right solution is not to configure a dependency on the .lib, as it will
automatically be added by the pragma in the include file. The search path
Vinay Sajip added the comment:
I don't feel it would be correct to allow long for log levels, since a log
level is not intended to have that range. It would be more sensible to get the
ESAPI developers to use a more appropriate boundary value; in general, negative
log values are not used
Antoine Pitrou added the comment:
This seems to miss the point that it's a regression. int and long are supposed
to be interchangeable for most purposes.
--
nosy: +pitrou
status: pending - open
___
Python tracker rep...@bugs.python.org
Richard Oudkerk added the comment:
After playing with the patch on Linux it seems that Linux much prefers the
realloc() scheme to the list-of-chunks scheme.
This new patch only does list-of-chunks on Windows.
--
Added file: http://bugs.python.org/file26985/readall-combined.patch
Richard Oudkerk added the comment:
Attached is an alternative benchmark program which does not use subprocess.
--
Added file: http://bugs.python.org/file26986/readall-benchmark.py
___
Python tracker rep...@bugs.python.org
Changes by Richard Oudkerk shibt...@gmail.com:
Removed file: http://bugs.python.org/file26952/push-thru-cat.py
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15758
___
Changes by Richard Oudkerk shibt...@gmail.com:
Removed file: http://bugs.python.org/file26970/readall-combined.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15758
___
Antoine Pitrou added the comment:
After playing with the patch on Linux it seems that Linux much prefers
the realloc() scheme to the list-of-chunks scheme.
What about the method call overhead in RawIO.readall(), and the
different progression of buffer sizes? (the realloc scheme uses larger
Martin v. Löwis added the comment:
Is this still blocking the release? If so, it should be resolved within the
next twelve hours, or else it may block the release until September.
--
___
Python tracker rep...@bugs.python.org
R. David Murray added the comment:
moijes12, thanks for the patch. Since this is a new feature we'll need to wait
until the 3.3 RC1 repo is branched before applying it.
--
nosy: +r.david.murray
___
Python tracker rep...@bugs.python.org
Martin v. Löwis added the comment:
Why is this blocking the release? It looks like a problem that can be solved in
a bug fix release, or else by requiring that Python 3.3 users use a recent
ncurses release.
--
nosy: +loewis
priority: release blocker - normal
Martin v. Löwis added the comment:
Stefan, your patch array_deprecate_u.diff is fine. If you get to it, please
also rephrase the clause Python's unicode type; not sure what the convention
is to refer to Py_UNICODE now (perhaps historical unicode type).
--
Trent Nelson added the comment:
Looks like this is a duplicate of issue 9185. The fix used for that just needs
to be extended to cover NetBSD as well. See proposed patch. I'll commit this
to 2.7 in a day or two if there are no objections.
Antoine: added you as nosy as you were involved
Roundup Robot added the comment:
New changeset 119c645f310e by R David Murray in branch '3.2':
#15249: Mangle From lines correctly when body contains invalid bytes.
http://hg.python.org/cpython/rev/119c645f310e
New changeset b6ee4e8c7a77 by R David Murray in branch 'default':
Merge #15249:
R. David Murray added the comment:
Thanks, LittleQ. And Petri.
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15249
Georg Brandl added the comment:
LGTM, please apply.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15776
___
___
Python-bugs-list mailing list
Georg Brandl added the comment:
I don't agree that this is a blocker; would be nice to fix it, of course.
--
priority: release blocker - critical
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15316
Antoine Pitrou added the comment:
os.path.isdir(foo) will return True if foo is a symlink to a directory, and
then shutil.rmtree(foo) will fail:
os.path.isdir(foo)
True
os.path.islink(foo)
True
shutil.rmtree(foo)
Traceback (most recent call last):
File stdin, line 1, in module
File
Roundup Robot added the comment:
New changeset 0668fc196ce5 by Andrew Svetlov in branch 'default':
Issue #15776: Allow pyvenv to work in existing directory with --clean.
http://hg.python.org/cpython/rev/0668fc196ce5
--
nosy: +python-dev
___
Python
Andrew Svetlov added the comment:
It is bug in shutil.rmtree, right?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15776
___
___
Stefan Krah added the comment:
I agree with Mark's arguments. Yesterday I tried to use as_decimal() in
a small program and it did not feel natural to me. I'll probably continue
to use Decimal(f.numerator) / f.denominator.
If this goes in, I'd prefer that as_decimal() always uses a
Brett Cannon added the comment:
It will get fixed today.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15316
___
___
Python-bugs-list mailing
Antoine Pitrou added the comment:
Senthil, either the module globals should be re-added for compatibility, or the
commits should be reverted, IMO.
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9374
Antoine Pitrou added the comment:
It is bug in shutil.rmtree, right?
Read the documentation:
shutil.rmtree(path, ignore_errors=False, onerror=None)
Delete an entire directory tree; path must point to a directory (but
not a symbolic link to a directory)
--
Roundup Robot added the comment:
New changeset a0b3cb52816e by Georg Brandl in branch '3.2':
Closes #9374: add back now-unused module attributes; removing them is a
backward compatibility issue, since they have a public-seeming name.
http://hg.python.org/cpython/rev/a0b3cb52816e
New changeset
Changes by Georg Brandl ge...@python.org:
--
priority: normal - release blocker
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15776
___
___
Georg Brandl added the comment:
Another *perfect* example how even the most innocuous-seeming patch can be
wrong.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15776
___
Andrew Svetlov added the comment:
Good point.
I will prepare the patch to fix this.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15776
___
___
New submission from Dave Malcolm:
In Python 3.2 and earlier:
str(ImportError(b'foo'))
b'foo'
In Python 3.3:
str(ImportError(b'foo'))
Traceback (most recent call last):
File stdin, line 1, in module
TypeError: __str__ returned non-string (type bytes)
This appears to be due to commit 76272.
Antoine Pitrou added the comment:
This appears to be due to commit 76272.
Could you give the changeset id? Revision numbers are not portable from one
repository clone to another.
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
Richard Oudkerk added the comment:
What about the method call overhead in RawIO.readall(), and the
different progression of buffer sizes? (the realloc scheme uses larger
and larger read() sizes, while RawIO.readall() uses a constant read()
size).
For this benchmark the call overhead does
Dave Malcolm added the comment:
Sorry, it's 6825fd9b00ed
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15778
___
___
Python-bugs-list mailing
Andrew Svetlov added the comment:
I like to leave fixes to 3.4.
Any change can produce side-effects, which can be nightmare for upcoming
release candidate.
Sure, Georg will share my opinion.
Though absence '\n' - '\r\n' for input if OS is Windows and
universal_newlines=True is not good.
Changes by Dave Malcolm dmalc...@redhat.com:
--
keywords: +patch
Added file: http://bugs.python.org/file26988/fix-str-of-bogus-ImportError.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15778
Antoine Pitrou added the comment:
For this benchmark the call overhead does not seem to be noticeable,
and using larger or adaptive read buffers does not seem to help
either. (I have tried both on Linux.)
Ok, thank you.
By the way, not every non-Windows OS is Linux, so the patch is
Dave Malcolm added the comment:
(patch added)
--
stage: - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15778
___
___
Changes by Brett Cannon br...@python.org:
--
nosy: +brian.curtin
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15778
___
___
Python-bugs-list
Éric Araujo added the comment:
Two common patterns are:
1. Use a single place for all venvs (virtualenvwrapper does this)
2. Use a venv in a subdirectory of a project directory
I’m a recent virtualenv user, but before I became a virtualenvwrapper fan I
used to create venvs in the project
Changes by Brett Cannon br...@python.org:
--
assignee: - brett.cannon
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15778
___
___
Eric Snow added the comment:
My patch was very similar. _cache_bytecode() is a good addition. Good point
about the cache, too.
I'm not convinced that source_path is the right thing to add to the API (even
just for SourceFileLoader). I would have thought mode would have been more
Roundup Robot added the comment:
New changeset 91909962d7f5 by Brett Cannon in branch 'default':
Issue #15778: Coerce ImportError.args to a string when it isn't
http://hg.python.org/cpython/rev/91909962d7f5
--
nosy: +python-dev
___
Python tracker
Changes by Brett Cannon br...@python.org:
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15778
___
Senthil Kumaran added the comment:
Oops. I had not seen Eric and Mattiahs comment to this issue, which
pointed out to the problem. Sorry for not acting on this.
Thanks Georg for adding those module attributes back.
On Fri, Aug 24, 2012 at 9:17 AM, Roundup Robot rep...@bugs.python.org wrote:
Antoine Pitrou added the comment:
Chris, I'm gonna take a look if nobody beats me to it.
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14674
___
Brett Cannon added the comment:
I agree with Eric: set_data() is meant to be generic so as to allow writing
arbitrary data, not just bytecode files. So while accepting a mode argument
makes sense and I'm fine with in terms of possible API change (in the future),
having a source_path argument
New submission from James Hutchison:
Windows 7 64-bit, Python 3.2.3
This is a very odd issue and I haven't figured out what caused it. I have a
python script that runs continuously. When it receives a request to do a task,
it creates a new thread (not a new process), does the task, then sends
Roundup Robot added the comment:
New changeset 0c661c5632e0 by Brett Cannon in branch 'default':
Issue #2051: Tweak last commit for this issue to pass in mode instead
http://hg.python.org/cpython/rev/0c661c5632e0
--
___
Python tracker
Stefan Holek added the comment:
Hm. What I am actually after is to bless an existing directory – source files
and all – with a virtualenv (or pyvenv). I am not interested in the command
deleting anything from anywhere, why thank you.
Workflow:
$ git clone g...@github.com:stefanholek/foo
Roundup Robot added the comment:
New changeset 132886ef135d by Antoine Pitrou in branch '3.2':
Issue #14674: Add a discussion of the json module's standard compliance.
http://hg.python.org/cpython/rev/132886ef135d
New changeset 16c0e26fc9cd by Antoine Pitrou in branch 'default':
Issue #14674:
1 - 100 of 154 matches
Mail list logo
