[issue1602] windows console doesn't print or input Unicode

2013-09-15 Thread Glenn Linderman

Glenn Linderman added the comment:

Hi Drekin. Thanks for your work in progressing this issue. There have been a 
variety of techniques proposed for this issue, but it sounds like yours has 
built on what the others learned, and is close to complete, together with issue 
17620.

Is this in a form that can be used with Python 3.3? or 3.4 alpha? Can it be 
loaded externally from a script, or must it be compiled into Python, or both?

I've been using a variant of davidsarah's patch since 2 years now, but would 
like to take yours out for a spin. Is there a Complete Idiot's guide to using 
your patch? :)

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1602
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1602] windows console doesn't print or input Unicode

2013-09-15 Thread Terry J. Reedy

Terry J. Reedy added the comment:

From reading the module,
  import stream; stream.enable()
replaces sys.stdin/out/err with new classes.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1602
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14927] add Do not supply 'int' argument to random.shuffle docstring

2013-09-15 Thread Georg Brandl

Georg Brandl added the comment:

I would propose a leading underscore for these methods; they should make it 
clear to the user that the parameter is meant to be private.

With this line in the docstring, I would wonder Why is the argument there in 
the first place?! (because most people don't know about the optimization 
technique).

--
nosy: +georg.brandl

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14927
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue12085] subprocess.Popen.__del__ raises AttributeError if __init__ was called with an invalid argument list

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

As Victor suggested in msg136939 we can use a class attribute. This technique 
is used in some other places in the stdlib (perhaps for other purposes). For 
example in subprocess.Handle.

We should check all defined __del__-s in stdlib and fix them all if they use a 
instance attribute created in __init__.

--
nosy: +serhiy.storchaka
resolution: fixed - 
stage: committed/rejected - 
status: closed - open

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12085
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19022] Improve handling of type.__abstractmethods__ descriptor

2013-09-15 Thread Nick Coghlan

Changes by Nick Coghlan ncogh...@gmail.com:


--
nosy: +Yaroslav.Halchenko, benjamin.peterson, daniel.urban

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19022
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19022] Improve handling of type.__abstractmethods__ descriptor

2013-09-15 Thread Nick Coghlan

New submission from Nick Coghlan:

Issue 10006 pointed out that attempting to access object.__abstractmethods__ 
raises AttributeError. The solution was to ensure type.__abstractmethods__ 
*also* raises AttributeError.

This is thoroughly confusing, since the name is clearly visible in the output 
of dir(type). While it's technically legal for a descriptor to always raise 
AttributeError, breaking the following typical assumed invariant is *highly* 
dubious:

all(hasattr(obj, x) for x in dir(obj))

I see three main alternatives for improving the situation:

1. Make __abstractmethods__ a read-only descriptor that returns something 
meaningful (like a frozenset(), the same as you get if you inherit from 
abc.ABCMeta without defining any abstract methods)

2. Throw a better exception message that explains the broken invariant rather 
than the generic AttributeError with attribute name that is currently thrown.

3. Implement type.__dir__ to filter out the uncooperative descriptor

--
messages: 197755
nosy: ncoghlan
priority: normal
severity: normal
stage: needs patch
status: open
title: Improve handling of type.__abstractmethods__ descriptor
type: behavior
versions: Python 3.4

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19022
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18994] Inside fcntl module, we does not check the return code of all_ins function

2013-09-15 Thread Charles-François Natali

Charles-François Natali added the comment:

I noticed this when converting the code to use PyModule_AddIntMacro().

Vajrasky, did you see Ezio's review on
http://bugs.python.org/review/18994/#ps9258 ?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18994
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18994] Inside fcntl module, we does not check the return code of all_ins function

2013-09-15 Thread Vajrasky Kok

Vajrasky Kok added the comment:

Charles-François Natali, sorry I just noticed Ezio's comment. The all_ins 
function return -1 on failure and 0 on success.

I use this form:

if (all_ins(m))
return NULL;

because I follow the example in Modules/posixmodule.c. If this form:

if (all_ins(m)  0)
return NULL;

is better, then so be it, here is the patch to accommodate Ezio's suggestion.

--
Added file: 
http://bugs.python.org/file31765/check_return_code_all_ins_in_fcntl_v2.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18994
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19020] Regression: Windows-tkinter-idle, unicode, and 0xxx filename

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

There are two methods: splitlist() and split(). The splitlist() method splits 
only one level and always returns a tuple (even if it is empty or one-element). 
The split() method splits recursive and can returns a string if it is not 
splittable. I.e. result type of split() depends on if the argument contains 
spaces.

Before issue18101 split() was broken on Python 3. It splitted recursively only 
bytes objects but not unicode strings. After fixing it becomes split already 
parsed tuples of strings.

Perhaps split() shouldn't recursive parse tuples. But in this case its purpose 
is questionable. Recursive parse only string argument? But how it can 
distinguish the list of two strings a and b c from the list of the a 
string and the [b, c] list? Both are spelled as {a {b c}} in Tcl.

The proposed patch gets rid from split() in the configure result parsing code 
and uses two-level splitline() instead. Perhaps we should replace all left 
split()-s to the appropriate number of splitline()-s.

--
assignee:  - serhiy.storchaka
components: +IDLE, Tkinter
keywords: +patch
nosy: +gpolo
stage: needs patch - patch review
Added file: http://bugs.python.org/file31766/tkinter_configure_splitlist.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19020
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19014] Allow memoryview.cast() for empty views

2013-09-15 Thread Stefan Krah

Stefan Krah added the comment:

I agree that this cast should work. Perhaps disallowing zero strides
is enough -- I have to look at this more closely though.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19014
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18950] Miscellaneous fixes for the sunau module

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

Update patch addresses Ezio's comment.

--
Added file: http://bugs.python.org/file31767/sunau.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18950
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18877] tkinter askopenfilenames does not work in Windows library folder

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

I guess tkinter.wantobjects is 0 for you?

--
nosy: +serhiy.storchaka

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18877
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18877] tkinter askopenfilenames does not work in Windows library folder

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

This patch should fix the issue (as many other issues).

--
assignee:  - serhiy.storchaka
keywords: +patch
stage: needs patch - patch review
versions: +Python 3.4
Added file: http://bugs.python.org/file31768/tkinter_Tkapp_CallResult.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18877
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue3405] Add support for the new data option supported by event generate (Tk 8.5)

2013-09-15 Thread Serhiy Storchaka

Changes by Serhiy Storchaka storch...@gmail.com:


--
nosy: +serhiy.storchaka
versions: +Python 3.4 -Python 3.3

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3405
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6181] Tkinter.Listbox several minor issues

2013-09-15 Thread Serhiy Storchaka

Changes by Serhiy Storchaka storch...@gmail.com:


--
nosy: +serhiy.storchaka
versions: +Python 3.3, Python 3.4 -Python 3.1, Python 3.2

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6181
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue13519] Tkinter rowconfigure and columnconfigure functions crash if minsize, pad, or weight is not None

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

I think this was fixed in issue16809.

--
nosy: +serhiy.storchaka
status: open - pending

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13519
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18141] tkinter.Image.__del__ can throw an exception if module globals are destroyed in the wrong order

2013-09-15 Thread Serhiy Storchaka

Changes by Serhiy Storchaka storch...@gmail.com:


--
nosy: +gpolo, serhiy.storchaka, terry.reedy
stage:  - patch review
type:  - behavior
versions: +Python 2.7, Python 3.3

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18141
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue12558] Locale-dependent exception for float width argument to Tkinter widget constructor

2013-09-15 Thread Serhiy Storchaka

Changes by Serhiy Storchaka storch...@gmail.com:


--
nosy: +serhiy.storchaka

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12558
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19023] ctypes docs: Unimplemented and undocumented features

2013-09-15 Thread Sye van der Veen

New submission from Sye van der Veen:

In the ctypes documentation, there's mention of a 
BigEndianUnion/LittleEndianUnion that isn't actually implemented, and the 
Arrays and pointers section just reads Not yet written.

The attached patch adds the (Big|Little)EndianUnion classes (with tests), 
finishes the Array/_Pointer class docs (and adds missing cases to the tests), 
and makes some stylistic improvements to docs.

The patch was made against default, but it's quite likely it could be 
back-ported all the way to Python 3.1.

--
assignee: docs@python
components: Documentation, Tests, ctypes
hgrepos: 209
messages: 197764
nosy: docs@python, syeberman
priority: normal
severity: normal
status: open
title: ctypes docs: Unimplemented and undocumented features
type: enhancement

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19023
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19023] ctypes docs: Unimplemented and undocumented features

2013-09-15 Thread Sye van der Veen

Changes by Sye van der Veen syeber...@gmail.com:


--
keywords: +patch
Added file: http://bugs.python.org/file31769/75843d82f6cf.diff

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19023
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue5712] tkinter - askopenfilenames returns string instead of tuple in windows 2.6.1 release

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

Is it still reproducible on 2.7?

--
nosy: +serhiy.storchaka
versions:  -Python 2.6

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue5712
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue12558] Locale-dependent exception for float width argument to Tkinter widget constructor

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

This is a bug in Tk. It uses locale-agnostic Tcl_GetDoubleFromObj() in most 
places, but in some places (especially in canvas) it uses locale-dependent 
strtod(). And that is even worse, strtod() is used even for parsing 
comma-delimited floats (@-indices).

We can't fix it from Python side (except perhaps switching to C locale for 
every Tcl/Tk operation). But in desires a warning in the documentation.

--
components: +Documentation
stage:  - needs patch
versions: +Python 3.3, Python 3.4 -Python 3.2

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue12558
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue3015] tkinter with wantobjects=False has been broken for some time

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

Looks as issue18877 is related to this. I have proposed for it a little 
different patch than first Guilherme's patch.

As for removing wantobjects, we perhaps can do this only in 3.4.

--
nosy: +serhiy.storchaka
versions: +Python 3.3, Python 3.4 -Python 3.0

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue3015
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18877] tkinter askopenfilenames does not work in Windows library folder

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

See also #3015 and #5712.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18877
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue6225] Fixing several minor bugs in Tkinter.Canvas and one in Misc._configure

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

See also #19020.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue6225
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1602742] itemconfigure returns incorrect text property of text items

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

The patch for issue19020 fixes this issue.

--
nosy: +serhiy.storchaka
versions: +Python 3.3, Python 3.4 -Python 3.1, Python 3.2

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1602742
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue8010] tkFileDialog.askopenfiles crashes on Windows 7

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

Looks as a duplicate of issue5712.

--
nosy: +serhiy.storchaka
status: open - pending

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue8010
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19024] Document asterisk (*), splat or star operator

2013-09-15 Thread anatoly techtonik

New submission from anatoly techtonik:

I'd say this is a critical documentation bug that leads to head bang when you 
try to figure out what does '*' in code means.

This bug is two fold:
1. Define a dedicated place in documentation for '*' operator with examples. I 
propose http://docs.python.org/3/library/stdtypes.html chapter with a name like 
Special Operations or Function Argument Operations or Function Argument 
Operators
 
2. Make '*', 'asterisk', 'splat', 'star' operator searchable
http://docs.python.org/3/search.html?q=*
http://docs.python.org/3/search.html?q=asterisk
http://docs.python.org/3/search.html?q=splat
http://docs.python.org/3/search.html?q=star+operator

Good?

References:
http://stackoverflow.com/questions/2322355/proper-name-for-python-operator
http://stackoverflow.com/questions/5239856/foggy-on-asterisk-in-python
http://stackoverflow.com/questions/2921847/python-once-and-for-all-what-does-the-star-operator-mean-in-python
http://stackoverflow.com/questions/287085/what-do-args-and-kwargs-mean
http://stackoverflow.com/search?q=[python]+%2Bkwargs+is%3Aquestion

--
assignee: docs@python
components: Documentation
messages: 197772
nosy: docs@python, techtonik
priority: normal
severity: normal
status: open
title: Document asterisk (*), splat or star operator
type: crash
versions: Python 2.6, Python 2.7, Python 3.1, Python 3.2, Python 3.3, Python 
3.4, Python 3.5

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19024
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1602] windows console doesn't print or input Unicode

2013-09-15 Thread Drekin

Drekin added the comment:

Glenn Linderman: Yes I have built on what the others learned. For your 
question, I made it and tested it in Python 3.3, it should also work in 3.4 and 
what I've tried, it actually works. As Terry J. Reedy says you can just load 
the module and enable the streams. I do this automatically on startup using 
sitecustomize. However as I said currently this meeses up the interactive 
session because of http://bugs.python.org/issue17620 . I have made some 
workaround – custom REPL built on stdlib module code. And also a helper script 
which runs the main script and then runs the custom REPL (I couldn't find any 
stadard hook which would run the custom REPL). I'm uploding full code. I will 
delete it if this isn't appropriate place.

Things like this could be fixed more easily if more core interpreter logic took 
place in stdlib. E. g. the code for interactive REPL. Few days ago I started 
some discussion on python ideas: 
https://mail.python.org/pipermail/python-ideas/2013-August/023000.html .

--
Added file: http://bugs.python.org/file31770/win_unicode_console.zip

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1602
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19024] Document asterisk (*), splat or star operator

2013-09-15 Thread anatoly techtonik

anatoly techtonik added the comment:

tag:easy

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19024
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19024] Document asterisk (*), splat or star operator

2013-09-15 Thread Ezio Melotti

Changes by Ezio Melotti ezio.melo...@gmail.com:


--
keywords: +easy
nosy: +ezio.melotti
stage:  - needs patch
type: crash - enhancement
versions:  -Python 2.6, Python 3.1, Python 3.2, Python 3.5

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19024
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19025] Deleting attribute of Enum gives misleading error message

2013-09-15 Thread Vajrasky Kok

New submission from Vajrasky Kok:

 from enum import Enum
 class MyPet(Enum):
...   CUTE_CAT = 1
...   VIGOROUS_DOG = 2
...   UGLY_BLOBFISH = 3
...   PROMISCUOUS_BONOBO = 4
...   def spam(cls): pass
... 
 del MyPet.CUTE_CAT
Traceback (most recent call last):
  File stdin, line 1, in module
AttributeError: CUTE_CAT

This is misleading. I have CUTE_CAT attribute in Enum. It should throw 
TypeError exception.

 del MyPet.LONELY_WOLF
Traceback (most recent call last):
  File stdin, line 1, in module
AttributeError: LONELY_WOLF

This is correct.

 del MyPet['CUTE_CAT']
Traceback (most recent call last):
  File stdin, line 1, in module
TypeError: 'EnumMeta' object does not support item deletion

I think this behaviour is correct.

 del MyPet['LONELY_WOLF']
Traceback (most recent call last):
  File stdin, line 1, in module
TypeError: 'EnumMeta' object does not support item deletion

This behaviour is somewhat misleading. I don't have LONELY_WOLF item.

 del MyPet.spam
 hasattr(MyPet, 'spam')
False

This is correct. We should be able to delete method from Enum class, or 
shouldn't we?

 cute_cat = MyPet.CUTE_CAT
 del cute_cat.name
Traceback (most recent call last):
  File stdin, line 1, in module
  File /home/sky/Code/python/programming_language/cpython/Lib/enum.py, line 
29, in __delete__
raise AttributeError(can't delete attribute)
AttributeError: can't delete attribute

This is *maybe correct. But shouldn't it be TypeError exception?

All the exception messages related with deleting attribute in Python codebase 
are using TypeError exception (with one exception in pyexpat on which it uses 
RuntimeError exception).

Attached the patch to handle the attribute deletion correctly.

For now, I don't change the exception for the last case (del 
MyPet.CUTE_CAT.name) because maybe there is an exception case for enum.

--
components: Library (Lib)
files: handling_attribute_deletion_correctly_in_enum.patch
keywords: patch
messages: 197775
nosy: ethan.furman, vajrasky
priority: normal
severity: normal
status: open
title: Deleting attribute of Enum gives misleading error message
type: behavior
versions: Python 3.4
Added file: 
http://bugs.python.org/file31771/handling_attribute_deletion_correctly_in_enum.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19025
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1565525] tracebacks eat up memory by holding references to locals and globals when they are not wanted

2013-09-15 Thread A.M. Kuchling

A.M. Kuchling added the comment:

Ping!  I'd like to change the function name to clear_frames() and then commit 
this.  Antoine or anyone, want to disagree with using clear_frames() as the 
name?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1565525
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue1565525] tracebacks eat up memory by holding references to locals and globals when they are not wanted

2013-09-15 Thread Antoine Pitrou

Antoine Pitrou added the comment:

 Ping!  I'd like to change the function name to clear_frames() and then
 commit this.  Antoine or anyone, want to disagree with using
 clear_frames() as the name?

clear_frames() sounds fine to me :-)

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1565525
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18704] IDLE: PEP8 Style Check Integration

2013-09-15 Thread R. Jayakrishnan

R. Jayakrishnan added the comment:

Thanks for showing great interest on this idea.
As usual, it seems this will going to be another nice project with IDLE for me. 
Better to close this issue then, and create an appropriate initiation with the 
suggested points(preferring a commencement by core developers).
 
Anyway to keep in touch with the Python analyzing and related tools, I have 
initiated a separate project with porting the works done on the patch here 
https://github.com/Jay-Krish/IDLEAnalyzer.
I Hope this will help further while working on this project.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18704
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16043] xmlrpc: gzip_decode has unlimited read()

2013-09-15 Thread Barry A. Warsaw

Changes by Barry A. Warsaw ba...@python.org:


--
versions:  -Python 2.6

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16043
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17104] Tk() not defined in Tkinter module

2013-09-15 Thread Serhiy Storchaka

Changes by Serhiy Storchaka storch...@gmail.com:


--
components: +Tkinter -None
status: open - pending

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17104
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread A.M. Kuchling

A.M. Kuchling added the comment:

The patch requires a little adjusting to apply against 2.6.

--
nosy: +akuchling
Added file: http://bugs.python.org/file31772/smtplib-2.6.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue15363] Idle/tkinter ~x.py 'save as' fails. closes idle

2013-09-15 Thread Serhiy Storchaka

Changes by Serhiy Storchaka storch...@gmail.com:


--
components: +Tkinter

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue15363
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19025] Deleting attribute of Enum gives misleading error message

2013-09-15 Thread R. David Murray

R. David Murray added the comment:

Doesn't this confusion (here and in 19011) arise from the fact that the enum 
class does *not* have CUTE_CAT attribute?  That is, the error message is 
correct, but surprising.  Because, frankly, Enums are surprising in many ways ;)

--
nosy: +r.david.murray

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19025
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16040] nntplib: unlimited readline() from connection

2013-09-15 Thread Barry A. Warsaw

Barry A. Warsaw added the comment:

Any more thoughts on this bug w.r.t. 2.6.9?  It seems that without a patch for 
any version of Python, and with 2.6.9 coming soon, a fix for this just won't 
make it into 2.6.9.  

That doesn't bother me too much, and I'm willing to just knock this off the 
2.6.9 radar unless objections (accompanied by patches? :) are raised.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16040
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19025] Deleting attribute of Enum gives misleading error message

2013-09-15 Thread Barry A. Warsaw

Changes by Barry A. Warsaw ba...@python.org:


--
nosy: +barry

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19025
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread A.M. Kuchling

A.M. Kuchling added the comment:

Here's a final proposed version of the patch for 2.6 that adds a test.  Changes 
made:

* code now raises SMTPResponseException instead of a new SMTPLineTooLong 
exception; bwarsaw deemed that adding a new exception class was changing the 
module API.

* we looked at Serhiy's suggestion to move the length check into the 'while' 
loop's condition and decided not to -- the code is more obvious with the 
separate if/break.

* the test class is a cut-and-paste and slight modification of the 
BadHELOServerTests class; I didn't try to unify them in some way.

--
Added file: http://bugs.python.org/file31773/smtplib-2.6-with-test.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19025] Deleting attribute of Enum gives misleading error message

2013-09-15 Thread Ethan Furman

Changes by Ethan Furman et...@stoneleaf.us:


--
assignee:  - ethan.furman

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19025
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17764] Support http.server passing bind address via commend line argument

2013-09-15 Thread Roundup Robot

Roundup Robot added the comment:

New changeset a248655c8261 by Senthil Kumaran in branch 'default':
Expose --bind argument for http.server, enable http.server to bind to a user
http://hg.python.org/cpython/rev/a248655c8261

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17764
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue17764] Support http.server passing bind address via commend line argument

2013-09-15 Thread Senthil Kumaran

Senthil Kumaran added the comment:

Found the patch suitable and had addressed Berker Pesag's review comments too. 
Thanks for contributing, Malte Swart and thanks for submitting the contributor 
agreement too.

--
nosy: +orsenthil
resolution:  - fixed
stage: patch review - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue17764
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18704] IDLE: PEP8 Style Check Integration

2013-09-15 Thread Raymond Hettinger

Raymond Hettinger added the comment:

[TJR]
 I am opposed to this specific proposal and 
 think this issue should be closed.

[Ramchandra Apte]
 Agree with Terry Reedy.

[R. Jayakrishnan]
 Better to close this issue then ...

I agree as well.

--
resolution:  - rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18704
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

It is not important in the context of this issue, but readline(0) is blocked 
and returns 1-character string. Move the length check above 
self.sslobj.read(1). For readability you can also move the chr != \n inside 
the loop:

 while size is None or len(str)  size:
 chr = self.sslobj.read(1)
 if not chr or chr == \n: break
 str += chr

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19026] OrderedDict should not accept dict as parameter

2013-09-15 Thread anatoly techtonik

New submission from anatoly techtonik:

http://stackoverflow.com/questions/15733558/python-ordereddict-not-keeping-element-order

I wonder why OrderedDict accepts dict as parameter in a first place? OD is used 
when order is important and if plain dict is supplied, the order is lost.

 d = {3:4, 1:2}
 OD(d)
OrderedDict([(1, 2), (3, 4)])

OrderedDict should not accept dict as parameter.

--
components: Library (Lib)
messages: 197787
nosy: techtonik
priority: normal
severity: normal
status: open
title: OrderedDict should not accept dict as parameter
versions: Python 2.6, Python 2.7, Python 3.1, Python 3.2, Python 3.3, Python 
3.4, Python 3.5

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19026
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

It is not important in the context of this issue, but readline(0) is blocked 
and returns 1-character string. Move the length check above 
self.sslobj.read(1). For readability you can also move the chr != \n inside 
the loop:

 while size is None or len(str)  size:
 chr = self.sslobj.read(1)
 if not chr or chr == \n: break
 str += chr

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread Serhiy Storchaka

Changes by Serhiy Storchaka storch...@gmail.com:


--
Removed message: http://bugs.python.org/msg197788

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread Barry A. Warsaw

Barry A. Warsaw added the comment:

Patch looks great, thanks Andrew.  All tests pass.  Feel free to commit to the 
2.6 branch along with a NEWS file entry.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19026] OrderedDict should not accept dict as parameter

2013-09-15 Thread Raymond Hettinger

Raymond Hettinger added the comment:

An OrderedDict is a dict subclass and needs to accept the same inputs as dict 
methods.  This is a guaranteed API and not a bug.

It is not the OrderedDict's fault if you supply an unordered input.  It can't 
add order after the fact.

--
assignee:  - rhettinger
nosy: +rhettinger
resolution:  - rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19026
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16038] ftplib: unlimited readline() from connection

2013-09-15 Thread A.M. Kuchling

A.M. Kuchling added the comment:

For 2.6 I'll make a revised version of Giampaolo's patch that doesn't add a new 
exception class.  

Rationale: Adding a new exception class changes the API of the module, which 
we'd like to avoid.  If someone is writing 2.6 code that wants to catch this 
exception, they can't write except ftplib.LineTooLong because the name isn't 
present.  Instead they'll have to catch the parent Error exception class and 
analyze either its type or the exception message.  My conclusion is that adding 
the new class isn't actually useful.

(bwarsaw and I are at a mini-sprint looking at the 2.6.9 blockers, so we're 
looking at all of these 'unlimited readline' issues and will continue to remove 
new exceptions introduced by patches.)

--
nosy: +akuchling

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16038
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread Barry A. Warsaw

Barry A. Warsaw added the comment:

On Sep 15, 2013, at 04:47 PM, Serhiy Storchaka wrote:

It is not important in the context of this issue, but readline(0) is blocked
and returns 1-character string. Move the length check above
self.sslobj.read(1). For readability you can also move the chr != \n inside
the loop:

 while size is None or len(str)  size:
 chr = self.sslobj.read(1)
 if not chr or chr == \n: break
 str += chr

Hi Serhiy.  Is there a functional difference to re-arranging this loop?
All things being equal, the minimal change is probably best.

Also, what do you mean by readline(0) is blocked?  Do you mean this is a
blocking call or something else?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18709] SSL module fails to handle NULL bytes inside subjectAltNames general names (CVE-2013-4238)

2013-09-15 Thread Barry A. Warsaw

Barry A. Warsaw added the comment:

I'm removing 2.6 from the Versions field since AFAIK we've resolved this issue 
for 2.6.  This way it'll be easier to scan the blockers for 2.6.9.

If anyone things we still have things to address for this issue in 2.6.9, 
please reassign it or follow up.

--
versions:  -Python 2.6

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18709
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 8a6def3add5b by Andrew Kuchling in branch '2.6':
#16042: CVE-2013-1752: Limit amount of data read by limiting the call to 
readline().
http://hg.python.org/cpython/rev/8a6def3add5b

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19026] OrderedDict should not accept dict as parameter

2013-09-15 Thread anatoly techtonik

anatoly techtonik added the comment:

I don't know if it is bug or feature. There are probably cases when order is 
not important and OrderedDict is used, but I don't remember any.

Too bad Python doesn't have first class ordered mapping type, so that it could 
report error if unordered arguments are supplied (such as dict or **kwargs).

tag:wart

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19026
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread R. David Murray

R. David Murray added the comment:

I'm not sure what Serhiy means by is blocked, but the second half makes 
sense: readline(0) on a file will return the empty string, but here it will 
read one character and return it.  Like he says, it doesn't break anything in 
the context of this bug fix, but it is an API bug.  Unless I'm missing 
something his replacement also has a bug, though: it won't add the \n to the 
returned string.

A minimal fix for the API bug would be to extend the 'if size' with an elif for 
0 that returns the empty string.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19027] undefined symbol: _PyParser_Grammar

2013-09-15 Thread Armin Rigo

New submission from Armin Rigo:

It is possible to get the working copy of Python 2.7 in a state such that any 
attempt to do make ends with the error shown in the title.  The working copy 
is perfectly valid; it is a matter of getting the wrong timestamps on the files.

(Despite the error message, this is not directly related to issue10013 or make 
-j.)

I didn't try Python 3.x, but the same issue might apply.

How to reproduce:

- get your checkout, let's say in python/src/

- go to your build directory, say python/build/, and run ../src/configure; 
make.

- then, in the src dir, give Include/graminit.h a more recent timestamp than 
Python/graminit.c, for example by saying touch Include/graminit.h (this is 
the crux: a common mercurial checkout will order them the other way, just 
because of alphabetical ordering).

- remove your python/build/ directory completely, create it anew, and run again 
../src/configure; make.

The cause of the message is that the Makefile rule for $(GRAMMAR_C) runs, 
because the GRAMMAR_H file is more recent; but the GRAMMAR_H rule does nothing, 
because the latter is already up-to-date.  So the only thing that runs is 
touch $(GRAMMAR_C).  But this creates an empty file build/Python/graminit.c!

--
components: Interpreter Core
messages: 197797
nosy: arigo
priority: normal
severity: normal
status: open
title: undefined symbol: _PyParser_Grammar
type: compile error
versions: Python 2.7

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19027
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19026] OrderedDict should not accept dict as parameter

2013-09-15 Thread anatoly techtonik

anatoly techtonik added the comment:

Is it possible to make strict OrderedDict an optional feature? Like `from 
features import strict_ordered_dict'?

--
status: closed - pending

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19026
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread A.M. Kuchling

A.M. Kuchling added the comment:

I took Serhiy's suggestion and just moved up the 'if size' check in the loop.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

 Also, what do you mean by readline(0) is blocked?  Do you mean this is a 
 blocking call or something else?

Yes, I mean this is a blocking call.

 Unless I'm missing something his replacement also has a bug, though: it won't 
 add the \n to the returned string.

Oh, right. The correct code should be as I proposed in msg173413 or... as 
Andrew has committed. Good.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19026] OrderedDict should not accept dict as parameter

2013-09-15 Thread Serhiy Storchaka

Changes by Serhiy Storchaka storch...@gmail.com:


--
status: pending - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19026
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19025] Deleting attribute of Enum gives misleading error message

2013-09-15 Thread Ethan Furman

Ethan Furman added the comment:

Perhaps a section in the docs about the differences from typical Python classes 
is warranted:

  - Enum members are virtual
  - Enum members are singletons
  - new Enum members (aka instances of an Enum class) cannot be created
  - during class creation Enum members cannot be overwritten, nor overwrite
other class attributes
  - Enum classes with members cannot be subclassed
  - Enum classes support iteration
  - Enum classes support containment

--
nosy: +eli.bendersky

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19025
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19025] Deleting attribute of Enum gives misleading error message

2013-09-15 Thread Ethan Furman

Ethan Furman added the comment:

As for the error messages (going in reverse order):

==
-- del cute_cat.name
Traceback (most recent call last):
...
AttributeError: can't delete attribute
==

This is the same error given by @property.


==
-- del MyPet.spam
-- hasattr(MyPet, 'spam')
False
==

This is correct.  Only Enum members get special treatment.


==
-- del MyPet['LONELY_WOLF']
Traceback (most recent call last):
  File stdin, line 1, in module
TypeError: 'EnumMeta' object does not support item deletion
==

There are two errors here: LONELY_WOLF does not exist, and an operation is 
being attempted that is not supported.  Of those two, attempting the 
unsupported operation is the more serious, and should be the reported error.  
(Yes, dealing with multiple errors at once can often be confusing.)


==
-- del MyPet.CUTE_CAT
Traceback (most recent call last):
  File stdin, line 1, in module
AttributeError: CUTE_CAT
==

This one I am not sure about (much as I am not sure about __getattr__ in 
#19011).  Enum members are virtual, and don't actually live in the class 
namespace.  I'm inclined to leave the existing behavior as is, but if we choose 
to change one, we should change both.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19025
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread R. David Murray

R. David Murray added the comment:

For the security fix, the check should only be done if the file is the the 
default .netrc.  (Which would also make your error message correct...otherwise 
it is not :)  Also, it would make more sense for the 'prop =' to be inside the 
'if posix'.

Barry, with that detail fixed should I apply this to 2.6?  (I'll tweak the 
error messages a bit, too.)

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread R. David Murray

R. David Murray added the comment:

Note that I'll test it by hand before applying, and will write a test for 3.3 
(where Mock is available to make testing practical).

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19026] OrderedDict should not accept dict as parameter

2013-09-15 Thread Raymond Hettinger

Raymond Hettinger added the comment:

In general, it is not possible for a hypothetical StrictOrderedDict to know 
whether its input was ordered or not.  For the specific case of dict, it is 
possible, but the general case is of course completely general (i.e. if the 
input has a keys() method, the pairs are loaded with:  for k in other.keys(): 
od[k] = other[k]).  Those semantics are guaranteed.

Remember, Armin's core concept for OrderedDict was to remember the order that 
keys were added, the order is determined by whoever does the adding.

FWIW, the stackoverflow question was resolved trivially.  The learning point is 
perfectly general (i.e. it explains why you write Decimal('1.1') instead of 
Decimal(1.1)).

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19026
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18594] C accelerator for collections.Counter is slow

2013-09-15 Thread Anoop Thomas Mathew

Anoop Thomas Mathew added the comment:

40% faster collections.Counter() . Removed C accelerator. Patch attached. 
Passes all tests. Results comparison follows.

--
keywords: +patch
nosy: +Anoop.Thomas.Mathew
Added file: 
http://bugs.python.org/file31774/collections_Counter_without_C_accelerator_with_comprehension.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18594
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18594] C accelerator for collections.Counter is slow

2013-09-15 Thread Anoop Thomas Mathew

Anoop Thomas Mathew added the comment:

Performance comparison with and without patch applied.

--
Added file: http://bugs.python.org/file31775/performance_comparision.csv

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18594
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread Barry A. Warsaw

Barry A. Warsaw added the comment:

On Sep 15, 2013, at 06:05 PM, R. David Murray wrote:

For the security fix, the check should only be done if the file is the the
default .netrc.  (Which would also make your error message
correct...otherwise it is not :) Also, it would make more sense for the 'prop
=' to be inside the 'if posix'.

Barry, with that detail fixed should I apply this to 2.6?  (I'll tweak the
error messages a bit, too.)

For the error message, I suggest including both os.getuid and prop.st_uid,
e.g. something like:

.netrc file is owned by (%d); should be (%d) % (prop.st_uid, os.getuid())

NetrcParseError seems a little odd but I suppose I could justify incorrect
ownership or mode as a parse error.  We definitely don't want to introduce a
new exception for 2.6.9, so the only other option is an OSError I think.

RDM, can you write any tests for this issue?  Also, are any documentation
changes necessary?  I think this should be a candidate for 2.6.9.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread Barry A. Warsaw

Changes by Barry A. Warsaw ba...@python.org:


--
nosy: +benjamin.peterson, georg.brandl, larry
priority: high - release blocker

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread Barry A. Warsaw

Barry A. Warsaw added the comment:

On Sep 15, 2013, at 05:34 PM, Serhiy Storchaka wrote:

Oh, right. The correct code should be as I proposed in msg173413 or... as
Andrew has committed. Good.

Excellent.  So we're good for this in 2.6.  Thanks!

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread R. David Murray

R. David Murray added the comment:

Here is a 2.6 specific patch.  I've hand tested this.

--
keywords: +patch
Added file: http://bugs.python.org/file31776/netrc-2.6.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread R. David Murray

R. David Murray added the comment:

I could write a 2.6 test for the permissions part, but not for the incorrect 
owner part.  Do you want one without the other?

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread Barry A. Warsaw

Barry A. Warsaw added the comment:

On Sep 15, 2013, at 06:51 PM, R. David Murray wrote:

I could write a 2.6 test for the permissions part, but not for the incorrect
owner part.  Do you want one without the other?

Yeah, I guess you can't mock os or stat in 2.6. ;)

Let's test the permission part, and if you can manually test the owner part,
that'll have to be good enough.  The former will at least test the changes to
NetrcParseError.__str__().

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19026] OrderedDict should not accept dict as parameter

2013-09-15 Thread anatoly techtonik

anatoly techtonik added the comment:

On Sun, Sep 15, 2013 at 9:25 PM, Raymond Hettinger
rep...@bugs.python.org wrote:

 In general, it is not possible for a hypothetical StrictOrderedDict to know 
 whether its input was ordered or not.

Right. That's why it should not accept input that can only be
unordered (including dict and **kwargs) - this is what I mean by
strict mode.

 Remember, Armin's core concept for OrderedDict was to remember the order 
 that keys were added, the order is determined by whoever does the adding.

IMHO the statement the order is determined by whoever does the
adding is false in 9/10 cases of passed dict. In 9/10 cases whoever
supplies dict or **kwargs argument is unaware of what mistake he is
making, and how many hour she will spend discovering the issue.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19026
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18945] Name collision handling in tempfile is not covered by tests

2013-09-15 Thread Roundup Robot

Roundup Robot added the comment:

New changeset fa54069eb8c4 by Eli Bendersky in branch '2.7':
Close #18945: Add tests for tempfile name collision handling.
http://hg.python.org/cpython/rev/fa54069eb8c4

--
resolution:  - fixed
stage: test needed - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18945
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread R. David Murray

R. David Murray added the comment:

Hmm.  Answering the doc question caused me to run into something that calls the 
whole patch into question: 
  
  
http://www.unix.com/unix-dummies-questions-answers/11326-netrc-refuses-password.html.

In that example, the ftp program only rejected reading the password from the 
.netrc file when the permissions were wrong, but otherwise happily read it.  
*That* would be a better backward compatibility fix.  And yes, in that case I 
think we should probably put a note about it in the docs.

I'll update my patch and add the permissions test.  I originally used OSError, 
but with the trigger on password only I think the parse error would actually be 
more appropriate, so I'll switch to that.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18998] iter() not working in ElementTree

2013-09-15 Thread Eli Bendersky

Changes by Eli Bendersky eli...@gmail.com:


--
resolution:  - works for me
stage: test needed - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18998
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18747] Re-seed OpenSSL's PRNG after fork

2013-09-15 Thread Barry A. Warsaw

Barry A. Warsaw added the comment:

After further contemplation and without objection from __ap__ and Crys on IRC, 
I am de-targeting this for 2.6.  I won't apply it for 2.6.9.

--
versions:  -Python 2.6

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18747
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16038] ftplib: unlimited readline() from connection

2013-09-15 Thread A.M. Kuchling

A.M. Kuchling added the comment:

2.6 version of the patch.  Changes from Giampaolo's version of the patch:

* 2.6 didn't have FTP over TLS, so the patch changes slightly to adapt.

* Removed the LineTooLong exception class and just raise Error instead.  (This 
repeats the message text for Line too long in several place.)

--
Added file: http://bugs.python.org/file31777/ftplib.patch

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16038
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue4636] bdist_wininst installer with install script raises exception

2013-09-15 Thread Dan Nicholson

Dan Nicholson added the comment:

Right, that's what makes this difficult. If the stub exe of the target python 
was used, then it wouldn't need to care about compatibility. However, what 
you're running is the stub of the build python. So, when I distribute a 
bdist_wininst exe, it's running the stub from my python on the user's machine. 
That introduces a few compatibility issues.

1. The exe needs the same CRT version installed on the target that it was built 
with. It would fail to run immediately in this case. This does make having the 
built python be newer than the target python a little difficult.

2. The exe loads the python dll on the target machine. This requires the python 
dll to have enough compatibility for the usage of the python API in the exe of 
the build version.

3. After loading the python dll, python code is run in the target python. This 
is the problem I'm trying to solve here. The python code is embedded in the exe 
of the installer, so the compatibility with the target python has to be 
considered at build time.

The situation you're describing where the wininst of the target is run could 
maybe be made to work, but it would be a larger project.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue4636
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18989] reuse of enum names in class creation inconsistent

2013-09-15 Thread Ethan Furman

Ethan Furman added the comment:

As David noted, this was discussed somewhere in the megathreads that was the 
Enum PEP discussion.

As Georg noted changes to such a thoroughly discussed API should not be taken 
lightly.

As Antoine noted the Enum class is more restricted than normal classes by 
design.

As Nick seconded the current situation is thoroughly confusing.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18989
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread Arfrever Frehtes Taifersar Arahesis

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
nosy: +Arfrever

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread Arfrever Frehtes Taifersar Arahesis

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
versions: +Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue13888] test_builtin failure when run after test_tk

2013-09-15 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

This is due PyOS_InputHook. After commenting out EnableEventHook() tests are 
passed.

--
nosy: +serhiy.storchaka

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue13888
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18989] reuse of enum names in class creation inconsistent

2013-09-15 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 54ddd1124df8 by Ethan Furman in branch 'default':
Close #18989: enum members will no longer overwrite other attributes, nor be 
overwritten by them.
http://hg.python.org/cpython/rev/54ddd1124df8

--
nosy: +python-dev
resolution:  - fixed
stage:  - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18989
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue19027] undefined symbol: _PyParser_Grammar

2013-09-15 Thread Arfrever Frehtes Taifersar Arahesis

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
nosy: +Arfrever

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue19027
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16037] httplib: header parsing is not unlimited

2013-09-15 Thread Arfrever Frehtes Taifersar Arahesis

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
title: httplib: header parsing is not delimited - httplib: header parsing is 
not unlimited
versions: +Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16037
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue14984] netrc module allows read of non-secured .netrc file

2013-09-15 Thread Barry A. Warsaw

Barry A. Warsaw added the comment:

FWIW, the Ubuntu manpage netrc(5) says:

 password string
   Supply a password.  If this token is present, the auto-login
   process will supply the specified string if the remote server
   requires a password as part of the login process.  Note that
   if this token is present in the .netrc file for any user other
   than anonymous, ftp will abort the auto-login process if the
   .netrc is readable by anyone besides the user.

On Ubuntu, /usr/bin/ftp comes from the netkit-ftp package, which has this code 
in ruserpass.c:

case PASSWD:
if (*aname==NULL) {
fprintf(stderr, Error: `password' must follow `login' in .netrc\n);
goto bad;
}
if (strcmp(*aname, anonymous) 
fstat(fileno(cfile), stb) = 0 
(stb.st_mode  077) != 0) {
fprintf(stderr, Error - .netrc file not correct permissions.\n);
fprintf(stderr, Remove password or correct mode (should be 600).\n);
goto bad;

So it looks like it's only doing a permission check too, and then only if it 
sees `password`. (FWIW, it does the same check, sans the anonymous check 
obviously, for `account`.)

Seems to me like only doing the permission check is sufficient, and in line 
with existing tools and documentation.  (Though technically, I suppose if you 
chowned ~/.netrc to someone other than yourself, it would be readable by 
anyone besides the user.)

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue14984
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16038] ftplib: unlimited readline() from connection

2013-09-15 Thread Arfrever Frehtes Taifersar Arahesis

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
versions: +Python 2.6, Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16038
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16039] imaplib: unlimited readline() from connection

2013-09-15 Thread Arfrever Frehtes Taifersar Arahesis

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
versions: +Python 2.6, Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16039
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16040] nntplib: unlimited readline() from connection

2013-09-15 Thread Arfrever Frehtes Taifersar Arahesis

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
versions: +Python 2.6, Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16040
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16041] poplib: unlimited readline() from connection

2013-09-15 Thread Arfrever Frehtes Taifersar Arahesis

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
nosy: +barry
priority: critical - release blocker
versions: +Python 2.6, Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16041
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16042] smtplib: unlimited readline() from connection

2013-09-15 Thread Arfrever Frehtes Taifersar Arahesis

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
versions: +Python 2.6, Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16042
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16043] xmlrpc: gzip_decode has unlimited read()

2013-09-15 Thread Arfrever Frehtes Taifersar Arahesis

Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:


--
versions: +Python 3.1

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16043
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue16039] imaplib: unlimited readline() from connection

2013-09-15 Thread A.M. Kuchling

A.M. Kuchling added the comment:

Updated version of the patch against 2.6 that adds a test.  Thanks for the fix, 
Emil!

--
nosy: +akuchling
Added file: http://bugs.python.org/file31778/imaplib.txt

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16039
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



[issue18515] zipfile._ZipDecryptor generates wasteful crc32 table on import

2013-09-15 Thread Daniel Holth

Daniel Holth added the comment:

I am withdrawing zipfile-no-crc32.patch. It did not work correctly. 

zdlazy.patch should go in. It avoids creating the rarely-needed crc32 table 
until the first time it is needed, saving some memory and the majority of time 
needed to import the module.

zdlazy.patch should not affect its C replacement at all, except that the 
rarely-needed CRC32 table that we are not generating becomes a never-needed 
table.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue18515
___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com



  1   2   >