[issue29406] asyncio SSL contexts leak sockets after calling close with certain Apache servers

2018-12-17 Thread Dima Tisnek


Dima Tisnek  added the comment:

ping...

--
nosy: +Dima.Tisnek

___
Python tracker 

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



[issue35482] can't open python368rc1.chm and python372rc1.chm

2018-12-17 Thread Ma Lin


Ma Lin  added the comment:

> I guess the next step is to try taking out those new index entries to figure 
> out which one causes the problem.

Very big amount of work..., I would suggest to revert them.
chm is a successful product, but it out of support, and no replacement yet, 
it's regrettable.

--

___
Python tracker 

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



[issue35522] os.stat().st_ctime and time.time() mismatch

2018-12-17 Thread Rohit Biswas


New submission from Rohit Biswas :

Related Stack Overflow Question: 
https://stackoverflow.com/questions/53810984/mismatch-between-file-creation-time-and-current-time-in-python

--
components: Library (Lib)
messages: 332040
nosy: belopolsky, rbiswas143
priority: normal
severity: normal
status: open
title: os.stat().st_ctime and time.time() mismatch
type: behavior
versions: Python 2.7, 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



[issue35497] Libary select docs enhance

2018-12-17 Thread Xiang Zhang


Change by Xiang Zhang :


--
assignee:  -> docs@python
components: +Documentation -Library (Lib)
nosy: +docs@python, xiang.zhang
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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Manjusaka


Manjusaka  added the comment:

> `selectors` makes underlying implementations irrelavant to most users since 
> we can simply use `DefaultSelector`

I agree with this. 

> If you know you want to add EPOLL_EXCLUSIVE, why not just use `select.epoll`?

I don't think so. 

If I use the `select` directly, that means I have to give up all the features 
what `selectors` support such as binding a `data` to a fd, that also means I 
have to do some repeat work myself. I don't think it's a good idea.

So, why not change this API when we can keep the compatibility?

--

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Xiang Zhang


Xiang Zhang  added the comment:

> I don't think to make selector be a public property is a good idea. It will 
> break the whole system integrity.

If exposing a private property is not a good idea, another choice may be 
construct a selector with a customized I/O multiplexer, adding an optional 
parameter to the __init__.

But actually I'm -1 to this change. `selectors` makes underlying 
implementations irrelavant to most users since we can simply use 
`DefaultSelector`(maybe why only read/write events are valid now?). But you are 
seeking to add implementation specific details back. If you know you want to 
add EPOLL_EXCLUSIVE, why not just use `select.epoll`? A single selector doesn't 
do much more than the underlying multiplexer.

--
nosy: +xiang.zhang

___
Python tracker 

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



[issue35518] test_timeout uses blackhole.snakebite.net domain which doesn't exist anymore

2018-12-17 Thread Benjamin Peterson


Benjamin Peterson  added the comment:

I think it might be better to rewrite the test to use a local tcp server. At 
least on Linux, I think trying to connect to a socket with a full accept queue 
should have equivalent beahvior. You could also be fancier, set up a network 
namespace, and actually drop all SYN packets with iptables.

--

___
Python tracker 

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



[issue35368] [2.7] Make PyMem_Malloc() thread-safe in debug mode

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

Let's continue discussion on serialno atomicity on bpo-31473.

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



[issue35469] [2.7] time.asctime() regression

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

The change is deliberate and is not a bug.

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



[issue35499] "make profile-opt" overrides CFLAGS_NODIST

2018-12-17 Thread STINNER Victor


Change by STINNER Victor :


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



[issue35501] "make coverage" should not leak coverage compiler flags to third party C extensions

2018-12-17 Thread STINNER Victor


Change by STINNER Victor :


--
title: "make coverage" should use leak coverage flags to third party C 
extensions -> "make coverage" should not leak coverage compiler flags to third 
party C extensions

___
Python tracker 

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



[issue33610] IDLE: Make multiple improvements to CodeContext

2018-12-17 Thread Cheryl Sabella


Cheryl Sabella  added the comment:

D1 in issue35521.

--
dependencies: +IDLE: Add doc section for Code Conext

___
Python tracker 

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



[issue35521] IDLE: Add doc section for Code Conext

2018-12-17 Thread Cheryl Sabella


Change by Cheryl Sabella :


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

___
Python tracker 

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



[issue35238] Alleviate memory reservation of fork_exec in subprocess.Popen via forkserver

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

See bpo-34663 for posix_spawn() & vfork.

--

___
Python tracker 

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



[issue35521] IDLE: Add doc section for Code Conext

2018-12-17 Thread Cheryl Sabella


New submission from Cheryl Sabella :

Item D1 from #33610.

D1: idle.rst subsection on Code Context

--
assignee: terry.reedy
components: IDLE
messages: 332032
nosy: cheryl.sabella, terry.reedy
priority: normal
severity: normal
status: open
title: IDLE: Add doc section for Code Conext
type: enhancement
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



[issue35482] can't open python368rc1.chm and python372rc1.chm

2018-12-17 Thread Steve Dower


Steve Dower  added the comment:

I did a quick look and couldn't find anything obvious in logs while building, 
so I guess the next step is to try taking out those new index entries to figure 
out which one causes the problem.

(Adding Serhiy for awareness, not necessarily trying to nominate him to do the 
work unless he wants to.)

--
nosy: +serhiy.storchaka

___
Python tracker 

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



[issue19217] Calling assertEquals for moderately long list takes too long

2018-12-17 Thread Emmanuel Arias


Change by Emmanuel Arias :


--
pull_requests: +10441

___
Python tracker 

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



[issue35238] Alleviate memory reservation of fork_exec in subprocess.Popen via forkserver

2018-12-17 Thread Oscar Esteban


Oscar Esteban  added the comment:

Hi Victor,

That would be great. However, we played a bit with an alternative 
implementation of posix_spawn (one I got from one related bpo), and it didn't 
seem to make any difference in terms of memory allocation.

Then, we found out that posix_spawn uses fork by default (Linux 
implementation). So the large memory allocations still happen. One can set the 
vFork option, but that is apparently a very bad idea, as far as we read.

Is that correct?

--

___
Python tracker 

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



[issue35498] Parents objects in pathlib.Path don't support slices as __getitem__ arguments

2018-12-17 Thread Brett Cannon


Change by Brett Cannon :


--
nosy: +brett.cannon

___
Python tracker 

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



[issue35488] pathlib Path.match does not behave as described

2018-12-17 Thread Brett Cannon


Change by Brett Cannon :


--
nosy: +brett.cannon

___
Python tracker 

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



[issue35486] subprocess module import hooks breaks back compatibility

2018-12-17 Thread Brett Cannon


Brett Cannon  added the comment:

RE: "PEP-302 and PEP-451 are the definitive specifications for how 
sys.meta_path is supposed to work"

That's actually not true. In the case of import the language reference is 
considered the reference for import: 
https://docs.python.org/3/reference/import.html (and true for PEPs in general 
once they are implemented).

--
nosy: +davin

___
Python tracker 

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



[issue35238] Alleviate memory reservation of fork_exec in subprocess.Popen via forkserver

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

> By the way you could open an issue so that subprocess uses posix_spawn() 
> where possible.

FYI I'm working on an implementation of this ;-)

--
nosy: +vstinner

___
Python tracker 

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



[issue35519] Can not run test without test module for tests which import random module

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

I renamed Lib/test/bisect.py to Lib/test/bisect_cmd.py ;-)

--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed
versions: +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



[issue35519] Can not run test without test module for tests which import random module

2018-12-17 Thread miss-islington


miss-islington  added the comment:


New changeset 05dfa0cc96f6b72b1e72f57b1b5f4b37764a382d by Miss Islington (bot) 
in branch '3.7':
bpo-35519: Rename test.bisect to test.bisect_cmd (GH-11200)
https://github.com/python/cpython/commit/05dfa0cc96f6b72b1e72f57b1b5f4b37764a382d


--
nosy: +miss-islington

___
Python tracker 

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



[issue34032] Add platlibdir to allow distinction between /usr/lib and /usr/lib64 for Linux

2018-12-17 Thread Matej Cepl


Matej Cepl  added the comment:

@carlos.velasco ... OK, so build of 3.7.2rc1 on x86_64 doesn't pass the 
testsuite (see the attached log).

--
Added file: https://bugs.python.org/file48002/_log.txt

___
Python tracker 

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



[issue35238] Alleviate memory reservation of fork_exec in subprocess.Popen via forkserver

2018-12-17 Thread Antoine Pitrou


Antoine Pitrou  added the comment:

By the way you could open an issue so that subprocess uses posix_spawn() where 
possible.

(or you could ask to reopen issue31814, which is basically that request but for 
a different reason than yours)

--

___
Python tracker 

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



[issue35238] Alleviate memory reservation of fork_exec in subprocess.Popen via forkserver

2018-12-17 Thread Oscar Esteban


Oscar Esteban  added the comment:

Thanks for your response.

The idea would be to enable ``subprocess.Popen`` to use an existing fork server 
in its fork_exec.

The rationale: I can start a pool of n workers very early in the execution 
flow. They will have ~350MB memory fingerprint in the beginning and they will 
be reset to that every ``maxtasksperchild``. So this is basically the amount of 
VM allocated (doubled) when forking. Pretty small.

Currently, as the fork is done from some process with all the python stack of 
the app loaded in memory (1.7GB in our case), then some additional 1.7GB of VM 
are allocated on each fork. This could be avoided if the fork was done from the 
forkserver pool.

As you mention, we have been considering such a "shell" server on top of 
asyncio, so your response just confirms our intuition.

I'll close this idea for now since I agree that any investment on this problem 
should be directed to the asyncio solution.

Please note that the idea proposed would work for Python < 3 (as opposed to 
anything based on asyncio).

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



[issue35519] Can not run test without test module for tests which import random module

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:


New changeset 1dd035954bb03c41b954ebbd63969b4bcb0e106e by Victor Stinner in 
branch 'master':
bpo-35519: Rename test.bisect to test.bisect_cmd (GH-11200)
https://github.com/python/cpython/commit/1dd035954bb03c41b954ebbd63969b4bcb0e106e


--

___
Python tracker 

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



[issue35519] Can not run test without test module for tests which import random module

2018-12-17 Thread miss-islington


Change by miss-islington :


--
pull_requests: +10440

___
Python tracker 

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



[issue35238] Alleviate memory reservation of fork_exec in subprocess.Popen via forkserver

2018-12-17 Thread Antoine Pitrou


Antoine Pitrou  added the comment:

At any rate, given the constraints you're working with (thousands of child 
processes, memory conservation issues), I suggest you abandon the idea of using 
multiprocessing and write your own subprocess-server instead.

I would suggest doing so using asyncio, which should allow you to control as 
many subprocesses as you want without spawning countless threads or 
intermediate processes:

https://docs.python.org/3/library/asyncio-subprocess.html

--

___
Python tracker 

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



[issue35238] Alleviate memory reservation of fork_exec in subprocess.Popen via forkserver

2018-12-17 Thread Antoine Pitrou


Antoine Pitrou  added the comment:

I'm not sure I understand the proposed solution. Do you mean you would replace 
this:

  Parent -> forkserver -> fork child then exec

with:

  Parent -> forkserver -> posix_spawn child?

--
nosy: +pitrou

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Manjusaka


Change by Manjusaka :


--
nosy:  -asvetlov

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Andrew Svetlov


Andrew Svetlov  added the comment:

Hmm, I forgot kqueue usage details: it operated not only with int flags but 
more complex structures.
Giampaolo, thanks for pointing on!

Please let me sleep on it.

--

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Guido van Rossum


Change by Guido van Rossum :


--
nosy:  -gvanrossum

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Manjusaka


Manjusaka  added the comment:

Actually, in my implementation, it also supports POLL with the different event.

I don't think to make selector be a public property is a good idea. It will 
break the whole system integrity.

Please think about it, if people want to use epoll/poll with some special 
event, they must use it like this.

>  s = selectors.EpollSelector()
>  s.selector.register(fd,select.EPOLLIN | select.EPOLLEXCLUSIVE)
>  s.selector.modify(fd,select.EPOLLOU | select.EPOLLEXCLUSIVE)

Here's a question, why we support the register?

I think it will make people don't care about the detail.

So, as you say, it's a little bit complicated, but it will help people don't 
care about the selector property detail, they just use the argument when they 
want to use it.

I think it's worth it.

--

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Giampaolo Rodola'


Giampaolo Rodola'  added the comment:

I took a look at your PR. As the PR currently stands it only works with epoll() 
selector. For the other selectors this is just an extra argument which does 
nothing, so it complicates the API of 2 methods for no real gain. Also, a 
single argument is not compatible with kqueue() because kqueue() requires two 
distinct `KQ_FILTER_*` and `KQ_EV_*` constants which cannot be ORed together. 

Instead, I think simply turning the underlying selector instance into a public 
property is more flexible and less complex. On Linux you'll do:

>>> s = selectors.EpollSelector()
>>> s.register(fd, EVENT_READ)
>>> s.selector.modify(fd, select.EPOLLEXCLUSIVE)
```

...and on BSD:

>>> s = selectors.KqueueSelector()
>>> s.register(fd, EVENT_READ)
>>> k = s.selector.kevent(fd, select.KQ_FILTER_READ, select.KQ_EV_CLEAR)
>>> s.selector.control([k], 0, 0)
```

Alternatively we could just keep `DefaultSelector._selector` private and 
document it. Personally I'd also be OK with that even though I'm not sure if 
there if there are precedents of documenting private APIs.

--
nosy: +asvetlov, gvanrossum

___
Python tracker 

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



[issue35348] Problems with handling the file command output in platform.architecture()

2018-12-17 Thread Marc-Andre Lemburg


Marc-Andre Lemburg  added the comment:

Guys, please read the doc-string of the platform.architecture() function (or 
ask the person who wrote most of the module). It clearly refers to inspecting a 
specific executable and only uses the Python interpreter as default.

The running process can provide some sane defaults, but is not necessarily 
using the same values as the given executable.

The function does not support multi-architecture executables. This is simply 
out of scope for the function.

Victor: AFAIK, I still own this module, so if you want to deprecate something, 
please ping me first.

--

___
Python tracker 

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



[issue35348] Problems with handling the file command output in platform.architecture()

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:


New changeset 0af9c33262fb43f39a6558e3f155689e83e10706 by Victor Stinner in 
branch 'master':
bpo-35348: Fix platform.architecture() (GH-11159)
https://github.com/python/cpython/commit/0af9c33262fb43f39a6558e3f155689e83e10706


--

___
Python tracker 

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



[issue35360] [Windows] Update SQLite dependency

2018-12-17 Thread Steve Dower


Steve Dower  added the comment:

SQLite updates and changes for us almost always get stuck on someone being 
willing to verify that nothing has broken (and stand by their analysis).

Without an active expert (I just nosied ghaering in case they are around), I'm 
not confident to make this change in any version prior to 3.8.

--
nosy: +ghaering

___
Python tracker 

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



[issue35475] Docs do not show PyImport_AddModuleObject() returns a borrowed reference.

2018-12-17 Thread miss-islington


miss-islington  added the comment:


New changeset 605ef6e534f05925ff826f65518abf163ed3900a by Miss Islington (bot) 
in branch '3.7':
bpo-35475: Add more PyImport* functions in refcounts.dat. (GH-11142)
https://github.com/python/cpython/commit/605ef6e534f05925ff826f65518abf163ed3900a


--
nosy: +miss-islington

___
Python tracker 

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



[issue33306] Improving SyntaxError for unmatched parentheses

2018-12-17 Thread Serhiy Storchaka


Change by Serhiy Storchaka :


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



[issue35519] Can not run test without test module for tests which import random module

2018-12-17 Thread STINNER Victor


Change by STINNER Victor :


--
keywords: +patch
pull_requests: +10439
stage: resolved -> patch review

___
Python tracker 

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



[issue35519] Can not run test without test module for tests which import random module

2018-12-17 Thread STINNER Victor


Change by STINNER Victor :


--
title: [2.7] Can not run test without test module for tests which import random 
module -> Can not run test without test module for tests which import random 
module

___
Python tracker 

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



[issue35519] [2.7] Can not run test without test module for tests which import random module

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

> Ah, I think because I use cpython2 directory name, you thought I used Python 
> 2. cpython2 is just a directory name. It has nothing to do with Python 2. I 
> used master when finding this bug.

Ah right, I can reproduce the issue in master:

vstinner@apu$ ./python Lib/test/test_xmlrpc.py 
Traceback (most recent call last):
  File "Lib/test/test_xmlrpc.py", line 8, in 
import xmlrpc.client as xmlrpclib
  File "/home/vstinner/prog/python/master/Lib/xmlrpc/client.py", line 136, in 

import http.client
  File "/home/vstinner/prog/python/master/Lib/http/client.py", line 71, in 

import email.parser
  File "/home/vstinner/prog/python/master/Lib/email/parser.py", line 12, in 

from email.feedparser import FeedParser, BytesFeedParser
  File "/home/vstinner/prog/python/master/Lib/email/feedparser.py", line 27, in 

from email._policybase import compat32
  File "/home/vstinner/prog/python/master/Lib/email/_policybase.py", line 9, in 

from email.utils import _has_surrogates
  File "/home/vstinner/prog/python/master/Lib/email/utils.py", line 28, in 

import random
  File "/home/vstinner/prog/python/master/Lib/random.py", line 47, in 
import bisect as _bisect
  File "/home/vstinner/prog/python/master/Lib/test/bisect.py", line 27, in 

import tempfile
  File "/home/vstinner/prog/python/master/Lib/tempfile.py", line 45, in 
from random import Random as _Random
ImportError: cannot import name 'Random' from 'random' 
(/home/vstinner/prog/python/master/Lib/random.py)

Hum. We should either rename Lib/test/bisect.py to Lib/test/bisect_cmd.py or 
move the feature into libregrtest.

--
nosy: +pablogsal
resolution: fixed -> 
status: closed -> open

___
Python tracker 

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



[issue33306] Improving SyntaxError for unmatched parentheses

2018-12-17 Thread Serhiy Storchaka


Serhiy Storchaka  added the comment:


New changeset 94cf308ee231bfbfaa9ddc50b9764545a1318773 by Serhiy Storchaka in 
branch 'master':
bpo-33306: Improve SyntaxError messages for unbalanced parentheses. (GH-6516)
https://github.com/python/cpython/commit/94cf308ee231bfbfaa9ddc50b9764545a1318773


--

___
Python tracker 

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



[issue35490] Remove the DecodeFSDefault return converter in Argument Clinic

2018-12-17 Thread Serhiy Storchaka


Change by Serhiy Storchaka :


--
resolution:  -> fixed
stage: patch review -> resolved
status: open -> closed
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



[issue35475] Docs do not show PyImport_AddModuleObject() returns a borrowed reference.

2018-12-17 Thread miss-islington


Change by miss-islington :


--
pull_requests: +10438

___
Python tracker 

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



[issue35475] Docs do not show PyImport_AddModuleObject() returns a borrowed reference.

2018-12-17 Thread miss-islington


Change by miss-islington :


--
pull_requests: +10437

___
Python tracker 

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



[issue35475] Docs do not show PyImport_AddModuleObject() returns a borrowed reference.

2018-12-17 Thread Serhiy Storchaka


Serhiy Storchaka  added the comment:


New changeset bdabb0737c631835b246c9823852d20331243315 by Serhiy Storchaka in 
branch 'master':
bpo-35475: Add more PyImport* functions in refcounts.dat. (GH-11142)
https://github.com/python/cpython/commit/bdabb0737c631835b246c9823852d20331243315


--

___
Python tracker 

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



[issue35504] `del OSError().characters_written` raises `SystemError`

2018-12-17 Thread Serhiy Storchaka


Change by Serhiy Storchaka :


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



[issue35504] `del OSError().characters_written` raises `SystemError`

2018-12-17 Thread miss-islington


miss-islington  added the comment:


New changeset cb272843f2d5dfc4ef996ba952b99a3e30c88bbc by Miss Islington (bot) 
in branch '3.7':
bpo-35504: Fix segfaults and SystemErrors when deleting certain attrs. 
(GH-11175)
https://github.com/python/cpython/commit/cb272843f2d5dfc4ef996ba952b99a3e30c88bbc


--
nosy: +miss-islington

___
Python tracker 

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



[issue21041] pathlib.PurePath.parents rejects negative indexes

2018-12-17 Thread Joshua Cannon


Joshua Cannon  added the comment:

I created issue35498 about .parents rejecting slices as well. (It was pointed 
out this discussion would probably decide that issue's fate)
I think that .parents looking like a duck, but not quacking like one isn't very 
pythonic.

Besides, the fact that p.parents[len(p.parents)-2] is allowed but p.parents[-2] 
is not just seems like extra steps. There's also list(p.parents)[-2], which is 
still not ideal. In either case, I'd imagine authors to put a comment like 
"PathLib .parents doesn't support negative indexes", which goes to show clients 
are expecting negative indices to work.

I see that this issue is several years old. I'm happy to shepherd it if it 
needs further contributions.

--
nosy: +thejcannon

___
Python tracker 

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



[issue35504] `del OSError().characters_written` raises `SystemError`

2018-12-17 Thread miss-islington


Change by miss-islington :


--
pull_requests: +10436

___
Python tracker 

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



[issue35504] `del OSError().characters_written` raises `SystemError`

2018-12-17 Thread Serhiy Storchaka


Serhiy Storchaka  added the comment:


New changeset 842acaab1376c5c84fd5966bb6070e289880e1ca by Serhiy Storchaka 
(Zackery Spytz) in branch 'master':
bpo-35504: Fix segfaults and SystemErrors when deleting certain attrs. 
(GH-11175)
https://github.com/python/cpython/commit/842acaab1376c5c84fd5966bb6070e289880e1ca


--

___
Python tracker 

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



[issue35490] Remove the DecodeFSDefault return converter in Argument Clinic

2018-12-17 Thread Serhiy Storchaka


Serhiy Storchaka  added the comment:


New changeset 4db62e115891425db2a974142a72d8eaaf95eecb by Serhiy Storchaka in 
branch 'master':
bpo-35490: Remove the DecodeFSDefault return converter in AC. (#11152)
https://github.com/python/cpython/commit/4db62e115891425db2a974142a72d8eaaf95eecb


--

___
Python tracker 

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



[issue35504] `del OSError().characters_written` raises `SystemError`

2018-12-17 Thread Serhiy Storchaka


Serhiy Storchaka  added the comment:


New changeset e2af34fcf84b41189b54e1f2912faded5daabaca by Serhiy Storchaka in 
branch 'master':
bpo-35504: Fix a SystemError when delete the characters_written attribute of an 
OSError. (GH-11172)
https://github.com/python/cpython/commit/e2af34fcf84b41189b54e1f2912faded5daabaca


--

___
Python tracker 

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



[issue18799] Resurrect and fix test_404 in Lib/test/test_xmlrpc.py

2018-12-17 Thread Serhiy Storchaka


Serhiy Storchaka  added the comment:

Thank you Vajrasky. The restored test is passed locally and on CI. Let try it 
on buildbots.

--
versions: +Python 3.8 -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



[issue19974] tarfile doesn't overwrite symlink by directory

2018-12-17 Thread Vajrasky Kok


Vajrasky Kok  added the comment:

Martin Panter, thank you for reviewing my patch. Let me rework it. It has been 
a while (4 years!!!).

--

___
Python tracker 

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



[issue18799] Resurrect and fix test_404 in Lib/test/test_xmlrpc.py

2018-12-17 Thread Vajrasky Kok


Change by Vajrasky Kok :


--
pull_requests: +10435

___
Python tracker 

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



[issue35519] [2.7] Can not run test without test module for tests which import random module

2018-12-17 Thread Vajrasky Kok


Vajrasky Kok  added the comment:

Okay, thanks, Victor. Your suggestion to rename Lib/test/bisect.py to 
Lib/test/bisect_cmd.py works. My question is why you fixed in 2.7 branch only? 
This problem persists in master (3.8).

Ah, I think because I use cpython2 directory name, you thought I used Python 2. 
cpython2 is just a directory name. It has nothing to do with Python 2. I used 
master when finding this bug.

--

___
Python tracker 

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



[issue35415] fileno argument to socket.socket is not validated

2018-12-17 Thread Andrew Svetlov


Change by Andrew Svetlov :


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



[issue35415] fileno argument to socket.socket is not validated

2018-12-17 Thread miss-islington


miss-islington  added the comment:


New changeset e991270363435da12049ecfe70bb69bd9c14b535 by Miss Islington (bot) 
(Dima Tisnek) in branch 'master':
bpo-35415: validate fileno argument to socket.socket (GH-10917)
https://github.com/python/cpython/commit/e991270363435da12049ecfe70bb69bd9c14b535


--
nosy: +miss-islington

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

I mean that using extra_events to get access to EPOLLEXCLUSIVE, there is no 
need later to add a new parameter for select.EPOLLONESHOT: it would be "future 
proof"!

--

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Manjusaka


Manjusaka  added the comment:

Vicor:

> Moreover, we directly support any EPOLL constant exposed in the select 
> module. No need to change the API.

I don't think so


In class _PollLikeSelector ,here's register method

def register(self, fileobj, events, data=None):
key = super().register(fileobj, events, data)
poller_events = 0
if events & EVENT_READ:
poller_events |= self._EVENT_READ
if events & EVENT_WRITE:
poller_events |= self._EVENT_WRITE

this code filter the event that people push in it. It only supports 
select.EPOLLIN and select.EPOLLOUT

--

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Manjusaka


Manjusaka  added the comment:

OK, I will change my code

--

___
Python tracker 

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



[issue35520] Python won't build with dtrace enabled on some systems.

2018-12-17 Thread Jakub Kulik


Change by Jakub Kulik :


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

___
Python tracker 

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



[issue35520] Python won't build with dtrace enabled on some systems.

2018-12-17 Thread Jakub Kulik


New submission from Jakub Kulik :

Python won't build on Solaris with dtrace support enabled.

Solaris is one of those systems where it is necessary to generate dtrace object 
files with dtrace -G. While this need is included in python configure and 
Makefiles, it doesn't work correctly.

First, configure tests -G support on file with not completely valid content of 
just BEGIN inside. Valid should have BEGIN{}. This is not a problem for systems 
that don't require dtrace object files as this test should fail for them 
anyway, however it incorrectly detects those like Solaris.

And second, Makefile is not ready for dtrace as the DTRACE_DEPS variable 
doesn't include all the necessary files.

--
components: Build
messages: 331997
nosy: kulikjak
priority: normal
severity: normal
status: open
title: Python won't build with dtrace enabled on some systems.
type: compile error
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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

I prefer Giampaolo since discussed flags are very specific to epoll(): select() 
doesn't support them for example, nor kqueue nor devpoll (not *yet*).

If we add a keyword-parameter, to me, it sounds like it's something "portable" 
working on multiple platforms and then you need hasattr():

if exclusive and hasattr(select, "EPOLLEXCLUSIVE"):
epoll_events |= select.EPOLLEXCLUSIVE

If the caller pass select.EPOLLEXCLUSIVE, hasattr() is useless.

Moreover, we directly support any EPOLL constant exposed in the select module. 
No need to change the API.

--

___
Python tracker 

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



[issue31562] snakebite.net is not available

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

Oh, I found the same bug: bpo-35518.

--
nosy: +vstinner

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Manjusaka


Manjusaka  added the comment:

In my opinion

selectors is an abstract for select, so I don't think allow people use select.* 
in selector is a good idea.

like this

> s.register(fd, EVENT_READ, extra_events=select.EPOLLEXCLUSIVE | 
> select.EPOLLONESHOT)

Because the multiple epoll's params are supported in different Python version 
and Linux Kernel version.

So, I think it's a good idea to support different epoll's params by 
keyword-only param in register method.

It's also convenient to check the params

--

___
Python tracker 

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



[issue35519] [2.7] Can not run test without test module for tests which import random module

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

Your problem is that you have the Lib/test/bisect.py file. I renamed it to 
Lib/test/bisect_cmd.py.

Workarounds:

* Run the test using: ./python -m test test_xmlrpc
* Rename Lib/test/bisect.py to Lib/test/bisect_cmd.py

This bug is already fixed in the 2.7. You have to wait for the next 2.7 release 
to get the fix.

--
nosy: +vstinner
resolution:  -> fixed
stage:  -> resolved
status: open -> closed
title: Can not run test without test module for tests which import random 
module -> [2.7] Can not run test without test module for tests which import 
random module
versions: +Python 2.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



[issue35519] Can not run test without test module for tests which import random module

2018-12-17 Thread Vajrasky Kok


New submission from Vajrasky Kok :

$ git clone g...@github.com:python/cpython.git cpython2
$ cd cpython2
$ ./configure --with-pydebug
$ make -j
$ ./python Lib/test/test_xmlrpc.py 
Traceback (most recent call last):
  File "Lib/test/test_xmlrpc.py", line 8, in 
import xmlrpc.client as xmlrpclib
  File "/opt/Code/python/cpython2/Lib/xmlrpc/client.py", line 136, in 
import http.client
  File "/opt/Code/python/cpython2/Lib/http/client.py", line 71, in 
import email.parser
  File "/opt/Code/python/cpython2/Lib/email/parser.py", line 12, in 
from email.feedparser import FeedParser, BytesFeedParser
  File "/opt/Code/python/cpython2/Lib/email/feedparser.py", line 27, in 
from email._policybase import compat32
  File "/opt/Code/python/cpython2/Lib/email/_policybase.py", line 9, in 
from email.utils import _has_surrogates
  File "/opt/Code/python/cpython2/Lib/email/utils.py", line 28, in 
import random
  File "/opt/Code/python/cpython2/Lib/random.py", line 47, in 
import bisect as _bisect
  File "/opt/Code/python/cpython2/Lib/test/bisect.py", line 27, in 
import tempfile
  File "/opt/Code/python/cpython2/Lib/tempfile.py", line 45, in 
from random import Random as _Random
ImportError: cannot import name 'Random' from 'random' 
(/opt/Code/python/cpython2/Lib/random.py)


I know about running test this way:
$ ./python -m test -v test_xmlrpc

And it works.

I am just wondering whether I should be able to run test this way: ./python 
Lib/test/test_blabla.py?

Because running other tests without test module works, for example: ./python 
Lib/test/test_ast.py. Only test which imports random module fails.

--
components: Tests
messages: 331992
nosy: vajrasky
priority: normal
severity: normal
status: open
title: Can not run test without test module for tests which import random 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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Giampaolo Rodola'


Giampaolo Rodola'  added the comment:

I see. Then I would say it's a matter of deciding what's the best API to 
provide. Another possibility is to promote the underlying epoll() instance as a 
public property, so one can do:

>>> s = selectors.EpollSelector()
>>> s.register(fd, EVENT_READ)
>>> s.selector.modify(fd, select.EPOLLEXCLUSIVE)

That raises the question whether all selector classes should have a public 
"selector" attribute. poll() and devpoll() related classes may need it for 
POLLPRI, POLLRDHUP, POLLWRBAND or others (whatever their use case is). kqueue() 
also has it's own specific constants (KQ_FILTER_* and KQ_EV_*). The only one 
where a public "selector" property would be useless is SelectSelector.

--
stage: patch review -> 

___
Python tracker 

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



[issue23451] Deprecation warnings building 3.5 Visual Studio Windows 8.1 64 bit

2018-12-17 Thread miss-islington


miss-islington  added the comment:


New changeset c367d52a74781b2c9ffd9e29722fbdfc0234408c by Miss Islington (bot) 
in branch '3.7':
bpo-23451: Update time.monotonic() documentation (GH-11190)
https://github.com/python/cpython/commit/c367d52a74781b2c9ffd9e29722fbdfc0234408c


--
nosy: +miss-islington

___
Python tracker 

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



[issue22117] Rewrite pytime.h to work on nanoseconds

2018-12-17 Thread miss-islington


miss-islington  added the comment:


New changeset c367d52a74781b2c9ffd9e29722fbdfc0234408c by Miss Islington (bot) 
in branch '3.7':
bpo-23451: Update time.monotonic() documentation (GH-11190)
https://github.com/python/cpython/commit/c367d52a74781b2c9ffd9e29722fbdfc0234408c


--
nosy: +miss-islington

___
Python tracker 

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



[issue35518] test_timeout uses blackhole.snakebite.net domain which doesn't exist anymore

2018-12-17 Thread Karthikeyan Singaravelan


Karthikeyan Singaravelan  added the comment:

Related : issue31562

--
nosy: +xtreak

___
Python tracker 

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



[issue35266] Add _PyPreConfig and rework _PyCoreConfig and _PyMainInterpreterConfig

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

When I looked again at this issue, I'm not sure how what should be done, what 
is the proper design, what should stay after Python initialization, etc. I 
prefer to abandon this change and maybe retry to write it later.

I have a more advanced version in this branch of my fork: 
https://github.com/vstinner/cpython/commits/pre_config_next

--
resolution:  -> out of date
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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Manjusaka


Change by Manjusaka :


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

___
Python tracker 

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



[issue35472] python 3.7.2 rc1 bumped sphinx requirements a bit too much

2018-12-17 Thread Julien Palard


Change by Julien Palard :


--
title: python 3.7.2 rc1 bumped the build requirements for no reason -> python 
3.7.2 rc1 bumped sphinx requirements a bit too much

___
Python tracker 

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



[issue35337] Check index in PyTuple_GET_ITEM/PyTuple_SET_ITEM in debug mode

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

Ok, I abandon my change.

--
resolution:  -> wont fix
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



[issue35472] python 3.7.2 rc1 bumped the build requirements for no reason

2018-12-17 Thread Julien Palard


Change by Julien Palard :


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

___
Python tracker 

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



[issue35472] python 3.7.2 rc1 bumped the build requirements for no reason

2018-12-17 Thread Julien Palard


Julien Palard  added the comment:

Hi Matthias,

I agree that for the smartquotes issue the 1.6.6 should be enough. But we had 
multiple small issues fixed in some versions of Sphinx. But mostly for 
translation and I don't think you're building the translations?

Also the needs_sphinx is only the minimum required version to build, it won't 
disallow me using sphinx 1.7 or 1.8 on docs.python.org to build the 
documentation.

The needs_sphinx flag only supports "micro" value since 1.4, but looks 
compatible with it before (04a8c26eabba3e728a5b74afd15f328f9235b2d0 in the 
sphinx repo).

So it looks OK to allow 1.6.6 as a minimum required version for 3.7, opening a 
PR on it.

--

___
Python tracker 

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



[issue35276] Document thread safety

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

About system-wide: see also PR 11190, "time.monotonic() is now always available 
and always system-wide."

--

___
Python tracker 

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



[issue22117] Rewrite pytime.h to work on nanoseconds

2018-12-17 Thread miss-islington


Change by miss-islington :


--
pull_requests: +10431

___
Python tracker 

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



[issue23451] Deprecation warnings building 3.5 Visual Studio Windows 8.1 64 bit

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:


New changeset 3ab064e80a9be1e6e9c62437fffb92bde9c5e1fb by Victor Stinner in 
branch 'master':
bpo-23451: Update time.monotonic() documentation (GH-11190)
https://github.com/python/cpython/commit/3ab064e80a9be1e6e9c62437fffb92bde9c5e1fb


--
nosy: +vstinner

___
Python tracker 

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



[issue22117] Rewrite pytime.h to work on nanoseconds

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:


New changeset 3ab064e80a9be1e6e9c62437fffb92bde9c5e1fb by Victor Stinner in 
branch 'master':
bpo-23451: Update time.monotonic() documentation (GH-11190)
https://github.com/python/cpython/commit/3ab064e80a9be1e6e9c62437fffb92bde9c5e1fb


--

___
Python tracker 

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



[issue23451] Deprecation warnings building 3.5 Visual Studio Windows 8.1 64 bit

2018-12-17 Thread miss-islington


Change by miss-islington :


--
pull_requests: +10430

___
Python tracker 

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



[issue35518] test_timeout uses blackhole.snakebite.net domain which doesn't exist anymore

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

CC Pablo who loves black holes ;-)

--
nosy: +pablogsal

___
Python tracker 

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



[issue35518] test_timeout uses blackhole.snakebite.net domain which doesn't exist anymore

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

We should either recreate this "blackhole" service on pythontest.net (server 
controlled by the PSF) or drop the test.

@Benjamin: Hi, do you know who maintain pythontest.net? Who can add a "black 
hole" service?

--
nosy: +benjamin.peterson

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Manjusaka


Manjusaka  added the comment:

Hello rodola

Here's detail:

In the server, we use multiprocess to handle the request with share the same 
FD. 

OK, when a request comes, all the process wake up and try to accept the request 
but only one process will accept success and the others will raise an Exception 
if we use the epoll without EPOLLEXCLUSIVE.

That means there will waste the performance. This effect is named thundering 
herd.

But if we use epoll with EPOLLEXCLUSIVE, when the envents happen, only one 
process will wake and try to handle the event.

So, I think it's necessary to support the EPOLLEXCLUSIVE in 
selectors.EpollSelector

Actually, Python support EPOLLEXCLUSIVE official since 3.7. It's time to 
support it in selectors

--

___
Python tracker 

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



[issue35518] test_timeout uses blackhole.snakebite.net domain which doesn't exist anymore

2018-12-17 Thread STINNER Victor


New submission from STINNER Victor :

snakebite.net and blackhole.snakebite.net domains don't exist anymore, but 
test_timeout uses it:

def testConnectTimeout(self):
# Testing connect timeout is tricky: we need to have IP connectivity
# to a host that silently drops our packets.  We can't simulate this
# from Python because it's a function of the underlying TCP/IP stack.
# So, the following Snakebite host has been defined:
blackhole = resolve_address('blackhole.snakebite.net', 5)
(...)

If I recall properly, snakebite.net was a service provided by Trent Nelson to 
test CPython on various operating systems (Solaris, IRIX, HP-UX, etc.). It 
seems like the service is now completely down.

Article from 2009:

https://conferences.oreilly.com/oscon/oscon2009/public/schedule/detail/8268

"Snakebite is a culmination of ten months of secretive work, seven trips to 
Michigan State University, six blown fuses and about $60,000. The end result? A 
network of around 37-ish servers of all different shapes and sizes, 
specifically geared towards the development needs of open source projects. Get 
the inside scoop from Snakebite's Founder, Trent Nelson, and MSU Director Dr. 
Titus Brown."

--
components: Tests
messages: 331978
nosy: vstinner
priority: normal
severity: normal
status: open
title: test_timeout uses blackhole.snakebite.net domain which doesn't exist 
anymore
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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Giampaolo Rodola'


Giampaolo Rodola'  added the comment:

I'm not sure I understand what EPOLLEXCLUSIVE is about. Could you provide a use 
case?

Also, there are other constants which may also be useful such as EPOLLWAKEUP 
and EPOLLONESHOT:
http://man7.org/linux/man-pages/man2/epoll_ctl.2.html
So perhaps it makes more sense to expose a lower-level "extra_events" argument 
instead, which is more flexible and also saves us from the hassle of describing 
what the new argument is about (which really is a Linux kernel thing) in the 
doc:

>>> s.register(fd, EVENT_READ, extra_events=select.EPOLLEXCLUSIVE | 
>>> select.EPOLLONESHOT)

That raises the question whether we should also have an "extra_events" argument 
for modify() method (probably yes). 

But again, I think it makes sense to understand what's the use case of these 
constants first, because if they are rarely used maybe who needs them can 
simply look at the source and use s._selector.modify() directly.

--
nosy: +neologix, yselivanov

___
Python tracker 

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



[issue35513] Lib/test/lock_tests.py should not use time.time(), but time.monotonic()

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

I searched from "time.time" and "from time import(...)time" in the Python 
standard library (in the master branch) and it seems like all usage of 
time.time() are now appropriate. I close the issue.

Sometimes, I'm not sure that if time.monotonic() or time.perf_counter() should 
be used, but at least both functions are monotonic and so are not affected by 
this issue (clock going backwards).

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



[issue35513] Lib/test/lock_tests.py should not use time.time(), but time.monotonic()

2018-12-17 Thread miss-islington


miss-islington  added the comment:


New changeset 9ade4cbc0f54fc0e2970e4e202f09ab83f5e3b77 by Miss Islington (bot) 
in branch '3.7':
bpo-35513, unittest: TextTestRunner uses time.perf_counter() (GH-11180)
https://github.com/python/cpython/commit/9ade4cbc0f54fc0e2970e4e202f09ab83f5e3b77


--

___
Python tracker 

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



[issue23451] Deprecation warnings building 3.5 Visual Studio Windows 8.1 64 bit

2018-12-17 Thread STINNER Victor


Change by STINNER Victor :


--
pull_requests: +10428
stage:  -> patch review

___
Python tracker 

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



[issue22117] Rewrite pytime.h to work on nanoseconds

2018-12-17 Thread STINNER Victor


Change by STINNER Victor :


--
pull_requests: +10429

___
Python tracker 

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



[issue31784] Implementation of the PEP 564: Add time.time_ns()

2018-12-17 Thread STINNER Victor


Change by STINNER Victor :


--
pull_requests: +10427

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Manjusaka


Manjusaka  added the comment:

"a keyword-only parameter" is good I'll take it done

--

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread Manjusaka


Manjusaka  added the comment:

I will work on a PR

--

___
Python tracker 

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



[issue35517] selector.EpollSelector: add new parameter to support EPOLLEXCLUSIVE

2018-12-17 Thread STINNER Victor


STINNER Victor  added the comment:

if exclusive and hasattr(select, "EPOLLEXCLUSIVE"):
epoll_events |= select.EPOLLEXCLUSIVE

Maybe NotImplementedError would be more appropriate rather than silently ignore 
the error?

--

___
Python tracker 

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



  1   2   >