[issue16040] nntplib: unlimited readline() from connection

[Roundup Robot]

Roundup Robot added the comment:

New changeset 985bda4edf9d by Georg Brandl in branch '3.2':
#16040: fix unlimited read from connection in nntplib.
https://hg.python.org/cpython/rev/985bda4edf9d

--

___
Python tracker 

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

[issue22571] Remove import * recommendations and examples in doc?

[Raymond Hettinger]

Raymond Hettinger added the comment:

Many of these (such as collections, collections.abc, decimal, and itertools) 
use import * as part of the doctest setup and are not part of the generated 
page shown to users.

Also, I don't think you should be aggressive about removing the other example 
that are visible users.   The "import *" construct is part of the language and 
will be encountered occasionally.

For similar reasons, the decimal module was specifically designed for be easily 
usable with "import *" because of the numerous exceptions and rounding flags 
and constants.

There are also some examples such the one in cmd.rst that I put there 
on-purpose.  Changing it would distract from the focus of the example.

With respect to Tkinter (and other gui toolkits as well), I believe that 
"import *" is a standard or common practice, in part because there are so many 
widget names names and widget options.  Changing the example might make it more 
pep8-ish but would result in a good deal of clutter that isn't the normal 
practice.


>  Should we finish the job?  Or leave the issue to
> individual chapter authors?

I vote for leaving any changes to the chapter authors who are in a better 
position to know that they were trying to communicate or who were just setting 
up the doctests.   As discussed above some of the import * are there for a 
reason.

We really need to start thinking of PEP 8 as something we do to our own code 
rather than something that gets inflicted on other people's code.

--
assignee: docs@python -> rhettinger
nosy: +rhettinger

___
Python tracker 

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

[issue22616] Allow connecting AST nodes with corresponding source ranges

[Aivar Annamaa]

New submission from Aivar Annamaa:

Currently lineno and col_offset attributes in AST nodes have confusing roles. 
According to documentation they indicate the starting position of node's source 
text but according to recent developments (#16795) they seem to indicate a 
position most suitable to use in error messages related to that node (rather 
narrow goal IMO).

Another problem is that currently the AST nodes don't contain any information 
about the end position of corresponding source text. Therefore it's very 
difficult to relate nodes with source. One could want to do this for example in 
advanced graphical debuggers (https://bitbucket.org/plas/thonny)

I propose adding new attributes to AST nodes which indicate the corresponding 
source range. If you want to keep nodes lightweight by default, then you could 
also introduce a flag in ast.parse for getting these attributes.

The range could be given either in token indices or in character positions (or 
both). This probably needs more discussion. (I would vote against pointers to 
UTF-8 bytes, as is the case with col_offset currently.)

--
components: Interpreter Core
messages: 229124
nosy: Aivar.Annamaa
priority: normal
severity: normal
status: open
title: Allow connecting AST nodes with corresponding source ranges
type: enhancement
versions: Python 3.5

___
Python tracker 

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

[issue22614] Idle: problem in PyShellEditorWindow.color_breakpoint_text

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

> In msg228954, Serhiy reported a new problem, which I an transferring to this
> new issue, with 2.7 and 3.4 but not 3.5.

I now can reproduce the issue on 3.5 too. But with Lib/test/test_decimal.py, 
not Lib/decimal.py. The difference between 3.4 and 3.5 is that Lib/decimal.py 
is large in 3.4, but it is tiny in 3.5.

I can easy reproduce this issue because my netbook is too slow. When I wait 
some time, and then close editor window, there is no traceback. This time is 
needed to colorize sources in editor window. May be you will reproduce this 
issue when open very large Python file and then quickly close editor window. If 
you have no such file, just concatenate large file with itself many times:

cp 
Lib/test/test_decimal.py+Lib/test/test_decimal.py+Lib/test/test_decimal.py+Lib/test/test_decimal.py+Lib/test/test_decimal.py
 testfile.py

> What system? I do not reproduce on Win 7 with an Oct 9 build.

I believe you will reproduce with enough large file.

> When?  If the command line worked for 2.7 in Jan and fails now, when did it
> start failing?  If the command worked for 3.4 after the Feb patches and
> fails now, same question.

At least with 3.4.0 I don't see this traceback. May it is hidden by the 'can't 
invoke "bind" command' error.

> How? You did not state, but in a later message implied that the file opened
> in the editor and the problem only happened when you closed, as with the
> January report.  However, the traceback says that the problem occurred
> during initialization.  According to Find in Files, the only call to
> color_breakpoint_text is that one __init__ call.  I therefore do not
> understand your msg228957 comment "The code runs up to self.text.update()
> in restore_file_breaks() and runs further only after windows closing."

Sorry for my ugly English. I meant that the interpreter enters in 
self.text.update(), then hangs inside it and exits from it only when the 
windows is closed. I added print's before and after this statement and the 
print after the statement was not executed until I had closed the window.

--

___
Python tracker 

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

[issue22490] Using realpath for __PYVENV_LAUNCHER__ makes Homebrew installs fragile

[Vinay Sajip]

Vinay Sajip added the comment:

> Ronald's change ... changed the stub launcher to no longer do a realpath() on 
> the executable name
> which means that sys.executable contains the path to the Python stub launcher 
> whether or not in a venv

Ronald's change doesn't do a realpath() on the executable name itself, but it 
*does* do a realpath() on the executable's directory. This seems to be what's 
causing the problem in Homebrew builds: there are two levels of symlink at play.

/usr/local/bin/python3.4 -> /usr/local/opt/python3/bin/python3.4 -> 
/usr/local/Cellar/python3/3.4.1_1/Frameworks/Python.framework/Versions/3.4/bin/python3.4

Because of the realpath(), the "wrong" stub launcher (from Homebrew's 
perspective) gets pointed to by the environment variable. At least, if my 
understanding is correct.

https://github.com/pypa/pip/issues/2031#issuecomment-56294599

--

___
Python tracker 

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

[issue22616] Allow connecting AST nodes with corresponding source ranges

[R. David Murray]

R. David Murray added the comment:

If it needs some discussion, perhaps it should be raised on python-ideas first. 
 My reading of #16795 (granted I only skimmed it) was that improving the error 
messages was a *side effect* of making the position information more accurate 
(in the context of static analysis tools).

Also, unless you intend to work on a patch this isn't likely to go anywhere :)

--
nosy: +r.david.murray

___
Python tracker 

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

[issue21603] IDLE SaveAs drops the extension in the prompted filename

[Raymond Hettinger]

Raymond Hettinger added the comment:

> Can you say which version of Tk was in use 
> (it should be in the About IDLE display) and on what platform?

I am using ActiveTcl8.5.15.1 and a latest Python for Macs taken from the 
python.org download page.  I'm running OS X 10.9.5.

--

___
Python tracker 

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

[issue22490] Using realpath for __PYVENV_LAUNCHER__ makes Homebrew installs fragile

[Tim Smith]

Tim Smith added the comment:

We would like to refer to python3 as /usr/local/opt/python3/bin/python3, where 
/usr/local/opt/python3 is a symlink to ../Cellar/python3/3.4.2, and 
/usr/local/Cellar/python3/3.4.2/bin/python3 is a symlink to 
/usr/local/Cellar/python3/3.4.2/Frameworks/Python.framework/Versions/3.4/bin/python3,
 which is the stub launcher for 
/usr/local/Cellar/python3/3.4.2/Frameworks/Python.framework/Versions/3.4/Python.

--

___
Python tracker 

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

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

[Terry J. Reedy]

Terry J. Reedy added the comment:

According to the tcl/tk manual, tcl does ~ expansion on non-'old' Windows.  
However, I cannot tell in what context that could be useful.  Trying to save to 
'~/x.py' fails. "The filename is not valid."  This might be because there is no 
HOME since Windows sometimes uses USERPROFILE instead.  But it seems that any 
name with a path separator is invalid.  At least on Windows, one must specify 
the directory my moving to it in the directory part of the dialog.

If dirname (passed to tk_getSaveFile as initialdir) is not "" after the split 
of self.filename (and I believe it never is), it seems to me possibly a bug 
that tk_getSaveFile looks for a leading ~ for base (passed as initialfile).  
But we cannot affect that.  Serhiy, your suggesting to check self.filename 
(full name) instead of base would be correct if tk did not check base after it 
is split off.  But it does, buggy or not.  The full name of the project file 
that failed was something like F:/python/~templete.py.

Martin and Serhiy.  If I understand correctly, you are both suggesting that 
'~x.py' should become './~x.py' instead of '\~x.py'.  This seems to work 
equally well.  It seems that after ~ processing, the basename gets changed back 
to and displayed as '~x.py' and saving as such works.

You also suggest that at least your version of the escaping should not be 
limited to Windows.  It seems like this might help, and should not hurt, but I 
would like the latter tested on Linux before I commit.  Attached is my patch 
with both changes.

--
stage: needs patch -> patch review
Added file: http://bugs.python.org/file36887/@tilde.patch

___
Python tracker 

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

[issue21603] IDLE SaveAs drops the extension in the prompted filename

[Ned Deily]

Ned Deily added the comment:

Thanks, Raymond.  Upon further investigation, I think you are running into a 
user-wide behavior of OS X rather than specifically a Tk or IDLE issue.  By 
tradition and by default, OS X tries to hide file extensions in file Open or 
Save dialogs; this dates back to the days of Classic Mac OS when there was no 
concept of file extensions.  There is a (non-obvious) user preference in the OS 
X Finder's preferences (Finder -> Preferences -> Advanced -> Show all filename 
extensions) that appears to control whether extensions are displayed by default 
in standard Open and Save dialogs in all applications, not just the Finder.  
(This is with OS X 10.9.x; I believe recent earlier systems behave similarly 
although the details might be slightly different).  When the preference is 
unchecked (the default), then in "Save As" dialogs (in IDLE and other apps) a 
"Hide Extension" option box appears in the lower-left corner of the "Save As" 
popup window and, if enabled, the extension is not displayed as pa
 rt of the file name.  Unchecking the "Hide Extension" box causes the extension 
to appear.  If the Finder "Show all filename extensions" advanced preference is 
checked, then the "Hide Extension" box does not appear on the "Save As" popup 
and the extension is always displayed.  I always have the "Show all filename 
extensions" preference checked so I forgot it was there and its effect on all 
file dialogs.  My guess is that you will want to have it enabled as well and 
then you should always see extensions.

Note that for the initial "Save" of a file in IDLE (as opposed to a "Save As"), 
Python 3.4 supplies a default extension of ".py" which is displayed regardless 
of the Finder preference setting; but Python 2.7 up through the current 2.7.8 
does not supply the default.  That changes in 2.7.9 as Terry has recently 
backported the 3.x change to 2.7 (Issue4832, although the commit message for 
the backport appears in Issue3832 due to a typo).

--

___
Python tracker 

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

[issue22614] Idle: problem in PyShellEditorWindow.color_breakpoint_text

[Terry J. Reedy]

Terry J. Reedy added the comment:

Makes sense now. Large file relative to machine speed + quick close click = 
close (at least to point of destruction of self.text) before initialization is 
complete.  Hence continued initialization fails. Yes, when closing crashed in 
Multicall, would not see initialization after close error.

Switch to .update_idletasks ignores close event, hence initialization gets to 
complete without error.  Do we want to do that?  I think not.  If one tries to 
edit gigabyte file, idle/tk will probably hang and one very well might want 
close event handled.  We could instead wrap test.tag_config with try: except 
AttributeError.  Original authors assumed this would never happen, but its 
occurrence seems like a rare but legitimate possibility that should be allowed 
for.

After sleeping, I will try to reproduce on my fast machine using Windows copy 
command.  3k EditorWindow.py is colorized too fast.

--
versions: +Python 3.5

___
Python tracker 

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

[issue3832] add force_shared Library option to create shared lib even with use_stub=False

[Terry J. Reedy]

Changes by Terry J. Reedy :


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

___
Python tracker 

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

[issue4832] IDLE does not supply a default ext of .py on Windows or OS X for new file saves

[Terry J. Reedy]

Terry J. Reedy added the comment:

Misdirected push message copied here.
New changeset 69cdf71bda12 by Terry Jan Reedy in branch '2.7':
Issue #3832: backport 677a9326b4d4 to 2.7 (and delete some obsolete code).
https://hg.python.org/cpython/rev/69cdf71bda12

--

___
Python tracker 

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

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

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Interesting, that "save as" ./~x.py works when the "~x.py" file exists and 
doesn't work (press the "Save" button has no any effect at all, even doesn't 
close a dialog) when it does not exist on Linux.

--

___
Python tracker 

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

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

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

I meant "save as ~x.py", without the "./" prefix.

--

___
Python tracker 

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

[issue21603] IDLE: Document SaveAs extension display on Mac

[Terry J. Reedy]

Terry J. Reedy added the comment:

(I moved the commit message)

I am assuming that Ned is correct.  I think we should start documenting system 
peculiarities like this.  I changed the title and will leave this open for this 
purpose.

--
title: IDLE SaveAs drops the extension in the prompted filename -> IDLE: 
Document SaveAs extension display on Mac

___
Python tracker 

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

[issue22614] Idle: problem in PyShellEditorWindow.color_breakpoint_text

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Here is a patch which should fix this issue. The same solution already is used 
in restore_file_breaks().

--
keywords: +patch
Added file: http://bugs.python.org/file36888/issue22614.patch

___
Python tracker 

___diff -r 55d50b544a3d Lib/idlelib/PyShell.py
--- a/Lib/idlelib/PyShell.pySun Oct 12 09:53:12 2014 +0200
+++ b/Lib/idlelib/PyShell.pySun Oct 12 13:32:30 2014 +0300
@@ -151,6 +151,9 @@ class PyShellEditorWindow(EditorWindow):
 
 def color_breakpoint_text(self, color=True):
 "Turn colorizing of breakpoint text on or off"
+if self.io is None:
+# can happen if IDLE closes due to the .update() call
+return
 if color:
 theme = idleConf.GetOption('main','Theme','name')
 cfg = idleConf.GetHighlight(theme, "break")
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue22616] Allow connecting AST nodes with corresponding source ranges

[Edward K. Ream]

Edward K. Ream added the comment:

I urge the Python development team to fix this and the related bugs given in 
the Post Script. The lack of an easy way of associating ast nodes with text 
ranges in the original sources is arguably the biggest hole in the Python api.

These bugs have immediate, severe, practical consequences for any tool that 
attempts to regularize (pep 8) or beautify Python code.

Consider the code for PythonTidy:
http://lacusveris.com/PythonTidy/PythonTidy-1.23.python

Every version has had bugs in this area arising from difficult workarounds to 
the hole in the API.  The entire Comments class is a horror directly related to 
these issues.

Consider Aivar's workaround to these bugs:
https://bitbucket.org/plas/thonny/src/8cdaa41aca7a5cc0b31618b6f1631d360c488196/src/ast_utils.py?at=default
See the docstring for def fix_ast_problems.  This is an absurdly difficult 
solution to what should be a trivial problem.

It's impossible to build reliable software using such heroic hacks.  The 
additional bugs listed below further complicate a nightmarish task.

In short, these bugs are *not* minor little nits.  They are preventing the 
development of reliable source-code tools.

Edward K. Ream

P.S. Here are the related bugs:

http://bugs.python.org/issue10769
Allow connecting AST nodes with corresponding source ranges

http://bugs.python.org/issue21295
Python 3.4 gives wrong col_offset for Call nodes returned from ast.parse

http://bugs.python.org/issue18374
ast.parse gives wrong position (col_offset) for some BinOp-s

http://bugs.python.org/issue16806
col_offset is -1 and lineno is wrong for multiline string expressions

EKR

--
nosy: +Edward.K..Ream

___
Python tracker 

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

[issue21986] Idle: disable pickleability of user code objects

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

It would be good to add a test of rpc.dumps().

Current code copies copyreg.dispatch_table at the moment of rpc import. But it 
can be changed later and these change will not affect private copy. For example 
the re module adds pickleability of compiled regex patterns. I think the use 
of ChainMap is more safe solution.

--

___
Python tracker 

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

[issue11664] Add patch method to unittest.TestCase

[Julian Berman]

Julian Berman added the comment:

My opinion is already here re: patch vs patch.object, so I won't repeat it, but 
@Michael, if you really want .patch, are you open to adding .patch_object as 
well?

(Regardless, thanks for working on this Julien.)

--

___
Python tracker 

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

[issue19232] Speed up _decimal import

[Roundup Robot]

Roundup Robot added the comment:

New changeset 75b5617b8dfc by Stefan Krah in branch 'default':
Issue #19232: Fix sys.modules lookup (--without-threads)
https://hg.python.org/cpython/rev/75b5617b8dfc

--

___
Python tracker 

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

[issue22617] Jobs Opportunities in France/Canada

[Georg Brandl]

Changes by Georg Brandl :


--
resolution:  -> not a bug
status: open -> closed

___
Python tracker 

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

[issue22617] spam

[Georg Brandl]

Changes by Georg Brandl :


--
title: Jobs Opportunities in France/Canada -> spam

___
Python tracker 

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

[issue22617] spam

[Georg Brandl]

Changes by Georg Brandl :


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

___
Python tracker 

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

[issue5700] io.FileIO calls flush() after file closed

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Yet one related bug is that flush() isn't called at all if the file was opened 
with closefd=False.

>>> import io, os
>>> class MyIO(io.FileIO):
... def flush(self):
... print('closed:', self.closed)
... 
>>> fd = os.open('test.out', os.O_WRONLY|os.O_CREAT)
>>> f = MyIO(fd, 'wb', closefd=False)
>>> f.close()

The proposed simple patch fixes both bugs.

--
assignee:  -> serhiy.storchaka
components: +IO
stage:  -> patch review
type:  -> behavior
versions: +Python 3.4, Python 3.5 -Python 3.2, Python 3.3
Added file: http://bugs.python.org/file36889/fileio_flush_closed.patch

___
Python tracker 

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

[issue22618] urllib.parse.parse_qsl different results after urllib.parse.unquote

[Alex Vaystikh]

New submission from Alex Vaystikh:

parsing query-string before and after cleaning with urllib.parse.unquote can 
have very different results: 
http://nbviewer.ipython.org/gist/bornio/e112e6d8d04dfed898c8

Perhaps it should be better documented, or make the method more idempotent?

--
components: Library (Lib)
messages: 229144
nosy: Alex.Vaystikh, orsenthil
priority: normal
severity: normal
status: open
title: urllib.parse.parse_qsl different results after urllib.parse.unquote
versions: Python 3.4

___
Python tracker 

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

[issue22618] urllib.parse.parse_qsl different results after urllib.parse.unquote

[Alex Vaystikh]

Changes by Alex Vaystikh :


--
type:  -> behavior

___
Python tracker 

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

[issue22586] urljoin allow_fragments doesn't work

[Roundup Robot]

Roundup Robot added the comment:

New changeset 9eed2e7fa764 by Georg Brandl in branch '3.4':
Closes #22586: clarify meaning of allow_fragments in urlparse.
https://hg.python.org/cpython/rev/9eed2e7fa764

New changeset c2eda29a8ccb by Georg Brandl in branch '2.7':
Closes #22586: clarify meaning of allow_fragments in urlparse.
https://hg.python.org/cpython/rev/c2eda29a8ccb

--
nosy: +python-dev
resolution:  -> fixed
stage:  -> resolved
status: open -> closed

___
Python tracker 

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

[issue22526] file iteration SystemError for huge lines (2GiB+)

[Roundup Robot]

Roundup Robot added the comment:

New changeset be600ea4ad13 by Serhiy Storchaka in branch '2.7':
Fixed and optimized a test of issue #22526.
https://hg.python.org/cpython/rev/be600ea4ad13

--

___
Python tracker 

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

[issue18873] "Encoding" detected in non-comment lines

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

I haven't fixed all bugs in handling encoding cookie yet (there are separate 
issues). Well, this issue can be closed, I'll open new issue about the PEP when 
will be needed. The PEP should be corrected because it affects how other Python 
implementations and other tools handle this.

--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed

___
Python tracker 

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

[issue10769] ast: provide more useful range information

[R. David Murray]

R. David Murray added the comment:

If there was a python-ideas or python-dev thread that resulted in consensus 
approval of this change, can someone post a link to it?

--
nosy: +r.david.murray

___
Python tracker 

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

[issue10769] ast: provide more useful range information

[Sven Brauch]

Sven Brauch added the comment:

Hi,

Mailing list thread: 
https://mail.python.org/pipermail/python-dev/2012-December/123320.html
Discussion on the patch: http://bugs.python.org/issue16795

Greetings,
Sven

--

___
Python tracker 

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

[issue21842] Fix IDLE in unicodeless build

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Most tests which detects no-unicode build, assume that the unicode doesn't 
exist. It can be confused to introduce faked public (non-underscored) name.

--

___
Python tracker 

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

[issue17293] uuid.getnode() MAC address on AIX

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

If there are no objections, I'll commit the patch soon.

--

___
Python tracker 

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

[issue13664] UnicodeEncodeError in gzip when filename contains non-ascii

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


--
assignee:  -> serhiy.storchaka

___
Python tracker 

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

[issue22616] Allow connecting AST nodes with corresponding source ranges

[R. David Murray]

R. David Murray added the comment:

The "python develop team" is a group of volunteers.  What you need to do to 
move this forward is to work on patches (or polishing patches, in the case of 
issue 16806) for open issues for which there has been no objection, and build a 
consensus for fixing the issues for which there has been objection.

Based on the issue 10769 discussion, it sounds like it would be worthwhile to 
go to python-ideas and advocate for the position that the ast parser should be 
enhanced for the kind of use cases you are talking about.  It is possible a PEP 
would be required, but it is also fairly likely that one will not be.

Personally I'm in favor of supporting such use cases, but I'm not involved in 
maintaining any of the relevant code nor do I have an interest in getting 
involved at this time.

--

___
Python tracker 

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

[issue17293] uuid.getnode() MAC address on AIX

[Charles-François Natali]

Charles-François Natali added the comment:

My only comment would be to use subprocess instead of os.popen().

--

___
Python tracker 

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

[issue17293] uuid.getnode() MAC address on AIX

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

This is different issue and can be applied only to 3.5.

--

___
Python tracker 

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

[issue13664] UnicodeEncodeError in gzip when filename contains non-ascii

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

I there are no objections I'll commit the patch soon.

--

___
Python tracker 

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

[issue22618] urllib.parse.parse_qsl different results after urllib.parse.unquote

[R. David Murray]

R. David Murray added the comment:

Isn't it obvious that you have to parse before you unquote?  That's the purpose 
of quoting, after all.  I suppose that is one of those "beginner mind" things 
that is obvious only because I'm an experienced programmer...and that there are 
web servers that do it wrong. 

The docs could definitely use some improvement, since they don't mention that 
parse_qs automatically unquotes the values...it is implicit in the fact that 
urlencode with doseq=true is the inverse, and it encodes them, but it should be 
made explicit.

You also might want to take note of the 'keep_blank_values' attribute, which 
could also use a doc improvement (it isn't obvious from the text what 'blank 
values' are):

>>> parse_qs('a=1&b=2&b=1&a%3Donly_appears_after_unquote', 
>>> keep_blank_values=True)
{'a': ['1'], 'b': ['2', '1'], 'a=only_appears_after_unquote': ['']}

--
assignee:  -> docs@python
components: +Documentation -Library (Lib)
nosy: +docs@python, r.david.murray
versions: +Python 2.7, Python 3.5

___
Python tracker 

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

[issue22618] urllib.parse.parse_qsl different results after urllib.parse.unquote

[R. David Murray]

R. David Murray added the comment:

Eh, that "Isn't it obvious" comment was uncalled for.  Of course it isn't 
obvious, especially when the docs are unclear.

--

___
Python tracker 

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

[issue10769] ast: provide more useful range information

[R. David Murray]

R. David Murray added the comment:

OK, so that patch was committed.  Does that mean this one can be close?  (I'm 
not familiar with the code in question.)

--

___
Python tracker 

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

[issue7665] test_urllib2 and test_ntpath fail if path contains "\"

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Oh-oh, I didn't attached my patch and cleared my working space.

--

___
Python tracker 

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

[issue17293] uuid.getnode() MAC address on AIX

[Charles-François Natali]

Charles-François Natali added the comment:

Why is that a different issue?
The code you *add in this patch* uses os.popen, why not use subprocess instead?

Furthermore, the code catches OSError when calling popen(), but
popen() doesn't raise an exception.

--

___
Python tracker 

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

[issue7665] test_urllib2 and test_ntpath fail if path contains "\"

[STINNER Victor]

STINNER Victor added the comment:

> Oh-oh, I didn't attached my patch and cleared my working space.

Too bad. I did the same mistake yesterday :-/

--

___
Python tracker 

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

[issue17293] uuid.getnode() MAC address on AIX

[STINNER Victor]

STINNER Victor added the comment:

FYI os.popen() now calls subprocess... So it's safe to call directly subprocess.

--
nosy: +haypo

___
Python tracker 

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

[issue17293] uuid.getnode() MAC address on AIX

[STINNER Victor]

STINNER Victor added the comment:

+cmd = 'LC_ALL=C %s %s 2>/dev/null' % (executable, '-ia')

It's safer to use the subprocess module instead of using a shell (see the 
recent Shellshock story) to change the environment variables and to redirect 
stderr. subprocess now has a convinient subprocess.DEVNULL. So it's something 
like:

env = os.environ.copy()
env['LC_ALL'] = 'C'
process = subprocess.Popen([executable, '-ia'], stderr=subprocess.DEVNULL)
...

--

___
Python tracker 

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

[issue22616] Allow connecting AST nodes with corresponding source ranges

[R. David Murray]

R. David Murray added the comment:

OK, reading the followon links just added to issue issue 10769, it seems clear 
you don't need to advocate for the idea of supporting external tools, there 
seems to be consensus agreement for that.  So now it is down to specific 
proposals and patches.  And reviews of patches, those are equally important :)

--

___
Python tracker 

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

[issue10769] ast: provide more useful range information

[Sven Brauch]

Sven Brauch added the comment:

Yes, this issue can be closed.

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

___
Python tracker 

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

[issue7665] test_urllib2 and test_ntpath fail if path contains "\"

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Here is an attempt to reproduce my patch.

--
Added file: http://bugs.python.org/file36890/issue7665.patch

___
Python tracker 

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

[issue22526] file iteration SystemError for huge lines (2GiB+)

[Benjamin Peterson]

Changes by Benjamin Peterson :


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

___
Python tracker 

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

[issue22619] Possible implementation of negative limit for traceback functions

[Dmitry Kazakov]

New submission from Dmitry Kazakov:

This is the possible patch for this proposal: 
https://mail.python.org/pipermail/python-ideas/2014-October/029826.html.

--
components: Library (Lib)
messages: 229167
nosy: vlth
priority: normal
severity: normal
status: open
title: Possible implementation of negative limit for traceback functions
type: enhancement
versions: Python 3.5

___
Python tracker 

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

[issue22619] Possible implementation of negative limit for traceback functions

[Dmitry Kazakov]

Dmitry Kazakov added the comment:

def _extract_tb_or_stack_iter(curr, limit, extractor):
# Distinguish frames from tracebacks (need to import types)
if limit is None:
limit = getattr(sys, 'tracebacklimit', None)
elif limit < 0 and isinstance(curr, types.TracebackType):
seq = list(_extract_tb_or_stack_iter(curr, None, extractor))
yield from seq[limit:]
else:
pass

--

___
Python tracker 

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

[issue22618] urllib.parse.parse_qsl different results after urllib.parse.unquote

[Alex Vaystikh]

Alex Vaystikh added the comment:

It is much clearer after your insight:
- I wasn't aware that 'parse_qs' unquotes values. That's most helpful!
- I had no idea what 'keep_blank_values' were before your example, but now it 
couldn't be more obvious. I know that adding that example to docs would've 
helped me a lot! Maybe others?

Thank you for the help!

--

___
Python tracker 

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

[issue10769] ast: provide more useful range information

[R. David Murray]

R. David Murray added the comment:

Thanks.

--
stage: needs patch -> resolved

___
Python tracker 

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

[issue6478] time.tzset does not reset _strptime's locale time cache

[R. David Murray]

R. David Murray added the comment:

Berker: the patch doesn't apply cleanly any more.  Also, about the 
test_bad_timezone modification...what about the previous check that the tzname 
wasn't in UTC/GMT?  Isn't that still needed?  Or perhaps better yet, an 
additional @run_with_tz decorator?  (Note that I don't really *understand* this 
code, so I may be totally off base here... :)

--
nosy: +r.david.murray
stage: commit review -> needs patch

___
Python tracker 

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

[issue17293] uuid.getnode() MAC address on AIX

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

> The code you *add in this patch* uses os.popen, why not use subprocess 
> instead?

Added code is just modified copy of existing code.

> Furthermore, the code catches OSError when calling popen(), but
popen() doesn't raise an exception.

It can raise an exception in rare cases. The manual of pipe mentions this. The 
code of posix_popen() contains error handling.

> FYI os.popen() now calls subprocess...

Not in 2.7.

> It's safer to use the subprocess module instead of using a shell (see the 
> recent Shellshock story) to change the environment variables and to redirect 
> stderr.

May be, but this is different issue, which is not related to this issue. If 
os.popen should be replaced with subprocess, it should be done even without 
applying the patch of this issue. Please open new issue.

However the recent Shellshock story affects only users of platforms which use 
bash or zsh as system shell (are there any?).

> subprocess now has a convinient subprocess.DEVNULL.

Not in 2.7.

--

___
Python tracker 

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

[issue22619] Possible implementation of negative limit for traceback functions

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

You forgot a patch.

--
nosy: +serhiy.storchaka

___
Python tracker 

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

[issue20596] Support for alternate wcstok syntax for Windows compilers

[R. David Murray]

R. David Murray added the comment:

Moving this back to 'patch review' stage since there's a new patch that hasn't 
been reviewed.  Perhaps this 'ping' will result in someone doing the review 
(so, no, the issue isn't dead, just sleeping :)

--
nosy: +r.david.murray
stage: commit review -> patch review
versions: +Python 3.5 -Python 3.4

___
Python tracker 

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

[issue20335] bytes constructor accepts more than one argument even if the first one is not a string

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


--
priority: normal -> low

___
Python tracker 

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

[issue22619] Possible implementation of negative limit for traceback functions

[Dmitry Kazakov]

Dmitry Kazakov added the comment:

Sorry, first time posting here; accidently pressed Enter.

--
hgrepos: +275

___
Python tracker 

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

[issue22619] Possible implementation of negative limit for traceback functions

[Dmitry Kazakov]

Changes by Dmitry Kazakov :


--
keywords: +patch
Added file: http://bugs.python.org/file36891/9f618a0c2880.diff

___
Python tracker 

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

[issue11973] kevent does not accept KQ_NOTE_EXIT (and other (f)flags)

[Roundup Robot]

Roundup Robot added the comment:

New changeset 35542a32cd54 by R David Murray in branch '3.4':
#11973: add test for previously fixed kevent signed/unsigned bug.
https://hg.python.org/cpython/rev/35542a32cd54

New changeset a028299c9bc2 by R David Murray in branch 'default':
Merge: #11973: add test for previously fixed kevent signed/unsigned bug.
https://hg.python.org/cpython/rev/a028299c9bc2

New changeset 38fe09772b4f by R David Murray in branch '2.7':
#11973: add test for previously fixed kevent signed/unsigned bug.
https://hg.python.org/cpython/rev/38fe09772b4f

--

___
Python tracker 

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

[issue11973] kevent does not accept KQ_NOTE_EXIT (and other (f)flags)

[R. David Murray]

R. David Murray added the comment:

Committed the test finally.  Thanks, David.

--
nosy: +r.david.murray
stage: commit review -> resolved
status: open -> closed

___
Python tracker 

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

[issue21603] IDLE: Document SaveAs extension display on Mac

[Saimadhav Heblikar]

Saimadhav Heblikar added the comment:

>>Saimadhav, can you quickly try Save As with x.py on Linux?

with files of type python: filename stored on disk -> x.py
with all files type : filename stored on disk -> x.py

--

___
Python tracker 

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

[issue17325] improve organization of the PyPI distutils docs

[Roundup Robot]

Roundup Robot added the comment:

New changeset 12536a991007 by R David Murray in branch '3.4':
#17325: Improve distutils PyPI documentation.
https://hg.python.org/cpython/rev/12536a991007

New changeset 8e4afcaa196c by R David Murray in branch 'default':
Merge: #17325: Improve distutils PyPI documentation.
https://hg.python.org/cpython/rev/8e4afcaa196c

New changeset 77098919f819 by R David Murray in branch '2.7':
#17325: Improve distutils PyPI documentation.
https://hg.python.org/cpython/rev/77098919f819

--
nosy: +python-dev

___
Python tracker 

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

[issue22619] Possible implementation of negative limit for traceback functions

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

I have added comments on Retvield (a system for patch reviewing). You should 
receive an email about this.

In general the patch is not optimal. There is no need to load source lines for 
dropped entries.

--

___
Python tracker 

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

[issue17325] improve organization of the PyPI distutils docs

[R. David Murray]

R. David Murray added the comment:

Committed.  I changed the urls as suggested by Antoine.  I also added a link to 
the test pypi page, though I'm not sure what I did is exactly what Antoine had 
in mind.

--
nosy: +r.david.murray
resolution:  -> fixed
stage: commit review -> resolved
status: open -> closed

___
Python tracker 

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

[issue20567] test_idle causes test_ttk_guionly 'can't invoke "event" command: application has been destroyed' messages from Tk

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

These messages are produced because there are non-handled events of destroyed 
root widget. They are handled when update() is called for other root widget. To 
get rid of them you should call update_idletasks() before destroying root 
widget.

--

___
Python tracker 

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

[issue22619] Possible implementation of negative limit for traceback functions

[Dmitry Kazakov]

Dmitry Kazakov added the comment:

Thanks. I replied to your comments at Retvield. I'll update the patch tomorrow.

--

___
Python tracker 

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

[issue22620] pythonw does not open on windows 8.1 x64

[Cristian Baboi]

Changes by Cristian Baboi :


--
components: Windows
nosy: Cristian.Baboi
priority: normal
severity: normal
status: open
title: pythonw does not open on windows 8.1 x64
type: crash
versions: Python 3.5

___
Python tracker 

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

[issue22620] pythonw does not open on windows 8.1 x64

[Cristian Baboi]

New submission from Cristian Baboi:

I've just installed the latest python on windows 8.1 x64 version and it does 
not start. The console version works

--

___
Python tracker 

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

[issue13664] UnicodeEncodeError in gzip when filename contains non-ascii

[Terry J. Reedy]

Terry J. Reedy added the comment:

fine with me

--
stage: patch review -> commit review

___
Python tracker 

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

[issue13096] ctypes: segfault with large POINTER type names

[Roundup Robot]

Roundup Robot added the comment:

New changeset e940bb13d010 by R David Murray in branch '3.4':
#13096: Fix segfault in CTypes POINTER handling of large values.
https://hg.python.org/cpython/rev/e940bb13d010

New changeset 02c9c3204a04 by R David Murray in branch 'default':
Merge: #13096: Fix segfault in CTypes POINTER handling of large values.
https://hg.python.org/cpython/rev/02c9c3204a04

New changeset ff59b0f9e142 by R David Murray in branch '2.7':
#13096: Fix segfault in CTypes POINTER handling of large values.
https://hg.python.org/cpython/rev/ff59b0f9e142

--
nosy: +python-dev

___
Python tracker 

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

[issue13096] ctypes: segfault with large POINTER type names

[R. David Murray]

R. David Murray added the comment:

Committed.

--
nosy: +r.david.murray
resolution:  -> fixed
stage: commit review -> resolved
status: open -> closed
versions: +Python 3.4

___
Python tracker 

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

[issue22620] pythonw does not open on windows 8.1 x64

[Cristian Baboi]

Cristian Baboi added the comment:

I tried both 2.7 and 3.5 versions and none of them works

--

___
Python tracker 

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

[issue22609] Constructors of some mapping classes don't accept `self` keyword argument

[Ethan Furman]

Ethan Furman added the comment:

I will take a look.

--

___
Python tracker 

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

[issue22620] pythonw does not open on windows 8.1 x64

[Mark Lawrence]

Mark Lawrence added the comment:

The purpose of pythonw is to run GUI scripts without having a console.  How are 
you trying to run it, what do you expect to happen and what actually happens?  
I'm assuming that you mean 3.4 as 3.5 hasn't been released yet.

--
nosy: +BreamoreBoy

___
Python tracker 

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

[issue22620] pythonw does not open on windows 8.1 x64

[Vandana Rao]

Vandana Rao added the comment:

I had the same problem. I assume that you are using IDLE. Hopefully it works.

--
nosy: +Vandana.Rao

___
Python tracker 

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

[issue20815] ipaddress unit tests PEP8

[Roundup Robot]

Roundup Robot added the comment:

New changeset 27a02eb9273f by R David Murray in branch '3.4':
#20815: small readability improvements in ipaddress tests.
https://hg.python.org/cpython/rev/27a02eb9273f

New changeset 4c9d27eef892 by R David Murray in branch 'default':
#20815: small readability improvements in ipaddress tests.
https://hg.python.org/cpython/rev/4c9d27eef892

--
nosy: +python-dev

___
Python tracker 

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

[issue20815] ipaddress unit tests PEP8

[R. David Murray]

R. David Murray added the comment:

Since ipaddress is relatively new, the package maintainers approved the patch, 
and it is mostly blank lines (and therefore not a problem with
annotate) I committed this.  Thank Michel.

--
nosy: +r.david.murray
resolution:  -> fixed
stage: commit review -> resolved
status: open -> closed
versions: +Python 3.4

___
Python tracker 

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

[issue13664] UnicodeEncodeError in gzip when filename contains non-ascii

[Roundup Robot]

Roundup Robot added the comment:

New changeset 272c78c9c47e by Serhiy Storchaka in branch '2.7':
Issue #13664: GzipFile now supports non-ascii Unicode filenames.
https://hg.python.org/cpython/rev/272c78c9c47e

--
nosy: +python-dev

___
Python tracker 

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

[issue13664] UnicodeEncodeError in gzip when filename contains non-ascii

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Thank you Terry for the review.

--
resolution:  -> fixed
stage: commit review -> resolved
status: open -> closed

___
Python tracker 

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

[issue22390] test.regrtest should complain if a test doesn't remove temporary files

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

How about regrtest_warn_lost_files.patch?

--

___
Python tracker 

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

[issue22620] pythonw does not open on windows 8.1 x64

[Cristian Baboi]

Cristian Baboi added the comment:

Sorry! I launched pythonw without ascript. I found idle and it works.

--

___
Python tracker 

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

[issue22620] pythonw does not open on windows 8.1 x64

[R. David Murray]

R. David Murray added the comment:

I'm going to close this, then.  If you think there's a documentation issue we 
can reopen it.

--
nosy: +r.david.murray
resolution:  -> not a bug
stage:  -> resolved
status: open -> closed
type: crash -> behavior

___
Python tracker 

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

[issue21603] IDLE: Document SaveAs extension display on Mac

[Ned Deily]

Ned Deily added the comment:

If Raymond concurs that either unchecking the Hide Extensions option or 
checking the Show All Filenmame Extensions preference solves the problem for 
him, we should just close this issue.  In this particular case, the file is 
actually saved with a .py extension; it's just that the extension part is not 
displayed in the standard OS file dialogs.  It is visible via, say, "ls" in a 
UNIX shell.  This is standard OS X application behavior and caters to two 
different styles of file interaction.  All OS X applications that use the 
standard file save dialogs work this way.  See, for example, 
http://support.apple.com/kb/PH13920?viewlocale=en_US&locale=en_US.

--

___
Python tracker 

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

[issue22390] test.regrtest should complain if a test doesn't remove temporary files

[STINNER Victor]

STINNER Victor added the comment:

> This allow other tests which leaks the same file to be reported too.

I don't understand your answer. The "fn.startswith(support.TESTFN)" test is not 
used in get_files().

If I understood correctly, your patch changes two things:

- it now reports files created for tests but not deleted (purpose of this issue)
- it removes more files than before

I don't understand why you want to remove more files than before. You may open 
a different issue, or at least explain the rationale.

I never see any forgotten test file after running tests, so I don't see why you 
are worried because of them. And with your first patch, we will now noticed 
forgotten files, so we can just fix tests.

--

___
Python tracker 

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

[issue20567] test_idle causes test_ttk_guionly 'can't invoke "event" command: application has been destroyed' messages from Tk

[Terry J. Reedy]

Terry J. Reedy added the comment:

You left this issue number off your tkinter test updates, such as f6f098bdb843. 
 By using .update_idletasks instead of .update, are you assuming that their are 
no user events, or that they should be ignormed?

I was wondering whether it would be Ok, if not a good idea, to run tests, and 
maybe Idle itself, with NoDefaultRoot.  I see that is already done for tkinter 
tests. I cannot use exactly the same design as AbstractTkTest since I sometimes 
do other things in setUp/tearDownClass, but I could define create_test_root() 
and delete_test_root() functions in a support file.

--

___
Python tracker 

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

[issue11664] Add patch method to unittest.TestCase

[Michael Foord]

Michael Foord added the comment:

The patch (including lazy import) looks good, and the test looks ok too. I 
still think that patch should be the default instead of patch.object - although 
I wouldn't object to a second method (name?) if there was significant demand.

--

___
Python tracker 

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

[issue22599] traceback: errors in the linecache module at exit

[STINNER Victor]

STINNER Victor added the comment:

> Alternative solution of this issue would be to use "from builtins import open 
> as _open" instead of "import builtins".

Right. I prefer your solution because it's much simpler, it doesn't make 
traceback less usable at exit, and it doesn't need to make assumption on the 
Python status.

Here is a patch implementing this idea with an unit test.

--
Added file: http://bugs.python.org/file36892/traceback_at_exit.patch

___
Python tracker 

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

[issue13664] UnicodeEncodeError in gzip when filename contains non-ascii

[koobs]

koobs added the comment:

This broke a FreeBSD buildbot (koobs-freebsd10), complete log attached.

--
nosy: +koobs
Added file: http://bugs.python.org/file36893/koobs-freebsd10-build-742.log

___
Python tracker 

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

[issue13664] UnicodeEncodeError in gzip when filename contains non-ascii

[Ezio Melotti]

Changes by Ezio Melotti :


--
status: closed -> open

___
Python tracker 

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

[issue11664] Add patch method to unittest.TestCase

[Robert Collins]

Robert Collins added the comment:

FWIW I'd really like to be reducing the TestCase API not extending it - 
particularly since there are lots of good convenient ways of doing this already 
(not least mock.patch/mock.patch.object).

So I'm -0.5 on adding this, as I don't see it adding value. That said, I'll 
happily review for correctness if there is consensus that we want it.

Relatedly I'd like to find some way to let regular functions tie into cleanups 
automatically, so that we don't need helpers like this *at all*. That probably 
needs a PEP though.

--

___
Python tracker 

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

[issue13664] UnicodeEncodeError in gzip when filename contains non-ascii

[koobs]

koobs added the comment:

koobs@10-STABLE-amd64:~ % locale
LANG=
LC_CTYPE="C"
LC_COLLATE="C"
LC_TIME="C"
LC_NUMERIC="C"
LC_MONETARY="C"
LC_MESSAGES="C"
LC_ALL=

--

___
Python tracker 

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

[issue11664] Add patch method to unittest.TestCase

[Ezio Melotti]

Ezio Melotti added the comment:

I'm -0.5 on this as well, and agree that we should try to keep the TestCase API 
small.

On one hand, a patch method available without extra imports would be handy, and 
having this as a generic function/method in unittest seems more natural to me 
than having it in unittest.mock.  On the other hand, adding it to unittest has 
downsides as well: it increases API complexity, adds duplication and possibly 
confusion (people might wonder if they should use TestCase.patch or 
unittest.mock.patch, and if there are any differences).  Adding both .patch and 
.patch_object makes things even worse.

--

___
Python tracker 

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

[issue22559] [backport] ssl.MemoryBIO

[Alex Gaynor]

Alex Gaynor added the comment:

Would you be ok with it going into 2.7.10? The biggest argument in favor of 
this is that it significantly reduces the diff between 2.x and 3.x's SSL 
module, specifically it removes the one major difference between the two of 
them.

--

___
Python tracker 

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

[issue13664] UnicodeEncodeError in gzip when filename contains non-ascii

[Terry J. Reedy]

Terry J. Reedy added the comment:

I rechecked and test_gzip passes on 2.7 Win7, I checked revision history and 
this is the only gzip or test_gzip patch for several months.

--

___
Python tracker 

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

[issue11664] Add patch method to unittest.TestCase

[Martin Panter]

Martin Panter added the comment:

[padding to avoid UTF-8 error with bug tracker]

See also Issue 22374, where an equivalent of “patch.object” is suggested as an 
example context manager for the “contextlib” documentation.

If we added a plain function or context manager rather than a new TestCase 
method, it might avoid the worries about bloating the API. Then it could be a 
generic thing for any kind of testing, and not coupled with the “unittest” 
framework.

About cleanup functions more generally, I think they already tie in well with 
the TestCase.addCleanup() API. Perhaps it could handle general context managers 
as well though, by inheriting an ExitStack.enter_context() method or providing 
an ExitStack attribute.

--

___
Python tracker 

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

[issue21986] Idle: disable pickleability of user code objects

[Terry J. Reedy]

Terry J. Reedy added the comment:

I agree that a test for dumps would be a good idea.  Ditto for hundreds of 
other untested functions.  I don't see this one as a priority.

PyShell imports non-idlelib modules, including re, before idlelib modules, such 
as rpc.  So the re addition is there, though I strongly doubt that compiled 
regexs are every sent to the user process.  Since about 10 different messages 
of different types are sent as part of startup, the fact that Idle does start 
is strong evidence that dumps and much of the rest of rpc is ok.  I would start 
a test of rpc by documenting the protocol and listing the messages types so at 
least one of each could be tested.

--

___
Python tracker 

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