[issue31378] Missing documentation for sqlite3.OperationalError

[Zackery Spytz]

Change by Zackery Spytz :


--
keywords: +patch
pull_requests: +7289
stage: needs patch -> patch review

___
Python tracker 

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

[issue33656] IDLE: Turn on DPI awareness on Windows

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

I don't know whether this is a theoretical or a real concern, but I think this 
fix can wait for 3.7.1.

--

___
Python tracker 

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

[issue33840] connection limit on listening socket in asyncio

[Lisa Guo]

Lisa Guo  added the comment:

Hi Yury, no, I'm not familiar with the other frameworks (libuv doesn't have 
this). I'll need to look into it. If anybody else knows, please comment as well.

--

___
Python tracker 

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

[issue2053] IDLE - standardize dialogs

[Terry J. Reedy]

Terry J. Reedy  added the comment:

I would rather leave this open for the moment as an information resource.  I 
wrote query.py with a base class and subclasses to standardize asking users for 
input.  So some of the patch may be obsolete, but it does have a catalog of 
popup uses.

The patch consistently replaces tkMessagebox.showerror with widget.showerror.  
I have done that in several places, although widget.showerror may just be 
messagebox.showerror.  At least some of my motivation was making testing 
easier, by adding a mock to the test instance instead of temporarily 
monkeypatching messagebox.  If a custom Showerror class were defined in, for 
instant, query.py, "self.showerror = tkMessagebox.showerror" could easily be 
replaced by "self.showerror = query.Showerror.

I also have some interest in consistently handling focus in response to key 
presses.  (There was recently a request for this on idle-dev.)

--
nosy:  -Todd.Rovito, gpolo, kbk, ned.deily, roger.serwy
stage: patch review -> test needed

___
Python tracker 

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

[issue26698] Tk DPI awareness

[Terry J. Reedy]

Terry J. Reedy  added the comment:

The fix for IDLE on Windows made it into 3.6.6rc1 and 3.7.0rc1.

Changing the python.exe and pythonw.exe default dpi settings is a possible 
issue for 3.8, and should have Steve.Dower nosy.

--
resolution:  -> duplicate
stage:  -> resolved
status: open -> closed
superseder:  -> IDLE: Turn on DPI awareness on Windows

___
Python tracker 

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

[issue33828] Add versionchanged notes for string.Formatter

[Xiang Zhang]

Xiang Zhang  added the comment:

Thanks for the review, Eric. :-)

--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed
type:  -> enhancement
versions: +Python 2.7

___
Python tracker 

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

[issue33828] Add versionchanged notes for string.Formatter

[Xiang Zhang]

Xiang Zhang  added the comment:


New changeset fc8ea20c6f8571de96791bc5f7f2d693406024c7 by Xiang Zhang in branch 
'2.7':
[2.7] bpo-33828: Clarify auto-numbering is not available to string.Formatter. 
(GH-7668) (GH-7676)
https://github.com/python/cpython/commit/fc8ea20c6f8571de96791bc5f7f2d693406024c7


--

___
Python tracker 

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

[issue33828] Add versionchanged notes for string.Formatter

[miss-islington]

miss-islington  added the comment:


New changeset 439a9b69985bd4757de1504d19b040f78e384ca4 by Miss Islington (bot) 
in branch '3.6':
bpo-33828: Add missing versionchanged note for string.Formatter. (GH-7668)
https://github.com/python/cpython/commit/439a9b69985bd4757de1504d19b040f78e384ca4


--

___
Python tracker 

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

[issue33828] Add versionchanged notes for string.Formatter

[miss-islington]

miss-islington  added the comment:


New changeset 29fdea5ec2054f18e68733ec83edbd0b855fd8bb by Miss Islington (bot) 
in branch '3.7':
bpo-33828: Add missing versionchanged note for string.Formatter. (GH-7668)
https://github.com/python/cpython/commit/29fdea5ec2054f18e68733ec83edbd0b855fd8bb


--
nosy: +miss-islington

___
Python tracker 

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

[issue33828] Add versionchanged notes for string.Formatter

[Xiang Zhang]

Change by Xiang Zhang :


--
pull_requests: +7288

___
Python tracker 

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

[issue33828] Add versionchanged notes for string.Formatter

[miss-islington]

Change by miss-islington :


--
pull_requests: +7287

___
Python tracker 

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

[issue33828] Add versionchanged notes for string.Formatter

[Xiang Zhang]

Xiang Zhang  added the comment:


New changeset b9d8ad5130e0f77be28a3dec6d468e6470835573 by Xiang Zhang in branch 
'master':
bpo-33828: Add missing versionchanged note for string.Formatter. (GH-7668)
https://github.com/python/cpython/commit/b9d8ad5130e0f77be28a3dec6d468e6470835573


--

___
Python tracker 

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

[issue33828] Add versionchanged notes for string.Formatter

[miss-islington]

Change by miss-islington :


--
pull_requests: +7286

___
Python tracker 

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

[issue33845] Update Doc\make.bat on 2.7 to bring it on par to master version

[Steve Dower]

Steve Dower  added the comment:

Merged!

We don't normally change the name (except to update the GH-7667 reference at 
the end), but once there's more than one commit GitHub will use the PR title 
instead of the commit message.

--
assignee:  -> steve.dower
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

[issue33845] Update Doc\make.bat on 2.7 to bring it on par to master version

[Steve Dower]

Steve Dower  added the comment:


New changeset 42ca8367ceb59021bbf5e5fb4fd0268d3411c0a4 by Steve Dower (Andrés 
Delfino) in branch '2.7':
bpo-33845: Update Doc\make.bat on 2.7 to bring it on par to master version 
(GH-7667)
https://github.com/python/cpython/commit/42ca8367ceb59021bbf5e5fb4fd0268d3411c0a4


--

___
Python tracker 

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

[issue31861] aiter() and anext() built-in functions

[Julian Mehnle]

Change by Julian Mehnle :


--
nosy: +jmehnle

___
Python tracker 

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

[issue33845] Update Doc\make.bat on 2.7 to bring it on par to master version

[Andrés Delfino]

Andrés Delfino  added the comment:

I think I wasn't clear: I see no reason to not merge this right now.

--

___
Python tracker 

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

[issue33845] Update Doc\make.bat on 2.7 to bring it on par to master version

[Andrés Delfino]

Andrés Delfino  added the comment:

Not really. I name all my first commits "Initial commit" as you gals/guys 
rename it anyways, and, also, it's usual to be asked for changes :)

I might change my mind after my initial commit, but I only make pull requests 
when the first commit _seems_ final to me at the time.

PS: Thank you for your words! :)

--

___
Python tracker 

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

[issue30317] test_timeout() of test_multiprocessing_spawn.WithManagerTestBarrier fails randomly on x86 Windows7 3.x buildbot

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

New failure on x86 Windows7 3.x:

http://buildbot.python.org/all/#/builders/58/builds/1000

--
nosy: +pablogsal

___
Python tracker 

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

[issue33845] Update Doc\make.bat on 2.7 to bring it on par to master version

[Steve Dower]

Steve Dower  added the comment:

Hopefully there aren't that many more fixes needed to this scripts - you've 
done a great job clearing up the issues!

Any reason not to merge right now? I notice your commit message simply reads 
"Initial commit"

--

___
Python tracker 

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

[issue33656] IDLE: Turn on DPI awareness on Windows

[Steve Dower]

Steve Dower  added the comment:

There's no option to omit ctypes, though I am aware of some custom builds of 
Python that exclude it. I don't think you need to worry about those being used 
for IDLE (but of course the general policy of the stdlib not relying on ctypes 
remains).

--

___
Python tracker 

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

[issue25246] Alternative algorithm for deque_remove()

[Pablo Galindo Salgado]

Change by Pablo Galindo Salgado :


--
pull_requests: +7285

___
Python tracker 

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

[issue33849] Caught infinite recursion resets the trace function

[Alexander Belopolsky]

Change by Alexander Belopolsky :


--
resolution:  -> duplicate
superseder:  -> Tracing disabled when a recursion error is triggered (even if 
properly handled)

___
Python tracker 

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

[issue33849] Caught infinite recursion resets the trace function

[Alexander Belopolsky]

Change by Alexander Belopolsky :


--
stage: needs patch -> 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

[issue33849] Caught infinite recursion resets the trace function

[Alexander Belopolsky]

Alexander Belopolsky  added the comment:

It looks like this has been known for quite some time.  See issue10933.

--

___
Python tracker 

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

[issue33849] Caught infinite recursion resets the trace function

[Alexander Belopolsky]

New submission from Alexander Belopolsky :

Consider the following code:

import sys

def trace(frame, event, arg):
pass

def f():
f()
sys.settrace(trace)
print(sys.gettrace())
try:
f()
except RuntimeError:
pass
print(sys.gettrace())

When I run it, I get


None

Apparently, the infinite recursion somehow resets the trace function.  This 
interferes with the coverage tools.

--
messages: 319409
nosy: belopolsky
priority: normal
severity: normal
stage: needs patch
status: open
title: Caught infinite recursion resets the trace function
type: behavior
versions: Python 3.6, Python 3.7, Python 3.8

___
Python tracker 

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

[issue33848] Incomplete format string syntax for Exceptions

[Eric V. Smith]

Eric V. Smith  added the comment:

This is because Exception does not define __format__, so object.__format__ is 
being called. object.__format__ does not allow any format specifier to be used. 
You get the same error for any object without its own __format__, when you 
supply a format spec.

>>> 'okay {0:s}'.format(sys)
Traceback (most recent call last):
  File "", line 1, in 
TypeError: unsupported format string passed to module.__format__

The reason this is enforced is because it allows an Execption.__format__ to be 
added in the future, without worrying what existing format specifiers are being 
used. We had a problem adding complex.__format__ because people had supplied 
format specs when formatting complex numbers, assuming they acted like strings. 
But now we've prevented this problem from happening in the future.

If you want to provide str formatting options to something like an exception, 
you should force-convert it to a str first, using !s:

>>> 'okay {0!s:*^10}'.format(Exception('test'))
'okay ***test***'

--
components: +Interpreter Core -ctypes
nosy: +eric.smith
resolution:  -> not a bug
stage:  -> resolved
status: open -> closed
type:  -> behavior

___
Python tracker 

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

[issue33656] IDLE: Turn on DPI awareness on Windows

[Terry J. Reedy]

Terry J. Reedy  added the comment:

I just ran the 3.7.0rc1 installer and here is no option to omit ctypes.

--

___
Python tracker 

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

[issue33839] IDLE tooltips.py: refactor and add docstrings and tests

[Terry J. Reedy]

Change by Terry J. Reedy :


--
Removed message: https://bugs.python.org/msg319406

___
Python tracker 

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

[issue33839] IDLE tooltips.py: refactor and add docstrings and tests

[Terry J. Reedy]

Terry J. Reedy  added the comment:

I just ran the 3.7.0rc1 installer and here is no option to omit ctypes.

--

___
Python tracker 

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

[issue33671] Efficient zero-copy for shutil.copy* functions (Linux, OSX and Win)

[Yury Selivanov]

Change by Yury Selivanov :


--
nosy:  -yselivanov

___
Python tracker 

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

[issue33671] Efficient zero-copy for shutil.copy* functions (Linux, OSX and Win)

[Giampaolo Rodola']

Giampaolo Rodola'  added the comment:

For future reference, as per https://github.com/python/cpython/pull/7160 
discussion, we decided not to use CopyFileEx on Windows and instead increase 
read() buffer size from 16KB to 1MB (Windows only) resulting in a 40.8% speedup 
(instead of 48%). Also copyfileobj() has been optimized on all platforms by 
using readinto()/memoryview()/bytearray().
Updated benchmarks on Windows:

128KB copy (+27%)

without patch:
50 loops, best of 5: 7.69 sec per loop
with patch:
50 loops, best of 5: 5.61 sec per loop

8MB copy (+45.6%)

without patch:
10 loops, best of 5: 20.8 sec per loop
with patch:
20 loops, best of 5: 11.3 sec per loop

512MB copy (+40.8%)

without patch:
1 loop, best of 5: 1.26 sec per loop
with patch:
1 loop, best of 5: 646 msec per loop

--
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

[issue31102] deheader: double #incude of the same file

[Mariatta Wijaya]

Mariatta Wijaya  added the comment:

Closing this. No action item from our side.

--
nosy: +Mariatta
resolution:  -> not a bug
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

[issue33656] IDLE: Turn on DPI awareness on Windows

[Terry J. Reedy]

Terry J. Reedy  added the comment:

I confirmed that IDLE did not import ctypes before this issue.  Serhiy, I 
presume that the new code can be skipped, you are suggesting replacing

import ctypes


with

try:
import ctypes
except ImportError:
pass  # The ctypes-using fix is not essential.
else:


But, in what sense is ctypes optional *on Windows* more than most other 
modules?  I don't see anything in the ctypes doc.  It is normal to depend on 
stdlib modules being present and not wrap imports.

Steve: does the Windows installer have an option to omit ctypes?  I don't 
remember one.  

Anyone: do you know of any CPython Windows distributions that we care about 
that omit ctypes?

Which of the following best describes the situation?
1. This is a theoretical concern that does not justify adding noise to the code.
2. This is a real concern, but so rare that the fix can be deferred to 3.6.7 
and 3.7.1.
3. This is likely common enough that we should ask Ned to cherry-pick the patch 
into 3.6.6 and 3.7.1.

--

___
Python tracker 

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

[issue33656] IDLE: Turn on DPI awareness on Windows

[Terry J. Reedy]

Change by Terry J. Reedy :


--
status: closed -> open

___
Python tracker 

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

[issue33848] Incomplete format string syntax for Exceptions

[Johannes Raggam]

New submission from Johannes Raggam :

The following is valid Python 2:

>>> 'okay {0:s}'.format(Exception('test'))
'okay test'
>>> 'okay {0}'.format(Exception('test'))
'okay test'

The following fails on Python 3.6:

>>> 'okay {0:s}'.format(Exception('test'))
Traceback (most recent call last):
  File "", line 1, in 
TypeError: unsupported format string passed to Exception.__format__

While this doesn't fail:

>>> 'okay {0}'.format(Exception('test'))
'okay test'

--
components: ctypes
messages: 319402
nosy: thet
priority: normal
severity: normal
status: open
title: Incomplete format string syntax for Exceptions
versions: Python 3.6

___
Python tracker 

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

[issue33671] Efficient zero-copy for shutil.copy* functions (Linux, OSX and Win)

[Giampaolo Rodola']

Giampaolo Rodola'  added the comment:


New changeset 4a172ccc739065bb658c75e8929774a8e94af9e9 by Giampaolo Rodola in 
branch 'master':
bpo-33671: efficient zero-copy for shutil.copy* functions (Linux, OSX and Win) 
(#7160)
https://github.com/python/cpython/commit/4a172ccc739065bb658c75e8929774a8e94af9e9


--

___
Python tracker 

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

[issue32108] configparser bug: section is emptied if you assign a section to itself

[Łukasz Langa]

Łukasz Langa  added the comment:


New changeset 33cd058f21d0673253c88cea70388282918992bc by Łukasz Langa (Cheryl 
Sabella) in branch 'master':
bpo-32108: Don't clear configparser values if key is assigned to itself 
(GH-7588)
https://github.com/python/cpython/commit/33cd058f21d0673253c88cea70388282918992bc


--

___
Python tracker 

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

[issue25246] Alternative algorithm for deque_remove()

[Tal Einat]

Tal Einat  added the comment:

IMO both approaches sound better than the existing implementation.  Better to 
choose one than to do nothing.

--
nosy: +taleinat

___
Python tracker 

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

[issue33847] doc: Add '@' operator entry to index

[Andrés Delfino]

Change by Andrés Delfino :


--
keywords: +patch
pull_requests: +7284
stage:  -> patch review

___
Python tracker 

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

[issue33847] doc: Add '@' operator entry to index

[Andrés Delfino]

New submission from Andrés Delfino :

Currently, the index only refers to decorators for the @ symbol. PR adds entry 
for the matrix multiplication operator.

--
assignee: docs@python
components: Documentation
messages: 319398
nosy: adelfino, docs@python
priority: normal
severity: normal
status: open
title: doc: Add '@' operator entry to index
type: enhancement
versions: Python 3.6, Python 3.7, Python 3.8

___
Python tracker 

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

[issue33846] Misleading error message in urllib.parse.unquote

[R. David Murray]

R. David Murray  added the comment:

We don't generally do type checks like that.  This is a specific case of a 
general issue with binary operators and mismatched string/bytes types.  I 
thought there was an open issue for it, but I haven't found it yet.

--
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

[issue33846] Misleading error message in urllib.parse.unquote

[Johannes Raggam]

New submission from Johannes Raggam :

urllib.parse.unquote gives an misleading error message when:

>>> import urllib
>>> urllib.parse.unquote(b'bytesurl')
*** TypeError: a bytes-like object is required, not 'str'

while:

>>> urllib.parse.unquote('texturl')
texturl

The correct behavior is to pass a string/text object to unquote. But passing a 
bytes object gives a misleading error message.

A fix would be to add an assertion in
https://github.com/python/cpython/blob/0250de48199552cdaed5a4fe44b3f9cdb5325363/Lib/urllib/parse.py#L614
 like:

>>> assert isinstance(string, str)

--
components: Library (Lib)
messages: 319396
nosy: thet
priority: normal
severity: normal
status: open
title: Misleading error message in urllib.parse.unquote
type: enhancement
versions: Python 3.6

___
Python tracker 

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

[issue33660] pathlib.Path.resolve() returns path with double slash when resolving a relative path in root directory

[Dong-hee Na]

Dong-hee Na  added the comment:

For reviewers,

I submitted a patch PR 7666.
Please take a look

--
nosy: +corona10

___
Python tracker 

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

[issue33630] test_posix: TestPosixSpawn fails on PPC64 Fedora 3.x

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

Serhiy, I am checking and the only temporary Python object (the result of 
encoding/decoding) that is passed to C library function that I can find is in:


if (!PyArg_ParseTuple(file_action, "OiO"
 ";A open file_action tuple must have 5 elements",
 _obj, , PyUnicode_FSConverter, ,
 , ))
 {
 goto fail;
 }
 errno = posix_spawn_file_actions_addopen(file_actionsp,
 fd, PyBytes_AS_STRING(path), oflag, (mode_t)mode);
  Py_DECREF(path);  /* addopen copied it. */


And according to the manpage of posix_spawn_file_actions_addopen: "The string 
described by path shall be copied by the posix_spawn_file_actions_addopen() 
function.".
The object created by `PyArg_ParseTuple` (path) is only freed after calling 
`posix_spawn_file_actions_addopen` and in that function the contents are copied 
inside 
`file_actionsp`. So it should not be a problem.

In case there is still a problem, could you elaborate more about what object is 
being freed and how is being passed to a C library function?

--

___
Python tracker 

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

[issue33844] Writing capital letters with csvwriter.writerow changes the csv file format

[Eric V. Smith]

Eric V. Smith  added the comment:

This is happening because the string "ID" is special to the SYLK format: 
https://en.wikipedia.org/wiki/SYmbolic_LinK_(SYLK)

--
nosy: +eric.smith

___
Python tracker 

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

[issue33828] Add versionchanged notes for string.Formatter

[Xiang Zhang]

Change by Xiang Zhang :


--
keywords: +patch
pull_requests: +7283
stage: needs patch -> patch review

___
Python tracker 

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

[issue33845] Update Doc\make.bat on 2.7 to bring it on par to master version

[Andrés Delfino]

Change by Andrés Delfino :


--
keywords: +patch
pull_requests: +7282
stage:  -> patch review

___
Python tracker 

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

[issue33845] Update Doc\make.bat on 2.7 to bring it on par to master version

[Andrés Delfino]

New submission from Andrés Delfino :

PR makes the necessary changes to make Doc\make.bat the same as the master 
version, except some small differences.

This should help backports.

--
components: Windows
messages: 319392
nosy: adelfino, paul.moore, steve.dower, tim.golden, zach.ware
priority: normal
severity: normal
status: open
title: Update Doc\make.bat on 2.7 to bring it on par to master version
type: enhancement
versions: Python 2.7

___
Python tracker 

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

[issue33660] pathlib.Path.resolve() returns path with double slash when resolving a relative path in root directory

[Dong-hee Na]

Change by Dong-hee Na :


--
keywords: +patch
pull_requests: +7281
stage: test needed -> patch review

___
Python tracker 

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

[issue33839] IDLE tooltips.py: refactor and add docstrings and tests

[Tal Einat]

Tal Einat  added the comment:

I've started work on this.  My progress can be followed on the bpo-33839 branch 
on my GitHub repo: https://github.com/taleinat/cpython/tree/bpo-33839

--

___
Python tracker 

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

[issue33630] test_posix: TestPosixSpawn fails on PPC64 Fedora 3.x

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

For reference: posix_spawn() was implemented in issue20104.

--

___
Python tracker 

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

[issue33630] test_posix: TestPosixSpawn fails on PPC64 Fedora 3.x

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

Should we merge PR6693 and PR6725 before fixing the memory problem?

--

___
Python tracker 

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

[issue33630] test_posix: TestPosixSpawn fails on PPC64 Fedora 3.x

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

A separate PR is needed for the apparent reusing of freed memory in the 
implementation of posix_spawn. I am currently looking into it.

--

___
Python tracker 

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

[issue33630] test_posix: TestPosixSpawn fails on PPC64 Fedora 3.x

[Pablo Galindo Salgado]

Pablo Galindo Salgado  added the comment:

In PR 7663 I am only fixing the race condition.

--

___
Python tracker 

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

[issue33630] test_posix: TestPosixSpawn fails on PPC64 Fedora 3.x

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

The part that is related to \udcdb\udcdb\udcdb... is a problem in posix_spawn() 
implementation, not in tests. It can't be fixed by changing tests.

--

___
Python tracker 

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

[issue33630] test_posix: TestPosixSpawn fails on PPC64 Fedora 3.x

[STINNER Victor]

STINNER Victor  added the comment:

I'm not sure that the attached PR fixes all race conditions.

--

___
Python tracker 

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

[issue23346] shutil.rmtree doesn't work correctly on FreeBSD.

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

I can't reproduce the problem on FreeBSD 11.1. If it is reproducible only on 
old versions, we can close this issue. If it depends on specific configuration, 
it is still a Python issue.

--

___
Python tracker 

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

[issue33844] Writing capital letters with csvwriter.writerow changes the csv file format

[Serhiy Storchaka]

Serhiy Storchaka  added the comment:

Both CSV and SYLK are text formats used by Excel.

--
nosy: +serhiy.storchaka
resolution:  -> not a bug
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

[issue27397] email.message.Message.get_payload(decode=True) raises AssertionError that "should never happen"

[Tal Einat]

Change by Tal Einat :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +Python 3.6

___
Python tracker 

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

[issue27397] email.message.Message.get_payload(decode=True) raises AssertionError that "should never happen"

[Tal Einat]

Tal Einat  added the comment:


New changeset 63dd1f70c1aa57494802c9b68f9db4918620fc8f by Tal Einat (Miss 
Islington (bot)) in branch '3.6':
bpo-27397: Make email module properly handle invalid-length base64 strings 
(GH-7583) (GH-7665)
https://github.com/python/cpython/commit/63dd1f70c1aa57494802c9b68f9db4918620fc8f


--

___
Python tracker 

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

[issue27397] email.message.Message.get_payload(decode=True) raises AssertionError that "should never happen"

[Tal Einat]

Tal Einat  added the comment:


New changeset 7b82281c80d0064559866afe92f19cae5978c841 by Tal Einat (Miss 
Islington (bot)) in branch '3.7':
bpo-27397: Make email module properly handle invalid-length base64 strings 
(GH-7583) (GH-7664)
https://github.com/python/cpython/commit/7b82281c80d0064559866afe92f19cae5978c841


--

___
Python tracker 

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

[issue33844] Writing capital letters with csvwriter.writerow changes the csv file format

[R. David Murray]

R. David Murray  added the comment:

It doesn't look like this has anything to do with Python.  This sounds like a 
question more appropriate for an Excel help forum.

--
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

[issue33587] inspect.getsource performs unnecessary filesystem stat call

[Pankaj Pandey]

Pankaj Pandey  added the comment:

I simply profiled the script with "python -m cProfile -s cumtime 
inspect_stack_perf.py"
And yes, I should move the linecache check first, that would be a better idea. 
Though getmodule() is also fast enough and reordering it does not significantly 
change the performance (for the common case of no PEP302 modules).

--

___
Python tracker 

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

[issue33844] Writing capital letters with csvwriter.writerow changes the csv file format

[Stefan Vasilev]

New submission from Stefan Vasilev :

import csv
csv.writer(open('./file.csv', 'a', newline='')).writerow(['ID', 0])

If you try to open file.csv with Excel on Windows 10 it says:
The file format and extension of 'file.csv' don't match. The file could be 
corrupted or unsafe. Unless you trust its source don't open it. Do you want to 
open it anyway.

When you click 'Yes', another msgbox appears saying:
Excel has detected that 'file.csv' is a SYLK file, but cannot load it. Either 
the file has errors or it is not a SYLK fyle format. Click OK to try to open 
the file in a different format.

When you click on 'OK' it opens it as expected.

The strange thing is that if you use the following code, there are no problems 
opening it:
import csv
csv.writer(open('./file.csv', 'a', newline='')).writerow(['Id', 0])

--
components: Windows
messages: 319378
nosy: Stefan Vasilev, paul.moore, steve.dower, tim.golden, zach.ware
priority: normal
severity: normal
status: open
title: Writing capital letters with csvwriter.writerow changes the csv file 
format
type: behavior
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

[issue27397] email.message.Message.get_payload(decode=True) raises AssertionError that "should never happen"

[miss-islington]

Change by miss-islington :


--
pull_requests: +7280

___
Python tracker 

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

[issue27397] email.message.Message.get_payload(decode=True) raises AssertionError that "should never happen"

[miss-islington]

Change by miss-islington :


--
pull_requests: +7279

___
Python tracker 

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

[issue27397] email.message.Message.get_payload(decode=True) raises AssertionError that "should never happen"

[Tal Einat]

Tal Einat  added the comment:


New changeset c3f55be7dd012b7e92901627d0b31c21e983ccb4 by Tal Einat in branch 
'master':
bpo-27397: Make email module properly handle invalid-length base64 strings 
(#7583)
https://github.com/python/cpython/commit/c3f55be7dd012b7e92901627d0b31c21e983ccb4


--

___
Python tracker 

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

[issue33630] test_posix: TestPosixSpawn fails on PPC64 Fedora 3.x

[Pablo Galindo Salgado]

Change by Pablo Galindo Salgado :


--
pull_requests: +7278

___
Python tracker 

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

[issue33819] Mention "ordered mapping" instead of "ordered dictionary" in email module

[R. David Murray]

R. David Murray  added the comment:

Message/EmailMessage are intentionally mimicing the full dictionary API, not 
just the Mapping API (with a couple exceptions and several extensions).  But it 
is important to mention that it is ordered, both for the reason INADA stgtes, 
and because the text goes on to discuss the handling of duplicate headers and 
what happens when you add and delete headers.  While this turns out to be 
almost the same as the rules used by a standard python dictionary, that is 
partially a coincidenceother ordering rules could have been chosen.

--
nosy: +r.david.murray
resolution:  -> not a bug
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

[issue23346] shutil.rmtree doesn't work correctly on FreeBSD.

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue27687] Linux shutil.move between mountpoints as root does not retain ownership

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue15100] Race conditions in shutil.copy, shutil.copy2 and shutil.copyfile

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue11989] deprecate shutil.copy2

[Giampaolo Rodola']

Giampaolo Rodola'  added the comment:

If such a thing goes in then copytree() should also be taken into account and 
expose the same parameter. That would probably be inconsistent with 
copytree()'s copy_function parameter though.

--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue10016] shutil.copyfile -- allow sparse copying

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue29699] shutil.rmtree should not fail with FileNotFoundError (race condition)

[Giampaolo Rodola']

Giampaolo Rodola'  added the comment:

+1. It looks reasonable to ignore FileNotFoundError on os.rmdir(),  os.unlink() 
and also os.open() and os.scandir().

--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue30400] Race condition in shutil.copyfile()

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue30044] shutil.copystat should (allow to) copy ownership, and other attributes

[Giampaolo Rodola']

Giampaolo Rodola'  added the comment:

This could be achieved at least on Windows with CopyFileEx [1] and on OSX with 
copyfile(3) + COPYFILE_ALL which copies ACLs (but not users/groups). These are 
(were, in case of CopyFileEx) exposed in 
https://github.com/python/cpython/pull/7160/. Such a new functionality would 
probably deserve a separate copy3() function but it would serve OSX and Windows 
only. I'm not sure how things would work on other POSIX platforms. 

[1] 
https://msdn.microsoft.com/en-us/library/windows/desktop/aa363852(v=vs.85).aspx
[2] http://www.manpagez.com/man/3/copyfile/

--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue33843] Remove deprecated stuff in cgi module

[INADA Naoki]

Change by INADA Naoki :


--
keywords: +patch
pull_requests: +7277
stage:  -> patch review

___
Python tracker 

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

[issue33843] Remove deprecated stuff in cgi module

[INADA Naoki]

New submission from INADA Naoki :

ref:

* cgi.escape is deprecated since 3.3. 67317750aff37489fd3fa279413ef20450e7c808
* parse_qs and parse_qsl are deprecated since 2008, before 3.0 
https://github.com/python/cpython/commit/c469d4c3aa0a66579d1927f0e5d9630b3ea4024f

--
components: Library (Lib)
messages: 319372
nosy: inada.naoki
priority: normal
severity: normal
status: open
title: Remove deprecated stuff in cgi module
versions: Python 3.8

___
Python tracker 

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

[issue20849] add exist_ok to shutil.copytree

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue32073] Add copy_directory_metadata parameter to shutil.copytree

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue32557] allow shutil.disk_usage to take a file path on Windows also

[Giampaolo Rodola']

Giampaolo Rodola'  added the comment:

+1

--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue32347] System Integrity Protection breaks shutil.copystat()

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue30235] Validate shutil supports path-like objects, update docs accordingly

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue33842] Remove tarfile.filemode

[INADA Naoki]

Change by INADA Naoki :


--
keywords: +patch
pull_requests: +7276
stage:  -> patch review

___
Python tracker 

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

[issue32453] shutil.rmtree can have O(n^2) performance on large dirs

[Giampaolo Rodola']

Change by Giampaolo Rodola' :


--
nosy: +giampaolo.rodola

___
Python tracker 

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

[issue33842] Remove tarfile.filemode

[INADA Naoki]

New submission from INADA Naoki :

ref: #14807, ffa1d0b8d53f426f1f9a0d47f25440b8994c4fb0

tarfile.filemode is deprecated since Python 3.3.
And it is not exported by __all__.

--
components: Library (Lib)
messages: 319370
nosy: inada.naoki
priority: normal
severity: normal
status: open
title: Remove tarfile.filemode
versions: Python 3.8

___
Python tracker 

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

[issue33819] Mention "ordered mapping" instead of "ordered dictionary" in email module

[INADA Naoki]

INADA Naoki  added the comment:

I think "dictionary" means just "dict-like", and it doesn't implies "ordered".  
Glossary doesn't say "dictionary" is ordered too.

So "regular dictionary is ordered now" is not enough reason to stop using 
"ordered dictionary".

--
nosy: +inada.naoki

___
Python tracker 

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

[issue33476] String index out of range in get_group(), email/_header_value_parser.py

[Dong-hee Na]

Dong-hee Na  added the comment:

@r.david.murray
Can I get a review for PR 7484?

--

___
Python tracker 

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

[issue1529353] Squeezer - squeeze large output in the interpreter

[Tal Einat]

Tal Einat  added the comment:

Cheryl, thanks for testing this and writing up your thoughts!

There is certainly much room for improvement.

Since it seems that you and Terry like the approach, I'll start working on the 
tooltips as Terry asked, and wait for more input from him before beginning 
additional work.

--

___
Python tracker 

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