[issue4179] pdb: Allow the "list" command to return to the currently debugged line

[Georg Brandl]

Georg Brandl  added the comment:

Implemented in r83260.  Thanks for the patch!

--
resolution:  -> accepted
status: open -> closed

___
Python tracker 

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

[issue9230] pdb.Pdb.checkline() throws AttributeError if called before starting a debug session

[Georg Brandl]

Georg Brandl  added the comment:

Thanks, fixed in r83261.

--
nosy: +georg.brandl
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue9214] Most Set methods of KeysView and ItemsView do not work right

[Eli Bendersky]

Eli Bendersky  added the comment:

Attaching an updated patch, with the trailing whitespace removed. Hope it's 
more acceptable now.

P.S. Please let me know how to detect such issues in future patches.

--
Added file: http://bugs.python.org/file18267/issue9214.2.patch

___
Python tracker 

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

[issue1437051] "continue" in .pdbrc has no effect

[Georg Brandl]

Georg Brandl  added the comment:

Thanks for the patience, this is now fixed in r83262.

I also added a -c option so that you can give the "continue" on the command 
line, not put it in the .pdbrc file.

--
nosy: +georg.brandl
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue9426] Explicitly state lack of support for keyword arguments in builtin functions

[Łukasz Langa]

New submission from Łukasz Langa :

Currently it's somewhat surprising that while the documentation often states 
the default values for certain builtins, invoking the builtin with keyword 
arguments as described in the documentation does not work.

Original discussion: #7447

I'm going through all builtins to see how the keyword arg support looks and I'm 
going to present a patch for Doc/library/builtins.rst that explicitly states 
that builtins don't support this kind of invocation.

Expect a patch later today.

--
assignee: d...@python
components: Documentation
messages: 112048
nosy: d...@python, lukasz.langa, tjreedy
priority: normal
severity: normal
status: open
title: Explicitly state lack of support for keyword arguments in builtin 
functions
versions: Python 2.7, Python 3.2

___
Python tracker 

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

[issue7964] "-m pdb" SyntaxError for "\r\n" formatted py files

[Georg Brandl]

Georg Brandl  added the comment:

This is fixed in py3k trunk by other means; exec() now accepts bytestrings with 
"unusual" newlines.

--
nosy: +georg.brandl
resolution:  -> out of date
status: open -> closed

___
Python tracker 

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

[issue8015] pdb "commands" command throws you into postmortem if you enter an empty command

[Georg Brandl]

Georg Brandl  added the comment:

Fixed in r83265. Thanks!

--
nosy: +georg.brandl
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue9417] Declaring a class creates circular references

[Ray.Allen]

Ray.Allen  added the comment:

> However here's a proposed solution:

* for the __mro__: instead of using a tuple, use a new object that inherits 
from it. This new object should use weak reference for the first item and 
should return the real object (if available) only in __getitem__().

* __objclass__ can should become a property based on weak references.



I'm afraid doing so could cause some public C API change. For example, the 
PyDescr_TYPE, if we implemented all the Descriptor object's d_type(that is the 
__objclass__ of all types of descriptors) based on weakref, we could have all 
the callers who call the descriptor functions to check weather the 
weak-referented class object has gone away.

--

___
Python tracker 

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

[issue1472251] pdb 'run' crashes when the it's first argument is non-string

[Georg Brandl]

Georg Brandl  added the comment:

Committed in trunk r83266; file objects are no longer allowed for exec(), 
therefore I dropped mentions of that.

--
nosy: +georg.brandl
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue7539] unicode exceptions terminate pdb.pm() loop

[Georg Brandl]

Georg Brandl  added the comment:

Thanks, applied to 2.7 branch in r83267.

--
nosy: +georg.brandl
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue8048] doctest assumes sys.displayhook hasn't been touched

[Georg Brandl]

Georg Brandl  added the comment:

Thanks, applied to 3k in r83268.

--
nosy: +georg.brandl
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue9417] Declaring a class creates circular references

[Andrea Colangelo]

Andrea Colangelo  added the comment:

I can confirm this bug should be addressed some way. On my high-traffic server, 
keeping GC enabled causes performance issues due to this bug, and disabling it 
causes an out-of-memory within hours.

--
nosy: +warp10

___
Python tracker 

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

[issue4938] Pdb cannot access doctest source in postmortem

[Georg Brandl]

Georg Brandl  added the comment:

I think this is just too complicated to implement in a sane way.  doctest+pdb 
is just a bit too hacky to be fully supported.

--
nosy: +georg.brandl
resolution:  -> wont fix
status: open -> closed

___
Python tracker 

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

[issue6719] pdb messes up when debugging an non-ascii program

[Georg Brandl]

Georg Brandl  added the comment:

Thanks, applied in r83269.

--
nosy: +georg.brandl
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue5727] doctest pdb readline broken

[Georg Brandl]

Georg Brandl  added the comment:

Fixed in r83271.  I think the first patch's approach is better, since it does 
not affect pdb, only doctest's adaption of pdb.  Consequently, I couldn't use 
the test case; thanks anyway for that!

--
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue9417] Declaring a class creates circular references

[Ray.Allen]

Ray.Allen  added the comment:

Is this due to your adopting of Django?
Are there other guys who using Django suffer the same problem?

--

___
Python tracker 

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

[issue5294] pdb "break" command messes up "continue"

[Georg Brandl]

Georg Brandl  added the comment:

Fixed in r83272 with a slightly different patch.  Thanks very much for the 
report!

--
nosy: +georg.brandl
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue5135] Expose simplegeneric function in functools module

[Nick Coghlan]

Nick Coghlan  added the comment:

A couple more relevant links.

I brought this issue up in the context of a JSON serialisation discussion on 
python-ideas: 
http://mail.python.org/pipermail/python-ideas/2010-July/007854.html

Andrey Popp mentioned his pure Python generic functions library in that thread: 
http://pypi.python.org/pypi/generic

--

___
Python tracker 

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

[issue1553375] Add traceback.print_full_exception()

[Thomas Guettler]

Changes by Thomas Guettler :


--
nosy: +guettli

___
Python tracker 

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

[issue1553375] Add traceback.print_full_exception()

[Thomas Guettler]

Thomas Guettler  added the comment:

It would be very nice if logging.info('...', exc_info=True)
shows the calling/upper frames, too.

--

___
Python tracker 

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

[issue9427] logging.error('...', exc_info=True) should display upper frames, too

[Thomas Guettler]

New submission from Thomas Guettler :

logging.error('...', exc_info=True) only displays the
frames downward. But I often need the upper frames, to debug a problem.

This example shows, that you don't see the "upper" frame in the stactrace. But 
that's information is important.


import logging

def foo():
try:
raise Exception()
except Exception, exc:
logging.error('Exception occured: %s' % exc, exc_info=True)

def upper():
foo()
upper()



===> python tmp/t.py
ERROR:root:Exception occured: 
Traceback (most recent call last):
  File "tmp/t.py", line 6, in foo
raise Exception()
Exception


--
components: Library (Lib)
messages: 112063
nosy: guettli
priority: normal
severity: normal
status: open
title: logging.error('...', exc_info=True) should display upper frames, too
versions: Python 2.6

___
Python tracker 

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

[issue9427] logging.error('...', exc_info=True) should display upper frames, too

[Thomas Guettler]

Thomas Guettler  added the comment:

Related: #1553375

--

___
Python tracker 

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

[issue1553375] Add traceback.print_full_exception()

[Thomas Guettler]

Thomas Guettler  added the comment:

Related #9427

--

___
Python tracker 

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

[issue3143] Make the left sidebar in the doc collapsible

[Georg Brandl]

Georg Brandl  added the comment:

Now used in trunk docs, r83274.

--
resolution: remind -> fixed
status: open -> closed

___
Python tracker 

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

[issue9427] logging.error('...', exc_info=True) should display upper frames, too

[Thomas Guettler]

Thomas Guettler  added the comment:

I tested it only on python 2.6. Can someone please look at more reset versions?

--

___
Python tracker 

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

[issue8797] urllib2 basicauth broken in 2.6.5: RuntimeError: maximum recursion depth exceeded in cmp

[Antoine Pitrou]

Changes by Antoine Pitrou :


--
stage: committed/rejected -> patch review

___
Python tracker 

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

[issue9426] Explicitly state lack of support for keyword arguments in builtin functions

[Éric Araujo]

Éric Araujo  added the comment:

Is this a duplicate of #8350?

--
nosy: +merwok

___
Python tracker 

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

[issue5135] Expose simplegeneric function in functools module

[Éric Araujo]

Changes by Éric Araujo :


--
nosy: +merwok

___
Python tracker 

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

[issue809887] Improve pdb breakpoint feedback

[Georg Brandl]

Georg Brandl  added the comment:

Thanks for the patch; I've committed a modified and extended version in r83275.

--
resolution:  -> accepted
status: open -> closed

___
Python tracker 

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

[issue9315] The trace module lacks unit tests

[Eli Bendersky]

Eli Bendersky  added the comment:

Some more unit tests show that a problem with method names exists in 
Lib/trace.py even in 2.7 (and probably earlier). When tracing an instance 
method with `runfunc`, its name is reported as follows:

ClassName'>.method_name

Instead of:

ClassName.method_name

Alexander, the part of your fix to trace.py in the file_module_function_of 
method fixes this problem. It should be applied to 2.7 as well, though.

--

___
Python tracker 

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

[issue8350] os.mkdir doc comment is incorrect

[Łukasz Langa]

Łukasz Langa  added the comment:

Closing as invalid would not be wise because using named arguments as keywords 
is in Python taken for granted. Cases that are exceptions from this rule should 
be explicitly noted as to avoid confusion, especially for less experienced 
programmers.

Similar case for builtins: #9426

--
nosy: +lukasz.langa

___
Python tracker 

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

[issue9426] Explicitly state lack of support for keyword arguments in builtin functions

[Łukasz Langa]

Łukasz Langa  added the comment:

Same problem, different context. Here we have an already accepted idea (from 
#7447) to extend the documentation by providing an explicit explanation that 
named arguments in builtins are usually positional only, contrary to the 
intuition. I've already commented in #8350 that I believe documentation should 
include a note that one cannot use specific arguments as keywords.

Both issues cover different parts of documentation so they're not duplicates.

--

___
Python tracker 

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

[issue9426] Explicitly state lack of support for keyword arguments in builtin functions

[Éric Araujo]

Éric Araujo  added the comment:

The title of #8350 is outdated. The discussion has shifted from one specific 
function to the general doc problem. Since it contains references and opinions, 
I think it supersedes this one.

--

___
Python tracker 

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

[issue9426] Explicitly state lack of support for keyword arguments in builtin functions

[Eric Smith]

Eric Smith  added the comment:

I agree this should be closed and moved to #8350.

--
nosy: +eric.smith

___
Python tracker 

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

[issue7964] "-m pdb" SyntaxError for "\r\n" formatted py files

[Jason R. Coombs]

Jason R. Coombs  added the comment:

Is it still worthwhile to add the test to the suite to catch a regression?

--

___
Python tracker 

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

[issue4841] io's close() not handling errors correctly

[Brian Curtin]

Changes by Brian Curtin :


--
status: pending -> open

___
Python tracker 

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

[issue9399] Provide a 'print' action for argparse

[Dennis Malcorps]

Dennis Malcorps  added the comment:

Here is a patch that adds a 'write' action to argparse. Usage example:

>>> parser = argparse.ArgumentParser()
>>> parser.add_argument("--license", action="write", message="This file\nis 
>>> licensed under \t GPL")
>>> parser.parse_args(['--license'])
This file
is licensed underGPL

A linebreak will be added after the message. The Output can be redirected with 
the optional 'file=' argument (it defaults to sys.stdout) The parser will then 
exit. 

This is my first patch ever written, so don't be too harsh if it's utter 
garbage^^

--
keywords: +patch
Added file: http://bugs.python.org/file18268/argparse.diff

___
Python tracker 

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

[issue9426] Explicitly state lack of support for keyword arguments in builtin functions

[Łukasz Langa]

Łukasz Langa  added the comment:

Let's do this then :)

Superseded by: #8350

--
status: open -> closed

___
Python tracker 

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

[issue9426] Explicitly state lack of support for keyword arguments in builtin functions

[Éric Araujo]

Changes by Éric Araujo :


--
resolution:  -> duplicate
stage:  -> committed/rejected
superseder:  -> os.mkdir doc comment is incorrect

___
Python tracker 

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

[issue8350] Explicitly state lack of support for keyword arguments in built-in functions

[Éric Araujo]

Changes by Éric Araujo :


--
resolution:  -> accepted
stage:  -> needs patch
title: os.mkdir doc comment is incorrect -> Explicitly state lack of support 
for keyword arguments in built-in functions
type:  -> behavior
versions: +Python 3.2

___
Python tracker 

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

[issue8350] Document lack of support for keyword arguments in C functions

[Éric Araujo]

Changes by Éric Araujo :


--
title: Explicitly state lack of support for keyword arguments in built-in 
functions -> Document lack of support for keyword arguments in C functions

___
Python tracker 

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

[issue8350] Document lack of support for keyword arguments in C functions

[Łukasz Langa]

Łukasz Langa  added the comment:

Currently it's somewhat surprising that while the documentation often states 
the default values for certain builtins, invoking the builtin with keyword 
arguments as described in the documentation does not work.

Original discussion: #7447

I'm going through all builtins to see how the keyword arg support looks and I'm 
going to present a patch for Doc/library/builtins.rst that explicitly states 
that builtins don't support this kind of invocation.

Expect a patch for that case later today. As for `os` et al, the discussion 
goes on.

--

___
Python tracker 

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

[issue8350] Document lack of support for keyword arguments in C functions

[Éric Araujo]

Éric Araujo  added the comment:

You could copy this notice from reference/expressions#calls:

“An implementation may provide built-in functions whose positional parameters 
do not have names, even if they are ‘named’ for the purpose of documentation, 
and which therefore cannot be supplied by keyword.”

An addition to documenting/rest or documenting/markup would be useful too.

--

___
Python tracker 

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

[issue9315] The trace module lacks unit tests

[Alexander Belopolsky]

Alexander Belopolsky  added the comment:

Eli,

Thanks a lot for your continuing effort.  I would like to check in  
your work before any further enhancement.  Please find a reasonable  
stop point and post a commit ready patch preferably with a news file  
entry describing the bug that is fixed.  Note that test additions  
don't get a NEWS entry - I'll describe those in the checking log  
entry.  Unless you think 2.7 backport is tricky, please post only py3k  
patch.  I'll merge the changes.

Thanks.

--

___
Python tracker 

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

[issue8350] Document lack of support for keyword arguments in C functions

[Eric Smith]

Changes by Eric Smith :


--
nosy: +eric.smith

___
Python tracker 

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

[issue7964] "-m pdb" SyntaxError for "\r\n" formatted py files

[Georg Brandl]

Georg Brandl  added the comment:

Yes, you're right. Committed in r83283.

--

___
Python tracker 

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

[issue9315] The trace module lacks unit tests

[Eli Bendersky]

Eli Bendersky  added the comment:

Attaching new patch. This patch is for 2.7 (as you suggested earlier, since 
it's easy to forward-port to py3k with 2to3) and affects Misc/NEWS, 
Lib/trace.py and Lib/test/test_trace.py (which is added).

In Lib/trace.py:
- Fixed the problem with class names (taken from Alexander's patch for py3k) 
when tracing methods. Documented this change in Misc/NEWS. Note that I did not 
include the __annotations__ change because I wasn't sure how it applies to 2.x

Lib/test/test_trace.py:
Added new unit-testing file for the trace module. Changes since the last patch:

- Implemented the changes suggested by Alexander (a few more nitpicks) in 
http://bugs.python.org/issue9315#msg111592, except changing fake* to test* 
because I thought test* in this context is confusing with the main purpose of 
the code (testing), and fake* conveys the meaning pretty well.

- Added tests for tracing instance methods (not yet class methods or static 
methods because the semantics of tracing these aren't 100% clear to me yet), 
and for generators and list comprehensions.

--
Added file: http://bugs.python.org/file18269/issue9315.1.patch

___
Python tracker 

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

[issue9415] SSL issues on "Ubuntu i386" buildbots

[Matthias Klose]

Matthias Klose  added the comment:

see https://lists.ubuntu.com/archives/ubuntu-devel/2010-July/031010.html

--

___
Python tracker 

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

[issue9415] SSL issues on "Ubuntu i386" buildbots

[Antoine Pitrou]

Antoine Pitrou  added the comment:

> see https://lists.ubuntu.com/archives/ubuntu-devel/2010-July/031010.html

Why are they doing that? SSLv2 ciphers are disabled by default anyway
(in newer OpenSSL versions, that is).

If Ubuntu is adding hacks to the libraries they're packaging, I'm not
willing to add a workaround to the test suite in order for the tests to
pass.

--

___
Python tracker 

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

[issue9325] Add an option to pdb/trace/profile to run library module as a script

[Alexander Belopolsky]

Changes by Alexander Belopolsky :


--
nosy: +georg.brandl

___
Python tracker 

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

[issue9428] profile.py bug with the main file being profiled

[Alexander Belopolsky]

New submission from Alexander Belopolsky :

Attached patch, profile-run.diff, fixes the same bug as in issue9323, only in 
the profile rather than the trace module.

Even though the affected code is small, it may be a good idea to share it 
between the trace and profile modules as well as unify command line interfaces. 
 See also issue9325.

--
assignee: belopolsky
components: Library (Lib)
files: profile-run.diff
keywords: easy, patch
messages: 112085
nosy: belopolsky
priority: normal
severity: normal
stage: patch review
status: open
title: profile.py bug with the main file being profiled
type: behavior
versions: Python 3.2
Added file: http://bugs.python.org/file18270/profile-run.diff

___
Python tracker 

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

[issue1503502] Pdb doesn't call flush on its stdout file descriptor

[Georg Brandl]

Georg Brandl  added the comment:

This has been implemented in the course of r83286.

--
nosy: +georg.brandl
resolution:  -> fixed
status: open -> closed

___
Python tracker 

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

[issue6246] Python debugger ignores exception if instructed to return from generator

[Georg Brandl]

Georg Brandl  added the comment:

This is no special behavior with generators.  If you remove the "yield" 
statement, you will get similar output with the exception bubbling up the stack.

--
nosy: +georg.brandl
resolution:  -> works for me
status: open -> closed

___
Python tracker 

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

[issue9264] trace.py documentation is incomplete

[Alexander Belopolsky]

Changes by Alexander Belopolsky :


--
nosy: +belopolsky

___
Python tracker 

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

[issue9423] Error in urllib2.do_open(self, http_class, req)

[Lyle Ross]

Lyle Ross  added the comment:

Thanks for the tip, Kumaran. I'll try it.

Lyle Ross

Office:735G
Email: lyle.r...@us-cert.gov
Ofc Phone: 703.235.5221

--

___
Python tracker 

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

[issue2001] Pydoc interactive browsing enhancement

[Ron Adam]

Ron Adam  added the comment:

I also put in a temporary fix to skip the test file that was causing it to 
crash when doing a search.  It's marked as such and can be removed once the bug 
is fixed.

--

___
Python tracker 

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

[issue9410] Add Unladden Swallow's optimizations to Python 3's pickle.

[Skip Montanaro]

Skip Montanaro  added the comment:

Look around the issues.  I'm pretty sure I worked on the
unbounded size issue at one point.

Skip

--
nosy: +skip.montanaro

___
Python tracker 

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

[issue2001] Pydoc interactive browsing enhancement

[Ron Adam]

Ron Adam  added the comment:

New diff file.

Removed the '-g' option and added a '-b' option.

Using the '-g' option will now bring up pydoc options help.

Added a simple server command prompt with 'b' and 'q' choices
to open a browser and quit the server.

Allow the '-p' option to be use along with the  '-b' option.

Catch the error when the port is already in use and print a nice error instead 
of a traceback with an exception.

The port number now defaults to port 0 and uses a random unused port.

Changed the name of the server to better describe what it does to
http._text_server_thread.py.  Used a leading underscore on the name to
indicate it is a private module and give it time to mature a bit more
before making it public.

Rewrote the _text_server_thread.py example in its __doc__ string so
it passes a doctest.

--
Added file: http://bugs.python.org/file18271/pydoc_server4.diff

___
Python tracker 

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

[issue9315] The trace module lacks unit tests

[Alexander Belopolsky]

Alexander Belopolsky  added the comment:

Eli,

Did you attach a wrong file?  New issue9315.1.patch looks the same as old 
issue9315.1.patch.

--

___
Python tracker 

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

[issue9337] Make float.__str__ behave identically to float.__repr__

[Mark Dickinson]

Mark Dickinson  added the comment:

Alexander Belopolsky pointed out another nice aspect of this change:  after the 
change, let 'numerictype' be any of int, float, complex, Decimal or Fraction, 
and let 'x' be an instance of numerictype.  Then

numerictype(str(x))

recovers x exactly.

See also additional discussion at:

http://mail.python.org/pipermail/python-dev/2010-July/102515.html

The consensus (so far) seems to be in favour of this change.  I'll leave it a 
few more days in case people haven't seen the python-dev thread yet, and then 
assuming that there's no dissent I'll commit.

--
stage: needs patch -> commit review

___
Python tracker 

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

[issue9377] socket, PEP 383: Mishandling of non-ASCII bytes in host/domain names

[David Watson]

David Watson  added the comment:

OK, here are new versions of the original patches.

I've tweaked the docs to make clear that ASCII-compatible
encodings actually *are* ASCII, and point to an explanation as
soon as they're mentioned.

You're right that PyUnicode_AsEncodedString() is the preferable
interface for the argument converter (I think I got
PyUnicode_AsEncodedObject() from an old version of
PyUnicode_FSConverter() :/), but for the ASCII step I've just
short-circuited it and used PyUnicode_EncodeASCII() directly,
since the converter has already checked that the object is of
Unicode type.  For the IDNA step, PyUnicode_AsEncodedString()
should result in a less confusing error message if the codec
returns some non-bytes object one day.

However, the PyBytes_Check isn't to check up on the codec, but to
check for a bytes argument, which the converter also supports.
For that reason, I think encode_hostname would be a misleading
name, but I've renamed it hostname_converter after the example of
PyUnicode_FSConverter, and renamed unicode_from_hostname to
decode_hostname.

I've also made the converter check for UnicodeEncodeError in the
ASCII step, but the end result really is UnicodeError if the IDNA
step fails, because the "idna" codec does not use
UnicodeEncodeError or UnicodeDecodeError.  Complain about that if
you wish :)


I think the example I gave in the previous comment was also
confusing, so just to be clear...

In /etc/hosts (in UTF-8 encoding):

127.0.0.2   €
127.0.0.3   xn--lzg


Without patches:

>>> from socket import *
>>> getnameinfo(("127.0.0.3", 0), 0)
('xn--lzg', '0')
>>> getnameinfo(("127.0.0.2", 0), 0)
('€', '0')
>>> getaddrinfo(*_)
[(2, 1, 6, '', ('127.0.0.3', 0)), (2, 2, 17, '', ('127.0.0.3', 0)), (2, 3, 0, 
'', ('127.0.0.3', 0))]
>>> '€'.encode("idna")
b'xn--lzg'


With patches:

>>> from socket import *
>>> getnameinfo(("127.0.0.3", 0), 0)
('xn--lzg', '0')
>>> getnameinfo(("127.0.0.2", 0), 0)
('\udce2\udc82\udcac', '0')
>>> getaddrinfo(*_)
[(2, 1, 6, '', ('127.0.0.2', 0)), (2, 2, 17, '', ('127.0.0.2', 0)), (2, 3, 0, 
'', ('127.0.0.2', 0))]
>>> '\udce2\udc82\udcac'.encode("idna")
Traceback (most recent call last):
  File "", line 1, in 
  File
  "/home/david/python-patches/python-3/Lib/encodings/idna.py",
  line 167, in encode
result.extend(ToASCII(label))
  File
  "/home/david/python-patches/python-3/Lib/encodings/idna.py",
  line 76, in ToASCII
label = nameprep(label)
  File
  "/home/david/python-patches/python-3/Lib/encodings/idna.py",
  line 38, in nameprep
raise UnicodeError("Invalid character %r" % c)
UnicodeError: Invalid character '\udce2'


The exception at the end demonstrates why surrogateescape strings
don't get confused with IDNs.

--
Added file: http://bugs.python.org/file18272/ascii-surrogateescape-2.diff

___
Python tracker 

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

[issue9377] socket, PEP 383: Mishandling of non-ASCII bytes in host/domain names

[David Watson]

Changes by David Watson :


Added file: http://bugs.python.org/file18273/try-surrogateescape-first-2.diff

___
Python tracker 

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

[issue9429] 2to3 does not rename test.test_support to test.support

[Alexander Belopolsky]

New submission from Alexander Belopolsky :

Even though test.support is an internal module, it is helpful to have a 2to3 
fixer for the 'test_support' to 'support' rename.

My naive attempt to add a fix (see patch pasted below), worked for "from test 
import test_support" but not for "import test.test_support.


Index: Lib/lib2to3/fixes/fix_imports.py
===
--- Lib/lib2to3/fixes/fix_imports.py(revision 83283)
+++ Lib/lib2to3/fixes/fix_imports.py(working copy)
@@ -55,6 +55,7 @@
'UserList' : 'collections',
'urlparse' : 'urllib.parse',
'robotparser' : 'urllib.robotparser',
+   'test.test_support' : 'test.support',
 }
 
 
Index: Lib/lib2to3/fixes/fix_renames.py
===
--- Lib/lib2to3/fixes/fix_renames.py(revision 83283)
+++ Lib/lib2to3/fixes/fix_renames.py(working copy)
@@ -11,6 +11,7 @@
 from ..fixer_util import Name, attr_chain
 
 MAPPING = {"sys":  {"maxint" : "maxsize"},
+   "test": {"test_support": "support"},
   }
 LOOKUP = {}

--
components: 2to3 (2.x to 3.0 conversion tool)
messages: 112095
nosy: belopolsky
priority: normal
severity: normal
stage: needs patch
status: open
title: 2to3 does not rename test.test_support to  test.support
type: behavior
versions: Python 3.2

___
Python tracker 

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

[issue9430] Document str() and repr() of timedelta.

[Alexander Belopolsky]

New submission from Alexander Belopolsky :

Attaching a tentative patch.  I think it is better to document these as 
operations rather than special methods.  It may need to be clarified that "-2 
days, 0:00:22" is timedelta(-2, 22) rather than -timedelta(2, 22), but I cannot 
come up with anything short enough for the ops table.  Maybe this should be 
done in a footnote.

--
assignee: d...@python
components: Documentation
files: datetime.rst.diff
keywords: easy, patch
messages: 112096
nosy: belopolsky, d...@python
priority: normal
severity: normal
status: open
title: Document str() and repr() of timedelta.
versions: Python 3.2
Added file: http://bugs.python.org/file18274/datetime.rst.diff

___
Python tracker 

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

[issue9430] Document str() and repr() of timedelta.

[Alexander Belopolsky]

Changes by Alexander Belopolsky :


--
nosy: +pitrou

___
Python tracker 

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

[issue9430] Document str() and repr() of timedelta.

[Ezio Melotti]

Ezio Melotti  added the comment:

Adding a footnote with an example sounds like a good idea.
There's an odd number of square brackets in the str(t) description.

--
nosy: +ezio.melotti
stage:  -> patch review

___
Python tracker 

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

[issue9301] urllib.quote(None) returns None in 2.7 (raised TypeError before)

[Senthil Kumaran]

Senthil Kumaran  added the comment:

Flox, agree to your patch on checking for unquote({}) and unquote(()).

AttributeError on unquote(None) was a mistake in previous releases and it was 
not intentional. We stand at a chance of correcting that when we are explicitly 
raising an Exception. Perhaps in another bug report to explicitly track it.

Ezio: It was conscious design decision to support both bytes and string for 
these functions. I think, discussed in issue3300

--

___
Python tracker 

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

[issue9431] 2to3 reapplies fix_dict

[George Boutsioukis]

New submission from George Boutsioukis :

This only happens on somewhat complex files, I haven't been able yet to isolate 
the source of this but here goes:

For django trunk, running 2to3 on django/contrib/admin/options.py yields the 
following:

@@ -282,7 +282,7 @@
 if self.declared_fieldsets:
 return self.declared_fieldsets
 form = self.get_form(request, obj)
-fields = [..].keys() + list([...])
+fields = list(list([...]keys())) + list([...])

instead of

+fields = list([...].keys()) + etc.

and there are a couple of more instances in the same file.

However running 2to3 on the single line above, out of the file gives the 
correct result. This is only an issue with the fix_dict fixer.

--
components: 2to3 (2.x to 3.0 conversion tool)
messages: 112099
nosy: gboutsioukis
priority: normal
severity: normal
status: open
title: 2to3 reapplies fix_dict
type: behavior
versions: Python 3.2

___
Python tracker 

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

[issue9432] unquote(None) raises AttributeError - Instead of TypeError

[Senthil Kumaran]

New submission from Senthil Kumaran :

>>> from urllib.parse import unquote
>>> unquote(None)
Traceback (most recent call last):
  File "", line 1, in 
  File "/home/senthil/python/release31-maint/Lib/urllib/parse.py", line 331, in 
unquote
res = string.split('%')
AttributeError: 'NoneType' object has no attribute 'split'

As obvious, the AttributeError above is not an Intentional one and is a 
mistake. TypeError is correct.

Opening this issue subsequent to the discussion in msg12098

--
assignee: orsenthil
messages: 112100
nosy: orsenthil
priority: normal
severity: normal
status: open
title: unquote(None) raises AttributeError - Instead of TypeError
type: behavior
versions: Python 3.2

___
Python tracker 

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

[issue9301] urllib.quote(None) returns None in 2.7 (raised TypeError before)

[Senthil Kumaran]

Senthil Kumaran  added the comment:

Fixed in revision 83294.

Opened Issue9432 just to track that change in Exception behavior.

--
status: open -> closed

___
Python tracker 

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

[issue9315] The trace module lacks unit tests

[Eli Bendersky]

Eli Bendersky  added the comment:

Ouch, sorry Here it is (issue9315.1.patch)

--
Added file: http://bugs.python.org/file18275/issue9315.2.patch

___
Python tracker 

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

[issue9432] unquote(None) raises AttributeError - Instead of TypeError

[Senthil Kumaran]

Senthil Kumaran  added the comment:

The previous hyperlink should have been: Issue9301

Fixed in revision 83294.

As this a change in Exception value, I am not sure, if backport is a good idea. 
If someone find change this breaks their existing setup, please comment here.

--
stage:  -> committed/rejected

___
Python tracker 

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

[issue9315] The trace module lacks unit tests

[Eli Bendersky]

Changes by Eli Bendersky :


Removed file: http://bugs.python.org/file18269/issue9315.1.patch

___
Python tracker 

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

[issue9315] The trace module lacks unit tests

[Eli Bendersky]

Eli Bendersky  added the comment:

[I wish I could edit/delete my older messages]

I've attached a new file, named issue9315.2.patch, with the updated patch. I 
usually add a numeric prefix before the .patch ending to distinguish 
consecutive versions of the same patch, and in this case I just submitted the 
wrong file. 

The new one should be correct, though.

--

___
Python tracker 

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

[issue5412] extend configparser to support mapping access(__*item__)

[Łukasz Langa]

Łukasz Langa  added the comment:

OK, please find attached a preliminary patch for the functionality. This patch 
is complete in terms of implementation with one significant ommision, 
__setitem__ on the parser (adding a section), which needs further discussion.

Because of the development state for this functionality, the patch includes 
some comments that are meant to aid reviewers and the developer (me). They are 
obviously not supposed to be included in the final patch. There is neither any 
ReST documentation and only the BasicTest was ported to check for the mapping 
behaviour as well at the moment. All of these will be corrected once there is 
agreement upon the interface and its implementation.

The mapping interface implementation is using the parser['section']['value'] 
notation that seems to be more popular amongst the developers. The alternative 
implementation presented in the first patch shall not be included because 
"There should be preferably only one obvious way to do it". Implementing both 
approaches in a consistent manner would be needlessly tricky and would bloat 
the code, the tests and the documentation. I believe it's not worth it.

The mapping interface implementation enables getting parser['section'] which 
returns a mutable view on the section. This means that:
- the values are not copied but they are taken from the original parser on 
demand
- the values mutated using this view are mutated in the original parser

The implementation is using the existing high-level API so that subclassing the 
original interface makes the mappings work as expected as well. This has the 
drawback of having potentially weak performance (especially __len__ and 
__iter__ on a section). One difference is the explicit lack of support for the 
`__name__` special key. This is because the existing behaviour of `__name__` is 
very inconsistent and supporting it would only lead to problems. Details here: 
http://mail.python.org/pipermail/python-dev/2010-July/102556.html

The mapping interface was implemented so that it behaves as close to an actual 
dictionary as possible. The existing differences are:
- all sections do include DEFAULTSECT values as well
  -- these values cannot be deleted from the section (because technically they 
are not there). If they are overriden in the section, deleting causes the 
default value to be visible again. Trying to delete a default value causes a 
KeyError
  -- because of this reason, .clear() on a section does not leave the section 
empty
- trying to delete the DEFAULTSECT throws ValueError
- the mapping interface is complete thanks to using the MutableMapping ABC. 
However, there are two methods in the old API that hide the dictionary 
interface:
  -- .get() - this one is unsolvable even when we get to implementing default= 
for it because invoking as .get('string1', 'string2') would be ambiguous. This 
difference shall be explicitly documented in the docs.
  -- .items() - this one could potentially be solvable by providing a special 
case when no arguments are passed. The problem is with existing subclasses 
(possibly 3rd party) which override this method without handling this case. I'm 
still on the fence whether to include a special case here.

As for adding a section with the mapping protocol access, I like the current 
behaviour used for instance in ConfigObj is good for us. They use something 
like:

parser['section'] = {'key1': 'value', 'key2': 'value'}

This should overwrite existing sections as well. If no-one objects, I'll 
implement this behaviour as well.

--
Added file: http://bugs.python.org/file18276/issue5412.diff

___
Python tracker 

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

[issue7567] Messed up terminal after calling curses.initscr() twice.

[Matthias Klose]

Matthias Klose  added the comment:

committed in r83306 (2.7) and r83307 (3.2); verified that the behaviour in 
offlineimap (reported in http://bugs.debian.org/586433 is fixed).
will apply it to 2.6 and 3.1 after a few days.

--
resolution:  -> fixed
status: open -> pending
versions: +Python 2.7, Python 3.1, Python 3.2

___
Python tracker 

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

[issue7245] better Ctrl-C support in pdb (program can be resumed) (issue216067)

[Georg Brandl]

Georg Brandl  added the comment:

I committed the "commands" part of the patch in r83308.

For the SIGINT part, I'm concerned that the SIGINT handler is set
in the Pdb constructor, since it's a processwide setting I think it
should a) be set only before sys.settrace() and b) be restored appropriately 
when debugging is finished.

--
nosy: +georg.brandl

___
Python tracker 

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

[issue7245] better Ctrl-C support in pdb (program can be resumed) (issue216067)

[Georg Brandl]

Georg Brandl  added the comment:

Adding the rest of the patch adapted for py3k trunk as of today.

--
keywords: +patch
Added file: http://bugs.python.org/file18277/pdb-keyboardinterrupt.diff

___
Python tracker 

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

[issue9227] can't import Tkinter / use IDLE after installing Python 2.7 on Mac OS X

[Danielle Dows]

Danielle Dows  added the comment:

I upgraded my working python from 2.5.1 (original install) to 2.7 (for 
macosx10.5) and got this:
** IDLE can't import Tkinter.  Your Python may not be configured for Tk. **

I tried installing the applications, with no change. I tried the macosx10.3 
version, same result. Installed the applications manually, same result. I 
downloaded 2.6.5 and again, same result. My next attempt will be the 2.5.5 
version, but I do not expect and different result. 

I am using Leopard 10.5.8 with all software up to date. Any suggestions?

--
nosy: +yandapanda
type:  -> crash
versions: +Python 2.6

___
Python tracker 

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

[issue9430] Document str() and repr() of timedelta.

[Alexander Belopolsky]

Alexander Belopolsky  added the comment:

First revision: issue9430.diff

--
Added file: http://bugs.python.org/file18278/issue9430.diff

___
Python tracker 

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

[issue9227] can't import Tkinter / use IDLE after installing Python 2.7 on Mac OS X

[Ned Deily]

Ned Deily  added the comment:

@Danielle: Are you sure you are launching the correct IDLE?  In a terminal 
window, enter the following shell commands and report the output:

  cd /Library/Frameworks/Python.framework/Versions/2.7/
  file lib/python2.7/lib-dynload/_tkinter.so
  otool -L lib/python2.7/lib-dynload/_tkinter.so
  bin/idle2.7

That last should launch idle for 2.7.  Quit it then in the Finder try 
double-clicking on the IDLE in /Applications/Python 2.7

--

___
Python tracker 

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

[issue9315] The trace module lacks unit tests

[Alexander Belopolsky]

Alexander Belopolsky  added the comment:

Eli,

I was about to commit issue9315.3.patch, which is a lightly modified version of 
issue9315.2.patch, but it turned out that test_trace cannot be run by 
regrtest.py:

$ ./python.exe Lib/test/regrtest.py test_trace
test_trace
test test_trace failed -- multiple errors occurred; run in verbose mode for 
details
1 test failed:
test_trace

It looks like your fake module machinery is a little too clever.

--
nosy:  -Alexander.Belopolsky

___
Python tracker 

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

[issue9315] The trace module lacks unit tests

[Alexander Belopolsky]

Changes by Alexander Belopolsky :


Added file: http://bugs.python.org/file18279/issue9315.3.patch

___
Python tracker 

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

[issue8620] wrong truncation of last line in cmd.Cmd

[R. David Murray]

R. David Murray  added the comment:

Here is a unit test that demonstrates the problem (against py3k trunk).  I'm 
not convinced that fixing this would be backward incompatible.  

(NB: please no one patch this bug, I'm expecting the patch to get created 
tomorrow during PyOhio.)

--
keywords: +patch
nosy: +r.david.murray
stage:  -> needs patch
Added file: http://bugs.python.org/file18280/cmd-noeol-test.patch

___
Python tracker 

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

[issue8620] wrong truncation of last line in cmd.Cmd

[R. David Murray]

Changes by R. David Murray :


Removed file: http://bugs.python.org/file18280/cmd-noeol-test.patch

___
Python tracker 

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

[issue8620] wrong truncation of last line in cmd.Cmd

[R. David Murray]

Changes by R. David Murray :


Added file: http://bugs.python.org/file18281/cmd-noeol-test.patch

___
Python tracker 

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

[issue9301] urllib.quote(None) returns None in 2.7 (raised TypeError before)

[Ezio Melotti]

Ezio Melotti  added the comment:

> All the quote*/unquote* functions accept both str and bytes (except 
> quote_from_bytes).

According to the documentation [0] unquote() only accepts str (not bytes), so 
checking for b'' in "if string in (b'', '')" seems wrong to me, especially if 
then it returns b''.

[0]: 
http://docs.python.org/dev/py3k/library/urllib.parse.html#urllib.parse.unquote

--
status: closed -> open

___
Python tracker 

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

[issue9315] The trace module lacks unit tests

[Eli Bendersky]

Eli Bendersky  added the comment:

Alexander,

Your issue9315.3.patch file doesn't contain the new test module at all.
 
I'm attaching the updated test_trace.py, fixing the problem with running via 
regrtest. The problem was very simple and not about the fake module - I fixed 
to runctx with globals() and vars() instead of run on tracer (since the called 
function is in my module). The fix is on line 218 (with a docstring fix on line 
213). 

Now `py27 regrtest.py -v test_trace` passes cleanly. I apologize for not trying 
it before. I'll learn for the future :-)

--
Added file: http://bugs.python.org/file18282/test_trace.py

___
Python tracker 

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