[issue28448] C implemented Future doesn't work on Windows

2016-10-17 Thread INADA Naoki
INADA Naoki added the comment: https://github.com/search?p=3&q=_schedule_callbacks&type=Code&utf8=%E2%9C%93 At least, Future class in uvloop have same API. Most of other results seems just copy of Python source tree. (But I didn't check all search result) -- ___

[issue28462] subprocess pipe can't see EOF from a child in case of a few children run with subprocess

2016-10-17 Thread Eryk Sun
Eryk Sun added the comment: Due to a race condition, the Popen call in the second process is inadvertently inheriting the handle for the write end of the pipe that's created for the first process. Thus stdout.readline() in the first thread doesn't see EOF until that handle is closed. For the

[issue26240] Docstring of the subprocess module should be cleaned up

2016-10-17 Thread Antony Lee
Changes by Antony Lee : -- nosy: -Antony.Lee ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.pyth

[issue26240] Docstring of the subprocess module should be cleaned up

2016-10-17 Thread Berker Peksag
Berker Peksag added the comment: Thanks for the patch, Tim. > Changed true -> :const:`True` in subprocess.rst. This is out of scope for this issue and I actually prefer the current form. Having method and class signatures in subprocess.__doc__ would make it less maintainable. I'd prefer havin

[issue26240] Docstring of the subprocess module should be cleaned up

2016-10-17 Thread Berker Peksag
Changes by Berker Peksag : -- nosy: +berker.peksag stage: needs patch -> patch review versions: +Python 3.7 ___ Python tracker ___ ___

[issue26240] Docstring of the subprocess module should be cleaned up

2016-10-17 Thread Tim Mitchell
Tim Mitchell added the comment: Have stripped down the module __doc__ to a list of contents. I chose to indent the descriptions of each argument to Popen. I know this is non-standard but it is such a long ramble otherwise. Changed true -> :const:`True` in subprocess.rst. -- keywords: +

[issue27931] Email parse IndexError <""@wiarcom.com>

2016-10-17 Thread Xiang Zhang
Xiang Zhang added the comment: Ping. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org

[issue28466] SIGALRM fails to interrupt time.sleep() call on Python 3.5

2016-10-17 Thread Martin Panter
Martin Panter added the comment: This is by design; see PEP 475, and the documentation . If you make your signal handler raise an exception, it will interrupt the sleep() call most of the time. But if the signal happens to be received j

[issue28465] python 3.5 magic number

2016-10-17 Thread Mariatta Wijaya
Mariatta Wijaya added the comment: Hi. imp.get_magic() is deprecated since 3.4, see the documentation here https://docs.python.org/3/library/imp.html?highlight=get_magic#imp.get_magic You should use importlib.util.MAGIC_NUMBER instead. Please try it. Thanks. -- nosy: +Mariatta ___

[issue27659] Check for the existence of crypt()

2016-10-17 Thread Martin Panter
Martin Panter added the comment: If there is an obscure platform where we don’t include the right header file for a function, changing the warning into an error would cause the build to fail. If we do make it an error, it should only be so for 3.7. -- _

[issue28466] SIGALRM fails to interrupt time.sleep() call on Python 3.6

2016-10-17 Thread Ryan Petrello
New submission from Ryan Petrello: I may have found a bug in SIGALRM handling in Python3.5. I've not been able to reproduce the same issue in Python2.7 or 3.4. Here's a simple example that illustrates the issue (which I'm able to reproduce on OS X 10.11.3 El Capitan and Ubuntu 14.04): $ pyt

[issue28466] SIGALRM fails to interrupt time.sleep() call on Python 3.5

2016-10-17 Thread Ryan Petrello
Changes by Ryan Petrello : -- title: SIGALRM fails to interrupt time.sleep() call on Python 3.6 -> SIGALRM fails to interrupt time.sleep() call on Python 3.5 ___ Python tracker

[issue28452] Remove _asyncio._init_module function

2016-10-17 Thread INADA Naoki
Changes by INADA Naoki : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___ Py

[issue28452] Remove _asyncio._init_module function

2016-10-17 Thread Roundup Robot
Roundup Robot added the comment: New changeset d32ec6591c49 by INADA Naoki in branch '3.6': Issue #28452: Remove _asyncio._init_module function https://hg.python.org/cpython/rev/d32ec6591c49 New changeset ce85a1f129e3 by INADA Naoki in branch 'default': Issue #28452: Remove _asyncio._init_module

[issue28444] Missing extensions modules when cross compiling python 3.5.2 for arm on Linux

2016-10-17 Thread Martin Panter
Martin Panter added the comment: PS: I agree it would be good to add more documentation for cross-compiling. I tried to suggest something in an outdated patch once before; see the bottom of . -- __

[issue28444] Missing extensions modules when cross compiling python 3.5.2 for arm on Linux

2016-10-17 Thread Martin Panter
Martin Panter added the comment: Well, I am not really an expert on the setup.py stuff, but I will ask a question anyway that may help the review process: Why do you remove the code that loops over Modules/Setup? Maybe is it redundant with the other code for removing the already-built-in modul

[issue28385] Bytes objects should reject all formatting codes with an error message

2016-10-17 Thread Eric V. Smith
Eric V. Smith added the comment: LGTM, although I'm not so sure about your #3. Maybe it should be a separate issue and raised on python-dev? But I don't feel strongly enough about it to argue the point. Thanks! -- ___ Python tracker

[issue28465] python 3.5 magic number

2016-10-17 Thread 曹忠
New submission from 曹忠: On debian 9 and python 3.5.2: >>> imp.get_magic() b'\x17\r\r\n' On windows and python 3.5.2 >>> imp.get_magic() b'\x16\r\r\n' the same python version, magic number is the same, why not? -- components: Build messages: 278830 nosy: 曹忠 priority: normal severity: no

[issue28464] BaseEventLoop.close should shutdown executor before marking itself closed

2016-10-17 Thread Chris Meyer
New submission from Chris Meyer: BaseEventLoop.close shuts down the executor associated with the event loop. It should do that BEFORE it sets self._closed = True, otherwise any pending executor futures will attempt to 'call_soon' on the event loop when they finish, resulting in a confusing err

[issue28452] Remove _asyncio._init_module function

2016-10-17 Thread Yury Selivanov
Yury Selivanov added the comment: LGTM -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.o

[issue28448] C implemented Future doesn't work on Windows

2016-10-17 Thread Yury Selivanov
Yury Selivanov added the comment: > C implemented Future should allow overriding _schedule_callbacks. I'm not so sure about this. Maybe we can just fix _WaitCancelFuture somehow? -- ___ Python tracker ___

[issue28462] subprocess pipe can't see EOF from a child in case of a few children run with subprocess

2016-10-17 Thread Martin Panter
Changes by Martin Panter : -- components: +Windows nosy: +paul.moore, steve.dower, tim.golden, zach.ware ___ Python tracker ___ ___ Py

[issue28409] test.regrtest does not support multiple -x flags

2016-10-17 Thread STINNER Victor
STINNER Victor added the comment: Serhiy Storchaka added the comment: > What is the difference between using parse_known_args() and nargs=argparse.REMAINDER? Using REMAINDER, args become ["b", "-c", "d"] for -a b -c d. I would be happy to not use parse_known_args(), but I didn't find how.

[issue28409] test.regrtest does not support multiple -x flags

2016-10-17 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: What is the difference between using parse_known_args() and nargs=argparse.REMAINDER? -- ___ Python tracker ___ _

[issue28448] C implemented Future doesn't work on Windows

2016-10-17 Thread INADA Naoki
Changes by INADA Naoki : -- keywords: +patch stage: needs patch -> patch review Added file: http://bugs.python.org/file45125/schedule_callbacks.patch ___ Python tracker ___ _

[issue27659] Check for the existence of crypt()

2016-10-17 Thread Chi Hsuan Yen
Chi Hsuan Yen added the comment: Agreed. Adding -Werror=implicit-function-declaration is much simpler. Feel free to close it as rejected. -- ___ Python tracker ___ _

[issue28463] Email long headers parsing/serialization

2016-10-17 Thread R. David Murray
R. David Murray added the comment: Ah, interesting case. Both the old folder/parser and the new folder/parser fail, in slightly different ways. I'll have to add this test case to the tests as I finish rewriting the folder. Thanks for the report. -- _

[issue28463] Email long headers parsing/serialization

2016-10-17 Thread Константин Волков
Константин Волков added the comment: Something with inserting long strings here. Its duplicating for some reason. Adding example as attachment. -- Added file: http://bugs.python.org/file45124/test.py ___ Python tracker

[issue28463] Email long headers parsing/serialization

2016-10-17 Thread Константин Волков
Константин Волков added the comment: Something with copy paste. x = '<147672320775.19544.6718708004153358...@mkren-spb.root.devdomain.local>' -- ___ Python tracker ___ __

[issue28463] Email long headers parsing/serialization

2016-10-17 Thread Константин Волков
New submission from Константин Волков: There is strange thing with long headers serialized, they have \n prefix. Example fails on Python3.4/3.5: from email.message import Message from email import message_from_bytes x = '<147672320775.19544.6718708004153358...@mkren-spb.root.devdomain.local>'

[issue28433] Add sorted (ordered) containers

2016-10-17 Thread Марк Коренберг
Марк Коренберг added the comment: @r.david.murray Please see answres at https://groups.google.com/forum/#!topic/python-ideas/nPOi2LtVsR4 No one say that adding sorted containers is bad idea. Some people say that specific use-cases require specific solutions, but they also said that it is goo

[issue27806] 2.7 32-bit builds fail on macOS 10.12 Sierra due to dependency on deleted header file QuickTime/QuickTime.h

2016-10-17 Thread Ned Deily
Ned Deily added the comment: Lapsang, sorry but this bug tracker is not a help forum. There are many other, more appropriate places to ask for help in building software, like Stack Overflow or the Python mailing list. But, if you are not comfortable patching and building software from source

[issue28455] argparse: convert_arg_line_to_args does not actually expect self argument

2016-10-17 Thread siccegge
siccegge added the comment: Looks quite helpfull indeed to me! Thanks! -- ___ Python tracker ___ ___ Python-bugs-list mailing list Uns

[issue27779] Sync-up docstrings in C version of the the decimal module

2016-10-17 Thread Lisa Roach
Lisa Roach added the comment: Anyone get the chance to look over this yet? -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue27778] PEP 524: Add os.getrandom()

2016-10-17 Thread STINNER Victor
STINNER Victor added the comment: Because of the lack of interest for getrandom_errno.patch, and Christian saying that it's not good to document specific errors, I now close the bug. Thank you all for your help on this nice security enhancement in Python 3.6! -- resolution: -> fixed s

[issue28409] test.regrtest does not support multiple -x flags

2016-10-17 Thread STINNER Victor
STINNER Victor added the comment: Thanks for the bug report Charalampos! It's now fixed. -- > Until python3.6.0a04 it was possible to pass multiple times the -x option at > regrtest. To be clear: the regrtest parser of command line argument was broken since Python 2.7 at least. "./python -m

[issue28409] test.regrtest does not support multiple -x flags

2016-10-17 Thread Roundup Robot
Roundup Robot added the comment: New changeset af06d9616c29 by Victor Stinner in branch '3.5': Issue #28409: regrtest: fix the parser of command line arguments. https://hg.python.org/cpython/rev/af06d9616c29 New changeset 26249f82c15d by Victor Stinner in branch '3.6': Merge 3.6: Issue #28409: r

[issue27806] 2.7 32-bit builds fail on macOS 10.12 Sierra due to dependency on deleted header file QuickTime/QuickTime.h

2016-10-17 Thread Lapsang Leaf
Lapsang Leaf added the comment: Yes, I understood that part, sorry. What I don’t understand is how to work/install the patch you have provided? So, again, please let me know, step by step, what I need to do to patch and get it running. Thanks > On 17 Oct 2016, at 17:04, Jeremy Sequoia wrote

[issue27806] 2.7 32-bit builds fail on macOS 10.12 Sierra due to dependency on deleted header file QuickTime/QuickTime.h

2016-10-17 Thread Jeremy Sequoia
Jeremy Sequoia added the comment: QuickTime/QuickTime.h was deprecated long ago. python fails to compile if QuickTime/QuickTime.h isn't present. QuickTime/QuickTime.h was removed in Sierra, causing the build to fail on Sierra. This patch fixes that issue. > On Oct 17, 2016, at 05:23, Lapsang

[issue21429] Input.output error with multiprocessing

2016-10-17 Thread Florijan Hamzic
Florijan Hamzic added the comment: Hi, i have a similiar issue: i have a really simple scenario Callee: ``` @classmethod def MoveDataToOtherSide(cls, model) data = { "FirstName": model.FirstName, "LastName": model.LastName, "Email": model.Email,

[issue28462] subprocess pipe can't see EOF from a child in case of a few children run with subprocess

2016-10-17 Thread Vyacheslav Grigoryev
New submission from Vyacheslav Grigoryev: I'm creating a master stand-alone module on Python which should run some children via subprocess module. Working with children is done in separate worker threads. Additionally I need to receive real-time output from a child so in a worker thread I also

[issue28461] Replacement of re with the regex package

2016-10-17 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: See issue12734 for adding the support of properties in re. See issue2636 for replacing re with regex. See issue22594 for encouraging the use of regex. -- nosy: +serhiy.storchaka resolution: -> duplicate stage: -> resolved status: open -> closed sup

[issue28461] Replacement of re with the regex package

2016-10-17 Thread Pierre Nugues
New submission from Pierre Nugues: I am using Unicode regexes in the form of properties: \p{} and these are not standard in the re module. I have to use the new regex module, which has to be installed separately. I would like to see the replacement of re with regex in the future Python versio

[issue28404] Logging SyslogHandler not appending '\n' to the end

2016-10-17 Thread José Manuel
José Manuel added the comment: Sorry to bother you again, but I've tested this not only with Fluentd, but with a RSYSLOG server and it does not work with TCP except if you manually add the trailer LF character. Other than that, UDP default transport protocol has no issues and works fine with b

[issue8376] Tutorial offers dangerous advice about iterators: “__iter__() can just return self”

2016-10-17 Thread loic rowe
loic rowe added the comment: A new proposal is to add at the end of the paragraph on the iterator those point: """ As you should have noticed this Reverse let you iterate over the data only once:: >>> rev = Reverse('spam') >>> for char in rev: ... print(char) ... m a p

[issue28460] Minidom, order of attributes, datachars

2016-10-17 Thread Petr Pulc
New submission from Petr Pulc: Hello, just an idea for improvement of minidom. Sometimes it is not convenient that the element attributes are sorted alphabetically. Usually, users do hack the minidom file themselves to force some behaviour, yet the order can be quite nicely defined by the DTD

[issue27806] 2.7 32-bit builds fail on macOS 10.12 Sierra due to dependency on deleted header file QuickTime/QuickTime.h

2016-10-17 Thread Lapsang Leaf
Lapsang Leaf added the comment: Can we have this in English, please, for noobs? I only use python in order to run zim on my mac. I do not have the same technical know-how as many users and I do not understand the parlance here. Currently I have OS X 10.12 with python 2.7 - updating to sierra

[issue28459] _pyio module broken on Cygwin / setmode not usable

2016-10-17 Thread Erik Bray
New submission from Erik Bray: Since the patch to #24881 was merged the _pyio module has been non-importable on Cygwin, due to an attempt to import from the msvcrt module. However, the msvcrt module is not built on Cygwin (Cygwin does not use MSVCRT). Cygwin's libc does, however, have _setmod

[issue24881] _pyio checks that `os.name == 'win32'` instead of 'nt'

2016-10-17 Thread STINNER Victor
STINNER Victor added the comment: "FWIW this patch broke the _pyio module on Cygwin," Please open a new issue, this one is closed. -- ___ Python tracker ___

[issue24881] _pyio checks that `os.name == 'win32'` instead of 'nt'

2016-10-17 Thread Erik Bray
Erik Bray added the comment: FWIW this patch broke the _pyio module on Cygwin, as the msvcrt module is not built on Cygwin. AFAICT this is only a problem for Python built with MSVCRT, which Cygwin does not use. When test case works as expected on Cygwin without this. -- nosy: +erik.

[issue28441] sys.executable is ambiguous on Cygwin without .exe suffix

2016-10-17 Thread Erik Bray
Erik Bray added the comment: I agree this has a slight change in behavior which I was at first hesitant about. But I think the previous behavior was wrong insofar as it was overly ambiguous. I agree it should apply on MSYS2 as well (I actually thought __CYGWIN__ was defined on MSYS2 but I co

[issue28458] from __future__ import print_function does not emulate the flush param from py3k

2016-10-17 Thread Berker Peksag
Berker Peksag added the comment: That's because the flush argument was added in Python 3.3 (after print() was backported to 2.7 via a future import) Thanks for the report. -- nosy: +berker.peksag resolution: -> not a bug stage: -> resolved status: open -> closed

[issue28458] from __future__ import print_function does not emulate the flush param from py3k

2016-10-17 Thread Attila-Mihaly Balazs
New submission from Attila-Mihaly Balazs: Doing the following in Python 2.7.12 does not work: from __future__ import print_function print(1, flush=True) It says: "'flush' is an invalid keyword argument for this function" While the following is a perfectly valid python 3k statement: print(1, f