[issue26612] test_ssl: use context manager (with) to fix ResourceWarning

2016-03-23 Thread Martin Panter
Martin Panter added the comment: My point about Python 2 was that this might add an annoying difference between the two branches. Even if you don’t apply this to Py 2, the next person adding or updating a test might be inclined to do it. Looking closer at your patch I admit that addCleanup() w

[issue22401] argparse: 'resolve' conflict handler damages the actions of the parent parser

2016-03-23 Thread paul j3
paul j3 added the comment: Is this the kind of scenario that you have problems with? parser = argparse.ArgumentParser() sp = parser.add_subparsers(dest='cmd') p1 = sp.add_parser('cmd1') p1.add_argument('-f') p2 = sp.add_parser('cmd2') p2.add_argument('-b') p3 = sp.a

[issue26632] __all__ decorator

2016-03-23 Thread Barry A. Warsaw
New submission from Barry A. Warsaw: This is probably terrible, but given how difficult it is to keep __all__'s up to date, maybe something like this can be useful. I literally whipped this up in about 5 minutes, so sit back and watch the bikeshedding! import sys def public(thing): if is

[issue26632] __all__ decorator

2016-03-23 Thread Barry A. Warsaw
Barry A. Warsaw added the comment: Oh, and it should be a built-in -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubs

[issue26632] __all__ decorator

2016-03-23 Thread Ethan Furman
Changes by Ethan Furman : -- nosy: +ethan.furman ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.p

[issue26632] __all__ decorator

2016-03-23 Thread Ethan Furman
Ethan Furman added the comment: def public(thing, value=None): if isinstance(thing, str): mdict = sys._getframe(1).f_globals name = thing mdict[name] = thing # no need for retyping! ;) else: mdict = sys.modules[thing.__module__].__dict__ name = thi

[issue26627] IDLE incorrectly labeling error as internal

2016-03-23 Thread Tadhg McDonald-Jensen
Tadhg McDonald-Jensen added the comment: other then bdb.py all of the excluded modules are imported into idlelib.run so that line could be replaced with: import bdb exclude = (__file__, rpc.__file__, threading.__file__, queue.__file__, RemoteDebugger.__file__, bdb.__file__) althoug

[issue9694] argparse required arguments displayed under "optional arguments"

2016-03-23 Thread paul j3
paul j3 added the comment: I can see changing the group title from 'optional arguments' to 'options' or 'optionals' parser._optionals.title 'optional arguments' But I don't think there's a need to change references in the code or its comments from 'optionals' to 'options'. I like the

[issue26633] multiprocessing behavior combining daemon with non-daemon children inconsistent with threading

2016-03-23 Thread Josh Rosenberg
New submission from Josh Rosenberg: Unclear if this is just unclear docs, or incorrect behavior. Per this Stack Overflow question ( https://stackoverflow.com/questions/36191447/why-doesnt-the-daemon-program-exit-without-join ), you get some rather odd behavior when you have both daemon and non

[issue26633] multiprocessing behavior combining daemon with non-daemon children inconsistent with threading

2016-03-23 Thread Josh Rosenberg
Josh Rosenberg added the comment: Oops, left the info log in my replacement code for the non-daemon case outside the if not daemon: block; it should be inside (since it's not joining the daemon threads). -- ___ Python tracker

[issue26631] Unable to install Python 3.5.1 on Windows 10 - Error 0x80070643: Failed to install MSI package.

2016-03-23 Thread Zachary Ware
Changes by Zachary Ware : -- components: +Windows nosy: +paul.moore, steve.dower, tim.golden, zach.ware ___ Python tracker ___ ___ Pyt

[issue26634] recursive_repr forgets to override __qualname__ of wrapper

2016-03-23 Thread Xiang Zhang
New submission from Xiang Zhang: In reprlib.recursive_repr, it seems __qualname__ is forgotten. Giving the example in reprlib document, it gives a strange result 'recursive_repr..decorating_function..wrapper' of Mylist.__repr__.__qualname__. I simply add the assignment of __qualname__. But I

[issue26634] recursive_repr forgets to override __qualname__ of wrapper

2016-03-23 Thread Raymond Hettinger
Raymond Hettinger added the comment: FWIW, qualname was invented *after* recursive_repr, so it would be more accurate to say that qualname forgot recursive_repr :-) If you don't mind, please add a test to your patch. -- assignee: -> rhettinger nosy: +rhettinger __

[issue26616] A bug in datetime.astimezone() method

2016-03-23 Thread Brian Guo
Brian Guo added the comment: >From the datetime documentation of astimezone(): https://docs.python.org/3.5/library/datetime.html#datetime.datetime.astimezone : 'If called without arguments (or with tz=None) the system local timezone is assumed. The tzinfo attribute of the converted datetime in

[issue26630] Windows EXE extension installers not finding 32bit Python 3.5 installation

2016-03-23 Thread Eryk Sun
Changes by Eryk Sun : -- components: +Windows nosy: +paul.moore, steve.dower, tim.golden, zach.ware resolution: -> duplicate stage: -> resolved status: open -> closed superseder: -> bdist_wininst created binaries fail to start and find 32bit Pythons _

[issue26634] recursive_repr forgets to override __qualname__ of wrapper

2016-03-23 Thread Xiang Zhang
Xiang Zhang added the comment: Hi, Raymond, I've added the test to test the assigned attributes. I use WRAPPER_ASSIGNMENTS in functools so we can keep them consistent. I don't know how to get the original func from the already decorated methods in this case so I define a new container. If there

<    1   2