Ben North added the comment:
New patch, fixing nit noted in msg193624:
non-error-recursive-dictview-3.3-1.patch
Otherwise same as previous
non-error-recursive-dictview-3.3.patch
--
Added file:
http://bugs.python.org/file31025/non-error-recursive-dictview-3.3-1.patch
anatoly techtonik added the comment:
I've made my own monster, attached.
--
Added file: http://bugs.python.org/file31026/DictRecord.py
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18535
___
Amaury Forgeot d'Arc added the comment:
You need a better use case though.
termios awful constant names ('TIOCGWINSZ') and values ('\x1b') won't become
more easy with a nametuple.
--
resolution: - works for me
status: open - closed
___
Python
ariel brunner added the comment:
I will, sorry. I thought it's possible reopen an issue. Since all the relevant
information is already contained here, and since the problem is with the fix to
the issue, I figured it's best to reopen this one.
--
Matt Hickford added the comment:
( Upstreamed from Pip https://github.com/pypa/pip/issues/942 )
There are lot of Python users on Windows. If they succeed in installing a
package manager (a struggle), this is typically their first experience with it
$ pip install numpy
blah blah blah
New submission from ariel brunner:
When defining a function with factional default arguments, the tuple text
replaces the correct values in the tooltip.
Here's an example -
def f(a=0.5):
pass
f(
tooltip shows - (a=0tuple), i.e. replaces the .5 with tuple.
This was found to happen
Changes by ariel brunner ariel...@mail.tau.ac.il:
--
versions: -Python 2.6
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18539
___
___
New submission from Matěj Stuchlík:
Traceback (most recent call last):
File stdin, line 1, in module
File /usr/lib64/python2.7/imaplib.py, line 163, in __init__
self.open(host, port)
File /usr/lib64/python2.7/imaplib.py, line 229, in open
self.sock = socket.create_connection((host,
New submission from Antoine Pitrou:
The idiomatic way to use a LoggerAdapter is to override the process() method.
However, the example in the cookbook (*) is some gobbledegook code overriding
__getitem__ and __iter__ on a separate class. It's a pity that users are
directed to use an insane
New submission from Sumitinder:
Mod_python is used in our application and continous load of mod_python result
in increase in httpd memory usage i.e upto 1.6 Gb.
Packets used:
mod_python-3.2.8-3.1
httpd-2.2.3-43.el5.centos
Centos 5.5 OS
Is there any memory leak in mod_python which is resulting
Ronald Oussoren added the comment:
mod_python isn't part of the cpython std lib. As such, this isn't the right
tracker to report problems with mod_python.
You may have more success by asking the project itself
(http://www.modpython.org).
--
nosy: +ronaldoussoren
resolution: -
Changes by Jakub Wilk jw...@jwilk.net:
--
nosy: +jwilk
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue2066
___
___
Python-bugs-list mailing list
Ronald Oussoren added the comment:
I don't know if multiple inheritance is explicitly supported at the C level,
but it is possible to create an extension type with multiple base classes in
tp_bases.
--
nosy: +ronaldoussoren
___
Python tracker
Changes by Bohuslav Slavek Kabrda bkab...@redhat.com:
--
nosy: +bkabrda
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10131
___
___
STINNER Victor added the comment:
Changing the default inheritance value of os.pipe() is not acceptable because
it would break backward compatibility.
Giving access to Windows extra parameter is nice, but we have to find a way to
propose a portable API. That's what I'm trying to do with the
anatoly techtonik added the comment:
Do not hijack the issue - value interpretation is the next step, which better
keep out of scope for this improvement. termios is a C interface, which
documents the meaning of TIOCGWINSZ and has defined names for structure
entries, such as lflag. This issue
anatoly techtonik added the comment:
If you need a better use case for DictRecord, urlparse is another one.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18535
___
New submission from Moritz Sichert:
If you pass any of cafile, capath or cadefault to urllib.parse.urlopen it
creates a new opener that contains the HTTPSHandler that was according to the
ca* arguments. It then uses this new opener to execute the request.
If you installed a custom opener with
João Bernardo added the comment:
ping.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18078
___
___
Python-bugs-list mailing list
Unsubscribe:
Antoine Pitrou added the comment:
Charles-François's point about the algorithmic complexity is legitimate, so I
think he was actually waiting for you to amend your patch ;)
--
___
Python tracker rep...@bugs.python.org
João Bernardo added the comment:
Charles-François's point about the algorithmic complexity is
legitimate, so I think he was actually waiting for you to amend
your patch ;)
This doesn't seems to be the actual issue as it would require a big change for
something that wasn't proven
Changes by Ronald Oussoren ronaldousso...@mac.com:
--
nosy: +ronaldoussoren
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6627
___
___
Vinay Sajip added the comment:
That's not quite right. The recommended way *is* to override the process()
method. From the page you linked to:
If you need a different method, e.g. if you want to prepend or append the
contextual information to the message string, you just need to subclass
Antoine Pitrou added the comment:
That does not have a specific example, as it seems simple enough to
understand as stated.
I think we should acknowledge that people often have difficulties
with the rather simple functionalities of the logging module,
not only the advanced ones.
The example
New submission from Robert O'Callahan:
Popen() ought to support redirection to/from more file descriptors than 0, 1,
and 2 when spawning processes. A clear use case is here:
http://stackoverflow.com/questions/6050187/write-to-file-descriptor-3-of-a-python-subprocess-popen-object
Instead of
R. David Murray added the comment:
Unless I'm misunderstanding something, the current implementation does match
the current documentation: if buffer-size n, returning buffer-size bytes
is returning up to n. Up to means it could be less than.
So you are advocating a change in behavior...but
New submission from Chris Lambacher:
Starting at line 153 in enum.py there is:
153 if not use_args:
154 enum_member = __new__(enum_class)
155 original_value = value
156 else:
157 enum_member =
Vinay Sajip added the comment:
I think we should acknowledge that people often have difficulties
with the rather simple functionalities of the logging module,
not only the advanced ones.
Perhaps some people do have difficulties, but that's always going to be the
case no matter what you do.
R. David Murray added the comment:
I'm in favor of treating this a bugfix. It is a bug for repr to raise a
recursion error, IMO.
--
nosy: +r.david.murray
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18533
Antoine Pitrou added the comment:
This means that read1() will only return up to n bytes if n is smaller
than the buffer size, otherwise it will return at most buffer-size
bytes.
Did you actually observe such behaviour? If so, this is a bug.
--
R. David Murray added the comment:
Anatoly: FYI Do not hijack the issue is not a collaborative conversation
style in English. A more collaborative conversation style would be to leave
out that part of the sentence, and just say that you think value interpretation
should be a separate, later
Antoine Pitrou added the comment:
But how is that necessary for the use case? Your LoggerAdapter-
derived class could take the existing class as a constructor
parameter (*), then inject the required info in its overriden
process() method.
If you can adapt an existing class to look
New submission from Derek Wilson:
the ssl.get_server_certificate function is very useful for just requesting a
cert. It would also be very useful to have a parallel function, possibly
ssl.get_server_cert_chain that does the same thing except but provides a tuple
of PEM encoded certs
Changes by R. David Murray rdmur...@bitdance.com:
--
nosy: +r.david.murray
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18543
___
___
R. David Murray added the comment:
So this represents a change in behavior of the socket library on Fedora 18 vs
earlier versions? Do any of the impalib tests fail?
--
components: +email
nosy: +barry, r.david.murray
versions: +Python 3.4
___
Python
Nick Coghlan added the comment:
Just noting for the record (since it appears it was never brought back to the
comments): it is expected that programs that manipulate the AST may require
updates before they will work on a new version of Python. Preserving AST
backwards compatbility is too
Nick Coghlan added the comment:
(That was the outcome of the suggested AST discussions on python-dev that were
mentioned earlier)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue11549
___
Christian Heimes added the comment:
I'm already working on the issue and I have a couple of patches in my work
queue, e.g. #18369, #18233, #18147, #16487 and #18293
--
nosy: +christian.heimes
stage: - needs patch
superseder: - SSLSocket.getpeercertchain()
Changes by Ethan Furman et...@stoneleaf.us:
--
assignee: - ethan.furman
nosy: +ethan.furman
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18545
___
Changes by Christian Heimes li...@cheimes.de:
--
assignee: - christian.heimes
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18546
___
___
Richard Oudkerk added the comment:
IMHO
1) It should check all predicates.
2) It should return a list of ready conditions.
3) It should *not* accept a list of conditions.
4) from_condition() should be removed.
Also notify() should try again if releasing a waiter raises RuntimeError
because it
João Bernardo added the comment:
1) It should check all predicates.
OK. Maybe later there could be an option for customization?
2) It should return a list of ready conditions.
OK.
3) It should *not* accept a list of conditions.
The list option would be to simplify the wait method...
Derek Wilson added the comment:
very cool - i had seen a couple of those patches and they look good. Most of
what I asked for is implemented in supporting bits for those issues.
Is their anything you could use some help on? I'm very interested in seeing
better ssl support in python.
New submission from Jonas Eriksson:
Only tested on marked python versions. Checked the code in hg (a5681f50bae2)
and did not see anything related to this in the current development version.
Essentially, what I see is this:
os.path.dirname(asdf)
''
os.path.dirname(./asdf)
'.'
What I expect
Matěj Stuchlík added the comment:
So this represents a change in behavior of the socket library on Fedora 18 vs
earlier versions?
I don't believe so. This happens on my Fedora 18 system, my Debian box with
Python 2.6.6, on Fedora 19
(https://bugzilla.redhat.com/show_bug.cgi?id=987340),
Roundup Robot added the comment:
New changeset 16c15d7d4480 by Vinay Sajip in branch '2.7':
Issue #18541: simplified LoggerAdapter example.
http://hg.python.org/cpython/rev/16c15d7d4480
New changeset adaecee37745 by Vinay Sajip in branch '3.3':
Issue #18541: simplified LoggerAdapter example.
R. David Murray added the comment:
OK, yeah, this seems to be long standing, untested behavior. I will take a
look at what should be done here.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18540
Nikolaus Rath added the comment:
On 07/24/2013 07:35 AM, Antoine Pitrou wrote:
Antoine Pitrou added the comment:
This means that read1() will only return up to n bytes if n is smaller
than the buffer size, otherwise it will return at most buffer-size
bytes.
Did you actually observe
New submission from py.user:
http://docs.python.org/3/library/unittest.html#organizing-test-code
def suite():
suite = unittest.TestSuite()
suite.addTest(WidgetTestCase('test_default_size'))
suite.addTest(WidgetTestCase('test_resize'))
return suite
--
assignee:
R. David Murray added the comment:
Oh, it would definitely be a backward compatibility issue. Imagine code that
does a dirname and branches like this:
d = dirname(somepath)
if d:
handle directory
else:
no directory case
Or to give a more concrete example:
path =
New submission from Christian Heimes:
http://hg.python.org/cpython/file/96d817f41c4c/Modules/socketmodule.c#l4642
The code has two checks if (x == (unsigned long) -1 PyErr_Occurred()).
Coverity claims that the second time is dead code and I agree. It looks like an
artifact from the time the
New submission from Christian Heimes:
http://hg.python.org/cpython/file/96d817f41c4c/Modules/socketmodule.c#l614
CID 715331 (#1 of 1): Unchecked return value from library (CHECKED_RETURN)
5. check_return: Calling function fcntl(s-sock_fd, 4, delay_flag) without
checking return value. This
Phil Webster added the comment:
Added tests for the rest of the SearchEngine module. I used the mock
tkMessageBox for testing the error display, but used the tkinter Text widget
for other tests because mark_set and tag_add/remove haven't been added to the
mock yet.
--
Added file:
New submission from Christian Heimes:
http://hg.python.org/cpython/file/96d817f41c4c/Modules/_posixsubprocess.c#l384
CID 715330 (#2 of 2): Unchecked return value from library (CHECKED_RETURN)
17. check_return: Calling function fcntl(c2pwrite, 2, old
0xfffe) without checking return
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18468
___
New submission from Christian Heimes:
CID 1046712 (#1 of 1): Unchecked return value (CHECKED_RETURN)
3. check_return: Calling function PyArena_AddPyObject(PyArena *, PyObject *)
without checking return value (as is done elsewhere 5 out of 6 times).
4. unchecked_value: No check of the return
Antoine Pitrou added the comment:
Le mercredi 24 juillet 2013 à 18:24 +, Nikolaus Rath a écrit :
There should be no problem reading 20 bytes with a single call to
BytesIO.read(), yet the buffered reader returns only 10 bytes.
Er. Your bug report seems to imply that read1(n) can return
Nikolaus Rath added the comment:
On 07/24/2013 02:28 PM, Antoine Pitrou wrote:
Antoine Pitrou added the comment:
Le mercredi 24 juillet 2013 à 18:24 +, Nikolaus Rath a écrit :
There should be no problem reading 20 bytes with a single call to
BytesIO.read(), yet the buffered reader
Antoine Pitrou added the comment:
Le mercredi 24 juillet 2013 à 21:45 +, Nikolaus Rath a écrit :
The documentation is correct that read1(n) never returns more than n bytes.
My complaint is that the actual bound is stricter than this, band
read1(n) will never return more than min(n,
Antoine Pitrou added the comment:
Le mercredi 24 juillet 2013 à 21:51 +, Antoine Pitrou a écrit :
- if the buffer is empty, 1 raw I/O call is issued and at most 1 byte is
returned
Make that at most n bytes are returned, of course.
--
___
Nikolaus Rath added the comment:
On 07/24/2013 02:51 PM, Antoine Pitrou wrote:
Antoine Pitrou added the comment:
Le mercredi 24 juillet 2013 à 21:45 +, Nikolaus Rath a écrit :
The documentation is correct that read1(n) never returns more than n bytes.
My complaint is that the actual
Antoine Pitrou added the comment:
Ah, well. This is already fixed, then:
Python 3.4.0a0 (default:ae769deb45b2, Jul 20 2013, 19:28:41)
[GCC 4.7.3] on linux
Type help, copyright, credits or license for more information.
import io
raw = io.BytesIO(bytes(200))
buffered = io.BufferedReader(raw,
Changes by Antoine Pitrou pit...@free.fr:
--
superseder: - Improve BufferedReader.read1()
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18524
___
Nikolaus Rath added the comment:
On 07/24/2013 03:04 PM, Antoine Pitrou wrote:
However, 3.2 didn't get that improvement, sorry. See changeset 27bf3d0b8e5f
and issue #13393.
--
resolution: - duplicate
status: open - closed
Duh. So much back and forth just to find it's already
Roundup Robot added the comment:
New changeset 4b33f74ab0f1 by Ezio Melotti in branch '3.3':
#16937: document that stdin is always buffered, even when -u is used. Patch by
Elena Oat.
http://hg.python.org/cpython/rev/4b33f74ab0f1
New changeset 52f0aa0d552a by Ezio Melotti in branch 'default':
Ezio Melotti added the comment:
Fixed, thanks for the patch!
--
assignee: docs@python - ezio.melotti
nosy: +ezio.melotti
resolution: - fixed
stage: - committed/rejected
status: open - closed
type: - enhancement
___
Python tracker
Roundup Robot added the comment:
New changeset 17542d36effd by Ezio Melotti in branch '2.7':
#14853: remove test that was making too many assumptions about stdin. Patch by
Elena Oat.
http://hg.python.org/cpython/rev/17542d36effd
New changeset c5d847ee354a by Ezio Melotti in branch '3.3':
Ezio Melotti added the comment:
Fixed, thanks for the patch!
--
assignee: - ezio.melotti
resolution: - fixed
stage: patch review - committed/rejected
status: open - closed
versions: +Python 3.3, Python 3.4
___
Python tracker rep...@bugs.python.org
Ezio Melotti added the comment:
FWIW more than once I tried to open a .py file on Windows just to see a black
box flashing by and promptly disappear. I agree that opening the file in an
editor (e.g. IDLE) is a better option, but I realize some users might prefer to
launch a script when they
Ezio Melotti added the comment:
LGTM.
This could go in 2.7 and 3.3 too IMHO.
--
nosy: +pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18376
___
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
keywords: -easy
nosy: +ezio.melotti
stage: needs patch - patch review
versions: +Python 3.3, Python 3.4 -Python 3.1, Python 3.2
___
Python tracker rep...@bugs.python.org
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
nosy: +ezio.melotti
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue7982
___
___
Python-bugs-list
Changes by Ezio Melotti ezio.melo...@gmail.com:
--
nosy: +ezio.melotti
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18529
___
___
Phil Webster added the comment:
This patch moves the comment_header code to reformat_comment, adds tests for
reformat_comment, and implements the fix that Todd mentioned in #18429. In
addition, I changed the get_comment_header and get_indent regular expressions
to only look for spaces and
74 matches
Mail list logo
