[issue32865] os.pipe creates inheritable FDs with a bad internal state on Windows

2018-02-17 Thread Eryk Sun
Eryk Sun added the comment: > The first is a bug in os.pipe (creation of an inheritable descriptor > with non-inheritable underlying handle). This can be fixed by using > _open_osfhandle() correctly. This is the only issue to be addressed here, and it's an easy fix. The

[issue32865] os.pipe creates inheritable FDs with a bad internal state on Windows

2018-02-17 Thread Nathaniel Smith
Change by Nathaniel Smith : -- nosy: +njs ___ Python tracker ___ ___ Python-bugs-list

[issue32867] argparse assertion failure with multiline metavars

2018-02-17 Thread MaT1g3R
New submission from MaT1g3R : If I run this script with -h -8<-- from argparse import ArgumentParser mapping = ['123456', '12345', '12345', '123'] p = ArgumentParser('11') p.add_argument('-v', '--verbose', help='verbose mode',

[issue32862] os.dup2(fd, fd, inheritable=False) behaves inconsistently

2018-02-17 Thread Alexey Izbyshev
Alexey Izbyshev added the comment: @eryksun: Thank you for the note! I've commented on #32865. This adds even more inconsistency to this corner case. -- ___ Python tracker

[issue32865] os.pipe creates inheritable FDs with a bad internal state on Windows

2018-02-17 Thread Alexey Izbyshev
Alexey Izbyshev added the comment: >Also, it has to skip this check if the FD is flagged as a pipe, because a pipe >is likely opened in synchronous mode and blocked on a read in the parent, in >which case calling GetFileType would deadlock. Does an FD get flagged as a

[issue32866] zipimport loader.get_data() requires absolute zip file path

2018-02-17 Thread Barry A. Warsaw
New submission from Barry A. Warsaw : Over in https://gitlab.com/python-devs/importlib_resources/issues/48 we have a report of a FileNotFoundError when trying to read a resource from a zip file. Upon further debugging, I found that zipimport's loader.get_data() raises an

[issue32862] os.dup2(fd, fd, inheritable=False) behaves inconsistently

2018-02-17 Thread Eryk Sun
Eryk Sun added the comment: In Windows the CRT file descriptor is actually still inheritable. This only makes the underlying OS handle non-inheritable. I don't think there's a way to make an existing FD non-inheritable using public CRT functions. See issue 32865.

[issue32865] os.pipe creates inheritable FDs with a bad internal state on Windows

2018-02-17 Thread Eryk Sun
Eryk Sun added the comment: Note that the CRT checks at startup whether an inherited FD is valid by calling GetFileType. If the handle is invalid or not a File, then the FD effectively is not inherited. This doesn't completely avoid the problem, since there's still a

[issue30638] Additional dependencies and rule for `make regen-all`

2018-02-17 Thread Zachary Ware
Zachary Ware added the comment: New changeset c1b8aedfbabf6e5460b09f4792d80f18051d43d3 by Zachary Ware in branch '3.6': [3.6] bpo-30638: Add clinic to `make regen-all` (GH-5671) https://github.com/python/cpython/commit/c1b8aedfbabf6e5460b09f4792d80f18051d43d3

[issue21060] Better error message for setup.py upload command without sdist/bdist

2018-02-17 Thread Éric Araujo
Éric Araujo added the comment: Thanks for the feedback, I went with the cleaner phrasing. -- versions: +Python 3.8 ___ Python tracker

[issue21060] Better error message for setup.py upload command without sdist/bdist

2018-02-17 Thread Éric Araujo
Change by Éric Araujo : -- pull_requests: +5511 ___ Python tracker ___ ___

[issue30638] Additional dependencies and rule for `make regen-all`

2018-02-17 Thread miss-islington
miss-islington added the comment: New changeset d5be8e13b0ccf4473300d3a1672fc113702cf66c by Miss Islington (bot) in branch '3.7': bpo-30638: Add clinic to `make regen-all` (GH-5671)

[issue32716] setup.py register --repository is broken

2018-02-17 Thread Éric Araujo
Éric Araujo added the comment: There are many open issues with .pypirc. I don’t think that -r URL is a supported use case though; the docs should say that the usage is to add a section in .pypirc to define a repo, then use -r repo-section-name. -- versions:

[issue30638] Additional dependencies and rule for `make regen-all`

2018-02-17 Thread Zachary Ware
Change by Zachary Ware : -- pull_requests: +5510 ___ Python tracker ___ ___

[issue30638] Additional dependencies and rule for `make regen-all`

2018-02-17 Thread miss-islington
Change by miss-islington : -- pull_requests: +5509 ___ Python tracker ___

[issue30638] Additional dependencies and rule for `make regen-all`

2018-02-17 Thread Zachary Ware
Zachary Ware added the comment: New changeset d6ff8a7037903497eff95fa32bdac2b6adf71505 by Zachary Ware in branch 'master': bpo-30638: Add clinic to `make regen-all` (GH-5671) https://github.com/python/cpython/commit/d6ff8a7037903497eff95fa32bdac2b6adf71505 --

[issue32604] Expose the subinterpreters C-API in Python for testing use.

2018-02-17 Thread Yury Selivanov
Yury Selivanov added the comment: Eric, it looks like your recent commit introduced a refleak. We need to fix it before beta2. ~/d/p/cpython (master $) » ./python.exe -m test -R3:3 test_multiprocessing_fork Run tests sequentially 0:00:00 load avg: 2.52 [1/1]

[issue31333] Implement ABCMeta in C

2018-02-17 Thread Guido van Rossum
Guido van Rossum added the comment: > Isn't 800 lines of C code too high price for speeding up ABCs creation? I think it's well worth it. This has long felt as a sore point to me. On Sat, Feb 17, 2018 at 10:27 AM, Ivan Levkivskyi wrote: > > Ivan

[issue20928] xml.etree.ElementInclude does not include nested xincludes

2018-02-17 Thread Stefan Behnel
Change by Stefan Behnel : -- pull_requests: +5508 stage: -> patch review ___ Python tracker ___

[issue20928] xml.etree.ElementInclude does not include nested xincludes

2018-02-17 Thread Stefan Behnel
Stefan Behnel added the comment: The latest "fixed2" patch looks good to me, but the author didn't sign a contributors agreement. However, I did, and I already wrote the same thing for lxml, so I put together an initial PR. -- type: behavior -> enhancement

[issue31333] Implement ABCMeta in C

2018-02-17 Thread Ivan Levkivskyi
Ivan Levkivskyi added the comment: > Isn't 800 lines of C code too high price for speeding up ABCs creation? 800 lines of C code is not something hard to notice, so I suppose the answer is obvious for all people involved in the work on PR :-) > ...this can save just

[issue31333] Implement ABCMeta in C

2018-02-17 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Isn't 800 lines of C code too high price for speeding up ABCs creation? This will save several microseconds per ABC creation. Even if the program creates 1000 ABCs, this can save just several milliseconds at start-up.

[issue32865] os.pipe creates inheritable FDs with a bad internal state on Windows

2018-02-17 Thread Eryk Sun
New submission from Eryk Sun : File descriptors in Windows are implemented by the C runtime library's low I/O layer. The CRT maps native File handles to Unix-style file descriptors. Additionally, in order to support inheritance for spawn/exec, the CRT passes inheritable FDs

[issue32849] Fatal Python error: Py_Initialize: can't initialize sys standard streams

2018-02-17 Thread Alexey Izbyshev
Alexey Izbyshev added the comment: ktrace shows that dup(0) succeeded but fstat(0) failed. The symptom is the same as in #30225. Could you check whether any of the following quick tests produces the same error? python3 -c 'import os, subprocess, sys; r, w = os.pipe();

[issue32864] Visual glitches when animating ScrolledText instances using place geometry manager

2018-02-17 Thread Victor Domingos
New submission from Victor Domingos : In the current Python 3.7.0b1, on macOS 10.12.6 Sierra (also on 10.11 El Capitan), which seems to include a newer Tcl/tk version, there seems to be some visual glitches when using the place() geometry manager to animate a

[issue32846] Deletion of large sets of strings is extra slow

2018-02-17 Thread Marc-Andre Lemburg
Marc-Andre Lemburg added the comment: Reminds of some experiments someone did a while ago as part of the GIL removal attempts where the ref count integers are all kept in a separate array. The intent there was to be able to do locking on a single array rather than on individual

[issue32863] Missing support for Emojis in tkinter

2018-02-17 Thread Victor Domingos
New submission from Victor Domingos : In the current Python 3.7.0b1, on macOS 10.12.6 Sierra (also on 10.11 El Capitan), which seems to include a newer Tcl/tk version, it still does not support a variety of UTF characters, including Emoji characters that nowadays

[issue32846] Deletion of large sets of strings is extra slow

2018-02-17 Thread Luis Pedro Coelho
Luis Pedro Coelho added the comment: I think some of this conversation is going off-topic, but there is no disk-swapping in my case. I realize ours is not a typical setup, but our normal machines have 256GB of RAM and the "big memory" compute nodes are >=1TB. Normally,

[issue32854] Add ** Map Unpacking Support for namedtuple

2018-02-17 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: I wandering if it is worth to add a general function that takes an object and a sequence of keys and return a mapping proxy that maps attribute names to values. -- ___ Python tracker

[issue32854] Add ** Map Unpacking Support for namedtuple

2018-02-17 Thread Raymond Hettinger
Raymond Hettinger added the comment: > Would it be worth adding an example of unpacking such as, > t(**a._asdict()), or something similar to the documentation ? There are a myriad of uses for dictionaries and the namedtuple docs don't seem like to right place to

[issue22167] iglob() has misleading documentation (does indeed store names internally)

2018-02-17 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Unfortunately issue25596 didn't change anything about this issue. iglob() still stores names (actually DirEntry objects) of all files in a directory before starting yielding the first of them. Otherwise we cold exceed the limit

[issue32852] trace changes sys.argv from list to tuple

2018-02-17 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Thank you for your contribution Kyle. Usually backports a made after merging the original PR. Thanks to bots this is done in half-automatical way. -- resolution: -> fixed stage: patch review -> resolved status: open ->