[issue16165] sched.scheduler.run() blocks scheduler

2012-12-08 Thread Serhiy Storchaka

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


Removed file: http://bugs.python.org/file27826/sched_unblock_2.patch

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



[issue16165] sched.scheduler.run() blocks scheduler

2012-12-08 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

Oh, I submitted wrong patch (it contained unrelated changes). Here is a right 
one.

--
Added file: http://bugs.python.org/file28254/sched_unblock_3.patch

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



[issue16640] Less code under lock in sched.scheduler

2012-12-08 Thread Serhiy Storchaka

New submission from Serhiy Storchaka:

In additional to issue16165 I think that sched.scheduler runs more code under a 
lock than needed. The proposed patch moves this code outside a lock. This 
unlikely a bug, may be only small enhancement and cleanup.

--
components: Library (Lib)
files: sched_less_lock.patch
keywords: patch
messages: 177147
nosy: giampaolo.rodola, pitrou, serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: Less code under lock in sched.scheduler
type: enhancement
versions: Python 3.4
Added file: http://bugs.python.org/file28255/sched_less_lock.patch

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



[issue16612] Integrate Argument Clinic specialized preprocessor into CPython trunk

2012-12-08 Thread Antoine Pitrou

Antoine Pitrou added the comment:

Can't you propose a PEP for this? I'd like to see the DSL officially discussed 
(or at least validated).

--
nosy: +pitrou

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



[issue16628] leak in ctypes.resize()

2012-12-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 1996171cfb96 by Antoine Pitrou in branch '3.2':
Issue #16628: Fix a memory leak in ctypes.resize().
http://hg.python.org/cpython/rev/1996171cfb96

New changeset cae5f3266b81 by Antoine Pitrou in branch '3.3':
Issue #16628: Fix a memory leak in ctypes.resize().
http://hg.python.org/cpython/rev/cae5f3266b81

New changeset def01022870a by Antoine Pitrou in branch 'default':
Issue #16628: Fix a memory leak in ctypes.resize().
http://hg.python.org/cpython/rev/def01022870a

New changeset df5a86a22310 by Antoine Pitrou in branch '2.7':
Issue #16628: Fix a memory leak in ctypes.resize().
http://hg.python.org/cpython/rev/df5a86a22310

--
nosy: +python-dev

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



[issue16628] leak in ctypes.resize()

2012-12-08 Thread Antoine Pitrou

Antoine Pitrou added the comment:

Should be fixed now.

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

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



[issue16641] sched.scheduler.enter arguments should be modifiable

2012-12-08 Thread Serhiy Storchaka

New submission from Serhiy Storchaka:

Example:

 import sched
 s=sched.scheduler()
 s.enter(10, 1, print)
Event(time=7452.676787873, priority=1, action=built-in function print, 
argument=[], kwargs={})
 next(s.queue).argument.append(spam)
 s.enter(5, 1, print)
Event(time=7473.326011725, priority=1, action=built-in function print, 
argument=['spam'], kwargs={})
 s.run()
spam
spam

Now every s.enter() call (even for other schedulers) without explicit 
argument use ['spam'] as argument.

Usually functions should not have modifiable arguments (unless it is 
intentional). We should use non-modifiable value or a sentinel for default. For 
example an empty tuple as default for argument and None as default for 
kwargs.

--
components: Library (Lib)
files: sched_nonmodifiable_args.patch
keywords: patch
messages: 177151
nosy: giampaolo.rodola, serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: sched.scheduler.enter arguments should be modifiable
type: behavior
versions: Python 3.3, Python 3.4
Added file: http://bugs.python.org/file28256/sched_nonmodifiable_args.patch

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



[issue16641] sched.scheduler.enter arguments should not be modifiable

2012-12-08 Thread Serhiy Storchaka

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


--
title: sched.scheduler.enter arguments should be modifiable - 
sched.scheduler.enter arguments should not be modifiable

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



[issue13390] Hunt memory allocations in addition to reference leaks

2012-12-08 Thread Antoine Pitrou

Antoine Pitrou added the comment:

 Minor bikeshedding/spelling nit: should
   _Py_AllocedBlocks
 be changed to
   _Py_AllocatedBlocks
 (and s/_Py_GetAllocedBlocks/_Py_GetAllocatedBlocks/)?

Right indeed. I'll do the change.

--

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



[issue16641] sched.scheduler.enter arguments should not be modifiable

2012-12-08 Thread Serhiy Storchaka

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


Removed file: http://bugs.python.org/file28256/sched_nonmodifiable_args.patch

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



[issue16641] sched.scheduler.enter arguments should not be modifiable

2012-12-08 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

Sorry, wrong patch.

--
Added file: http://bugs.python.org/file28257/sched_nonmodifiable_args.patch

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



[issue4395] Document auto __ne__ generation; provide a use case for non-trivial __ne__

2012-12-08 Thread Nick Coghlan

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


--
nosy: +ncoghlan

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



[issue4395] Document auto __ne__ generation; provide a use case for non-trivial __ne__

2012-12-08 Thread Nick Coghlan

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


--
nosy: +antocuni

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



[issue16642] Mention new kwargs named tuple parameter in sched module

2012-12-08 Thread Serhiy Storchaka

New submission from Serhiy Storchaka:

Here is a patch which adds missed mentions of kwargs named tuple parameter.

--
assignee: docs@python
components: Documentation
files: sched_docs_kwargs.patch
keywords: patch
messages: 177154
nosy: docs@python, giampaolo.rodola, serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: Mention new kwargs named tuple parameter in sched module
type: behavior
versions: Python 3.3, Python 3.4
Added file: http://bugs.python.org/file28258/sched_docs_kwargs.patch

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



[issue16643] Wrong documented default value for timefunc parameter in sched.scheduler()

2012-12-08 Thread Serhiy Storchaka

New submission from Serhiy Storchaka:

sched.scheduler() prefers to use time.monotonic as a default value for timefunc 
parameter, and time.time used only as a fallback. But time.time documented as a 
default value.

--
assignee: docs@python
components: Documentation
messages: 177155
nosy: docs@python, giampaolo.rodola, haypo, serhiy.storchaka
priority: normal
severity: normal
stage: needs patch
status: open
title: Wrong documented default value for timefunc parameter in 
sched.scheduler()
type: behavior
versions: Python 3.3, Python 3.4

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



[issue4395] Document auto __ne__ generation; provide a use case for non-trivial __ne__

2012-12-08 Thread Chris Jerdonek

Changes by Chris Jerdonek chris.jerdo...@gmail.com:


--
nosy: +chris.jerdonek

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



[issue16612] Integrate Argument Clinic specialized preprocessor into CPython trunk

2012-12-08 Thread Chris Jerdonek

Chris Jerdonek added the comment:

It seems the clinic.txt DSL document should be proofread for proper/consistent 
use of argument/parameter (e.g. as described in the recently added 
http://docs.python.org/dev/glossary.html#term-parameter ).  To choose a couple 
random examples--

+Argument declaration |   type name = default
+Argument flags   |   flag flag2 flag3=value

Shouldn't this be Type declaration, etc?

+Functions With Positional-Only Arguments
+

Positional-Only Parameters

--
nosy: +chris.jerdonek

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



[issue16612] Integrate Argument Clinic specialized preprocessor into CPython trunk

2012-12-08 Thread Chris Jerdonek

Changes by Chris Jerdonek chris.jerdo...@gmail.com:


--
nosy: +ezio.melotti

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



[issue15209] Re-raising exceptions from an expression

2012-12-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 8ba3c975775b by Nick Coghlan in branch '3.3':
Issue #15209: Clarify exception chaining description
http://hg.python.org/cpython/rev/8ba3c975775b

New changeset 5854101552c2 by Nick Coghlan in branch 'default':
Merge from 3.3 (Issue #15209)
http://hg.python.org/cpython/rev/5854101552c2

--
nosy: +python-dev

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



[issue15209] Re-raising exceptions from an expression

2012-12-08 Thread Nick Coghlan

Nick Coghlan added the comment:

I rewrote the relevant section of the module docs (since they were a bit murky 
in other ways as well).

Since I didn't answer the question earlier, the main reason a bare raise is 
permitted is because it's designed to be used to a bare except clause (e.g. 
when rolling back a database transaction as a result of an error). While you 
could achieve the same thing now with except BaseException, the requirement 
for all exceptions to inherit from BaseException is relatively recent - back in 
the days of string exceptions there was simply no way to catch arbitrary 
exceptions *and* give them a name.

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

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



[issue16267] order of decorators @abstractmethod and @classmethod is significant (is not documented to be in @abstractclassmethod which advises their combined use)

2012-12-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 3345afd6dc61 by Nick Coghlan in branch '3.3':
Close issue #16267: better docs for @abstractmethod composition
http://hg.python.org/cpython/rev/3345afd6dc61

New changeset be7202c38089 by Nick Coghlan in branch 'default':
Merge from 3.3 (issue #16267)
http://hg.python.org/cpython/rev/be7202c38089

--

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



[issue7741] Allow multiple statements in code.InteractiveConsole.push

2012-12-08 Thread Nick Coghlan

Nick Coghlan added the comment:

OK, after a long detour that delved deep into codeop and the vagaries of 
PyCF_DONT_IMPLY_DEDENT (due to a problem that turned out to be due to a missing 
\n in a test case I added), my main conclusion is:

Coupling the single vs multiple statement decision to the implicit print 
after every call decision is *really* annoying. The latter should be its own 
flag *or else* also implied by the DONT_IMPLY_DEDENT flag that is already 
passed to the compiler by codeop.

If *that* gets fixed, then the code module could simply switch over to 
compiling in exec mode always, without any side effects on the implicit display 
of expression results.

--

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



[issue16598] Docs: double newlines printed in some file iteration examples

2012-12-08 Thread Andrew Svetlov

Andrew Svetlov added the comment:

Committed, except whatsnew files.
That ones should be untouched from my perspective — the issue is not bugfix but 
improvement and I see no reason why we need to backedit whatsnew for old 
outdated versions.
Changesetes fed7306f26ce, 268ead8ae46b, 9e4b003a4d7a, 29627bd5b333.
Thanks for patches!

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

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



[issue16612] Integrate Argument Clinic specialized preprocessor into CPython trunk

2012-12-08 Thread Stefan Krah

Stefan Krah added the comment:

I second Antoine's request for a PEP. I'm not only concerned about the DSL:

This approach may work for modules like posixmodule.c, where each function
is largely a self-contained unit wrapping some API function.

On the other hand, I don't want to imagine how _decimal.c will look like:
There are  100 functions that are closely related. Many of those functions
are *already* generated by macros. I think that vertical space is a precious
resource, so all docstrings (cruft as far as understanding the program is
concerned) are in a header file.

The clinic.py preprocessor would add at least 3000 lines to an already 
large module, tearing apart groups of functions that form logical units.


Apologies for sounding negative, but IIRC this hasn't been brought up in 
the python-dev discussion at all.

--
nosy: +skrah

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



[issue15209] Re-raising exceptions from an expression

2012-12-08 Thread Ethan Furman

Ethan Furman added the comment:

There is one typo and one error in the first paragraph of the patch:

 When raising a new exception (rather than
 using to bare ``raise`` to re-raise the
 ^ should be an 'a'

 exception currently being handled), the
 implicit exception chain can be made explicit
 by using :keyword:`from` with :keyword:`raise`.
 The single argument to :keyword:`from` must be
 an exception or ``None``. It will be set as
 :attr:`__cause__` on the raised exception.

 Setting :attr:`__cause__` also implicitly sets
 the :attr:`__suppress_context__` attribute to ``True``.

The last sentence is incorrect -- __suppress_context__ is only set to True if 
__cause__ is set to None; if __cause__ is set to any other exception 
__suppress_context__ remains False and the new exception chain will be printed:

 try:
...   raise ValueError
... except:
...   raise NameError from KeyError
...
KeyError

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File stdin, line 4, in module
NameError

This is easily fixed by adding 'to ``None``':

 Setting :attr:`__cause__` to ``None`` also implicitly sets
 the :attr:`__suppress_context__` attribute to ``True``.

--

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



[issue16644] Wrong code in ContextManagerTests.test_invalid_args() in test_subprocess.py

2012-12-08 Thread Serhiy Storchaka

New submission from Serhiy Storchaka:

Test ContextManagerTests.test_invalid_args() in file 
Lib/test/test_subprocess.py contains code which has no sense. When Popen() fail 
then this code will not reachable. When Popen() success then c.exception will 
not assigned.

Here is a patch with right test.

--
components: Tests
files: test_subprocess_test_invalid_args.patch
keywords: patch
messages: 177164
nosy: ezio.melotti, michael.foord, pitrou, serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: Wrong code in ContextManagerTests.test_invalid_args() in 
test_subprocess.py
type: behavior
versions: Python 3.2, Python 3.3, Python 3.4
Added file: 
http://bugs.python.org/file28259/test_subprocess_test_invalid_args.patch

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



[issue16645] Wrong test_extract_hardlink() in test_tarfile.py

2012-12-08 Thread Serhiy Storchaka

New submission from Serhiy Storchaka:

Test test_extract_hardlink() in Lib/test/test_tarfile.py ran 4 times. First 
time (uncompressed tarfile) it extract symbolic and hard links successfully, 
but in other 3 times (gzipped, bzipped and lzma'ed tarfile) extracting fails 
becouse this links already exists (left from first test). The test just ignore 
this error and other possible errors (error EEXIST raised, but ignored all 
except ENOENT).

Here is a patch, which adds a right cleanup of extracted links.

--
components: Tests
files: test_tarfile_test_extract_hardlink.patch
keywords: patch
messages: 177166
nosy: ezio.melotti, michael.foord, pitrou, serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: Wrong test_extract_hardlink() in test_tarfile.py
type: behavior
versions: Python 2.7, Python 3.2, Python 3.3, Python 3.4
Added file: 
http://bugs.python.org/file28260/test_tarfile_test_extract_hardlink.patch

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



[issue16612] Integrate Argument Clinic specialized preprocessor into CPython trunk

2012-12-08 Thread Larry Hastings

Larry Hastings added the comment:

Antoine, Stefan: There doesn't appear to be a bright line separating this 
should get a PEP from this doesn't need a PEP.  That said, changes to the C 
API for CPython don't seem to merit PEPs very often, the recent Stable ABI 
being the lone exception to the rule I can recall.  Also, Guido already said 
(in python-dev) he didn't think it merited a PEP, and I tend to agree.

What I don't understand is what advantage writing a PEP would bring.  If you're 
worried that it needs discussion--okay, let's discuss it.  Surely we can do 
that here?  Or in python-dev?  What's on your mind?


Stefan: It sounds like Argument Clinic won't work well for _cdecimal, nor would 
any alternate DSL grammar work any better, really.  But have no fear--I doubt 
anyone is going to force you to accept Argument-Clinic-ization of _cdecimal 
against your will.  If you're comfortable maintaining the _cdecimal code 
without it, you certainly have my blessing.  Though I do hope you'll add the 
data needed to generate Signature objects once we figure out what that looks 
like.


Chris: clinic.txt is available for review as part of the patch I posted, please 
feel free to make your comments there.  I could file a new patch with the two 
changes you already suggest if you prefer.  (Though I'm not changing the name 
of the project to Parameter Clinic--the Monty Python reference is simply 
irresistible.)


Finally, an admission.  I wrote Argument Clinic thinking that the 
inspect.Signature Parameter objects had a per-Parameter docstring.  Well, they 
don't.  (I feel like an idiot.)  So we could theoretically remove the 
per-parameter docstrings from the DSL.  However, I'm pretty happy with the 
current structure of the DSL.  My inclination therefore is to leave them in.  
(And who knows, maybe the Parameter object could grow a docstring.)  Still, if 
you're going to propose changes to the DSL, I submit that removing the 
per-parameter docstring is wholly viable.

--

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



[issue16612] Integrate Argument Clinic specialized preprocessor into CPython trunk

2012-12-08 Thread Antoine Pitrou

Antoine Pitrou added the comment:

 Antoine, Stefan: There doesn't appear to be a bright line separating
 this should get a PEP from this doesn't need a PEP.  That said,
 changes to the C API for CPython don't seem to merit PEPs very often,
 the recent Stable ABI being the lone exception to the rule I can
 recall.  Also, Guido already said (in python-dev) he didn't think it
 merited a PEP, and I tend to agree.

My argument for requiring a PEP is that this change will not only affect
the people maintaining the clinic code. It will affect everyone
contributing and maintaining code in the CPython codebase. It's much
more than just an addition to the C API, it's a change in how CPython
is developed.

Writing and proposing a PEP brings public scrutiny in a much more
visible, and also better-recorded, way than a bug entry on a tracker.

Note that writing a PEP doesn't mean that there'll be a huge discussion
about it. And you needn't post it on python-ideas, you can post it on
python-dev instead.

--

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



[issue16612] Integrate Argument Clinic specialized preprocessor into CPython trunk

2012-12-08 Thread Alex Gaynor

Alex Gaynor added the comment:

For what it's worth, I'm  not as concerned with the process of the PEP, as 
having a single document we can review and discuss.

--

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



[issue16612] Integrate Argument Clinic specialized preprocessor into CPython trunk

2012-12-08 Thread Antoine Pitrou

Antoine Pitrou added the comment:

(and, oh, I agree there isn't a bright line separating this should get
a PEP from this doesn't need a PEP)

--

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



[issue16076] xml.etree.ElementTree.Element and xml.etree.ElementTree.TreeBuilder are no longer pickleable

2012-12-08 Thread Daniel Shahaf

Daniel Shahaf added the comment:

@Eli, thanks.  In the meantime, attaching a patch addressing Ezio's review; the 
caveats from msg 177134 still apply.

--
Added file: http://bugs.python.org/file28261/i16076-v4-combined.diff

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



[issue16646] FTP.makeport() loses socket error details

2012-12-08 Thread Serhiy Storchaka

New submission from Serhiy Storchaka:

In FTP.makeport() a socket error catched and saved but then raised a new error 
with saved error as argument.

Here is a patch which reraises an original error.

--
components: Library (Lib)
files: ftplib_makeport_raise_err.patch
keywords: patch
messages: 177171
nosy: serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: FTP.makeport() loses socket error details
type: behavior
versions: Python 2.7, Python 3.2, Python 3.3, Python 3.4
Added file: http://bugs.python.org/file28262/ftplib_makeport_raise_err.patch

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



[issue16646] FTP.makeport() loses socket error details

2012-12-08 Thread Serhiy Storchaka

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


Removed file: http://bugs.python.org/file28262/ftplib_makeport_raise_err.patch

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



[issue16646] FTP.makeport() loses socket error details

2012-12-08 Thread Serhiy Storchaka

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


Added file: http://bugs.python.org/file28263/ftplib_makeport_raise_err.patch

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



[issue16647] LMTP.connect() loses socket error details

2012-12-08 Thread Serhiy Storchaka

New submission from Serhiy Storchaka:

In LMTP.connect() a socket error catched and saved but then raised a new error 
with saved error as argument.

Here is a patch which reraises an original error.

--
components: Library (Lib)
files: smtplib_LMTP_connect_raise_err.patch
keywords: patch
messages: 177172
nosy: serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: LMTP.connect() loses socket error details
type: behavior
versions: Python 2.7, Python 3.2, Python 3.3, Python 3.4
Added file: 
http://bugs.python.org/file28264/smtplib_LMTP_connect_raise_err.patch

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



[issue11159] Sax parser crashes if given unicode file name

2012-12-08 Thread Carsten Grohmann

Changes by Carsten Grohmann carstengrohm...@gmx.de:


--
nosy: +cgrohmann

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



[issue16644] Wrong code in ContextManagerTests.test_invalid_args() in test_subprocess.py

2012-12-08 Thread Antoine Pitrou

Antoine Pitrou added the comment:

The test comes from e02021fbfe22 (Fix #10554. Added context manager support to 
Popen objects.).

--
nosy: +brian.curtin

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



[issue16632] Enable DEP and ASLR

2012-12-08 Thread Antoine Pitrou

Changes by Antoine Pitrou pit...@free.fr:


--
nosy: +brian.curtin, tim.golden

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



[issue16648] stdib should use new exception types from PEP 3151

2012-12-08 Thread Andrew Svetlov

New submission from Andrew Svetlov:

http://www.python.org/dev/peps/pep-3151 define exception classes like 
ConnectionError and BrokenPipeError

We need to refactor stdlib to use that shiny new exceptions instead of checking 
errno values if possible.

Also maybe we need to add some absent exceptions.
I guess NotConnectedError for ENOTCONN, which should be ConnectionError 
subclass.

--
components: Library (Lib)
messages: 177174
nosy: asvetlov
priority: normal
severity: normal
stage: needs patch
status: open
title: stdib should use new exception types from PEP 3151
type: enhancement
versions: Python 3.4

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



[issue16648] stdib should use new exception types from PEP 3151

2012-12-08 Thread Serhiy Storchaka

Serhiy Storchaka added the comment:

Right now I'm working on a patch. ;) A pack of issues were produced in process.

--
nosy: +serhiy.storchaka

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



[issue16648] stdib should use new exception types from PEP 3151

2012-12-08 Thread Andrew Svetlov

Andrew Svetlov added the comment:

Please, feel free to add me in nosylist for any issue related to this one.

--

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



[issue16602] weakref can return an object with 0 refcount

2012-12-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 0748c22b37e5 by Antoine Pitrou in branch '3.2':
Issue #16602: When a weakref's target was part of a long deallocation chain, 
the object could remain reachable through its weakref even though its refcount 
had dropped to zero.
http://hg.python.org/cpython/rev/0748c22b37e5

New changeset 259c1636c884 by Antoine Pitrou in branch '3.3':
Issue #16602: When a weakref's target was part of a long deallocation chain, 
the object could remain reachable through its weakref even though its refcount 
had dropped to zero.
http://hg.python.org/cpython/rev/259c1636c884

New changeset 17e5acad302e by Antoine Pitrou in branch 'default':
Issue #16602: When a weakref's target was part of a long deallocation chain, 
the object could remain reachable through its weakref even though its refcount 
had dropped to zero.
http://hg.python.org/cpython/rev/17e5acad302e

--
nosy: +python-dev

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



[issue16602] weakref can return an object with 0 refcount

2012-12-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 7e771f0363e2 by Antoine Pitrou in branch '2.7':
Issue #16602: When a weakref's target was part of a long deallocation chain, 
the object could remain reachable through its weakref even though its refcount 
had dropped to zero.
http://hg.python.org/cpython/rev/7e771f0363e2

--

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



[issue16602] weakref can return an object with 0 refcount

2012-12-08 Thread Antoine Pitrou

Antoine Pitrou added the comment:

Thank you very much for reporting this issue. I believe it is now fixed.

--
resolution:  - fixed
stage: needs patch - committed/rejected
status: open - closed

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



[issue11159] Sax parser crashes if given unicode file name

2012-12-08 Thread Daniel Urban

Changes by Daniel Urban urban.dani...@gmail.com:


--
type: crash - behavior

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



[issue16602] weakref can return an object with 0 refcount

2012-12-08 Thread Eugene Toder

Eugene Toder added the comment:

Thank you, Antoine.
This change has one effect that's worth highlighting in NEWS at least -- the 
PyWeakref_GET_OBJECT() macro now evaluates its argument twice. This can break 
existing code where the argument has side-effects, e.g. o = 
PyWeakref_GET_OBJECT(p++). I found one such case in our code base, but I don't 
know how common this is. So this is something to watch out for when upgrading.
I don't think there's a way to write PyWeakref_GET_OBJECT() in standard C90 
without double evaluation.

--

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



[issue16602] weakref can return an object with 0 refcount

2012-12-08 Thread Antoine Pitrou

Antoine Pitrou added the comment:

 I don't think there's a way to write PyWeakref_GET_OBJECT() in standard 
 C90 without double evaluation.

Agreed. We generally don't document whether our macros are compatible with side 
effects in parameters, and I think we'd like to keep it that way. People should 
simply avoid doing this kind of thing, as it's knowingly fragile, and trivial 
to avoid anyway.

--

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



[issue16602] weakref can return an object with 0 refcount

2012-12-08 Thread Eugene Toder

Eugene Toder added the comment:

 People should simply avoid doing this kind of thing, as it's knowingly 
 fragile, and trivial to avoid anyway.
Is this documented in the C API guide, or somewhere else?
In any case, notifying people so they can quickly check their code seems much 
nicer than having them figure this out the hard way.

--

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



[issue16602] weakref can return an object with 0 refcount

2012-12-08 Thread Antoine Pitrou

Antoine Pitrou added the comment:

  People should simply avoid doing this kind of thing, as it's
  knowingly fragile, and trivial to avoid anyway.
 Is this documented in the C API guide, or somewhere else?

I don't think so, but it's common C wisdom that you shouldn't pass
arguments which have side effects to a macro, except if you are sure the
macro allows it.

 In any case, notifying people so they can quickly check their code
 seems much nicer than having them figure this out the hard way.

I see it as a double-edged sword: if we start adding a warning for this
macro, people will expect us to do it for every other macro, which we
aren't doing right now.

--

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



[issue16602] weakref can return an object with 0 refcount

2012-12-08 Thread Antoine Pitrou

Antoine Pitrou added the comment:

  In any case, notifying people so they can quickly check their code
  seems much nicer than having them figure this out the hard way.
 
 I see it as a double-edged sword: if we start adding a warning for this
 macro, people will expect us to do it for every other macro, which we
 aren't doing right now.

And also, as this issue shows, we may have to change it in a bugfix
release, which means people shouldn't trust the fact that there's no
such warning, anyway.

--

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



[issue7741] Allow multiple statements in code.InteractiveConsole.push

2012-12-08 Thread Aaron Iles

Aaron Iles added the comment:

Should a new issue be created to decouple print after every call from the
single vs multiple statement condition that is a blocker for this issue? Or
can it be resolved here?

On Sunday, 9 December 2012, Nick Coghlan wrote:


 Nick Coghlan added the comment:

 OK, after a long detour that delved deep into codeop and the vagaries of
 PyCF_DONT_IMPLY_DEDENT (due to a problem that turned out to be due to a
 missing \n in a test case I added), my main conclusion is:

 Coupling the single vs multiple statement decision to the implicit
 print after every call decision is *really* annoying. The latter should be
 its own flag *or else* also implied by the DONT_IMPLY_DEDENT flag that is
 already passed to the compiler by codeop.

 If *that* gets fixed, then the code module could simply switch over to
 compiling in exec mode always, without any side effects on the implicit
 display of expression results.

 --

 ___
 Python tracker rep...@bugs.python.org javascript:;
 http://bugs.python.org/issue7741
 ___


--

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



[issue16612] Integrate Argument Clinic specialized preprocessor into CPython trunk

2012-12-08 Thread Bradley Froehle

Changes by Bradley Froehle brad.froe...@gmail.com:


--
nosy: +bfroehle

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



[issue16346] readline problem

2012-12-08 Thread Éric Araujo

Éric Araujo added the comment:

On a Ubuntu system you don’t need to install readline from PyPI: CPython has a 
readline module, which gets compiled if the headers are available.

That said, the error you report is still strange.  Can you attach the full 
console output as a text file?

--

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



[issue16326] distutils build_ext fails to set library_dirs in 2.7.2 on Linux

2012-12-08 Thread Éric Araujo

Éric Araujo added the comment:

Vinay, do you think dereferencing sys.executable could lead to trouble with 
venvs?

--
nosy: +vinay.sajip

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



[issue14894] distutils.LooseVersion fails to compare number and a word

2012-12-08 Thread Éric Araujo

Éric Araujo added the comment:

Thanks Dwayne.  I think that most tools (e.g. setuptools) generate things like 
1.0.2+gitab25c6fe95ee92fac3187dcd90e0560ccacb084a i.e. real version set by 
maintainer + hash, not just the hash, so I am still not convinced a change is 
needed.

--
stage: needs patch - 

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



[issue16406] move the Uploading Packages section to distutils/packageindex.rst

2012-12-08 Thread Éric Araujo

Éric Araujo added the comment:

I’m reluctant to break links.  I think the issues you report could be fixed or 
mitigated with changes to text, clarifications, interlinks.  What do you think?

--

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



[issue16406] move the Uploading Packages section to distutils/packageindex.rst

2012-12-08 Thread Chris Jerdonek

Chris Jerdonek added the comment:

I've been meaning to ask Georg or anyone else knowledgeable about what options 
we have for redirects, etc. -- specifically for this issue.  Maybe something 
similar can even be done using Sphinx.

If we can't redirect, I was thinking even a stub page with a manual link to the 
PyPI page might be an improvement (to prevent the broken external link issue 
you mention).

--

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



[issue13614] setup.py register fails if long_description contains ReST

2012-12-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 692be1f9fa1d by Éric Araujo in branch '2.7':
Fix setup.py register failure with invalid rst in description (#13614).
http://hg.python.org/cpython/rev/692be1f9fa1d

--

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



[issue7719] distutils: ignore .nfsXXXX files

2012-12-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset d978828bfd0e by Éric Araujo in branch '3.2':
Ignore .nfs* files in distutils (#7719).
http://hg.python.org/cpython/rev/d978828bfd0e

New changeset 10ab746f55fb by Éric Araujo in branch '3.3':
Merge fixes for #13614, #13512 and #7719 from 3.2
http://hg.python.org/cpython/rev/10ab746f55fb

New changeset b10c1c6f869f by Éric Araujo in branch 'default':
Merge fixes for #13614, #13512 and #7719 from 3.3
http://hg.python.org/cpython/rev/b10c1c6f869f

--

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



[issue13614] setup.py register fails if long_description contains ReST

2012-12-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 99c80e8a721e by Éric Araujo in branch '3.2':
Fix setup.py register failure with invalid rst in description (#13614).
http://hg.python.org/cpython/rev/99c80e8a721e

New changeset 10ab746f55fb by Éric Araujo in branch '3.3':
Merge fixes for #13614, #13512 and #7719 from 3.2
http://hg.python.org/cpython/rev/10ab746f55fb

New changeset b10c1c6f869f by Éric Araujo in branch 'default':
Merge fixes for #13614, #13512 and #7719 from 3.3
http://hg.python.org/cpython/rev/b10c1c6f869f

--

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



[issue13512] ~/.pypirc created insecurely

2012-12-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 4a2814f24a10 by Éric Araujo in branch '3.2':
Create ~/.pypirc securely (#13512).
http://hg.python.org/cpython/rev/4a2814f24a10

New changeset 10ab746f55fb by Éric Araujo in branch '3.3':
Merge fixes for #13614, #13512 and #7719 from 3.2
http://hg.python.org/cpython/rev/10ab746f55fb

New changeset b10c1c6f869f by Éric Araujo in branch 'default':
Merge fixes for #13614, #13512 and #7719 from 3.3
http://hg.python.org/cpython/rev/b10c1c6f869f

--

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



[issue7719] distutils: ignore .nfsXXXX files

2012-12-08 Thread Éric Araujo

Éric Araujo added the comment:

Thanks all!

--
resolution:  - fixed
stage: commit review - committed/rejected
status: open - closed
versions: +Python 3.4

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



[issue13614] setup.py register fails if long_description contains ReST

2012-12-08 Thread Éric Araujo

Changes by Éric Araujo mer...@netwok.org:


--
resolution:  - fixed
stage: commit review - committed/rejected
status: open - closed

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



[issue13512] ~/.pypirc created insecurely

2012-12-08 Thread Éric Araujo

Changes by Éric Araujo mer...@netwok.org:


--
resolution:  - fixed
stage: patch review - committed/rejected
status: open - closed
versions: +Python 3.4

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



[issue16267] order of decorators @abstractmethod and @classmethod is significant (is not documented to be in @abstractclassmethod which advises their combined use)

2012-12-08 Thread Nick Coghlan

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


--
resolution:  - fixed
stage: needs patch - committed/rejected
status: open - closed

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



[issue15209] Re-raising exceptions from an expression

2012-12-08 Thread Roundup Robot

Roundup Robot added the comment:

New changeset 3b67247f0bbb by Nick Coghlan in branch '3.3':
Issue #15209: Fix typo and some additional wording tweaks
http://hg.python.org/cpython/rev/3b67247f0bbb

New changeset 04eb89e078b5 by Nick Coghlan in branch 'default':
Merge from 3.3 (issue #15209)
http://hg.python.org/cpython/rev/04eb89e078b5

--

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



[issue15209] Re-raising exceptions from an expression

2012-12-08 Thread Nick Coghlan

Nick Coghlan added the comment:

On Sun, Dec 9, 2012 at 2:59 AM, Ethan Furman rep...@bugs.python.org wrote:


 Ethan Furman added the comment:

 There is one typo and one error in the first paragraph of the patch:

  When raising a new exception (rather than
  using to bare ``raise`` to re-raise the
  ^ should be an 'a'


Fixed.

  Setting :attr:`__cause__` also implicitly sets
  the :attr:`__suppress_context__` attribute to ``True``.

 The last sentence is incorrect -- __suppress_context__ is only set to True
 if __cause__ is set to None; if __cause__ is set to any other exception
 __suppress_context__ remains False and the new exception chain will be
 printed:

  try:
 ...   raise ValueError
 ... except:
 ...   raise NameError from KeyError
 ...
 KeyError

 The above exception was the direct cause of the following exception:

 Traceback (most recent call last):
   File stdin, line 4, in module
 NameError


Not true: __suppress_context__ is always set as a side effect of setting
__cause__ (it's built into the setter for the __cause__ descriptor). What
you're seeing in the traceback above is the explicit cause, not the
implicit context.

 e = Exception()
 e.__cause__ = Exception()
 e.__suppress_context__
True

The only mechanism we offer to suppress an explicit __cause__ is setting
__cause__ to None.

Cheers,
Nick.

--

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



[issue13349] Non-informative error message in index() and remove() functions

2012-12-08 Thread Sean Ochoa

Sean Ochoa added the comment:

Update based on Taggnostr's (Ezio on IRC, if I recall correctly) feedback from 
11/12/2012 around 11:57 PST.  

* Added check for index result in positive tests.
* Added assertIn check for remove result in positive tests.
* Removed extra whitespace.
* Formatted comments to be more concise.

--
Added file: http://bugs.python.org/file28265/issue13349.patch.6

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



[issue16649] Add a PyCF_PRINT_EXPRESSION_RESULTS flag

2012-12-08 Thread Nick Coghlan

New submission from Nick Coghlan:

code.InteractiveConsole.push attempts to support multi-line input when using an 
input method that doesn't reliably split on newlines.

This support fails as soon as the multi-line input contains multiple statements 
(including when the last statement is incomplete).

However, switching to 'exec' instead of 'single' causes other problems - 
specifically, it turns off the expression printing.

While expression printing could be implied by the PyCF_DONT_IMPLY_DEDENT flag 
in addition to the single evaluation mode, it seems cleaner to break out the 
expression printing behaviour into its own flag and setting it appropriately in 
the code and/or codeop modules.

--
components: Interpreter Core
messages: 177199
nosy: aliles, kristjan.jonsson, ncoghlan
priority: low
severity: normal
status: open
title: Add a PyCF_PRINT_EXPRESSION_RESULTS flag
versions: Python 3.4

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



[issue16649] Add a PyCF_PRINT_EXPRESSION_RESULTS flag

2012-12-08 Thread Nick Coghlan

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


--
nosy: +benjamin.peterson

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



[issue16649] Add a PyCF_DISPLAY_EXPRESSION_RESULTS flag

2012-12-08 Thread Nick Coghlan

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


--
title: Add a PyCF_PRINT_EXPRESSION_RESULTS flag - Add a 
PyCF_DISPLAY_EXPRESSION_RESULTS flag

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



[issue16649] Add a PyCF_DISPLAY_EXPRESSION_RESULTS flag

2012-12-08 Thread Nick Coghlan

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


--
stage:  - needs patch
type:  - enhancement

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



[issue7741] Allow multiple statements in code.InteractiveConsole.push

2012-12-08 Thread Nick Coghlan

Nick Coghlan added the comment:

More implicit magic seems like a bad idea, so I've split out a proposal for an 
explicit PyCF_DISPLAY_EXPRESSION_RESULTS flag as #16649.

The behaviour would then be selectable regardless of the compilation mode, but 
would remain the default for single.

--
dependencies: +Add a PyCF_DISPLAY_EXPRESSION_RESULTS flag
priority: normal - low
versions: +Python 3.4 -Python 2.7

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



[issue16650] Popen._internal_poll() references errno.ECHILD outside of the local scope

2012-12-08 Thread Serhiy Storchaka

New submission from Serhiy Storchaka:

As noted in Popen._internal_poll() docstring this method cannot reference 
anything outside of the local scope. However it references errno.ECHILD. The 
proposed patch fixes this.

Is it good that Popen._handle_exitstatus() references building SubprocessError?

--
components: Library (Lib)
files: subprocess_reference_nonlocal.patch
keywords: patch
messages: 177201
nosy: asvetlov, pitrou, serhiy.storchaka
priority: normal
severity: normal
stage: patch review
status: open
title: Popen._internal_poll() references errno.ECHILD outside of the local scope
type: behavior
versions: Python 2.7, Python 3.2, Python 3.3, Python 3.4
Added file: http://bugs.python.org/file28266/subprocess_reference_nonlocal.patch

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



[issue16346] readline problem

2012-12-08 Thread Mathieu Dutour Sikiric

Mathieu Dutour Sikiric added the comment:

Well, that was the points, the headers were not available.

  Mathieu

On Sunday, December 9, 2012, Éric Araujo wrote:


 Éric Araujo added the comment:

 On a Ubuntu system you don’t need to install readline from PyPI: CPython
 has a readline module, which gets compiled if the headers are available.

 That said, the error you report is still strange.  Can you attach the full
 console output as a text file?

 --

 ___
 Python tracker rep...@bugs.python.org javascript:;
 http://bugs.python.org/issue16346
 ___


--

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



[issue13349] Non-informative error message in index() and remove() functions

2012-12-08 Thread Sean Ochoa

Sean Ochoa added the comment:

* Fixed issue with test name in Lib/test/test_tuple.py
* Fixed issue with test_remove in Lib/test/test_list.py to assertNotIn instead 
of assertIn for positive case.

Confirmed with Ezio that issue #7330 will need to be fixed/approved before this 
issue can be closed.

--
Added file: http://bugs.python.org/file28267/issue13349.patch.7

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



[issue7330] PyUnicode_FromFormat: implement width and precision for %s, %S, %R, %V, %U, %A

2012-12-08 Thread Sean Ochoa

Changes by Sean Ochoa sean.m.oc...@gmail.com:


--
nosy: +Sean.Ochoa

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



[issue16406] move the Uploading Packages section to distutils/packageindex.rst

2012-12-08 Thread Georg Brandl

Georg Brandl added the comment:

Redirects are no problem in principle, but it should not get out of hand.  A 
simple this was here but now isn't page might be better.

--

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