Vinay Sajip added the comment:
In this case couldn't symlinks be automatically used on Windows Vista or
newer?
It seems simpler if the default behaviour is the same on all Windows flavours -
you can specify --symlinks if you're on Windows Vista or later.
--
Larry Hastings added the comment:
Okay, I got inspired and (in the words of Barry Warsaw) JFDI. Attached is my
revised patch. I took Serhiy's patch and reworked it quite a bit:
* I think it's now easier to follow. In particular:
* The most common case (no overflow) is now first. In
Changes by Nick Coghlan ncogh...@gmail.com:
--
assignee: - ncoghlan
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17731
___
___
Python-bugs-list
Mark Dickinson added the comment:
To answer Serhiy's question: I'd say that this level of cleanup is probably
only appropriate for 3.4. Larry?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15301
Roundup Robot added the comment:
New changeset 193e7ad92900 by Vinay Sajip in branch 'default':
Issue #17713: Added failure diagnostics to test.
http://hg.python.org/cpython/rev/193e7ad92900
--
___
Python tracker rep...@bugs.python.org
Vinay Sajip added the comment:
This looks to me as if it will need a patch in distutils. Unlike virtualenv,
which contains a patched copy of distutils (and hence allows having a .cfg
adjacent to it), pyvenv does not create patched modules in the venv. It does
not make sense to change this
Larry Hastings added the comment:
See my comment above (dated 2013-04-14 04:30). I'm passing the buck.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15301
___
Marc-Andre Lemburg added the comment:
Checked the patch: it fixes the problem. Thanks.
Will this go into Python 2.7.5 ?
I'm asking because we need to issue a patch level release of egenix-mx-base and
if Python 2.7.5 will fix the problem, we'll just add the work-around for Python
2.7.4.
Giampaolo Rodola' added the comment:
Maybe I'm misinterpreting what you wrote but the test fails before the patch
and succeeds after it so what's the point in adding multiple tests with
different timeouts?
Also, rathr than using an harcoded delta, we could maybe use a fudger
factor, like
Nick Coghlan added the comment:
spam is a fairly generic name, so I'm guessing something else is leaving a
spam module around in sys.modules - when I run the tests with the order given
in RDM's original report, I get the same error.
I also get a failure in test_builtin though, which is a
Changes by Christian Heimes li...@cheimes.de:
--
nosy: +christian.heimes
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17708
___
___
Nick Sloan added the comment:
That's along the lines of what I've been thinking as I dig into this. I'd love
to take a stab at a patch for this if no one else has done so already.
--
___
Python tracker rep...@bugs.python.org
New submission from Nick Coghlan:
I'm getting a failure in test_builtin when running the following:
./python -m test -w test_genexps test_builtin
==
FAIL: test_input_tty_non_ascii (test.test_builtin.BuiltinTest)
Nick Coghlan added the comment:
Created #17734 for the weird interference between test_genexps and test_builtin
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17731
___
Changes by Pascal Chambon chambon.pas...@gmail.com:
--
nosy: +Pascal.Chambon
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17636
___
___
Ronald Oussoren added the comment:
I agree that plistlib shouldn't raise an exception for data that can
represented as a valid plist file.
I've checked that the Cocoa class for generating plist files will happily
create a plist file when the data is nested 100 levels deep. In that case
Ronald Oussoren added the comment:
The attach patch should fix the issue (but there needs to be a unittest as
well).
--
Added file: http://bugs.python.org/file29863/issue-17353.txt
___
Python tracker rep...@bugs.python.org
New submission from Kyle Simpson:
Here is one way to reproduce this bug:
1. Create a module file (bug.py in this example)
def func():
pass
2. Run Python
import bug
help(bug)
3. Edit bug.py
def func():
pass
def newfunc():
pass
4. Use the same Python interpreter as in step 2
Roundup Robot added the comment:
New changeset 73c79022977b by Nick Coghlan in branch '3.3':
Close #17731: Clean up properly in test_import
http://hg.python.org/cpython/rev/73c79022977b
New changeset 5d4001e32a31 by Nick Coghlan in branch 'default':
Merge fix for #17731 from 3.3
New submission from Jonas Wagner:
The attached patch corrects a wrong method comment in _elementtree.c. It
happened to be at Line 316, and was thus discovered by random sampling. [1]
[1] http://www-cs-faculty.stanford.edu/~uno/316.html
--
components: Extension Modules
files:
Pascal Chambon added the comment:
(sorry for the long post, but it's a complex issue I guess)
I forgot to precise that I have this behaviour with the latest python2.7, as
well as python3.3 (I guess other versions behave the same).
I agree that having side effects in script imports looks
New submission from Bohuslav Slavek Kabrda:
Hi,
it seems that test_gdb fails on armv7hl on Fedora 19 and 20 [1] (I'm also
tracking my notes of the bug there). Basically, the problem seems to come down
to PyObjectPtr.subclass_from_type (file python-gdb.py) returning different
values for
Changes by Benjamin Peterson benja...@python.org:
--
nosy: +dmalcolm
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17708
___
___
Python-bugs-list
Pam McA'Nulty added the comment:
Here's a patch. I needed to handle the fact that the repr of a single byte can
be 1, 2 or 4 characters long and did not want to wrap in the middle of a byte
representation. Note also that bytes literals require a continuation
character. In the pathological
Pam McA'Nulty added the comment:
oops, forgot to add some samples:
pprint.pprint(b\n\n\n\n\n\n, width=5)
b'\n'\
b'\n'\
b'\n'\
b'\n'\
b'\n'\
b'\n'
pprint.pprint({a: b\x00\xff * 20})
{'a': b'\x00\xff\x00\xff\x00\xff\x00\xff\x00\xff\x00\xff\x00\xff\x00\xff\x00'\
Roundup Robot added the comment:
New changeset 725d6347ac7e by Eric V. Smith in branch '2.7':
Issue #17728: Specify default precision for float.format for presentation types
e, f, and g.
http://hg.python.org/cpython/rev/725d6347ac7e
New changeset ad481c95a1d4 by Eric V. Smith in branch '3.3':
Changes by Eric V. Smith e...@trueblade.com:
--
resolution: - fixed
stage: - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17728
___
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +dmalcolm
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17737
___
___
Python-bugs-list mailing
R. David Murray added the comment:
Can you explain what makes this one a different problem? It looks like the
same one to me. Or is your intent in this issue just to avoid the exception?
In that case it seems to me it would better to fix issue 1218234 if we can.
--
nosy:
New submission from Jonas Wagner:
I'm puzzled by the following code in SHA1_copy (at
python/Modules/sha1module.c:320
if (Py_TYPE(self) == SHA1type) {
if ( (newobj = newSHA1object())==NULL)
return NULL;
} else {
if ( (newobj = newSHA1object())==NULL)
Antoine Pitrou added the comment:
I don't understand why you say that bytes literals require a continuation
character:
(bx
... by)
b'xy'
[bx
... by]
[b'xy']
I think the len caching is a misoptimization, it's useless here (most CPU
time will be sent creating and wrapping the
sorin added the comment:
Can we have this merged, it prevents us form using distutil, especially in a
continuous integration environment where you do not have control over the build
server.
See: https://drone.io/github.com/pycontribs/tendo/1
--
nosy: +sorin
Pam McA'Nulty added the comment:
- eval expects bytes to have a continuation character and test_str_wrap did an
eval check so I figured test_bytes_wrap should as well:
# repr some bytes:
b = b\x00\xff * 5
b
b'\x00\xff\x00\xff\x00\xff\x00\xff\x00\xff'
r = repr(b)
r
Éric Araujo added the comment:
I’ll get this in the next bugfix releases.
--
keywords: -needs review
nosy: +benjamin.peterson, georg.brandl, larry
priority: normal - release blocker
versions: +Python 3.4 -Python 3.2
___
Python tracker
Charles-François Natali added the comment:
Maybe I'm misinterpreting what you wrote but the test fails before the patch
and succeeds after it so what's the point in adding multiple tests with
different timeouts?
Well, the test you added tests explicitely for a value 1s because
this
Antoine Pitrou added the comment:
Well, but eval works if you put parentheses as required by the grammar:
s = (b'xy'\nb'za')
eval(s)
b'xyza'
Yes, _str_parts and _bytes_parts should probably remain separate. It's the
higher-level routine that would deserve sharing.
Also, perhaps the other
New submission from David D Lowe:
The documentation for ssl.SSLSocket.getpeercert states:
If the binary_form parameter is True, and a certificate was provided, this
method returns the DER-encoded form of the entire certificate as a sequence
of bytes, or None if the peer did not provide a
Kyle Roberts added the comment:
I think `copy_from` should be included for mkstemp as well. It provides similar
functionality to TemporaryFile and NamedTemporaryFile, but it doesn't delete
the temp file on close as the other two do by default. Thoughts?
--
Ezio Melotti added the comment:
Isn't this the same as #13886?
--
nosy: +ezio.melotti
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17734
___
___
Dan Riti added the comment:
Agreed Ezio, I've updated the patch to include the change to
Doc/library/io.rst:readlines.
--
Added file: http://bugs.python.org/file29868/demote-readlines-v3.patch
___
Python tracker rep...@bugs.python.org
Zachary Ware added the comment:
Here's another new version of the patch, addressing Ezio's review comments and
a few things I found after giving operator.py a closer look myself.
Things changed in operator.py in this version:
- all ``__func__ = func`` assignments are moved to the end, after
Changes by Zachary Ware zachary.w...@gmail.com:
--
nosy: -Arfrever, brett.cannon, eric.araujo, ezio.melotti, jcea, meador.inge,
pitrou, serhiy.storchaka, zach.ware
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16694
Changes by Zachary Ware zachary.w...@gmail.com:
Removed file: http://bugs.python.org/file28327/operator.py
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16694
___
Changes by Zachary Ware zachary.w...@gmail.com:
Removed file: http://bugs.python.org/file28328/py_operator.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16694
___
Changes by Zachary Ware zachary.w...@gmail.com:
Removed file: http://bugs.python.org/file28374/py_operator.v3.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16694
___
Changes by Zachary Ware zachary.w...@gmail.com:
Removed file: http://bugs.python.org/file28388/py_operator.v5.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16694
___
Changes by Zachary Ware zachary.w...@gmail.com:
Removed file: http://bugs.python.org/file28383/py_operator.v4.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16694
___
Changes by Zachary Ware zachary.w...@gmail.com:
Removed file: http://bugs.python.org/file28522/py_operator.v7.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16694
___
Changes by Zachary Ware zachary.w...@gmail.com:
Removed file: http://bugs.python.org/file28524/py_operator.v8.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16694
___
Changes by Zachary Ware zachary.w...@gmail.com:
Removed file: http://bugs.python.org/file28532/py_operator.v9.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16694
___
Changes by Zachary Ware zachary.w...@gmail.com:
--
nosy: +Arfrever, brett.cannon, eric.araujo, ezio.melotti, jcea, meador.inge,
pitrou, serhiy.storchaka, zach.ware
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16694
Nick Sloan added the comment:
Here is a patch that seems to fix the problem. It simply short-circuits
distutils options that change directories. This is my first python patch ever,
so I'm eager for comments. Is this the right approach?
--
keywords: +patch
Added file:
Roundup Robot added the comment:
New changeset 1e8be05a4039 by Ezio Melotti in branch '3.3':
#13510: clarify that f.readlines() is note necessary to iterate over a file.
Patch by Dan Riti.
http://hg.python.org/cpython/rev/1e8be05a4039
New changeset 7f4325dc4256 by Ezio Melotti in branch
Ezio Melotti added the comment:
Fixed, thanks for the patch!
I also realized I missed Terry suggestion about file.readlines() == list(file),
so I added that too.
--
assignee: docs@python - ezio.melotti
resolution: - fixed
stage: needs patch - committed/rejected
status: open - closed
Zachary Ware added the comment:
A change that I mentioned in a Rietveld comment on v10, but not in my last
message: __all__ in operator.py no longer includes all of the __func__s, as
currently doing from operator import * does not import all of the __func__s.
--
Zachary Ware added the comment:
That is indeed simpler than what I wrote, and it does work as expected. But,
is it preferable to do it this way, or with Ezio's suggested method
(``skip_unless_gzip = unittest.skipUnless(gzip, gzip not available)``, and
for bz2 and lzma)? I can see merits to
New submission from Zachary Ware:
In Doc/library/socket.rst, :func:`socket` links to #module-socket, not
#socket.socket. The attached patch changes all occurances of :func:`socket` to
:func:`~socket.socket`, except the first one which keeps the explicit module
name (no ~).
--
Changes by Zachary Ware zachary.w...@gmail.com:
Added file: http://bugs.python.org/file29872/socket_func_link_2.7.diff
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17740
___
Nick Sloan added the comment:
On second thought, there is probably no good reason to ignore the build-*
settings. Here is an updated patch.
--
Added file: http://bugs.python.org/file29873/distutilsvenv.patch
___
Python tracker rep...@bugs.python.org
Ezio Melotti added the comment:
I think :func:`.socket` should work too.
--
assignee: docs@python - ezio.melotti
nosy: +ezio.melotti
stage: - patch review
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17740
Vinay Sajip added the comment:
On second thought, there is probably no good reason to ignore the build-*
settings.
I was just about to mention this. I'm not an expert on distutils internals and
whether this is the best way to accomplish what's needed, but the approach
seems reasonable to
Zachary Ware added the comment:
So it does. Would you like a new pair of patches? It is just a
search-and-replace from :func:`socket` to :func:`.socket`.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17740
Orion Poplawski added the comment:
Despite numpy not calling the library properly, it stills seems to me that
python should not segfault due to not handling a null pointer. But thanks for
the help.
--
___
Python tracker rep...@bugs.python.org
Changes by Antoine Pitrou pit...@free.fr:
--
title: Trash can mechanism segfault during interpreter finalization in Python
2.7.4 - Trashcan mechanism segfault during interpreter finalization in Python
2.7.4
___
Python tracker rep...@bugs.python.org
Changes by Antoine Pitrou pit...@free.fr:
--
stage: - commit review
type: - crash
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17703
___
___
Antoine Pitrou added the comment:
I was targeting head, not the release branches.
Perhaps, but I don't see the point of choosing a different fix in the default
branch than in the bugfix branches.
--
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset e814fbd470bf by Antoine Pitrou in branch '2.7':
Issue #17703: Fix a regression where an illegal use of Py_DECREF() after
interpreter finalization can cause a crash.
http://hg.python.org/cpython/rev/e814fbd470bf
--
nosy: +python-dev
Antoine Pitrou added the comment:
Committed!
--
resolution: - fixed
stage: commit review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17703
___
Nick Sloan added the comment:
My thought was that perhaps there will be other circumstances where we may want
to ignore options in the future. The idea was that by providing an
ignore_options list that can be extended, multiple conditions with different
sets of options can be stacked together
Serhiy Storchaka added the comment:
* The most common case (no overflow) is now first. In Serhiy's patch
the most common case is buried in the middle of the second if.
It's because my implementation is a simplified version of more complicated
patch for issue2005, which supports signed uid_t
Marc-Andre Lemburg added the comment:
On 15.04.2013 21:21, Antoine Pitrou wrote:
Committed!
Cool, thanks for the quick turnaround.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17703
Éric Araujo added the comment:
I would ignore options from all config files, and do the simplest thing (i.e.
not add the ignore_options attribute).
--
assignee: vinay.sajip - eric.araujo
components: +Distutils -Library (Lib)
___
Python tracker
Roundup Robot added the comment:
New changeset 527b7f88b53c by Antoine Pitrou in branch '2.7':
Issue #17710: Fix cPickle raising a SystemError on bogus input.
http://hg.python.org/cpython/rev/527b7f88b53c
--
nosy: +python-dev
___
Python tracker
Serhiy Storchaka added the comment:
I think Antoine is more appropriate for committing this patch. I waited so long
with this because I do not dare to take responsibility for themselves (it's
almost like adding a new module).
--
assignee: serhiy.storchaka -
Georg Brandl added the comment:
Thanks!
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17728
___
___
Python-bugs-list mailing list
Unsubscribe:
Changes by Serhiy Storchaka storch...@gmail.com:
--
assignee: serhiy.storchaka - alexandre.vassalotti
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
Brett Cannon added the comment:
Have to think about the whole rollback situation in terms of a failure to
import a parent. Whenever you want to change the semantics of import you will
break someone's code, it's just a question of how wide the breakage would be
and how much of an improvement
Roundup Robot added the comment:
New changeset 4e412cbaaf96 by Antoine Pitrou in branch '3.3':
Issue #17710: Fix pickle raising a SystemError on bogus input.
http://hg.python.org/cpython/rev/4e412cbaaf96
New changeset 5a16d2992112 by Antoine Pitrou in branch 'default':
Issue #17710: Fix pickle
Antoine Pitrou added the comment:
I've committed the patches. Feel free to improve the default branch if you like.
--
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
Antoine Pitrou added the comment:
About the test:
+self.assertRaises(Exception, pair.read)
+self.assertRaises(Exception, pair.write)
First, you should check for the actual RuntimeError.
Second, you need to pass the right arguments for these method calls: for
example read(1)
New submission from Antoine Pitrou:
iterparse() is a blocking operation. It is not really suitable for event-driven
applications (e.g. non-blocking I/O). Here is a patch adding a
IncrementalParser class.
--
components: Library (Lib)
files: etree_incparser.patch
keywords: patch
Antoine Pitrou added the comment:
Note that basing iterparse() on IncrementalParser and removing the API
discrepancy between the Python and C modules helps make the etree code smaller.
--
___
Python tracker rep...@bugs.python.org
Ned Deily added the comment:
Thanks for the suggested. Here's a revised wording and a slightly more legible
example:
Return a copy of the string where all tab characters are replaced by zero or
more spaces, depending on the current tab column and the given tab size.
Starting at the
Antoine Pitrou added the comment:
Thanks for reporting this. This is a documentation issue. As stated in the
OpenSSL docs:
Due to the protocol definition, a TLS/SSL server will always send a
certificate, if present. A client will only send a certificate when explicitly
requested to do so by
Zachary Ware added the comment:
I caught a small oversight. This new patch changes the example in README.txt
to ``make html PYTHON2=C:\Python27\python.exe`` (PYTHON-PYTHON2). Also, I
added a bit of backward compatibility to make.bat; %PYTHON2% will default to
%PYTHON% if it is set.
Changes by Antoine Pitrou pit...@free.fr:
--
nosy: +gregory.p.smith
stage: - patch review
versions: +Python 2.7 -Python 3.1, Python 3.2, Python 3.5
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17738
Piotr Dobrogost added the comment:
It's simpler but is it better this way? I doubt. I think we should take
advantage of symlinks whenever we can and only fallback to copying if they are
not available.
--
___
Python tracker rep...@bugs.python.org
New submission from STINNER Victor:
In Python 3.3, I added _PyUnicodeWriter API to factorize code handling a
Unicode buffer, just the code to allocate memory and resize the buffer if
needed.
I propose to do the same with a new _PyBytesWriter API. The API is very similar
to _PyUnicodeWriter:
Changes by STINNER Victor victor.stin...@gmail.com:
Added file: http://bugs.python.org/file29878/bytes_writer_cjkcodecs.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17742
___
STINNER Victor added the comment:
See also #17694.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17742
___
___
Python-bugs-list mailing list
New submission from Piotr Dobrogost:
This makes it possible to handle paths/usernames with special characters - see
https://github.com/pypa/virtualenv/pull/352
--
components: Library (Lib)
messages: 187037
nosy: piotr.dobrogost, vinay.sajip
priority: normal
severity: normal
status:
New submission from Piotr Dobrogost:
activate.bat sets VIRTUAL_ENV environment variable. This variable is treated as
a sign that virtualenv is active. For this reason deactivate.bat should unset
this variable. See https://github.com/pypa/virtualenv/pull/364
--
components: Library
Changes by Barry A. Warsaw ba...@python.org:
--
nosy: +barry
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17012
___
___
Python-bugs-list mailing
Barry A. Warsaw added the comment:
From a documentation standpoint, path='' is not the same as When no path is
specified, so indeed it should return None when path=''. Serhiy's patch
looks good to me.
--
___
Python tracker rep...@bugs.python.org
Changes by Barry A. Warsaw ba...@python.org:
--
assignee: - barry
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17012
___
___
Python-bugs-list
New submission from Tshepang Lekhonkhobe:
attached patch reflects current reality (me assuming that PEPs are living
documents)
--
assignee: docs@python
components: Documentation
files: diff
messages: 187040
nosy: docs@python, tshepang
priority: normal
severity: normal
status: open
Changes by Jesús Cea Avión j...@jcea.es:
--
nosy: +jcea
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17741
___
___
Python-bugs-list mailing list
Eli Bendersky added the comment:
It's better, although the distinction between tab column and tab position
is not entirely clear.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17670
___
Eli Bendersky added the comment:
Thanks Antoine. This looks interesting - I'm somewhat swamped ATM but will try
to review the patch in the next few days.
Incidentally, since it is a new feature would it be worthwhile to discuss it on
python-ideas?
--
Raymond Hettinger added the comment:
I would like to spend some time with this before it goes forward (especially
the attrgetter, itemgetter, methodgetter group).
Right now, it looks like a nice effort but I don't see how it makes Python any
better for adding it. The odds are that this code
1 - 100 of 107 matches
Mail list logo