[issue32164] IDLE: delete tabbedpages.py

2017-11-28 Thread Terry J. Reedy
Terry J. Reedy added the comment: New changeset 957aef7c41ccae8a60d3dbbe8863401bd270dc86 by Terry Jan Reedy (Miss Islington (bot)) in branch '3.6': bpo-32164: Delete unused file idlelib/tabbedpages.py (GH-4628) (#4631)

[issue32164] IDLE: delete tabbedpages.py

2017-11-28 Thread Terry J. Reedy
Change by Terry J. Reedy : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue9334] argparse does not accept options taking arguments beginning with dash (regression from optparse)

2017-11-28 Thread Evan Driscoll
Evan Driscoll added the comment: One last comment for the time being. I actually think *both* changes are valuable. Fixing the bug, well, fixes the bug if you can set the appropriate flag. The improved error message still helps for existing code and new code that *doesn't*

[issue9334] argparse does not accept options taking arguments beginning with dash (regression from optparse)

2017-11-28 Thread Evan Driscoll
Evan Driscoll added the comment: > I also think that nargs=## could maybe be special-cased to just ignore > the A/O designation completely and only check there are enough, but I > haven't tried this out. Does this seem like a viable approach? Would a > patch that does that,

[issue32164] IDLE: delete tabbedpages.py

2017-11-28 Thread Roundup Robot
Change by Roundup Robot : -- pull_requests: +4547 ___ Python tracker ___

[issue32164] IDLE: delete tabbedpages.py

2017-11-28 Thread Terry J. Reedy
Terry J. Reedy added the comment: New changeset 041efd292ebff46060fc1680b0608b1d4c876a48 by Terry Jan Reedy in branch 'master': bpo-32164: Delete unused file idlelib/tabbedpages.py (#4628) https://github.com/python/cpython/commit/041efd292ebff46060fc1680b0608b1d4c876a48

[issue30781] IDLE: configdialog -- switch to ttk widgets.

2017-11-28 Thread Terry J. Reedy
Terry J. Reedy added the comment: New changeset a8de6964ec46cdadb12cf2d45ebf1473659c by Terry Jan Reedy (Miss Islington (bot)) in branch '3.6': bpo-30781: Remove unused imports in idlelib.configdialog. (GH-4627) (#4630)

[issue30780] IDLE: configdialog - add tests for ConfigDialog GUI.

2017-11-28 Thread Terry J. Reedy
Terry J. Reedy added the comment: Review of overview: PR-2706 and PR-2613 fixed one line in test_configdialog for this issue. PR-2046, #30617 and PR-3238, #30781 are merged cross-references. PR-3222 was closed in favor of PR-3220, #31287, merged. PR-3592 completes highlights

[issue32156] Fix flake8 warning F401: ... imported but unused

2017-11-28 Thread Terry J. Reedy
Terry J. Reedy added the comment: Idlelib deletions done. pathbrowser, #32100, PR-4626 configdialog, #30781, PR-4627 -- nosy: +terry.reedy ___ Python tracker

[issue32100] IDLE: PathBrowser isn't working

2017-11-28 Thread Terry J. Reedy
Terry J. Reedy added the comment: New changeset 38ecf57306ef25874ef7c44a9875771e90faee97 by Terry Jan Reedy (Miss Islington (bot)) in branch '3.6': bpo-32100: Delete unneeded import in idlelib.pathbrowser. (GH-4626) (#4629)

[issue32122] Improve -x option documentation

2017-11-28 Thread Steve Dower
Steve Dower added the comment: Those examples look fine to me -- ___ Python tracker ___

[issue32165] PyEval_InitThreads is called before Py_Initialize in LoadPython in Modules/_ctypes/callbacks.c

2017-11-28 Thread Kenta Murata
New submission from Kenta Murata : As described in Doc/c-api/init.rst, PyEval_InitThreads() cannot be called before Py_Initialize() function. But in Modules/_ctypes/callbacks.c, PyEval_InitThreds() is called before Py_Initialize() in LoadPython function. --

[issue30781] IDLE: configdialog -- switch to ttk widgets.

2017-11-28 Thread Roundup Robot
Change by Roundup Robot : -- pull_requests: +4545 ___ Python tracker ___

[issue30781] IDLE: configdialog -- switch to ttk widgets.

2017-11-28 Thread Terry J. Reedy
Terry J. Reedy added the comment: New changeset e8f7c78a868834bb53fa0ac903fc87785112f49e by Terry Jan Reedy in branch 'master': bpo-30781: Remove unused imports in idlelib.configdialog. (#4627) https://github.com/python/cpython/commit/e8f7c78a868834bb53fa0ac903fc87785112f49e

[issue32100] IDLE: PathBrowser isn't working

2017-11-28 Thread Roundup Robot
Change by Roundup Robot : -- pull_requests: +4544 ___ Python tracker ___

[issue32100] IDLE: PathBrowser isn't working

2017-11-28 Thread Terry J. Reedy
Terry J. Reedy added the comment: New changeset fd6f8c56b9a757210e8ceaea9edc50e502dc2085 by Terry Jan Reedy in branch 'master': bpo-32100: Delete unneeded import in idlelib.pathbrowser. (#4626) https://github.com/python/cpython/commit/fd6f8c56b9a757210e8ceaea9edc50e502dc2085

[issue32164] IDLE: delete tabbedpages.py

2017-11-28 Thread Terry J. Reedy
Change by Terry J. Reedy : -- keywords: +patch pull_requests: +4543 stage: needs patch -> patch review ___ Python tracker ___

[issue32164] IDLE: delete tabbedpages.py

2017-11-28 Thread Terry J. Reedy
New submission from Terry J. Reedy : idlelib.tabbedpages.TabbedPageSet is a custom widget only used for the tabbed config dialog. #30781 converted to using tkinter.ttk.Notebook. The latter has proven to be an improvement and the former has no other use, nor would I

[issue30781] IDLE: configdialog -- switch to ttk widgets.

2017-11-28 Thread Terry J. Reedy
Change by Terry J. Reedy : -- pull_requests: +4542 ___ Python tracker ___ ___

[issue32163] getattr() returns None even when default is given

2017-11-28 Thread Steven D'Aprano
Steven D'Aprano added the comment: That's not a bug. That's because the file object does have an encoding attribute, which is set to None. getattr only returns the default when the attribute doesn't exist, not if it exists but is None. -- nosy:

[issue32100] IDLE: PathBrowser isn't working

2017-11-28 Thread Terry J. Reedy
Change by Terry J. Reedy : -- pull_requests: +4541 ___ Python tracker ___ ___

[issue32163] getattr() returns None even when default is given

2017-11-28 Thread darkdragon-001
New submission from darkdragon-001 : # fail.py def main(): patch = './a' f = open(patch, 'r') a = getattr(f,'encoding','ascii') print(str(a)) if __name__ == "__main__": main() --- $ touch a $ python fail.py It still prints out 'None' instead of

[issue32030] PEP 432: Rewrite Py_Main()

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +4540 ___ Python tracker ___ ___

[issue32110] Make codecs.StreamReader.read() more compatible with read() of other files

2017-11-28 Thread Serhiy Storchaka
Change by Serhiy Storchaka : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue32110] Make codecs.StreamReader.read() more compatible with read() of other files

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: New changeset fc73c54dae46e6c47dcd4a535f7bc68a46b8e398 by Serhiy Storchaka (Miss Islington (bot)) in branch '2.7': bpo-32110: codecs.StreamReader.read(n) now returns not more than n (GH-4499) (#4623)

[issue32101] Add PYTHONDEVMODE=1 to enable the developer mode

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: While implementing a new PYTHONDEVMODE environment variable, I found that it would help to also add a new sys.flags.dev_mode flag. So I added it. So for example, asyncio uses sys.flags.dev_mode rather than checking for 'dev' in

[issue32101] Add PYTHONDEVMODE=1 to enable the developer mode

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +4539 ___ Python tracker ___ ___

[issue9334] argparse does not accept options taking arguments beginning with dash (regression from optparse)

2017-11-28 Thread Evan Driscoll
Evan Driscoll added the comment: I ran into this issue today. (Or rather a couple weeks ago, and I just diagnosed it today.) Reading through the thread and from the bug's age it looks like a fix is probably not to promising, but Cherniavsky Beni's 2016-04-11 22:03 comment

[issue32110] Make codecs.StreamReader.read() more compatible with read() of other files

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: New changeset 230ffeae0a3961b1769806bd722c26227c84e8da by Serhiy Storchaka (Miss Islington (bot)) in branch '3.6': bpo-32110: codecs.StreamReader.read(n) now returns not more than n (GH-4499) (#4622)

[issue32162] typing.Generic breaks __init_subclass__

2017-11-28 Thread Ilya Kulakov
Ilya Kulakov added the comment: That's a better workaround: class X(typing.Generic[T]): def __init_subclass__(cls, **kwargs): super(typing.GenericMeta, cls).__setattr__('_gorg', cls) super().__init_subclass__(**kwargs) --

[issue32110] Make codecs.StreamReader.read() more compatible with read() of other files

2017-11-28 Thread Roundup Robot
Change by Roundup Robot : -- pull_requests: +4538 ___ Python tracker ___

[issue32110] Make codecs.StreamReader.read() more compatible with read() of other files

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: New changeset 219c2de5ad0fdac825298bed1bb251f16956c04a by Serhiy Storchaka in branch 'master': bpo-32110: codecs.StreamReader.read(n) now returns not more than n (#4499)

[issue32110] Make codecs.StreamReader.read() more compatible with read() of other files

2017-11-28 Thread Roundup Robot
Change by Roundup Robot : -- pull_requests: +4537 ___ Python tracker ___

[issue32122] Improve -x option documentation

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: I don't know if "@path\to\python -x %0 %* & exit /b" and "@py -3 -x %0 %* & exit /b" are correct and well supported on all recent Windows versions. Apart of that, I like your new documentation :-) -- nosy: +vstinner

[issue9334] argparse does not accept options taking arguments beginning with dash (regression from optparse)

2017-11-28 Thread Evan Driscoll
Change by Evan Driscoll : -- nosy: +evaned ___ Python tracker ___ ___ Python-bugs-list

[issue32124] Document functions safe to be called before Py_Initialize()

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +4536 ___ Python tracker ___ ___

[issue32162] typing.Generic breaks __init_subclass__

2017-11-28 Thread Ilya Kulakov
Ilya Kulakov added the comment: Nah, that's a bad one: you cannot use Generic classes as intended by specifying types. It looks like it happens because cls._grog is not yet set properly by the time __init_subclass__ is called. --

[issue32161] Python 2.7.14 installation on Ubuntu 16.04/GCC 5.4 throws "internal compiler error" in "PyFloat_GetMax" function

2017-11-28 Thread Rahul
Rahul added the comment: Thanks for your response. I'll follow up with GCC. On Nov 28, 2017 14:48, "STINNER Victor" wrote: > > STINNER Victor added the comment: > > "internal compiler error" is a bug in GCC, not

[issue31279] Squash new gcc warning (-Wstringop-overflow)

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +4535 ___ Python tracker ___ ___

[issue30923] Add -Wimplicit-fallthrough=0 to Makefile ?

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +4534 ___ Python tracker ___ ___

[issue32137] Stack overflow in repr of deeply nested dicts

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- nosy: +vstinner ___ Python tracker ___ ___

[issue32162] typing.Generic breaks __init_subclass__

2017-11-28 Thread Ilya Kulakov
Ilya Kulakov added the comment: Current workaround is class X(typing.Generic[T] if typing.TYPE_CHECKING else object): -- ___ Python tracker

[issue32161] Python 2.7.14 installation on Ubuntu 16.04/GCC 5.4 throws "internal compiler error" in "PyFloat_GetMax" function

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: Once you reported the bug to GCC, please add a link to your bug report here, please ;-) -- ___ Python tracker

[issue32162] typing.Generic breaks __init_subclass__

2017-11-28 Thread Ilya Kulakov
Ilya Kulakov added the comment: This issue is more server that I expected: it doesn't just propagate value to superclasses, but overrides them. The last subclass created by Python runtime will overwrite value for the whole chain. --

[issue32161] Python 2.7.14 installation on Ubuntu 16.04/GCC 5.4 throws "internal compiler error" in "PyFloat_GetMax" function

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: "internal compiler error" is a bug in GCC, not in Python. Please follow instructions in the error message: Please submit a full bug report, with preprocessed source if appropriate. See for instructions. See

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread Ned Deily
Ned Deily added the comment: Sorry, I didn't see that the change included the deletion of svnmap.txt. However, I don't know where or by what that file might still be used, particularly by the lookup function used by the bug tracker. Presumably someone (Ezio, RDM, or Brett)

[issue32162] typing.Generic breaks __init_subclass__

2017-11-28 Thread Ilya Kulakov
New submission from Ilya Kulakov : When superclass inherits from Generic, attributes set for a subclass are incorrectly propagated to its superclass. Without Generic attribute access raises an exception: class X: def __init_subclass__(cls, **kwargs):

[issue32107] Improve MAC address calculation and fix test_uuid.py

2017-11-28 Thread Barry A. Warsaw
Barry A. Warsaw added the comment: New changeset 23df2d1304ece169d7e0dfc843dfb8026b413d9f by Barry Warsaw in branch 'master': bpo-32107 - Improve MAC address calculation and fix test_uuid.py (#4600)

[issue32155] Fix flake8 warning F841: local variable ... is assigned to but never used

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: The main bugs have been fixed in Python 3.6 and 3.7 (master). Python 2.7 only has the bug in Tools/scripts/treesync.py, but I don't think that it's worth it to fix this old script in the stable 2.7 branch. I just removed this script

[issue32155] Fix flake8 warning F841: local variable ... is assigned to but never used

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: New changeset 71bd588646b282c9eebc390b31184a5bdb54e712 by Victor Stinner in branch 'master': bpo-32155: Revert distutils.config change (#4618) https://github.com/python/cpython/commit/71bd588646b282c9eebc390b31184a5bdb54e712

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: svnmap.txt was added in a single revision and was not changed anymore. commit 533c79dc67824d95b0aff5a252f9415d3bc80716 Author: Antoine Pitrou Date: Sat Mar 5 17:46:03 2011 +0100 Add the map of SVN

[issue32155] Fix flake8 warning F841: local variable ... is assigned to but never used

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: New changeset 6979fcdc91114b1ccb16345e26734d6df4cccbc3 by Victor Stinner in branch '3.6': bpo-32155: Bugfixes found by flake8 F841 warnings (#4619) https://github.com/python/cpython/commit/6979fcdc91114b1ccb16345e26734d6df4cccbc3

[issue32161] Python 2.7.14 installation on Ubuntu 16.04/GCC 5.4 throws "internal compiler error" in "PyFloat_GetMax" function

2017-11-28 Thread Rahul
New submission from Rahul : Attempt to `make` Python 2.7.14 on Ubuntu 16.04/GCC 5.4 throws the following error: Objects/floatobject.c: In function ‘PyFloat_GetMax’: Objects/floatobject.c:59:5: internal compiler error: Illegal instruction return DBL_MAX; ^

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: Serhiy: "Is svnmap.txt used for mapping Subversion references to Mercurial references on the tracker? r88752" Brett, David, Ezio: any idea for Serhiy's question? -- nosy: +brett.cannon, ezio.melotti, r.david.murray

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Is svnmap.txt used for mapping Subversion references to Mercurial references on the tracker? r88752 -- ___ Python tracker

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: > The PR was approved by Brett Cannon and Ned Deily. Oh, and Mariatta after I added a NEWS entry. -- ___ Python tracker

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: > Are you sure that these scripts are not used outside of CPython? How would someone use it? There are only part of the code base, they are not installed on the system (by "make install"). If someone uses these scripts on another

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Oh, you are VERY fast in committing Victor. Please restore these scripts. -- ___ Python tracker

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Are you sure that these scripts are not used outside of CPython? They were here when CPython used Subversion, they were here when CPython used Mercurial, what is wrong with Git? The support of the svn:eol property is a thing I

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: New changeset fe2d5babba5d26de2093b6518316b268488187be by Victor Stinner in branch 'master': bpo-32159: Remove tools for CVS and Subversion (#4615) https://github.com/python/cpython/commit/fe2d5babba5d26de2093b6518316b268488187be

[issue32157] Remove explicit quotes around %r and {!r}

2017-11-28 Thread Serhiy Storchaka
Change by Serhiy Storchaka : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue32155] Fix flake8 warning F841: local variable ... is assigned to but never used

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +4533 ___ Python tracker ___ ___

[issue32155] Fix flake8 warning F841: local variable ... is assigned to but never used

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: I proposed PR 4618 to revert the distutils.config changes. -- ___ Python tracker ___

[issue32155] Fix flake8 warning F841: local variable ... is assigned to but never used

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- pull_requests: +4532 ___ Python tracker ___ ___

[issue32157] Remove explicit quotes around %r and {!r}

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: New changeset a4a3020abc065d40f57069d6c69d0ddc85d6 by Serhiy Storchaka in branch 'master': bpo-32157: Removed explicit quotes around %r and {!r}. (#4582)

[issue32160] lzma documentation: example to XZ compress file on disk

2017-11-28 Thread Daniel Himmelstein
Daniel Himmelstein added the comment: > we could consider an API addition Just brainstorming here... perhaps an API addition would be most appropriate in the shutil module (https://docs.python.org/3.6/library/shutil.html) which already contains

[issue27535] Ignored ResourceWarning warnings leak memory in warnings registries

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: I ran a quick benchmark on Python 3.6: current code (3.6 branch, ref) => PR 4587 (patch): vstinner@apu$ python3 -m perf compare_to ref.json patch.json Mean +- std dev: [ref] 597 ns +- 10 ns -> [patch] 830 ns +- 15 ns: 1.39x slower

[issue32160] lzma documentation: example to XZ compress file on disk

2017-11-28 Thread Daniel Himmelstein
Change by Daniel Himmelstein : -- assignee: -> docs@python components: +Documentation nosy: +docs@python type: -> enhancement ___ Python tracker

[issue31732] Add TRACE level to the logging module

2017-11-28 Thread Antoine Pitrou
Antoine Pitrou added the comment: Perhaps a recipe should be published to explain how to add your own levels? >>> import logging >>> logging.NOTE = logging.INFO + 5 >>> logging.addLevelName(logging.INFO + 5, 'NOTE') >>> class MyLogger(logging.getLoggerClass()): ...:def

[issue32160] lzma documentation: example to XZ compress file on disk

2017-11-28 Thread Daniel Himmelstein
New submission from Daniel Himmelstein : The documentation for the lzma module currently contains 6 examples (https://docs.python.org/3.6/library/lzma.html#examples). However, it does not include an example to XZ compress a file on disk. The functionality I'm

[issue31732] Add TRACE level to the logging module

2017-11-28 Thread Antoine Pitrou
Change by Antoine Pitrou : -- Removed message: https://bugs.python.org/msg307162 ___ Python tracker ___

[issue31732] Add TRACE level to the logging module

2017-11-28 Thread Antoine Pitrou
Antoine Pitrou added the comment: Perhaps a recipe should be published to explain how to add your own levels? e.g.: >>> import logging >>> logging.NOTE = logging.INFO + 5 >>> logging.addLevelName(logging.INFO + 5, 'NOTE') >>> class MyLogger(logging.Logger): ...:def

[issue27535] Ignored ResourceWarning warnings leak memory in warnings registries

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: Serhiy: "I meant the original issue. AFAIK ResourceWarningis not raised in 2.7. And other warnings less likely have unique messages. Seems the problem is less critical in 2.7." Oh yes, sorry, I forgot your comment and I forgot that

[issue32154] asyncio: Don't export selectors and _overlapped in asyncio namespace

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue32154] asyncio: Don't export selectors and _overlapped in asyncio namespace

2017-11-28 Thread STINNER Victor
STINNER Victor added the comment: New changeset ac577d7d0bd27a69921ced14c09172235ceebab5 by Victor Stinner in branch 'master': bpo-32154: Remove asyncio.windows_utils.socketpair (#4609) https://github.com/python/cpython/commit/ac577d7d0bd27a69921ced14c09172235ceebab5

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread STINNER Victor
Change by STINNER Victor : -- keywords: +patch pull_requests: +4531 stage: -> patch review ___ Python tracker ___

[issue32159] Remove tools for CVS and Subversion

2017-11-28 Thread STINNER Victor
New submission from STINNER Victor : CPython migrated from CVS to Subversion, to Mercurial, and then to Git. CVS and Subversion are no more used to develop CPython. Changes of attached PR: * platform module: drop support for sys.subversion. The sys.subversion

[issue32158] Suppress (and other contextlib context managers) should work as decorators (where appropriate)

2017-11-28 Thread Barry A. Warsaw
Change by Barry A. Warsaw : -- nosy: +barry ___ Python tracker ___ ___ Python-bugs-list

[issue32158] Suppress (and other contextlib context managers) should work as decorators (where appropriate)

2017-11-28 Thread Jason R. Coombs
Jason R. Coombs added the comment: Fair enough. For an example, here's the case where I wanted to use the decorator to avoid excess indentation and keep the most meaningful part of the function at the base of the body: @suppress(KeyError) def v12_to_13(manager, case):

[issue32158] Suppress (and other contextlib context managers) should work as decorators (where appropriate)

2017-11-28 Thread Raymond Hettinger
Raymond Hettinger added the comment: > But it got me thinking - couldn't suppress (and possibly other) > contextlib decorators support this usage out of the box? They possibly could but probably shouldn't. My experience is that giving them a dual role makes them

[issue32071] Add py.test-like "-k" test selection to unittest

2017-11-28 Thread Antoine Pitrou
Antoine Pitrou added the comment: New changeset 4d193bcc2560b824389e4d98d9d8b9b34e33dbaf by Antoine Pitrou (Jonas Haag) in branch 'master': bpo-32071: Fix regression and add What's New entry (#4589)

[issue32158] Suppress (and other contextlib context managers) should work as decorators (where appropriate)

2017-11-28 Thread Jason R. Coombs
New submission from Jason R. Coombs : The contextlib docs explain why a decorator is nice: > It makes it clear that the cm applies to the whole function, rather than just > a piece of it (and saving an indentation level is nice, too). However, the built-in context managers

[issue16487] Allow ssl certificates to be specified from memory rather than files.

2017-11-28 Thread Senthil Kumaran
Senthil Kumaran added the comment: Hello JP, I had been little inactive for a while (>2 years). However, I think, I can take now some load in CPython world and I will take this ticket/feature forward this Sunday (1-Dec-2017). If Christian (or currently active committers)

[issue16487] Allow ssl certificates to be specified from memory rather than files.

2017-11-28 Thread Jan-Philip Gehrcke
Jan-Philip Gehrcke added the comment: Hello! I would very much appreciate if we can find a way for us to get another review for the last patch. I did most of the work in August 2016 and got a review from Senthil and Christian which I processed. When I got back to the

[issue32157] Remove explicit quotes around %r and {!r}

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Yes, these are the only instances of quotes used with %r or {!r}. -- ___ Python tracker

[issue32157] Remove explicit quotes around %r and {!r}

2017-11-28 Thread Ethan Furman
Change by Ethan Furman : -- Removed message: https://bugs.python.org/msg307149 ___ Python tracker ___

[issue32157] Remove explicit quotes around %r and {!r}

2017-11-28 Thread Ethan Furman
Ethan Furman added the comment: Yup, I sure did. :/ Looks like a good change to me. Are these the only instances of quotes used with %r? -- ___ Python tracker

[issue32157] Remove explicit quotes around %r and {!r}

2017-11-28 Thread Yury Selivanov
Change by Yury Selivanov : -- nosy: -yselivanov ___ Python tracker ___ ___

[issue32157] Remove explicit quotes around %r and {!r}

2017-11-28 Thread Antoine Pitrou
Antoine Pitrou added the comment: Ethan, I think you misunderstood Serhiy's proposal. -- ___ Python tracker ___

[issue32157] Remove explicit quotes around %r and {!r}

2017-11-28 Thread Ethan Furman
Ethan Furman added the comment: I make sure to use %r in error-reporting and debugging messages precisely because it's output is (usually) valid python. If I see a 1 without quotes I know it's a number, and with quotes I know it's a string. -1 -- nosy:

[issue32157] Remove explicit quotes around %r and {!r}

2017-11-28 Thread Serhiy Storchaka
New submission from Serhiy Storchaka : It is common to use quotes around %s. If the formatted argument is a string this will produce a string in qoutes, that look as a string literal in Python and other programming languages or just as a quoted string in English.

[issue31440] wrong default module search path in help message

2017-11-28 Thread Christian Heimes
Christian Heimes added the comment: New changeset cb79c2203953bd465f2c53b7a09a51071717575f by Christian Heimes (Miss Islington (bot)) in branch '3.6': bpo-31440: Changed default module search path for windows (#4613)

[issue32157] Remove explicit quotes around %r and {!r}

2017-11-28 Thread Serhiy Storchaka
Change by Serhiy Storchaka : -- keywords: +patch pull_requests: +4530 ___ Python tracker ___

[issue31440] wrong default module search path in help message

2017-11-28 Thread Christian Heimes
Christian Heimes added the comment: I'll do a backport tomorrow. -- assignee: -> christian.heimes ___ Python tracker ___

[issue32072] Issues with binary plists

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Ronald, could you please make a review? I want to merge this before 3.7.0a3. -- ___ Python tracker

[issue32155] Fix flake8 warning F841: local variable ... is assigned to but never used

2017-11-28 Thread Éric Araujo
Éric Araujo added the comment: The best course of action would be to revert the distutils changes. There are a handful of tickets related to register/upload commands, which are funky commands with their own config file not 100% in line with other distutils commands.

[issue32046] 2to3 fix for operator.isCallable()

2017-11-28 Thread Éric Araujo
Éric Araujo added the comment: Merged in master, I suppose stable branches should be left alone? -- ___ Python tracker ___

[issue32046] 2to3 fix for operator.isCallable()

2017-11-28 Thread Éric Araujo
Éric Araujo added the comment: New changeset a489599793f9b00ddc2c68e2ce3bce9cbb2c09a2 by Éric Araujo (Dong-hee Na) in branch 'master': bpo-32046: Update 2to3 when converts operator.isCallable(obj). (#4417)

[issue32156] Fix flake8 warning F401: ... imported but unused

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: I suggest to split your patch on several PR for different domains, and let the domain experts review changes separately. IDLE changes look mostly harmless to me. But changes in distutils should be made with a large care.

[issue32150] Expand tabs to spaces in C files

2017-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Thanks Guido! I think now we can revive issue8912. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

  1   2   >