[issue29119] Unintentional hard reference assignment in Python version of OrderedDict.move_to_end

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

od_proxy2a.diff LGTM.

--

___
Python tracker 

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

[issue29062] hashlib documentation link error

[Serhiy Storchaka]

Changes by Serhiy Storchaka :


--
nosy: +martin.panter, r.david.murray

___
Python tracker 

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

[issue29102] Add an id field to PyInterpreterState.

[Eric Snow]

Eric Snow added the comment:

int_fast64_t it is then. :)  I vacillated between the options and went with the 
bigger space.  However, you're right that following convention is worth it.

--

___
Python tracker 

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

[issue28815] test_socket fails if /proc/modules is existent but not readable

[patrila]

patrila added the comment:

@Martin Panter: Thanks for your support and merging.

--

___
Python tracker 

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

[issue29102] Add an id field to PyInterpreterState.

[Eric Snow]

Eric Snow added the comment:

I've updated the patch to address Nick's review.  Thanks!

--
Added file: http://bugs.python.org/file46100/interpreter-id-3.diff

___
Python tracker 

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

[issue29093] Windows launcher breaks history in Git Bash

[Evan]

Evan added the comment:

This appears to be a bug with Git Bash, not Python or the launcher. Closing.

--
resolution:  -> third party
status: open -> closed

___
Python tracker 

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

[issue29105] code or doc improvement for logging.handlers.RotatingFileHandler

[Roundup Robot]

Roundup Robot added the comment:

New changeset 4255c7aae85b by Vinay Sajip in branch '3.6':
Closes #29105: Updated RotatingFileHandler documentation.
https://hg.python.org/cpython/rev/4255c7aae85b

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

[issue29120] Rename Python/random.c to Python/bootstrap_hash.c

[Nick Coghlan]

New submission from Nick Coghlan:

This is the last major structural refactoring proposal to come out of the prep 
work on the PEP 432 modular bootstrapping feature development branch: 

* rename Python/random.c -> Python/hash_randomization.c
* rename _PyRandom_Init -> __Py_HashRandomization_Init
* rename _PyRandom_Fini -> __Py_HashRandomization_Fini

The current naming is confusing when working on the interpreter startup 
sequence as the file and API naming looks like it relates to initialising the 
random module, when it has nothing to do with that.

--
assignee: ncoghlan
messages: 284383
nosy: brett.cannon, christian.heimes, eric.snow, ncoghlan, steve.dower
priority: normal
severity: normal
stage: needs patch
status: open
title: Rename Python/random.c to Python/bootstrap_hash.c
type: enhancement
versions: Python 3.7

___
Python tracker 

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

[issue28524] Set default argument of logging.disable() to logging.CRITICAL

[Roundup Robot]

Roundup Robot added the comment:

New changeset 459500606560 by Vinay Sajip in branch 'default':
Closes #28524: added default level for logging.disable().
https://hg.python.org/cpython/rev/459500606560

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

[issue22490] Using realpath for __PYVENV_LAUNCHER__ makes Homebrew installs fragile

[Vinay Sajip]

Vinay Sajip added the comment:

> I tried making a Framework build of Python with the environment variable 
> removed entirely, but I'm not experienced enough with builds on a Mac to know 
> what I need to do to create a proper build.

Given Jason's comment above, it's probably best for one of the Mac-expert 
committers to remove the environment variable altogether from 
Mac/Tools/pythonw.c:main(), and confirm everything still builds and runs OK, 
and then for Tim to confirm that Homebrew isn't adversely impacted by the 
change.

--

___
Python tracker 

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

[issue28443] Logger methods never use kwargs

[Vinay Sajip]

Vinay Sajip added the comment:

I'll close this - as indicated in my comment, there's no need for this 
enhancement, as the already available "extra" parameter performs the same 
function.

--
resolution:  -> wont fix
status: open -> closed

___
Python tracker 

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

[issue15452] Improve the security model for logging listener()

[Vinay Sajip]

Changes by Vinay Sajip :


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

[issue29093] Windows launcher breaks history in Git Bash

[Paul Moore]

Paul Moore added the comment:

Thanks for letting us know

--

___
Python tracker 

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

[issue29120] Rename Python/random.c to Python/bootstrap_hash.c

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

This is not just about the hash randomization. The file also contains 
implementations of _PyOS_URandom() and _PyOS_URandomNonblock().

--
nosy: +serhiy.storchaka

___
Python tracker 

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

[issue29119] Unintentional hard reference assignment in Python version of OrderedDict.move_to_end

[Andra Bogildea]

Andra Bogildea added the comment:

od_proxy.diff and od_proxy2a.diff are the same. Did you meant soft_link = 
proxy(link) in od_proxy2a.diff?

I believe Technique 1 (od_proxy.diff) is preferable, because it reuses the 
proxy object.

Regarding automated tests, I agree with what’s been said. The usage of weak 
references is an internal detail (and name mangling makes it even more 
inconvenient).

--
nosy: +andrabogildea

___
Python tracker 

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

[issue29120] Move hash randomisation initialisation out of Python/random.c

[Nick Coghlan]

Nick Coghlan added the comment:

You're right, I forgot about that part (I was mainly looking at the diffs 
today).

Spelling out the *problem*, rather than just jumping to a specific proposed 
solution, the confusing code layout we currently have is:

* Modules/_randommodule.c provides _random.Random
* Include/pylifecycle.c declares (amongst other things):
  * _PyRandom_Init
  * _PyRandom_Fini
  * _PyOS_URandom
  * _PyOS_URandomNonBlock
* Python/random.c implements all four of those APIs

I don't believe the latter two APIs existed back when I started the PEP 432 
feature branch, which is why I had forgotten about them in this context.

Having the OS independent _PyOS_URandom APIs in a file called "Python/random.c" 
is reasonable, and there's even a legitimate connection to the random module 
there by way of random.SystemRandom.

So the only confusing aspect is also having the hash randomisation 
initialisation functions in there, rather than in their own file, with more 
appropriate API names.

--
title: Rename Python/random.c to Python/bootstrap_hash.c -> Move hash 
randomisation initialisation out of Python/random.c

___
Python tracker 

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

[issue29120] Move hash randomisation initialisation out of Python/random.c

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Actually _PyRandom_Fini() has a relation to _PyOS_URandom APIs. It releases 
resources acquired by _PyOS_URandom APIs. It shouldn't be renamed to 
_Py_HashRandomization_Fini.

_PyRandom_Init() just initializes _Py_HashSecret. But it can't be implemented 
via _PyOS_URandom APIs. It uses a static function from this file. If move it to 
other place, we should add new _PyOS_URandom API function. I don't think this 
would make the code cleaner.

--

___
Python tracker 

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

[issue28518] execute("begin immediate") throwing OperationalError

[Marcus]

Changes by Marcus :


--
nosy: +callidomus

___
Python tracker 

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

[issue28518] execute("begin immediate") throwing OperationalError

[Berker Peksag]

Berker Peksag added the comment:

I only did a quick review, but Aviv's latest patch looks reasonable to me. 
Thanks!

And yes, we can add CheckVacuum back if it's not already tested.

--

___
Python tracker 

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

[issue29093] Windows launcher breaks history in Git Bash

[Evan]

Evan added the comment:

No problem, thanks for your time.

--

___
Python tracker 

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

[issue29120] Move hash randomisation initialisation out of Python/random.c

[STINNER Victor]

Changes by STINNER Victor :


--
nosy: +haypo

___
Python tracker 

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

[issue29120] Move hash randomisation initialisation out of Python/random.c

[STINNER Victor]

STINNER Victor added the comment:

Nick: As Serhiy explained, IMHO you misunderstood _PyRandom_Init() and 
_PyRandom_Fini(), I suggest to close the issue.

--

___
Python tracker 

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

[issue29102] Add an id field to PyInterpreterState.

[STINNER Victor]

STINNER Victor added the comment:

I would prefer to not use "fast" C types because they are not well supported. 
For example, ctypes has ctypes.c_int64 but no ctypes.c_int_fast64.

Previous work adding an unique identifier: PEP 509
https://www.python.org/dev/peps/pep-0509/#integer-overflow

--
nosy: +haypo

___
Python tracker 

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

[issue29121] sqlite3 Controlling Transactions documentation not updated

[Aviv Palivoda]

New submission from Aviv Palivoda:

commit 284676cf2ac8 changed the sqlite3 module so it will no longer implicitly 
commit an open transaction before DDL statements. The docs have been updated 
but there is still an incorrect paragraph that has not been removed. 
Attached is a patch that remove the old paragraph.

--
files: sqlite-transaction-doc.patch
keywords: patch
messages: 284396
nosy: berker.peksag, ghaering, palaviv, serhiy.storchaka
priority: normal
severity: normal
status: open
title: sqlite3 Controlling Transactions documentation not updated
versions: Python 3.6, Python 3.7
Added file: http://bugs.python.org/file46101/sqlite-transaction-doc.patch

___
Python tracker 

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

[issue28518] execute("begin immediate") throwing OperationalError

[Aviv Palivoda]

Aviv Palivoda added the comment:

Adding the vacuum test can't actually happen because the change in commit 
284676cf2ac8 changed the API and we don't commit before non DML statements. I 
opened a issue that will clarify the docs (#29121).

--

___
Python tracker 

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

[issue29082] In 2.7.13, _ctypes.LoadLibrary no longer accepts Unicode objects

[Chi Hsuan Yen]

Changes by Chi Hsuan Yen :


Removed file: 
http://bugs.python.org/file46055/LoadLibrary_revert_arg_parsing.patch

___
Python tracker 

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

[issue29082] In 2.7.13, _ctypes.LoadLibrary no longer accepts Unicode objects

[Chi Hsuan Yen]

Chi Hsuan Yen added the comment:

I finally get Windows builds working. Here's the patch.

--
Added file: http://bugs.python.org/file46102/issue29082.patch

___
Python tracker 

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

[issue29082] In 2.7.13, _ctypes.LoadLibrary no longer accepts Unicode objects

[Chi Hsuan Yen]

Chi Hsuan Yen added the comment:

Oops, Parser/asdl_c.py shouldn't be included. Here's the correct patch

--
Added file: http://bugs.python.org/file46103/issue29082.patch

___
Python tracker 

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

[issue26267] UUID docs should say how to get "standard form"

[Berker Peksag]

Berker Peksag added the comment:

Thanks, Ammar.

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

___
Python tracker 

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

[issue26267] UUID docs should say how to get "standard form"

[Roundup Robot]

Roundup Robot added the comment:

New changeset 1a25c639f81e by Berker Peksag in branch '3.5':
Issue #26267: Improve uuid.UUID documentation
https://hg.python.org/cpython/rev/1a25c639f81e

New changeset c9e944cc6301 by Berker Peksag in branch '3.6':
Issue #26267: Merge from 3.5
https://hg.python.org/cpython/rev/c9e944cc6301

New changeset 8b19c2a1b197 by Berker Peksag in branch 'default':
Issue #26267: Merge from 3.6
https://hg.python.org/cpython/rev/8b19c2a1b197

--
nosy: +python-dev

___
Python tracker 

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

[issue29122] set() bug

[styg]

New submission from styg:

set() method leads to this error:
a=[1,2,3,3]
b=set(a)
print b
>> set([1,2,3]) # should be [1,2,3]

--
messages: 284402
nosy: styg
priority: normal
severity: normal
status: open
title: set() bug
type: behavior
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

[issue29122] set() bug

[Ammar Askar]

Ammar Askar added the comment:

Why would the expected output be [1, 2, 3]? That is the list notation for 
having a list that contains the elements 1, 2 and 3.

--
nosy: +ammar2

___
Python tracker 

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

[issue29123] CheckSqlTimestamp is fragile

[Berker Peksag]

New submission from Berker Peksag:

==
FAIL: CheckSqlTimestamp (sqlite3.test.types.DateTimeTests)
--
Traceback (most recent call last):
  File 
"/root/buildarea/3.6.angelico-debian-amd64/build/Lib/sqlite3/test/types.py", 
line 391, in CheckSqlTimestamp
self.assertEqual(ts.year, now.year)
AssertionError: 2016 != 2017

--

See 
http://buildbot.python.org/all/builders/AMD64%20Debian%20root%203.6/builds/93/steps/test/logs/stdio
 for details.

In Lib/sqlite3/test/types.py, there is a comment saying

# SQLite's current_timestamp uses UTC time, while datetime.datetime.now() 
uses local time.

Unless I'm missing something, there is no reason not to use datetime.utcnow(). 
That would make the test more robust.

--
components: Tests
messages: 284404
nosy: berker.peksag
priority: normal
severity: normal
stage: needs patch
status: open
title: CheckSqlTimestamp is fragile
type: behavior
versions: Python 3.5, Python 3.6, Python 3.7

___
Python tracker 

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

[issue29124] Freeze fails to compile on windows

[Liran Ringel]

New submission from Liran Ringel:

The patch fixes 3 bugs:
1. Include Python.h in frozen_dllmain.c - needed for Py_ARRAY_LENGTH symbol
2. DL_IMPORT is used in checkextensions_win32.py, but it's deprecated and not 
defined anymore. Replaced it with __declspec(dllimport)
3. Fix python lib path in winmakemakefile.py

There is one more thing left to do to make it work, but it already has an open 
issue:
http://bugs.python.org/issue28068

--
components: Demos and Tools, Windows
files: fix-freeze-on-windows.patch
keywords: patch
messages: 284405
nosy: Liran Ringel, paul.moore, steve.dower, tim.golden, zach.ware
priority: normal
severity: normal
status: open
title: Freeze fails to compile on windows
type: compile error
versions: Python 3.5, Python 3.6, Python 3.7
Added file: http://bugs.python.org/file46104/fix-freeze-on-windows.patch

___
Python tracker 

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

[issue29102] Add an id field to PyInterpreterState.

[Eric Snow]

Eric Snow added the comment:

Thanks for pointing that out, Victor.  Given the precedent I switched to using 
int64_t.  The patch actually uses PY_INT64_T, but I didn't see a reason to use 
int64_t directly.  FWIW, there *are* a few places that use int_fast64_t, but 
they are rather specialized and I didn't want this patch to be a place where I 
had to deal with setting a more general precedent. :)

--
Added file: http://bugs.python.org/file46105/interpreter-id-4.diff

___
Python tracker 

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

[issue29122] set() bug

[Raymond Hettinger]

Raymond Hettinger added the comment:

Sorry Styg, but this is the expected behavior.

To achieve what you wanted, write this is instead:

   print list(set(a))

--
nosy: +rhettinger
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

[issue29125] Shell injection via TIX_LIBRARY when using tkinter.tix

[symphorien]

New submission from symphorien:

The tkinter.tix module looks for a Tix installation in the directory specified 
by the TIX_LIBRARY environment variable, but blindly trusts that it is a path 
in the filesystem. This enables a shell injection :

TIX_LIBRARY='/dev/null}; exec gsimplecal;' python2 -c "from Tix import Tk; Tk()"

or 

TIX_LIBRARY='/dev/null}; exec gsimplecal;' python3 -c "from tkinter.tix import 
Tk; Tk()"

Python execs gsimplecal, waits on its completion and then raises a 
tkinter.TclError.

The offending code is here : 
https://github.com/python/cpython/blob/master/Lib/tkinter/tix.py#L204-L208

--
components: Tkinter
messages: 284408
nosy: symphorien
priority: normal
severity: normal
status: open
title: Shell injection via TIX_LIBRARY when using tkinter.tix
type: security
versions: Python 2.7, Python 3.6

___
Python tracker 

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

[issue29120] Move hash randomisation initialisation out of Python/random.c

[Raymond Hettinger]

Raymond Hettinger added the comment:

FWIW, I concur with Nick's initial observation that the current "API naming 
looks like it relates to initialising the random module, when it has nothing to 
do with that."

It would be nice if Python/random.c were renamed to something else that 
indicated its relation to urandom and to hash secrets.

--
nosy: +rhettinger

___
Python tracker 

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

[issue29119] Unintentional hard reference assignment in Python version of OrderedDict.move_to_end

[Roundup Robot]

Roundup Robot added the comment:

New changeset 19376765d7c3 by Raymond Hettinger in branch '3.5':
Issue #29119: Fix weakref in OrderedDict.move_to_end(). Work by Andra Bogildea.
https://hg.python.org/cpython/rev/19376765d7c3

--
nosy: +python-dev

___
Python tracker 

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

[issue29119] Unintentional hard reference assignment in Python version of OrderedDict.move_to_end

[Roundup Robot]

Roundup Robot added the comment:

New changeset cd811b867393 by Raymond Hettinger in branch '3.6':
Issue #29119:  Merge in ACK from 3.5 branch
https://hg.python.org/cpython/rev/cd811b867393

--

___
Python tracker 

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

[issue29119] Unintentional hard reference assignment in Python version of OrderedDict.move_to_end

[Raymond Hettinger]

Raymond Hettinger added the comment:

Nice work Andra.  Thank you.

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

[issue29123] CheckSqlTimestamp is fragile

[Roundup Robot]

Roundup Robot added the comment:

New changeset bb64ae455490 by Berker Peksag in branch '3.5':
Issue #29123: Make CheckSqlTimestamp more robust
https://hg.python.org/cpython/rev/bb64ae455490

New changeset 0331420d6cba by Berker Peksag in branch '3.6':
Issue #29123: Merge from 3.5
https://hg.python.org/cpython/rev/0331420d6cba

New changeset ad06f9acff92 by Berker Peksag in branch 'default':
Issue #29123: Merge from 3.6
https://hg.python.org/cpython/rev/ad06f9acff92

--
nosy: +python-dev

___
Python tracker 

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

[issue29123] CheckSqlTimestamp is fragile

[Berker Peksag]

Changes by Berker Peksag :


--
resolution:  -> fixed
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

[issue29093] Windows launcher breaks history in Git Bash

[Eryk Sun]

Eryk Sun added the comment:

git-bash.exe runs bash.exe via cmd.exe, which means the console (conhost.exe) 
ends up with 5 attached processes:

cmd.exe
bash.exe
bash.exe
py.exe
python.exe

The default console configuration uses 4 history buffers, so by the time 
python.exe runs, they've all been used up. You can change this in the Alt+Space 
control menu's "Properties" dialog. This should modify the settings associated 
with cmd.exe, which are stored at "HKCU\Console\%SystemRoot%_system32_cmd.exe". 
In the upper right section increase the number of buffers to 32. You can also 
increase the size. I increase it to 500 commands. Start a new console via "Git 
Bash Here" to confirm that the new settings are remembered.  

Also modify the default configuration used for new programs, which is stored at 
"HKCU\Console". Back on the Alt+Space control menu, choose "Defaults" and 
similarly modify the size and number of buffers. 

Note that if you run a console program using a shortcut, such as the default 
"Command Prompt" shortcut, the console configuration is stored in the .LNK 
shortcut file instead of the registry.

--
nosy: +eryksun

___
Python tracker 

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

[issue29126] Fix comments about _PyThreadState_Current

[Jim Nasby]

New submission from Jim Nasby:

Attached patch fixes two comments in dictobject.c referring to no longer 
exposed _PyThreadState_Current variable. I also tweaked the grammar for that 
issue in NEWS, since it came up in my search. Apologies if that's considered 
gratuitous noise.

--
components: Interpreter Core
files: _PyThreadState_Current_comments.diff
keywords: patch
messages: 284415
nosy: Jim Nasby
priority: normal
severity: normal
status: open
title: Fix comments about _PyThreadState_Current
type: enhancement
versions: Python 3.7
Added file: 
http://bugs.python.org/file46106/_PyThreadState_Current_comments.diff

___
Python tracker 

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

[issue29123] CheckSqlTimestamp is fragile

[Roundup Robot]

Roundup Robot added the comment:

New changeset 1d5b4426fe87 by Berker Peksag in branch '2.7':
Issue #29123: Make CheckSqlTimestamp more robust
https://hg.python.org/cpython/rev/1d5b4426fe87

--

___
Python tracker 

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

[issue29126] Fix comments about _PyThreadState_Current

[Berker Peksag]

Changes by Berker Peksag :


--
nosy: +haypo

___
Python tracker 

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

[issue29120] Move hash randomisation initialisation out of Python/random.c

[Nick Coghlan]

Nick Coghlan added the comment:

On 1 Jan 2017 05:00, "Raymond Hettinger"  wrote:

It would be nice if Python/random.c were renamed to something else that
indicated its relation to urandom and to hash secrets.

Right, based on Victor & Serhiy's feedback, the split I would now propose
 is:

- Include/pyurandom.h and Python/pyurandom.c for the OS urandom APIs (with
related init/fini functions)
- Python/hash_randomization.c for the hash seeding functions  (while this
process *uses* pyurandom, it isn't part of *providing* that interface)

--

___
Python tracker 

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

[issue29127] Incorrect reference names in asyncio.subprocess documentation

[Eric Ahn]

New submission from Eric Ahn:

On this page of the documentation 
https://docs.python.org/3/library/asyncio-subprocess.html it seems that some of 
the reference names are incorrect.

Namely, asyncio.subprocess.PIPE is referred to as 
asyncio.asyncio.subprocess.PIPE (along with STDOUT and DEVNULL), plus 
asyncio.subprocess.Process is referred to as asyncio.asyncio.subprocess.Process.

This is reflected in the permalinks as well as when one tries to reference 
these via intersphinx (which is how I discovered it; I was trying to link to 
this page and was failing).

--
assignee: docs@python
components: Documentation
messages: 284418
nosy: Eric Ahn, docs@python
priority: normal
severity: normal
status: open
title: Incorrect reference names in asyncio.subprocess documentation
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

[issue28911] Clarify the behaviour of assert_called_once_with

[Berker Peksag]

Changes by Berker Peksag :


--
nosy: +berker.peksag, michael.foord -153957
stage:  -> patch review
type: enhancement -> behavior
versions:  -Python 3.3, Python 3.4

___
Python tracker 

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

[issue28964] AST literal_eval exceptions provide no information about line number

[Berker Peksag]

Berker Peksag added the comment:

Thanks for the patch, Steve!.

This looks like a reasonable request to me, but 1) we can only make this change 
in 3.7 2) we need a test case for the new message 3) you can use f-strings 
instead of str.format()

--
nosy: +berker.peksag
stage:  -> patch review
type: behavior -> enhancement
versions: +Python 3.7 -Python 2.7

___
Python tracker 

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

[issue29071] IDLE doesn't highlight f-strings properly

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

The regular expression can be simplified using local flags:

-stringprefix = (r"(\br|R|u|U|f|F|fr|Fr|fR|FR|rf|rF|Rf|RF"
-"|b|B|br|Br|bR|BR|rb|rB|Rb|RB)?")
+stringprefix = r"(?i:\br|u|f|fr|rf|b|br|rb)?"

--
nosy: +serhiy.storchaka

___
Python tracker 

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

[issue29094] Regression in zipfile writing in 2.7.13

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

The specification of ZIP files is not pretty clear, but the unzip utility first 
try to interpret offsets relatively to the start of the archive that can be 
different from the start of the file. Go's standard archive/zip package 
interprets offsets as absolute file positions and doesn't support concatenated 
ZIP files.

In any case there is a regression. Proposed patch partially reverts issue26293. 
Offsets in created ZIP file are now absolute when create with modes 'w' and 
'x'. But they are relative when create with mode 'a'. This should fix a 
regression in pex, but keep the case of issue26293.

I'm going to push the patch before tagging 3.5.3 rc1.

--
keywords: +patch
nosy: +benjamin.peterson, larry, ned.deily
priority: normal -> release blocker
stage:  -> patch review
type:  -> behavior
versions: +Python 3.5, Python 3.6, Python 3.7
Added file: 
http://bugs.python.org/file46107/zipfile_write_absolute_offsets.patch

___
Python tracker 

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

[issue28964] AST literal_eval exceptions provide no information about line number

[Serhiy Storchaka]

Serhiy Storchaka added the comment:

Usually literal_eval() is used with short one-line input. For what large 
documents do you use it?

--
nosy: +serhiy.storchaka

___
Python tracker 

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