[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +10742, 10743 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailing list Unsub

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +10742, 10743, 10744 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailin

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +10742 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35550] Some define guards for Solaris are wrong

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- assignee: -> gregory.p.smith resolution: -> fixed stage: patch review -> commit review versions: +Python 3.8 ___ Python tracker <https://bugs.python.or

[issue35550] Some define guards for Solaris are wrong

2018-12-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 6f9bc72c79c3262e5d0f2c0e96b016477399cfb1 by Gregory P. Smith (Jakub KulĂ­k) in branch 'master': bpo-35550: Fix incorrect Solaris define guards (GH-11275) https://github.com/python/cpython/commit/6f9bc72c79c3262e5d0f2c0e96b016477399cfb1

[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2018-12-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: I'm going to close this as 3.7 and 3.8 are good now, but if someone wants to see the same thing done in 2.7 it should be possible for them make a PR. This is primarily just a configure.ac change. -- resolution: -> fixed stage: commit rev

[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2018-12-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset a144feeb7ec501aaf30072d50e70d54b200e5ef0 by Gregory P. Smith (Miss Islington (bot)) in branch '3.7': bpo-28503: Use crypt_r() when available instead of crypt() (GH-11373) (GH-11376) https://github.com/python/cpython/commit

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-12-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 01b9664740307b39c2907bd84cbb0b2c35be9df4 by Gregory P. Smith (Miss Islington (bot)) in branch '3.7': bpo-35214: MSan workarounds for socket, time, and test_faulthandler. (GH-11375) (GH-11378) https://github.com/python/cpython/commit

[issue35225] test_faulthandler fails under ubsan

2018-12-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: These are all related to the output of the child process containing sanitizer stuff so the stdout/stderr based tests are failing. i skipped two of these for memory sanitizers, the third looks easy enough to fix via the regex. PR attached

[issue35225] test_faulthandler fails under ubsan

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- assignee: -> gregory.p.smith nosy: +gregory.p.smith ___ Python tracker <https://bugs.python.org/issue35225> ___ ___ Python-

[issue35225] test_faulthandler fails under ubsan

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- keywords: +patch, patch pull_requests: +10730, 10731 stage: -> patch review ___ Python tracker <https://bugs.python.org/issu

[issue35225] test_faulthandler fails under ubsan

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- keywords: +patch pull_requests: +10730 stage: -> patch review ___ Python tracker <https://bugs.python.org/issue35225> ___ _

[issue35225] test_faulthandler fails under ubsan

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- keywords: +patch, patch, patch pull_requests: +10730, 10731, 10732 stage: -> patch review ___ Python tracker <https://bugs.python.org/issu

[issue31122] SSLContext.wrap_socket() throws OSError with errno == 0

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- stage: -> needs patch ___ Python tracker <https://bugs.python.org/issue31122> ___ ___ Python-bugs-list mailing list Unsubscrib

[issue31122] SSLContext.wrap_socket() throws OSError with errno == 0

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- versions: -Python 3.5, Python 3.6 ___ Python tracker <https://bugs.python.org/issue31122> ___ ___ Python-bugs-list mailin

[issue31122] SSLContext.wrap_socket() throws OSError with errno == 0

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- versions: +Python 3.6, Python 3.7, Python 3.8 ___ Python tracker <https://bugs.python.org/issue31122> ___ ___ Python-bugs-list m

[issue31122] SSLContext.wrap_socket() throws OSError with errno == 0

2018-12-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: This flakiness just caused a PR merge to be blocked by AppVeyor for me: == ERROR: test_with_statement (test.test_nntplib.NetworkedNNTP_SSLTests

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-12-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset b474e6774d60fa67d5373e361a0ed53c18b24f53 by Gregory P. Smith in branch 'master': bpo-35214: MSan workarounds for socket, time, and test_faulthandler. (GH-11375) https://github.com/python/cpython/commit/b474e6774d60fa67d5373e361a0ed53c18b24f53

[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- stage: patch review -> commit review ___ Python tracker <https://bugs.python.org/issue28503> ___ ___ Python-bugs-list mai

[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2018-12-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: As this is internal only rather than a feature, i'll bring this into 3.7 as well. -- versions: +Python 3.7 ___ Python tracker <https://bugs.python.org/issue28

[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2018-12-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 387512c7ecde6446f2e29408af2e16b9fc043807 by Gregory P. Smith in branch 'master': bpo-28503: Use crypt_r() when available instead of crypt() (GH-11373) https://github.com/python/cpython/commit/387512c7ecde6446f2e29408af2e16b9fc043807

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +10715, 10716, 10717 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailin

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +10715, 10716 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailing list Unsub

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +10715 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +10712, 10713, 10714 ___ Python tracker <https://bugs.python.org/issue28503> ___ ___ Python-bugs-list mailin

[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +10712, 10713 ___ Python tracker <https://bugs.python.org/issue28503> ___ ___ Python-bugs-list mailing list Unsub

[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +10712 ___ Python tracker <https://bugs.python.org/issue28503> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- versions: +Python 3.8 -Python 3.7 ___ Python tracker <https://bugs.python.org/issue28503> ___ ___ Python-bugs-list mailin

[issue28503] [Patch] '_crypt' module: fix implicit declaration of crypt(), use crypt_r() where available

2018-12-30 Thread Gregory P. Smith
Change by Gregory P. Smith : -- assignee: -> gregory.p.smith nosy: +gregory.p.smith ___ Python tracker <https://bugs.python.org/issue28503> ___ ___ Python-

[issue35537] use os.posix_spawn in subprocess

2018-12-26 Thread Gregory P. Smith
Gregory P. Smith added the comment: Thanks for all your research and reference links on this! As a _posixsubprocess maintainer, I am not against either posix_spawn or vfork being used directly in the future when feasible. A challenge, especially with platform specific vfork, is making sure

[issue34836] test_ssl.test_default_ecdh_curve needs no tls1.3 flag in 2.7, for now

2018-12-26 Thread Gregory P. Smith
Change by Gregory P. Smith : -- assignee: -> christian.heimes ___ Python tracker <https://bugs.python.org/issue34836> ___ ___ Python-bugs-list mailing list Un

[issue35499] "make profile-opt" overrides CFLAGS_NODIST

2018-12-18 Thread Gregory P. Smith
Gregory P. Smith added the comment: But the `build_all_generate_profile` build is an intermediate instrumented interpreter build, it isn't shipped and things like PGO often require flags that the linker sees in order to generate the instrumented binary. If those are left off of the link

[issue31823] Opaque default value for close_fds argument in Popen.__init__

2018-12-12 Thread Gregory P. Smith
Gregory P. Smith added the comment: Only if someone can point to a concrete problem to be fixed in the 3.7+ docs. Looking at all references to close_fds in https://docs.python.org/3/library/subprocess.html I don't see one. (I'm not going to touch the 3.6 docs

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-12-11 Thread Gregory P. Smith
Gregory P. Smith added the comment: I believe there are still some issues to deal with. I don't want to close the issue until I've got my buildbot running. -- versions: -Python 3.6 ___ Python tracker <https://bugs.python.org/issue35

[issue31823] Opaque default value for close_fds argument in Popen.__init__

2018-12-10 Thread Gregory P. Smith
Gregory P. Smith added the comment: In 3.6 this help(subprocess.Popen.__init__) is accurate and encourages looking at the docs. In 3.7 and later it'll simply report close_fds=True in the siguature as that is the case in 3.7 onwards. I see nothing to fix here. -- nosy

[issue31823] Opaque default value for close_fds argument in Popen.__init__

2018-12-10 Thread Gregory P. Smith
Change by Gregory P. Smith : -- versions: -Python 3.6 ___ Python tracker <https://bugs.python.org/issue31823> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue22005] datetime.__setstate__ fails decoding python2 pickle

2018-12-09 Thread Gregory P. Smith
Gregory P. Smith added the comment: Serhiy: should this one be marked fixed? -- assignee: -> serhiy.storchaka stage: patch review -> commit review ___ Python tracker <https://bugs.python.org/i

[issue22005] datetime.__setstate__ fails decoding python2 pickle

2018-12-09 Thread Gregory P. Smith
Gregory P. Smith added the comment: It is fundamentally impossible for pickled data to magically cross the 2 and 3 language boundary unscathed. The basic str/bytes/unicode types in the language changed meaning. Code must be written manually by the data owners to fix that up based on what

[issue22005] datetime.__setstate__ fails decoding python2 pickle

2018-12-09 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset e328753d91379274b699b93decff45de07854617 by Gregory P. Smith in branch 'master': bpo-22005: Document the reality of pickle compatibility. (GH-11054) https://github.com/python/cpython/commit/e328753d91379274b699b93decff45de07854617

[issue22005] datetime.__setstate__ fails decoding python2 pickle

2018-12-09 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +10289 ___ Python tracker <https://bugs.python.org/issue22005> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34850] Emit a syntax warning for "is" with a literal

2018-12-01 Thread Gregory P. Smith
Gregory P. Smith added the comment: Oh neat, I didn't realize that Nathaniel. That makes such warnings much more useful. I'm fine if you go ahead with this change. In the unlikely event it turns out to cause user annoyance problems in betas due to third party code that can't be updated

[issue34850] Emit a syntax warning for "is" with a literal

2018-11-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: Given that linters like pylint can already detect the common case of this issue when using `is` and `is not` to compare to a literal where == or != seems more appropriate, I don't think a warning is very useful. In my experience people are more likely

[issue33015] Fix function cast warning in thread_pthread.h

2018-11-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: I don't think it is crazy for 2.7, but I'd move that to its own issue as you already nicely addressed the problem related to this PR without needing that. -- ___ Python tracker <https://bugs.python.

[issue33015] Fix function cast warning in thread_pthread.h

2018-11-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: why was that only an issue on 2.7? -- ___ Python tracker <https://bugs.python.org/issue33015> ___ ___ Python-bugs-list mailin

[issue24209] Allow IPv6 bind in http.server

2018-11-18 Thread Gregory P. Smith
Change by Gregory P. Smith : -- versions: +Python 3.8 -Python 3.6 ___ Python tracker <https://bugs.python.org/issue24209> ___ ___ Python-bugs-list mailin

[issue25438] document what codec PyMemberDef T_STRING decodes the char * as

2018-11-17 Thread Gregory P. Smith
Change by Gregory P. Smith : -- resolution: -> fixed stage: patch review -> commit review status: open -> closed versions: +Python 3.7, Python 3.8 -Python 3.4, Python 3.5 ___ Python tracker <https://bugs.python.or

[issue31354] Fixing a bug related to LTO only build

2018-11-14 Thread Gregory P. Smith
Change by Gregory P. Smith : -- versions: +Python 3.6, Python 3.8 ___ Python tracker <https://bugs.python.org/issue31354> ___ ___ Python-bugs-list mailin

[issue31354] Fixing a bug related to LTO only build

2018-11-14 Thread Gregory P. Smith
Change by Gregory P. Smith : -- priority: normal -> release blocker ___ Python tracker <https://bugs.python.org/issue31354> ___ ___ Python-bugs-list mai

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-14 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset b6f4472dc4190e2fd668490d86aeefd2ab0df935 by Gregory P. Smith in branch '2.7': [2.7] bpo-35214: Fix OOB memory access in unicode escape parser (GH-10506) (GH-10538) https://github.com/python/cpython/commit

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-13 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +9789 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue23734] zipimport should not check pyc timestamps against zipped py files

2018-11-13 Thread Gregory P. Smith
Gregory P. Smith added the comment: Correct, this issue was filed before we had the new pyc options (which came out of our September core dev. sprint in 2016 iirc). I'll just close it, the way forward for people using zip files on modern CPython interpreters is to explicitly use the new

[issue31354] Fixing a bug related to LTO only build

2018-11-13 Thread Gregory P. Smith
Gregory P. Smith added the comment: Is it an actual problem to compile extension modules with -flto? (I realize as an extension module there isn't a huge benefit to the concept unless it happens to have multiple source files or link against a non-shared library

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-13 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 746b2d35ea47005054ed774fecaed64fab803d7d by Gregory P. Smith in branch 'master': bpo-35214: Fix OOB memory access in unicode escape parser (GH-10506) https://github.com/python/cpython/commit/746b2d35ea47005054ed774fecaed64fab803d7d

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-13 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +9766 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-12 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 3015fb8ce4d25603434b9b44bb7effb98a481532 by Gregory P. Smith in branch 'master': bpo-35214: Add _Py_ prefix to MEMORY_SANITIZER def. (GH-10503) https://github.com/python/cpython/commit/3015fb8ce4d25603434b9b44bb7effb98a481532

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-12 Thread Gregory P. Smith
Gregory P. Smith added the comment: > Can we prefix MEMORY_SANITIZER with _Py_? Yes, I wondered if I should do that. not that I expect anyone would ever define it to mean anything else, but that seems like the right thing to do. https://github.com/python/cpython/pull/10

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-12 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +9763 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-12 Thread Gregory P. Smith
Gregory P. Smith added the comment: yeah, i've been surprised how far i was able to get from an oss CPython tree and pre-built clang 7 binary installation. If you have headers installed for common libraries like libssl-dev and libreadline-dev you can't produce an interpreter that'll work

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-12 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +9760 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-12 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 3b5b1c0931cd32253bbf2f3bd74e90ff821e026e by Gregory P. Smith in branch '3.6': [3.6] bpo-35214: Initial clang MemorySanitizer support (GH-10479) (GH-10493) https://github.com/python/cpython/commit/3b5b1c0931cd32253bbf2f3bd74e90ff821e026e

[issue25438] document what codec PyMemberDef T_STRING decodes the char * as

2018-11-12 Thread Gregory P. Smith
Gregory P. Smith added the comment: it still seems relevant, having better docs is always good. -- ___ Python tracker <https://bugs.python.org/issue25

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-12 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +9751 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-12 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 5f4d05d83fde05fed0b6554d0beee4f1c1f1d8f1 by Gregory P. Smith in branch '3.7': [3.7] bpo-35214: Initial clang MemorySanitizer support (GH-10479) (GH-10492) https://github.com/python/cpython/commit/5f4d05d83fde05fed0b6554d0beee4f1c1f1d8f1

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-12 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +9749 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-12 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 1584a0081500d35dc93ff88e5836df35faf3e3e2 by Gregory P. Smith in branch 'master': bpo-35214: Initial clang MemorySanitizer support (GH-10479) https://github.com/python/cpython/commit/1584a0081500d35dc93ff88e5836df35faf3e3e2

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-11 Thread Gregory P. Smith
Change by Gregory P. Smith : -- keywords: +patch pull_requests: +9746 ___ Python tracker <https://bugs.python.org/issue35214> ___ ___ Python-bugs-list mailin

[issue35214] Get the test suite passing with clang Memory Sanitizer enabled

2018-11-11 Thread Gregory P. Smith
New submission from Gregory P. Smith : clang's memory sanitizer (-fsanitize=memory) turns up useful problems in code. I'm working on getting a CPython buildbot running it setup but would like our build to be cleaner to start with before I run that. These are the initial fixes required

[issue35193] Off by one error in peephole call to find_op on case RETURN_VALUE

2018-11-09 Thread Gregory P. Smith
Gregory P. Smith added the comment: I'm working on getting a memory-sanitizer buildbot setup. I'm so happy it made finding and debugging this relatively easy. -- resolution: -> fixed stage: patch review -> resolved status: open -&g

[issue35193] Off by one error in peephole call to find_op on case RETURN_VALUE

2018-11-08 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 65e1a1fd311943866361fcb288c0df65dadbe092 by Gregory P. Smith in branch '3.6': bpo-35193: Fix an off by one error in the RETURN_VALUE case. (GH-10418) (GH-10422) https://github.com/python/cpython/commit

[issue35193] Off by one error in peephole call to find_op on case RETURN_VALUE

2018-11-08 Thread Gregory P. Smith
Change by Gregory P. Smith : -- pull_requests: +9703 ___ Python tracker <https://bugs.python.org/issue35193> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35193] Off by one error in peephole call to find_op on case RETURN_VALUE

2018-11-08 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 49fa4a9f1ef387e16596f271414c855339eadf09 by Gregory P. Smith in branch 'master': bpo-35193: Fix an off by one error in the RETURN_VALUE case. (GH-10418) https://github.com/python/cpython/commit/49fa4a9f1ef387e16596f271414c855339eadf09

[issue35193] Off by one error in peephole call to find_op on case RETURN_VALUE

2018-11-08 Thread Gregory P. Smith
Change by Gregory P. Smith : -- keywords: +patch pull_requests: +9698 stage: -> patch review ___ Python tracker <https://bugs.python.org/issue35193> ___ _

[issue28517] Dead code in wordcode

2018-11-08 Thread Gregory P. Smith
Gregory P. Smith added the comment: the off by one error fix here introduced a new off by one error. PR coming, follow https://bugs.python.org/issue35193 for that. -- nosy: +gregory.p.smith ___ Python tracker <https://bugs.python.org/issue28

[issue35193] Off by one error in peephole call to find_op on case RETURN_VALUE

2018-11-08 Thread Gregory P. Smith
New submission from Gregory P. Smith : An off by one error was introduced to peephole.c by the "off by one error fix" in https://bugs.python.org/issue28517. Clang's memory sanitizer detects it (msan). find_op is ultimately called with h == codelen so it accesses one byte out of

[issue35107] untokenize() fails on tokenize output when a newline is missing

2018-10-29 Thread Gregory P. Smith
Gregory P. Smith added the comment: Interesting! I have a 3.6.2 sitting around and cannot reproduce that "x=1" behavior. I don't know what the behavior _should_ be. It just feels natural that untokenize should be able to round trip anything tokenize or generate_tokens emi

[issue33899] Tokenize module does not mirror "end-of-input" is newline behavior

2018-10-29 Thread Gregory P. Smith
Gregory P. Smith added the comment: https://bugs.python.org/issue35107 filed to track further fallout from this API change. -- ___ Python tracker <https://bugs.python.org/issue33

[issue35107] untokenize() fails on tokenize output when a newline is missing

2018-10-29 Thread Gregory P. Smith
New submission from Gregory P. Smith : The behavior change introduced in 3.6.7 and 3.7.1 via https://bugs.python.org/issue33899 has further consequences: ```python >>> tokenize.untokenize(tokenize.generate_tokens(io.StringIO('#').readline)) Traceback (most recent call last): File &

[issue33015] Fix function cast warning in thread_pthread.h

2018-10-24 Thread Gregory P. Smith
Gregory P. Smith added the comment: This is presumably also present in 3.6 and 2.7 so I've tagged those on the issue, but for this kind of change i'd leave it up to release managers to see if they want to backport something of this nature that late in those release cycles. Observable side

[issue33015] Fix function cast warning in thread_pthread.h

2018-10-24 Thread Gregory P. Smith
Gregory P. Smith added the comment: I left comments on the github PRs with reasons why, but PR 6008 seems correct. PR 10057 would leave us with undefined behavior. -- ___ Python tracker <https://bugs.python.org/issue33

[issue33012] Invalid function cast warnings with gcc 8 for METH_NOARGS

2018-10-24 Thread Gregory P. Smith
Change by Gregory P. Smith : -- nosy: +gregory.p.smith versions: +Python 3.7 ___ Python tracker <https://bugs.python.org/issue33012> ___ ___ Python-bugs-list m

[issue33899] Tokenize module does not mirror "end-of-input" is newline behavior

2018-10-24 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset dfba1f67e7f1381ceb7cec8fbcfa37337620a9b0 by Gregory P. Smith (Tal Einat) in branch 'master': bpo-33899: Mention tokenize behavior change in What's New (GH-10073) https://github.com/python/cpython/commit

[issue33899] Tokenize module does not mirror "end-of-input" is newline behavior

2018-10-24 Thread Gregory P. Smith
Gregory P. Smith added the comment: some pylint fallout appears to be addressed in https://github.com/PyCQA/pylint/commit/2698cbe56b44df7974de1c3374db8700296c6fad -- ___ Python tracker <https://bugs.python.org/issue33

[issue33899] Tokenize module does not mirror "end-of-input" is newline behavior

2018-10-24 Thread Gregory P. Smith
Gregory P. Smith added the comment: FYI, An example of other fallout from this change - patsy broke and needed this fix: https://github.com/pydata/patsy/commit/4f53bbaf58c0bf1a9bed73fc67c7c6d0aa7f4e20#diff-53c70e68c6dfd4fe9b08427792cb2bd6 -- nosy: +gregory.p.smith

[issue34912] Update overflow checks in resize_buffer

2018-10-19 Thread Gregory P. Smith
Gregory P. Smith added the comment: correct, i don't see an obvious problem in the existing code. -- nosy: +gregory.p.smith ___ Python tracker <https://bugs.python.org/issue34

[issue35011] expat: Restore the use of pyexpatns.h to avoid link time conflicts vs other versions

2018-10-17 Thread Gregory P. Smith
Change by Gregory P. Smith : -- resolution: -> fixed stage: patch review -> commit review status: open -> closed ___ Python tracker <https://bugs.python.or

[issue35011] expat: Restore the use of pyexpatns.h to avoid link time conflicts vs other versions

2018-10-17 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 9d4712bc8f26bf1d7e626b53ab092fe030bcd68d by Gregory P. Smith in branch 'master': bpo-35011: Restore use of pyexpatns.h in libexpat (GH-9939) https://github.com/python/cpython/commit/9d4712bc8f26bf1d7e626b53ab092fe030bcd68d

[issue35011] expat: Restore the use of pyexpatns.h to avoid link time conflicts vs other versions

2018-10-17 Thread Gregory P. Smith
Change by Gregory P. Smith : -- keywords: +patch pull_requests: +9290 stage: needs patch -> patch review ___ Python tracker <https://bugs.python.org/issu

[issue35011] expat: Restore the use of pyexpatns.h to avoid link time conflicts vs other versions

2018-10-17 Thread Gregory P. Smith
Gregory P. Smith added the comment: This appears to not have been shipped in a release yet. It is new in 3.6.7. cc'ing ned daily to see if he wants to include the fix (the PR is trivial, coming ASAP). I don't have a good feel for how this impacts the real world or not. We noticed because

[issue35011] expat: Restore the use of pyexpatns.h to avoid link time conflicts vs other versions

2018-10-17 Thread Gregory P. Smith
Gregory P. Smith added the comment: Not a release blocker as most users probably do not run into this problem, but the pyexpatns.h mechanics should be restored. -- assignee: -> gregory.p.smith components: +Build, Extension Modules nosy: +benjamin.peterson stage: -> needs

[issue35011] Update to expat removed the pyexpatns.h, causing link time symbol conflicts vs other versions in an application

2018-10-17 Thread Gregory P. Smith
New submission from Gregory P. Smith : These lines used to exist in Modules/expat/expat_external.h: /* Namespace external symbols to allow multiple libexpat version to co-exist. */ #include "pyexpatns.h" https://github.com/python/cpython/commit/5dc3f23b5fb0b510926012cb3732dae63cd

[issue6721] Locks in the standard library should be sanitized on fork

2018-10-07 Thread Gregory P. Smith
Gregory P. Smith added the comment: New changeset 3b699932e5ac3e76031bbb6d700fbea07492641d by Gregory P. Smith (Miss Islington (bot)) in branch '3.7': bpo-6721: Hold logging locks across fork() (GH-4071) (#9291) https://github.com/python/cpython/commit

[issue34867] Add mode to disable small integer and interned string caches

2018-10-04 Thread Gregory P. Smith
Gregory P. Smith added the comment: The intent is to use only enable this during testing / continuous integration. -- ___ Python tracker <https://bugs.python.org/issue34

[issue34850] Emit a syntax warning for "is" with a literal

2018-09-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: If we were to ship a "chaos" mode in the CPython interpreter itself, I assume you envision an interpreter flag and/or env var? If it required someone compiling the interpreter a special way I don't think it would be widely adopted within

[issue34850] Emit a syntax warning for "is" with a literal

2018-09-30 Thread Gregory P. Smith
Gregory P. Smith added the comment: The problem with a SyntaxWarning is that the wrong people will see it. It gets in the way of users of applications that happen to be written in Python. scenarios: (a) A Python interpreter gets upgraded, and suddenly the _users_ of an application start

[issue15994] memoryview to freed memory can cause segfault

2018-09-28 Thread Gregory P. Smith
Change by Gregory P. Smith : -- nosy: +gregory.p.smith versions: +Python 3.7, Python 3.8 -Python 3.5 ___ Python tracker <https://bugs.python.org/issue15

[issue34826] io.BufferedReader crashes in 2.7 on memoryview attribute access

2018-09-27 Thread Gregory P. Smith
New submission from Gregory P. Smith : The following test code segfaults on Python 2.7: ```python import io class X(io.RawIOBase): def readable(self): return True def readinto(self, b): if isinstance(b, memoryview): print(b.format) # XXX boom, crashes here. return 0

[issue29988] with statements are not ensuring that __exit__ is called if __enter__ succeeds

2018-09-24 Thread Gregory P. Smith
Change by Gregory P. Smith : -- versions: +Python 3.8 ___ Python tracker <https://bugs.python.org/issue29988> ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34776] Postponed annotations break inspection of dataclasses

2018-09-24 Thread Gregory P. Smith
Gregory P. Smith added the comment: fwiw, agreed that this should wait for 3.7.2. -- ___ Python tracker <https://bugs.python.org/issue34776> ___ ___ Python-bug

[issue34776] Postponed annotations break inspection of dataclasses

2018-09-24 Thread Gregory P. Smith
Change by Gregory P. Smith : -- nosy: +gregory.p.smith versions: +Python 3.8 ___ Python tracker <https://bugs.python.org/issue34776> ___ ___ Python-bugs-list m

[issue34663] Support POSIX_SPAWN_USEVFORK flag in posix_spawn

2018-09-19 Thread Gregory P. Smith
Gregory P. Smith added the comment: Agreed on not exposing it. It seems obsolete in recent glibc and the older glibc implementations that had it may have made questionable decisions. :) Thanks for chiming in Florian, and thanks Pablo for your detailed investigation

<    8   9   10   11   12   13   14   15   16   17   >