[issue25455] Some repr implementations don't check for self-referential structures

Serhiy Storchaka added the comment: Added ElementTree and io modules experts to the nosy list. -- nosy: +benjamin.peterson, eli.bendersky, pitrou, scoder, stutzbach ___ Python tracker

[issue25539] python3 fail on parsing http header

Martin Panter added the comment: Are you able to print out the repr() of the header or the entire HTTP response so we can see exactly what characters are there? Or provide a URL if it is a public server. I suspect it may not be a completely blank line, but may have whitespace there. Both

[issue25539] python3 fail on parsing http header

Changes by Martin Panter : -- type: crash -> behavior ___ Python tracker ___ ___

[issue25455] Some repr implementations don't check for self-referential structures

Serhiy Storchaka added the comment: Recursive partial objects are legitimate. Here is a patch that makes partial's repr to support recursive partial objects. Also added a test for pickling. Cases for Element and file-like objects are questionable. Recursive Element.tag and TextIOWrapper.name

[issue25540] virtual conf file encoding error on windows

Serhiy Storchaka added the comment: Already fixed in issue25185. -- nosy: +serhiy.storchaka resolution: -> out of date stage: -> resolved status: open -> closed superseder: -> Inconsistency between venv and site ___ Python tracker

[issue25300] Enable Intel MPX (Memory protection Extensions) feature

Florin Papa added the comment: Please see the updated patch after Zachary Ware's review. Thank you for the feedback. -- Added file: http://bugs.python.org/file40931/mpx_enable_3_6_v6.patch ___ Python tracker

[issue25532] infinite loop when running inspect.unwrap over unittest.mock.call

Michael Foord added the comment: Have you read the docs for call? :-) call.foo("foo") generates a call object representing a method call to the method foo. So you can do. m = Mock() m.foo("foo") self.assertEqual(m.mock_calls, call.foo("foo")) (etc) See also the call.call_list

[issue25532] infinite loop when running inspect.unwrap over unittest.mock.call

Michael Foord added the comment: Preventing the use of "__" attributes would limit the usefulness of call with MagicMock and the use of magic methods. That might be an acceptable trade-off, but would break existing tests for people. (i.e. it's a backwards incompatible change.) --

[issue25537] Call `isinstance` instead of `issubclass` during exception handling

R. David Murray added the comment: Ah, now I understand. That is definitely not something that could go into 2.7. It is also a level of change that would need to go to python-ideas first, and possibly even a PEP. Closing this pending such a discussion. -- dependencies: -Catching

[issue25539] python3 fail on parsing http header

Shin Dong-won added the comment: I had a mistake. That's not CRLF but just two '\n'. -- ___ Python tracker ___

[issue25541] Wrong usage of sockaddr_un struct for abstract namespace unix sockets

Changes by PrzemeK : -- components: IO, Library (Lib) nosy: soutys priority: normal severity: normal status: open title: Wrong usage of sockaddr_un struct for abstract namespace unix sockets type: behavior versions: Python 2.7, Python 3.4

[issue25538] Traceback from __exit__ method is misleading

R. David Murray added the comment: What caught me out was more like this (but more complicated so it wasn't obvious the line mentioned wasn't part of the exception chain; in fact it looked like it was since it was a line that could very well have raised the exception in question):

[issue25532] infinite loop when running inspect.unwrap over unittest.mock.call

Chris Withers added the comment: Indeed, I guess Venusian will get confused, but not sure thats solvable as there will be obvious bugs indicating that call shouldn't be imported at module-level. This does feel like the problem is actually with inspect.unwrap: there's evidence of an attempt

[issue25542] tuple unpacking on assignment should be lazy for unpacked generators

Martin Panter added the comment: But when the starred element is not the final one, it couldn’t be lazy, so you would end up with an inconsistency. a, *rest, z = Also, it would be inconsistent with other kinds of iterables, which get turned into list objects. >>> a, *aa = (1, 2) >>> aa [2]

[issue25542] tuple unpacking on assignment should be lazy for unpacked generators

New submission from Mario Wenzel: if I have an assignment a = then a is the generator. But if I do any kind of unpacking *a, = # a list of all items a, *aa = # first item in a, list of rest in as If the right-hand side is a generator expression, I would expect that a, *aa unpacks the

[issue25300] Enable Intel MPX (Memory protection Extensions) feature

Raymond Hettinger added the comment: [ Stefan Krah] > I'd suggest to wait until the mpx support is stable in gcc. I concur. -- ___ Python tracker ___

[issue25543] locale.atof keep '.' even if not part of the localeconv

New submission from Cédric Krier: If a value containing '.' is passed to locale.atof, but '.' is not the locale decimal_point (and not the thousands_sep), it is anyway parsed as the decimal_point. For me, it should raise a ValueError -- components: Library (Lib) messages: 253989 nosy:

[issue25388] tokenizer crash/misbehavior -- heap use-after-free

Changes by Serhiy Storchaka : -- assignee: -> serhiy.storchaka ___ Python tracker ___

[issue25539] python3 fail on parsing http header

Shin Dong-won added the comment: I found that Python3 no more uses header format from rfc822, and the data b'P3P : CP="ALL CURa ADMa DEVa TAIa OUR BUS IND PHY ONL UNI PUR FIN COM NAV INT DEM CNT STA POL HEA PRE LOC OTC"\r\n' not match with `headerRE = re.compile(r'^(From

[issue25539] python3 fail on parsing http header

Shin Dong-won added the comment: b'Date: Tue, 03 Nov 2015 10:05:42 GMT\nServer: Apache/2.2.20 (Unix) mod_ssl/2.2.20 OpenSSL/0.9.8e-fips-rhel5 DAV/2\n PHP/5.2.16 mod_fastcgi/2.4.6\nX-Powered-By: PHP/5.2.16\nP3P: CP="NOI CURa ADMa DEVa TAIa OUR DELa BUS IND PHY ONL UNI COM NAV INT DEM\n

[issue25546] python 3.5 installation problem; Error 0x80240017: Failed to execute MSU package

New submission from Ryan Franklin: Python 3.5 will not install on my Windows 7 Pro machine. The observed failure is : Error 0x80240017: Failed to execute MSU package. This appears to be an open problem with Python 3.5. The same does not have any problems with the Python 3.4.1 installations.

[issue25483] Improve f-string implementation: FORMAT_VALUE opcode

Roundup Robot added the comment: New changeset 1ddeb2e175df by Eric V. Smith in branch 'default': Issue 25483: Add an opcode to make f-string formatting more robust. https://hg.python.org/cpython/rev/1ddeb2e175df -- nosy: +python-dev ___ Python

[issue25483] Improve f-string implementation: FORMAT_VALUE opcode

Roundup Robot added the comment: New changeset 4734713a31ed by Eric V. Smith in branch 'default': Issue 25483: Update dis.rst with FORMAT_VALUE opcode description. https://hg.python.org/cpython/rev/4734713a31ed -- ___ Python tracker

[issue25547] Improve repr for files to show whether the file is open or closed.

Serhiy Storchaka added the comment: >>> f = open('/dev/null', 'rb', buffering=0) >>> f <_io.FileIO name='/dev/null' mode='rb' closefd=True> >>> f.close() >>> f <_io.FileIO [closed]> -- nosy: +serhiy.storchaka ___ Python tracker

[issue892902] problem with pickling newstyle class instances

Serhiy Storchaka added the comment: Could you please make a review Alexandre? -- ___ Python tracker ___ ___

[issue25061] Add native enum support for argparse

desbma added the comment: I came up with something that satisfies my needs (no boilerplate code, and intuitive add_argument call). I modified your factory, and defined a simple action class (this is a quick and dirty prototype for discussion, I am in no way asking that such thing should be

[issue25548] Show the address in the repr for class objects

New submission from Raymond Hettinger: In old-style classes, the memory address of the class object is shown for both the class and for the instance. In new-style classes, only the instance shows the address. I and other Python instructors have found the memory address display to be useful

[issue25483] Improve f-string implementation: FORMAT_VALUE opcode

Eric V. Smith added the comment: Brett: https://docs.python.org/devguide/compiler.html#introducing-new-bytecode looks correct (and reminded me to update dis.rst!). -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___

[issue25542] tuple unpacking on assignment should be lazy for unpacked generators

Mario Wenzel added the comment: You are right. I didn't even know head, *middle, end = worked. Thanks -- resolution: -> rejected status: open -> closed ___ Python tracker

[issue25547] Improve repr for files to show whether the file is open or closed.

New submission from Raymond Hettinger: Good news in Python 3.5, the __repr__ for locks now indicates whether the lock is in a locked state. Bad news for Python 3.x, the __repr__ for files lost the indication of the file's open/closed status: Python 3: - $ python3.5 Python 3.5.0

[issue25495] binascii documentation incorrect

Mouse added the comment: The harm in mentioning the 57-byte chunking is that so far it successfully confused people. b2a_base64() function is not coupled to MIME. It has no constraints on either its input, or its output. *IF* it is used by (in) a MIME application, then the caller may want

[issue25061] Add native enum support for argparse

Raymond Hettinger added the comment: IMO, this adds more complexity than it solves. Argparse already has more options than people can remember. Also, it isn't clear whether this logic should go into the parser or into the business logic (consider for example that the requests package

[issue25543] locale.atof keep '.' even if not part of the localeconv

Stéphane Wirtel added the comment: @eric.smith maybe we can add a new strict function. -- nosy: +matrixise ___ Python tracker ___

[issue25483] Improve f-string implementation: FORMAT_VALUE opcode

Roundup Robot added the comment: New changeset 93fd7adbc7dd by Eric V. Smith in branch 'default': Issue 25483: Fix doc typo and added versionadded. Thanks, Berker Peksag. https://hg.python.org/cpython/rev/93fd7adbc7dd -- ___ Python tracker

[issue12300] Document pydoc.help

Éric Araujo added the comment: Works for me. -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue24663] ast.literal_eval does not handle empty set literals

R. David Murray added the comment: Like I said, you should take your use case to python-ideas, for a safe_eval that would handle it. literal_eval is not the place for handing 'set()', in my opinion. (There is always a chance other other devs will disagree, but none have so far :). FYI,

[issue13601] sys.stderr should always be line-buffered

Terry J. Reedy added the comment: A month after this discussion, the flush keyword was added to print, which cover partial lines sent to either stdout or stderr via print. https://bugs.python.org/issue13761 -- ___ Python tracker

[issue12300] Document pydoc.help

Stéphane Wirtel added the comment: What can we do for this issue ? -- nosy: +matrixise ___ Python tracker ___

[issue12300] Document pydoc.help

Terry J. Reedy added the comment: Close it, which I think Éric implicitly agreed to in paired issue #12299, where I said more in opposition to this one. What I meant above is that help() is 'listed' in https://docs.python.org/3/library/pydoc.html#module-pydoc. While it does not have usual

[issue21243] Auto-generate exceptions.c from a Python file

Changes by Stéphane Wirtel : -- versions: +Python 3.6 -Python 3.5 ___ Python tracker ___

[issue13601] sys.stderr should always be line-buffered

Terry J. Reedy added the comment: As I read this, there was agreement that the status quo is sufficient. That would imply that this should be closed. Correct? -- nosy: +terry.reedy ___ Python tracker

[issue24663] ast.literal_eval does not handle empty set literals

Hrvoje Abraham added the comment: For now I'll implement my_literal_eval via AST filtering, reproducing ast.literal_eval + 'set()' functionality. -- ___ Python tracker

[issue24663] ast.literal_eval does not handle empty set literals

Hrvoje Abraham added the comment: I use communication protocol based on Python literals and ast.literal_eval for deserialization. I'm avoiding sets because empty set value is not supported in a clean consistent manner on language level. If I write repr(set()) i get 'set()', this should

[issue24663] ast.literal_eval does not handle empty set literals

Hrvoje Abraham added the comment: I believe this is the issue. ast.literal_eval sets support can not be considered complete until it also handles empty sets. I do not consider it valid for me to explicitly handle this case in my projects using some weird hacks. Python community settled on

[issue25545] email.message.get_payload returns wrong encoding

R. David Murray added the comment: Your problem is that your input email is ia unicode string. A unicode string has no RFC defintion as an email, so things do not work right, as you observed. Whether or not email should throw an error when fed a non-ascii unicode string is an interesting

[issue25483] Improve f-string implementation: FORMAT_VALUE opcode

Berker Peksag added the comment: + * ``(flags & 0x03) == 0x00``: *value* is formattedd as-is. Just noticed a small typo: formattedd Also, needs ``.. versionadded:: 3.6`` -- nosy: +berker.peksag ___ Python tracker

[issue25546] python 3.5 installation problem; Error 0x80240017: Failed to execute MSU package

Changes by Berker Peksag : -- nosy: +steve.dower ___ Python tracker ___ ___

[issue25548] Show the address in the repr for class objects

Stéphane Wirtel added the comment: Hi Raymond, I just executed the code with python 3.5 and I don't have this result: Python 3.5.0 (default, Sep 23 2015, 04:41:38) [GCC 4.2.1 Compatible Apple LLVM 7.0.0 (clang-700.0.72)] on darwin Type "help", "copyright", "credits" or "license" for more

[issue24663] ast.literal_eval does not handle empty set literals

R. David Murray added the comment: You haven't presented an actual use case, it would be interesting to see one. Regardless of that, however, ast.literal_eval is an exposure of the literal value parsing part of the AST. set() is not part of that, because it is not a literal, it is a function

[issue25537] Call `isinstance` instead of `issubclass` during exception handling

Martin Panter added the comment: FWIW I don’t see it as a drastic change. If the current patch for Issue 12029 went ahead, I imagine the change for instance checking could look like @@ PyErr_GivenExceptionMatches() -/* err might be an instance, so check its class. */ -if

[issue25537] Call `isinstance` instead of `issubclass` during exception handling

R. David Murray added the comment: The code change may be minor, but the semantic change is not. It needs discussion. -- ___ Python tracker ___

[issue24379] operator.subscript

Josh Rosenberg added the comment: Martin: I think the primary reason I suggested an empty __slots__ was to avoid people doing dumb stuff like attaching attributes to the singleton, and then getting annoyed if support for them was removed in some future release (e.g. if this ever moved to C

[issue25388] tokenizer crash/misbehavior -- heap use-after-free

Brian Cain added the comment: Sorry, the report would have been clearer if I'd included a build with symbols and a stack trace. The test was inspired by the test from issue24022 (https://hg.python.org/cpython/rev/03b2259c6cd3), it sounds like it should not have been. But indeed it seems

[issue25388] tokenizer crash/misbehavior -- heap use-after-free

Brian Cain added the comment: Here is a more useful ASan report: = ==12168==ERROR: AddressSanitizer: heap-use-after-free on address 0x6251e110 at pc 0x00697238 bp 0x7fff412b9240 sp 0x7fff412b9238 READ of size 1 at

[issue21243] Auto-generate exceptions.c from a Python file

Changes by Emanuel Barry : -- nosy: +ebarry ___ Python tracker ___ ___ Python-bugs-list

[issue25446] smtplib.py AUTH LOGIN code messed up sending login and password data since 3.5

Larry Hastings added the comment: Okay, I'm scheduling 3.5.1rc1 on the assumption that you'll check in by next weekend. If you're going to slip please let me know and I'll slip accordingly. -- ___ Python tracker

[issue25537] Call `isinstance` instead of `issubclass` during exception handling

R. David Murray added the comment: Let me rephrase that. The semantic change *may* not be. That's what needs discussion :) -- ___ Python tracker ___

[issue25539] python3 fail on parsing http header

Martin Panter added the comment: Okay, now I understand the problem. There is a quirky header line with a space in the field name “P3P ”. The HTTP client’s parse_headers() treats this as any other header line, but then it passes the header to the email package, which interprets this line as

[issue1650090] doctest doesn't find nested functions

Changes by John Mark Vandenberg : -- nosy: +John.Mark.Vandenberg ___ Python tracker ___

[issue2275] urllib/httplib header capitalization

Changes by John Mark Vandenberg : -- nosy: +John.Mark.Vandenberg ___ Python tracker ___ ___

[issue24925] Allow doctest to find line number of __test__ strings if formatted as a triple quoted string.

Changes by John Mark Vandenberg : -- nosy: +John.Mark.Vandenberg ___ Python tracker ___ ___

[issue12790] doctest.testmod does not run tests in functools.partial functions

Changes by John Mark Vandenberg : -- nosy: +John.Mark.Vandenberg ___ Python tracker ___ ___

[issue25549] call sum on list of timedelta throws TypeError

New submission from Kevin Jing Qiu: Calling sum() on a list of timedelta objects results in TypeError: unsupported operand type(s) for +: 'int' and 'datetime.timedelta' Here's a script that illustrates this behaviour: (also attached) import datetime x = [datetime.timedelta(1),

[issue25549] call sum on list of timedelta throws TypeError

Emanuel Barry added the comment: `sum` has an optional `start` parameter, which defaults to 0, and is used as the first item to add. Since timedeltas and ints are not interoperable, that means you have to explicitly tell sum what to use. The following code works: >>>

[issue25546] python 3.5 installation problem; Error 0x80240017: Failed to execute MSU package

Steve Dower added the comment: Can you try installing http://www.microsoft.com/en-us/download/details.aspx?id=48145 and then install Python again? That installer is installing the same update for your machine as Python 3.5 requires, so our installer should skip it once it is on.

[issue25545] email.message.get_payload returns wrong encoding

New submission from Christian Tanzer: For an email message with `Content-type: text/plain; charset=utf-8`, in Python 3.5, get_payload returns a bytes object encoded with `latin-1`. Python 2.7 returns a str object encoded with `utf-8` as expected. Running the attached test script

[issue25543] locale.atof keep '.' even if not part of the localeconv

Cédric Krier added the comment: Example: >>> import locale >>> locale.setlocale(locale.LC_ALL, 'fr_FR.UTF-8') 'fr_FR.UTF-8' >>> locale.atof('2.5') 2.5 >>> locale.atof('2,5') 2.5 -- ___ Python tracker

[issue25543] locale.atof keep '.' even if not part of the localeconv

STINNER Victor added the comment: locale.atof() is implemented as float(locale.delocalize(string)). locale.delocalize() replaces the locale numeric dot with ".": >>> locale.setlocale(locale.LC_ALL, 'fr_FR.UTF-8') 'fr_FR.UTF-8' >>> locale.delocalize("1,2") '1.2' >>>

[issue25543] locale.atof keep '.' even if not part of the localeconv

Cédric Krier added the comment: But you can have some strange behaviour: >>> import locale >>> locale.setlocale(locale.LC_ALL, 'fr_FR.UTF-8') 'fr_FR.UTF-8' >>> locale.atof('2.500,5') Traceback (most recent call last): File "", line 1, in File "/usr/lib64/python2.7/locale.py", line 316, in

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

Changes by Albert White : -- nosy: +Albert White ___ Python tracker ___ ___

[issue25544] cleanup temporary files in distutils.has_function

New submission from Min RK: One of the nits noted in http://bugs.python.org/issue717152, which introduced ccompiler.has_function, was that it does not clean up after itself. This patch uses a TemporaryDirectory context to ensure that the files created during has_function are cleaned up.

[issue25533] Make pkgutil.iter_modules() yield built-in modules

Brett Cannon added the comment: The lack of a way to list frozen modules was why I brought it up. :) And built-in packages are not currently supported, but they theoretically could be if someone put in the work (I think there's an issue for that somewhere). --

[issue25543] locale.atof keep '.' even if not part of the localeconv

Eric V. Smith added the comment: That's an unfortunate implementation choice. I agree we probably can't break locale.atof, but how about adding another function with a better implementation, that's strictly locale-aware? -- nosy: +eric.smith ___

[issue25539] python3 fail on parsing http header

Shin Dong-won added the comment: I think so but same http request works well in Python2 -- ___ Python tracker ___

[issue25539] python3 fail on parsing http header

STINNER Victor added the comment: > It contains two line of continuous CRLF between the data. It looks like a bug at server side. \n\n is the separator between headers and data. An header must not end with \n\n. -- nosy: +haypo ___ Python tracker

[issue25541] Wrong usage of sockaddr_un struct for abstract namespace unix sockets

New submission from PrzemeK: Case: http://stackoverflow.com/questions/33465094/connection-refused-when-using-abstract-namespace-unix-sockets The code that does not work (python3 example): ... sock_path = b"/var/tmp/sock.tmp" server.bind(b"\0" + sock_path) ... and strace shows: connect(3,

[issue25541] Wrong usage of sockaddr_un struct for abstract namespace unix sockets

PrzemeK added the comment: Errata - 1st paragraph should be: The code that does not work (python3 example): ... sock_path = b"/var/tmp/sock.tmp" server.bind(b"\0" + sock_path) ... and strace shows: bind(3, {sa_family=AF_LOCAL, sun_path=@"/var/tmp/sock.tmp"...}, 20) = 0 --

[issue25539] python3 fail on parsing http header

Shin Dong-won added the comment: I monkey patched the method `parse_headers` to print what the fp read and finally arrived at that http.client is not a problem. email.parser.Parser.parsestr adds two '\n' between two lines of "P3P: ~~". end: b'POST /bbs/login_check.php HTTP/1.1\r\nHost:

[issue24097] Use after free in PyObject_GetState

Changes by Serhiy Storchaka : -- assignee: pitrou -> serhiy.storchaka ___ Python tracker ___

[issue24097] Use after free in PyObject_GetState

Serhiy Storchaka added the comment: In 2.7 the test doesn't crash, but __reduce__ produces invalid recursive structure. -- versions: +Python 2.7 ___ Python tracker

[issue24097] Use after free in PyObject_GetState

Serhiy Storchaka added the comment: Here is backported to 2.7 patch with test. Can you review the test and commit the patch Antoine? -- assignee: serhiy.storchaka -> pitrou Added file: http://bugs.python.org/file40936/getstate_borrowed_ref_with_test-2.7.patch

[issue24097] Use after free in PyObject_GetState

Changes by Serhiy Storchaka : Added file: http://bugs.python.org/file40937/getstate_borrowed_ref_with_test-3.4.patch ___ Python tracker ___