[issue35928] socket makefile read-write discards received data

Ammar Askar added the comment: Recreatable on master as well, also Martin your suspicion seems correct, reverting https://github.com/python/cpython/commit/23db935bcf258657682e66464bf8512def8af830 fixes it. -- nosy: +ammar2, serhiy.storchaka stage: -> needs patch versions: +Python

[issue35928] socket makefile read-write discards received data

Palle Ravn added the comment: >>> f = TextIOWrapper(BufferedRWPair(BytesIO(b"Hello\nYou\n"), BytesIO())) >>> f.readline() 'Hello\n' >>> f.write(_) 6 >>> f.readline() # Returns empty string '' -- ___ Python tracker

[issue35813] shared memory construct to avoid need for serialization between processes

Davin Potts added the comment: @giampaolo.rodola: Your patch from 3 days ago in issue35917 included additional tests around the SharedMemoryManager which are now causing test failures in my new PR. This is my fault because I altered SharedMemoryManager to no longer support functionality

[issue35813] shared memory construct to avoid need for serialization between processes

Davin Potts added the comment: Docs and tests are now available in a new PR. I have stayed focused on getting these docs and tests to everyone without delay but that means I have not yet had an opportunity to respond to the helpful comments, thoughtful questions, and threads that have

[issue35813] shared memory construct to avoid need for serialization between processes

Change by Davin Potts : -- pull_requests: +11834 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35813] shared memory construct to avoid need for serialization between processes

Change by Davin Potts : -- pull_requests: +11834, 11835, 11836 ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue35813] shared memory construct to avoid need for serialization between processes

Change by Davin Potts : -- pull_requests: +11834, 11835 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35951] os.renames() creates directories if original name doesn't exist

Ammar Askar added the comment: Aah, I interpreted the combination of the warning and the fact that it says "attempted first" to mean that any failures in the actual renaming will leave the new directory in place. That is, no cleanup is ever performed. A quick glance at the code seems to

[issue35951] os.renames() creates directories if original name doesn't exist

Chris Jerdonek added the comment: Lacking permissions seems very different to me from the source directory or file not existing. For example, in the example I provided, I did have the needed permissions. Incidentally (and this is a separate documentation issue), the note seems unclear as

[issue35960] dataclasses.field does not preserve empty metadata object

Change by Karthikeyan Singaravelan : -- nosy: +eric.smith ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35961] gc_decref: Assertion "gc_get_refs(g) > 0" failed: refcount is too small

Karthikeyan Singaravelan added the comment: I too saw this a week back but couldn't reproduce it. There is one another test that fails like this and passes on verbose mode issue35809 . Travis doesn't report tests that fail in the complete run and pass during verbose run like buildbots do.

[issue35460] Add PyDict_GetItemStringWithError

Raymond Hettinger added the comment: -1 for expanding this API. As Serhiy pointed-out, PyDict_GetItemString() is an old API kept just for backward compatibility. For your use case, it is easy to call PyUnicode_FromString(key) and then follow-up with PyDict_GetItemWithError(). The latter

[issue35889] sqlite3.Row doesn't have useful repr

Raymond Hettinger added the comment: +1 from me. We're already made regular expression match objects less opaque and that has been useful. There's no need for a python-ideas discussion for this. If a repr doesn't round-trip, we generally put it angle brackets (see PEP 8): >>>

[issue35796] time.localtime returns error for negative values

Ammar Askar added the comment: Victor is correct, this is a limitation in Windows. As the documentation for time notes: >Although this module is always available, not all functions are available on >all platforms. Most of the functions defined in this module call platform C >library

[issue35960] dataclasses.field does not preserve empty metadata object

Change by Christopher Hunt : -- keywords: +patch, patch, patch pull_requests: +11831, 11832, 11833 stage: -> patch review ___ Python tracker ___

[issue35960] dataclasses.field does not preserve empty metadata object

Change by Christopher Hunt : -- keywords: +patch, patch pull_requests: +11831, 11832 stage: -> patch review ___ Python tracker ___

[issue35960] dataclasses.field does not preserve empty metadata object

Change by Christopher Hunt : -- keywords: +patch pull_requests: +11831 stage: -> patch review ___ Python tracker ___ ___

[issue31940] copystat on symlinks fails for alpine -- faulty lchmod implementation?

Change by Benjamin Peterson : -- resolution: -> duplicate stage: patch review -> resolved status: open -> closed superseder: -> never enable lchmod on Linux ___ Python tracker

[issue28627] [alpine] shutil.copytree fail to copy a direcotry with broken symlinks

Change by Benjamin Peterson : -- resolution: -> duplicate stage: -> resolved status: open -> closed ___ Python tracker ___ ___

[issue35956] Sort documentation could be improved for complex sorting

Raymond Hettinger added the comment: It seems to me that the sorting howto covers this topic. If I'm reading the OP's task correctly, it isn't complex at all: points.sort(key=attrgetter('x', 'y')) # x is primary key; y is secondary -- assignee: docs@python -> rhettinger nosy:

[issue35951] os.renames() creates directories if original name doesn't exist

Ammar Askar added the comment: It seems this behavior is somewhat documented: https://docs.python.org/3/library/os.html#os.renames >Works like rename(), except creation of any intermediate directories needed to >make the new pathname good is attempted first. >This function can fail with the

[issue35961] gc_decref: Assertion "gc_get_refs(g) > 0" failed: refcount is too small

New submission from Pablo Galindo Salgado : I am seeing some failures in travis and some buildbots: 0:02:24 load avg: 3.30 [147/420/1] test_slice crashed (Exit code -6) -- running: test_multiprocessing_spawn (32 sec 523 ms), test_asyncio (45 sec 433 ms), test_multiprocessing_forkserver (47

[issue35960] dataclasses.field does not preserve empty metadata object

New submission from Christopher Hunt : The metadata argument to dataclasses.field is not preserved in the resulting Field.metadata attribute if the argument is a mapping with length 0. The docs for dataclasses.field state: > metadata: This can be a mapping or None. None is treated as an empty

[issue35808] Let's retire pgen

Change by Pablo Galindo Salgado : -- pull_requests: -11829, 11830 ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue35808] Let's retire pgen

Change by Pablo Galindo Salgado : -- pull_requests: -11830 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35808] Let's retire pgen

Change by Pablo Galindo Salgado : -- keywords: +patch, patch, patch pull_requests: +11828, 11829, 11830 stage: needs patch -> patch review ___ Python tracker ___

[issue35808] Let's retire pgen

Change by Pablo Galindo Salgado : -- keywords: +patch, patch pull_requests: +11828, 11829 stage: needs patch -> patch review ___ Python tracker ___

[issue35808] Let's retire pgen

Change by Pablo Galindo Salgado : -- keywords: +patch pull_requests: +11828 stage: needs patch -> patch review ___ Python tracker ___

[issue23460] Decimals do not obey ':g' exponential notation formatting rules

Brennan D Baraban <3...@holbertonschool.com> added the comment: What is the status of this issue? I can submit a PR based on Tuomas Suutari's patch. -- nosy: +bdbaraban ___ Python tracker

[issue35892] Fix awkwardness of statistics.mode() for multimodal datasets

Steven D'Aprano added the comment: Thanks Raymond for the interesting use-case. The original design of mode() was support only the basic form taught in secondary schools, namely a single unique mode for categorical data or discrete numerical data. I think it is time to consider a richer

[issue35904] Add statistics.fmean(seq)

Steven D'Aprano added the comment: > Would you like me to submit a PR with docs and tests? Yes please! I'm happy with the name fmean. -- ___ Python tracker ___

[issue35899] '_is_sunder' function in 'enum' module fails on empty string

Brennan D Baraban <3...@holbertonschool.com> added the comment: Got it, makes sense. Thank you. New contributor here :) -- ___ Python tracker ___

[issue12317] inspect.getabsfile() is not documented

Terry J. Reedy added the comment: Given the absence of agreement among core-devs, a PR is a bit premature. Looking again at the existing docstring, I think it must be revised before copying and *not* copied as is. 0. The existing first sentence mislead me. The 'source or compiled file'

[issue35899] '_is_sunder' function in 'enum' module fails on empty string

Ethan Furman added the comment: Let's give Maxwell until the 14th (so a week from when I asked him to turn his code into a patch) and if nothing from him by then you are welcome to take it over. -- ___ Python tracker

[issue34652] never enable lchmod on Linux

Anthony Sottile added the comment: I believe this also closes https://bugs.python.org/issue31940 and https://bugs.python.org/issue28627 -- nosy: +Anthony Sottile ___ Python tracker

[issue35899] '_is_sunder' function in 'enum' module fails on empty string

Brennan <3...@holbertonschool.com> added the comment: I'm not sure if Maxwell is still working on this issue, but may I pick it up? I can submit a PR within the day. -- nosy: +bdbaraban ___ Python tracker

[issue35940] multiprocessing manager tests fail in the Refleaks buildbots

Giampaolo Rodola' added the comment: Pablo thanks a lot for taking care of this. -- ___ Python tracker ___ ___ Python-bugs-list

[issue35278] [security] directory traversal in tempfile prefix

Cheryl Sabella added the comment: Adding Łukasz to the nosy list as release manager. -- nosy: +cheryl.sabella, lukasz.langa ___ Python tracker ___

[issue9584] fnmatch, glob: Allow curly brace expansion

Matúš Valo added the comment: Hi All, this is a humble ping. -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue35959] math.prod(range(10)) caues segfault

Change by Pablo Galindo Salgado : -- pull_requests: +11825, 11826, 11827 ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue35959] math.prod(range(10)) caues segfault

Change by Pablo Galindo Salgado : -- pull_requests: +11825 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35959] math.prod(range(10)) caues segfault

Change by Pablo Galindo Salgado : -- pull_requests: +11825, 11826 ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue35959] math.prod(range(10)) caues segfault

Change by Pablo Galindo Salgado : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue35940] multiprocessing manager tests fail in the Refleaks buildbots

Change by Pablo Galindo Salgado : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue35959] math.prod(range(10)) caues segfault

Pablo Galindo Salgado added the comment: PR11808 fixes the error and add some basic test. Please review the PR instead :) -- ___ Python tracker ___

[issue35959] math.prod(range(10)) caues segfault

Change by Pablo Galindo Salgado : -- keywords: +patch, patch, patch pull_requests: +11822, 11823, 11824 stage: -> patch review ___ Python tracker ___

[issue35959] math.prod(range(10)) caues segfault

Change by Pablo Galindo Salgado : -- keywords: +patch, patch pull_requests: +11822, 11823 stage: -> patch review ___ Python tracker ___

[issue35959] math.prod(range(10)) caues segfault

Change by Pablo Galindo Salgado : -- keywords: +patch pull_requests: +11822 stage: -> patch review ___ Python tracker ___ ___

[issue35936] Give modulefinder some much-needed updates.

Brandt Bucher added the comment: Alright, I've gotten all of the tests passing for the new importlib-only implementation. I broke these modifications out into a new private function, _find_module, to make it clear that this fix is a simple drop-in replacement for imp.find_module. Let me

[issue35959] math.prod(range(10)) caues segfault

Rémi Lapeyre added the comment: Could it be https://github.com/python/cpython/blob/master/Modules/mathmodule.c#L2565 When 0 is in the iterator, i_result get sets to 0 and then on the next iteration x/i_result is 0/0 which is undefined behavior? C99 6.5.5p5 - The result of the / operator is

[issue35959] math.prod(range(10)) caues segfault

Pablo Galindo Salgado added the comment: The problem is that the intermediate result (i_result) can be 0 when doing the overflow check: x / i_result != b i am working on a fix. -- ___ Python tracker

[issue35959] math.prod(range(10)) caues segfault

New submission from Karthikeyan Singaravelan : math.prod introduced with issue35606 seems to segfault when zero is present on some cases like start or middle of the iterable. I couldn't find the exact cause of this. This also occurs in optimized builds. # Python information built with

[issue35956] Sort documentation could be improved for complex sorting

Cheryl Sabella added the comment: Take a look at issue 35020 which discusses multisort. -- nosy: +cheryl.sabella ___ Python tracker ___

[issue35958] io.IOBase subclasses don't play nice with abc.abstractmethod

New submission from Jon McMahon : Subclasses of io.IOBase can be instantiated with abstractmethod()s, even though ABCs are supposed to prevent this from happening. I'm guessing this has to do with io using the _io C module because the alternative pure-python implementation _pyio doesn't seem

[issue35957] Indentation explanation is unclear

New submission from Jérôme LAURENS : https://docs.python.org/3/reference/lexical_analysis.html#indentation reads Point 1: "Tabs are replaced (from left to right) by one to eight spaces such that the total number of characters up to and including the replacement is a multiple of eight" and in

[issue35956] Sort documentation could be improved for complex sorting

SilentGhost added the comment: Is this not equivalent to the following? sorted(points, key=lambda p: (p.y, p.x)) -- nosy: +SilentGhost ___ Python tracker ___

[issue35956] Sort documentation could be improved for complex sorting

New submission from fabrice salvaire : I just implemented Graham Scan algorithm in Python 3 and have to read carefully the sort documentation. Notice this is a good algorithm for a large audience language like Python. Since Python 3, the old order function cmp is depicted as an old way to

[issue21269] Provide args and kwargs attributes on mock call objects

Karthikeyan Singaravelan added the comment: This feels safer to me with respect to backwards compatibility and also that it might be easier to backport this to mock on GitHub which works with Python 2.7. I have less knowledge on difference between tuple and namedtuple internals so I might

[issue21269] Provide args and kwargs attributes on mock call objects

Rémi Lapeyre added the comment: @xtreak, couldn't we have made `_Call` inherit from namedtuple to achieve a similar result (albeit the handling of name would be weird)? -- ___ Python tracker

[issue21269] Provide args and kwargs attributes on mock call objects

Rémi Lapeyre added the comment: I like this patch, working with calls often feels weird and this change simplify attribute access. -- nosy: +remi.lapeyre ___ Python tracker

[issue35460] Add PyDict_GetItemStringWithError

Ronald Oussoren added the comment: There are more reasons to implement in C than just speed ;-). In my code I have two usecases for using PyDict_GetItemString, both not in performance critical code 1) Look up a hardcoded name in a dictionary. These could be switched to the Id API if that

[issue35878] ast.c: end_col_offset may be used uninitialized in this function

Change by Ivan Levkivskyi : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue35955] unittest assertEqual reports incorrect location of mismatch

Karthikeyan Singaravelan added the comment: Sorry to comment on a closed issue. I see the following behavior with difflib.ndiff which is used under the hood by unittest. The strings that differ by '-' and 'w' generate different output compared to 'a' and 'w'. I find the output for diff

[issue35460] Add PyDict_GetItemStringWithError

Stefan Behnel added the comment: The overhead of calling PyErr_Occurred() is definitely negligible in something as involved as PyDict_GetItemStringWithError(), where a mere key lookup first has to fire up the string decoder on a C character buffer to create a new string object and then

[issue35955] unittest assertEqual reports incorrect location of mismatch

Jason R. Coombs added the comment: I was able to replicate the issue using pytest and not unittest, so I've [reported the issue with that project](https://github.com/pytest-dev/pytest/issues/4765). -- ___ Python tracker

[issue35955] unittest assertEqual reports incorrect location of mismatch

Change by Jason R. Coombs : -- resolution: -> third party stage: -> resolved status: open -> closed ___ Python tracker ___ ___

[issue35955] unittest assertEqual reports incorrect location of mismatch

Jason R. Coombs added the comment: I should acknowledge that I'm using pytest here also... and pytest may be the engine that's performing the reporting of the failed assertion. In fact, switching to simple assertions, I see the same behavior, so I now suspect the issue may lie with pytest

[issue35955] unittest assertEqual reports incorrect location of mismatch

New submission from Jason R. Coombs : In [this job](https://travis-ci.org/jaraco/cmdix/jobs/491246158), a project is using assertEqual to compare two directory listings that don't match in the group. But the hint markers pointing to the mismatch are pointing at positions that match: E

[issue35953] crosscompilation fails with clang on android

Change by Karthikeyan Singaravelan : -- nosy: +xdegaye ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35460] Add PyDict_GetItemStringWithError

Ronald Oussoren added the comment: Adding PyDict_GetItemStringWithError makes is possible to migrate existing code to a design that does not ignore errors. I have a significant number of calls to PyDict_GetItemString that cannot be switch toed PyDict_GetItem without increasing the number

[issue12317] inspect.getabsfile() is not documented

Dong-hee Na added the comment: @terry.reedy I submited the PR 11786 for this issue. If you are okay can you take a look please? Thanks! -- ___ Python tracker ___

[issue35954] Incoherent type conversion in configparser

Change by Rémi Lapeyre : -- nosy: +remi.lapeyre ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35954] Incoherent type conversion in configparser

Change by Karthikeyan Singaravelan : -- nosy: +lukasz.langa ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35954] Incoherent type conversion in configparser

Adeokkuw added the comment: Btw: The name "read_dict" [1] as well as its docstring say exactly the opposite of what it does. It acts as a "save_dict". Maybe that can be fixed on the go ... The docstring """ [...] All types held in the dictionary are converted to strings during reading,

[issue35954] Incoherent type conversion in configparser

New submission from Adeokkuw : configparser interface implicitly converts all objects to str (read_dict [sic] on line 724 of "3.7/Lib/configparser.py") while saving but not while lookup (__getitem__ on line 956). MWE: ``` config = configparser.ConfigParser() config[123] = {}

[issue35953] crosscompilation fails with clang on android

New submission from muhzi : Trying to cross compile python for android using NDK r19. but I keep facing a linker error. x86_64-linux-android21-clang -pie -L~/Android/Sdk/ndk-bundle/platforms/android-21/arch-x86_64/usr/lib64 -Xlinker -export-dynamic -o python \

[issue35953] crosscompilation fails with clang on android

Change by muhzi : -- components: Cross-Build nosy: Alex.Willmer, muhzi priority: normal severity: normal status: open title: crosscompilation fails with clang on android type: compile error versions: Python 2.7 ___ Python tracker

[issue35055] Error when we try to download the epub archive

Julien Palard added the comment: @ned the issue with builds in 3.8 come from our use of column and row spanning in the 3.8 documentation, breaking the plain text builder. I fixed it in https://github.com/sphinx-doc/sphinx/pull/5559 which has to be release in march IIRC. --

[issue35892] Fix awkwardness of statistics.mode() for multimodal datasets

Francis MB added the comment: >> There may be better names for the flag. "tie_goes_to_first_encountered" >> seemed a bit long though ;-) Could it may be an alternative to set the mode tie case in a form like: def mode(seq, *, case=CHOOSE_FIRST): [...] (or TIE_CHOOSE_FIRST, ...) where

[issue21269] Provide args and kwargs attributes on mock call objects

Kumar Akshay added the comment: Thanks @xtreak! I've added a PR with the following API ➜ cpython git:(fix-issue-21269) ✗ ./python.exe Python 3.8.0a0 (heads/fix-issue-21269-dirty:2433a2ab70, Feb 10 2019, 14:24:54) [Clang 10.0.0 (clang-1000.10.44.4)] on darwin Type "help", "copyright",

[issue21269] Provide args and kwargs attributes on mock call objects

Change by Kumar Akshay : -- keywords: +patch, patch, patch pull_requests: +11819, 11820, 11821 stage: needs patch -> patch review ___ Python tracker ___

[issue21269] Provide args and kwargs attributes on mock call objects

Change by Kumar Akshay : -- keywords: +patch pull_requests: +11819 stage: needs patch -> patch review ___ Python tracker ___ ___

[issue21269] Provide args and kwargs attributes on mock call objects

Change by Kumar Akshay : -- keywords: +patch, patch pull_requests: +11819, 11820 stage: needs patch -> patch review ___ Python tracker ___

[issue35924] curses segfault resizing window

Change by SilentGhost : -- nosy: +twouters ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue35500] Align expected and actual calls on mock.assert_called_with error message

Tal Einat added the comment: After more thought, I agree that it isn't worth changing the current wording from "Actual" to something else. -- ___ Python tracker ___