[issue31096] asyncio.stream.FlowControlMixin._drain_helper may lead to a blocking behavior

Dmitry Malinovsky added the comment: I've changed the implementation significantly since August 2017, futures are not involved anymore so please ignore that part. However, such drain behavior is still present - but I don't think anymore that yielding to the event loop is

[issue33598] ActiveState Recipes links in docs, and the apparent closure of Recipes

Raymond Hettinger added the comment: > I have seen no official announcement of a shutdown date for > the code.activestate.com website, but it's future has to be > in question considering the migration. We can check with ActiveState but my understanding is that

[issue33647] Add re.replace(string, replacement_map)

Raymond Hettinger added the comment: > I would like an example added to the sub(pattern, function, > string) examples. That seems like the most reasonable way to go. -- nosy: +rhettinger ___ Python tracker

[issue33651] Add get() method to sqlite3.Row class

Raymond Hettinger added the comment: How is this needed? SQL queries return rows that are homogenous and with known fields. This is isn't like other dictionaries when you might not know in advance whether a given key is present. -- nosy: +rhettinger

[issue33521] Add 1.32x faster C implementation of asyncio.isfuture().

Yury Selivanov added the comment: IMO this particular patch is OK and should go in. I'll take another look at the PR in a few days (and will run some benchmarks myself before making the decision). -- ___ Python tracker

[issue33521] Add 1.32x faster C implementation of asyncio.isfuture().

STINNER Victor added the comment: I disagree with your rationale. At least, it is not how we decide to optimize something or not in Python. Python is first made of people who will have to maintain the code for the next 5 years if not longer. C code is harder to maintain

[issue33510] [logging] add JSON log formatter

Wilfredo Sanchez added the comment: Please consider using RFC 7464 (JSON Text Sequences). This allows detection of truncated JSON events without losing the following event. It allows newlines in the JSON data. All this involves is adding an RS between events.

[issue33651] Add get() method to sqlite3.Row class

New submission from Wilfredo Sanchez : The sqlite3.Row class has mapping-like behavior but does not implement the get() method, so providing default values in code requires a bit of boilerplate that is not necessary with dictionaries. -- components: Library

[issue32911] Doc strings no longer stored in body of AST

Matthias Bussonnier added the comment: > Just to quickly touch on Matthias' question about opt-in or deprecations, a > key thing to note is the ast module is automatically generated from the ASDL > file, so either of those approaches would require developing a

[issue33650] Prohibit adding a signal handler for SIGCHLD

New submission from Yury Selivanov : Doing that will break subprocesses. -- components: asyncio messages: 317717 nosy: asvetlov, yselivanov priority: normal severity: normal status: open title: Prohibit adding a signal handler for SIGCHLD versions: Python 3.8

[issue32911] Doc strings no longer stored in body of AST

Brett Cannon added the comment: Just to quickly touch on Matthias' question about opt-in or deprecations, a key thing to note is the ast module is automatically generated from the ASDL file, so either of those approaches would require developing a new mechanism in the code

[issue32841] Asyncio.Condition prevents cancellation

Bar Harel added the comment: Yup. Thanks Yuri :-) -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue33598] ActiveState Recipes links in docs, and the apparent closure of Recipes

Terry J. Reedy added the comment: The Python Cookbook, based on the site, says that copyright of each recipe is retained by original authors. If so authors who have signed the PSF CLA can contribute their own work. Since Active State must have collectively licensed the

[issue33598] ActiveState Recipes links in docs, and the apparent closure of Recipes

Marc-Andre Lemburg added the comment: I'd suggest to contact ActiveState first before jumping to conclusions. -- nosy: +lemburg ___ Python tracker

[issue33598] ActiveState Recipes links in docs, and the apparent closure of Recipes

Alex Walters added the comment: All recipes are MIT licensed unless otherwise noted on the recipe itself. However, AFAICT, all the recipes that are linked to in the docs are written by Raymond Hettinger, so I don't think licensing will be that big of an issue. We

[issue33649] asyncio docs overhaul

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

[issue33598] ActiveState Recipes links in docs, and the apparent closure of Recipes

Terry J. Reedy added the comment: Can you check the copyright and license of the recipe text and code? -- nosy: +terry.reedy ___ Python tracker

[issue33647] Add re.replace(string, replacement_map)

Terry J. Reedy added the comment: Your proposal is underspecified. (And 'yours' is inconsistent with 'your';-). If you want sequential replacememt in multiple scans, make sequential replace calls. Use a loop if the number of replacement scans is variable or large. To be

[issue33649] asyncio docs overhaul

New submission from Yury Selivanov : An overhaul of asyncio documentation is long overdue. Here's the structure for it that I have in mind: - Introduction (what is asyncio and async/await) - A quick tutorial (show how to use asyncio.run() and basic functions like

[issue30145] Create a How to or Tutorial documentation for asyncio

Yury Selivanov added the comment: Closing this issue. I'll open a new one for the planned asyncio docs overhaul. -- resolution: -> out of date stage: -> resolved status: open -> closed ___ Python tracker

[issue31647] asyncio: StreamWriter write_eof() after close raises mysterious AttributeError

Yury Selivanov added the comment: If this issue isn't yet fixed could you please submit a PR? -- ___ Python tracker ___

[issue31096] asyncio.stream.FlowControlMixin._drain_helper may lead to a blocking behavior

Yury Selivanov added the comment: Andrew, what are your thoughts on this one? -- nosy: +asvetlov versions: +Python 3.8 -Python 3.5, Python 3.6 ___ Python tracker

[issue32528] Change base class for futures.CancelledError

Yury Selivanov added the comment: Closing this issue as I, personally, don't see this happening and there's no point in keeping it open. -- resolution: -> rejected stage: -> resolved status: open -> closed ___ Python tracker

[issue30698] asyncio sslproto do not shutdown ssl layer cleanly

Yury Selivanov added the comment: Is this issue resolved now, or do we need to work on this? -- ___ Python tracker ___

[issue32672] .then execution of actions following a future's completion

Yury Selivanov added the comment: Closing this one. I don't see us adding Future.then. Feel free to discuss on the MLs. -- components: +asyncio -Library (Lib) nosy: +asvetlov, yselivanov resolution: -> rejected stage: patch review -> resolved

[issue32841] Asyncio.Condition prevents cancellation

Yury Selivanov added the comment: Should this issue be closed now? -- ___ Python tracker ___

[issue1366311] SRE engine should release the GIL when/if possible

Barry A. Warsaw added the comment: > Did you try to use regex which has this feature? For short strings and simple > patterns there may be no benefit. In my use case, it’s not possible, since the problematic API is glob.iglob() through multiple levels of calls. --

[issue1366311] SRE engine should release the GIL when/if possible

Serhiy Storchaka added the comment: It is possible to implement this feature, but it has some limitations. Did you try to use regex which has this feature? For short strings and simple patterns there may be no benefit. -- assignee: -> serhiy.storchaka

[issue33595] Fix references to lambda "arguments"

Terry J. Reedy added the comment: Thanks for the fix. -- resolution: -> fixed stage: patch review -> resolved status: open -> closed title: FIx references to lambda "arguments" -> Fix references to lambda "arguments" ___ Python

[issue33595] FIx references to lambda "arguments"

Terry J. Reedy added the comment: New changeset 804fcf66559992db9d23695e501c502ab20b7712 by Terry Jan Reedy in branch '2.7': [2.7] bpo-33595: Fix lambda parameters being refered as arguments (GH-7037) (GH-7122)

[issue33595] FIx references to lambda "arguments"

Change by Terry J. Reedy : -- pull_requests: +6757 ___ Python tracker ___ ___

[issue33645] Importing bs4 fails with -3 option in Python 2.7.15

Florian Schulze added the comment: You are right, this actually happens with Python 2.7.14 as well. I was fooled by a warnings.filterwarnings matching to a path in my codebase. That one did match for my Python 2.7.15 testing but didn't for my Python 2.7.14 testing,

[issue32911] Doc strings no longer stored in body of AST

Serhiy Storchaka added the comment: PR 7121 is based on PR 5927 and goes one step further. It doesn't introduce a new AST type DocString. Third-party code should not need any adaptation for 3.7. -- ___ Python tracker

[issue32911] Doc strings no longer stored in body of AST

Change by Serhiy Storchaka : -- keywords: +patch pull_requests: +6756 stage: commit review -> patch review ___ Python tracker

[issue33595] FIx references to lambda "arguments"

Change by miss-islington : -- pull_requests: +6755 ___ Python tracker ___

[issue33595] FIx references to lambda "arguments"

Change by miss-islington : -- pull_requests: +6754 stage: backport needed -> patch review ___ Python tracker

[issue33594] add deprecation since 3.5 for a few methods of inspect.

Terry J. Reedy added the comment: See also #33582 for 'formatargspec'. -- nosy: +terry.reedy, yselivanov versions: -Python 3.5 ___ Python tracker

[issue33587] inspect.getsource performs unnecessary filesystem stat call

Change by Terry J. Reedy : -- stage: -> patch review type: -> performance versions: +Python 3.8 ___ Python tracker ___

[issue33587] inspect.getsource performs unnecessary filesystem stat call

Change by Terry J. Reedy : -- nosy: +yselivanov title: inspect.getsource performs unnecessary filesystem stat call which is ignored in some circumstances -> inspect.getsource performs unnecessary filesystem stat call ___ Python

[issue33582] formatargspec deprecated but does nto emit DeprecationWarning.

Change by Terry J. Reedy : -- nosy: +yselivanov versions: -Python 3.5 ___ Python tracker ___

[issue1366311] SRE engine should release the GIL when/if possible

Change by Armin Rigo : -- nosy: -arigo ___ Python tracker ___ ___

[issue33573] statistics.median does not work with ordinal scale, add doc

Terry J. Reedy added the comment: I agree. -- assignee: -> docs@python components: +Documentation nosy: +docs@python, terry.reedy resolution: not a bug -> stage: -> needs patch title: statistics.median does not work with ordinal scale -> statistics.median does not

[issue33521] Add 1.32x faster C implementation of asyncio.isfuture().

Jimmy Lai added the comment: @vstinner In general, we would like to make all asyncio common functions efficient with C implementation because CPython asyncio overhead is very expensive. In our application, overall it costs about 10% global CPU instructions after we used

[issue1366311] SRE engine should release the GIL when/if possible

Barry A. Warsaw added the comment: I'm reopening this bug even though it's been long closed and even though the attached patch is no longer relevant. We recently found a degenerative performance problem with entrypoints and threads. As the number of threads increases, it

[issue18872] platform.linux_distribution() doesn't recognize Amazon Linux

Petr Viktorin added the comment: linux_distribution is removed in Python 3.8. -- nosy: +petr.viktorin resolution: -> out of date stage: -> resolved status: open -> closed ___ Python tracker

[issue19213] platform.linux_distribution detects Oracle Linux as Red Hat Enterprise Linux

Petr Viktorin added the comment: Er, sorry, I meant in 3.8. -- ___ Python tracker ___

[issue20454] platform.linux_distribution() returns empty value on Archlinux and python 2.7

Petr Viktorin added the comment: Er, sorry, I meant in 3.8. -- ___ Python tracker ___

[issue19213] platform.linux_distribution detects Oracle Linux as Red Hat Enterprise Linux

Petr Viktorin added the comment: linux_distribution is removed in Python 3.7. Closing. -- nosy: +petr.viktorin resolution: -> out of date stage: patch review -> resolved status: open -> closed ___ Python tracker

[issue20454] platform.linux_distribution() returns empty value on Archlinux and python 2.7

Petr Viktorin added the comment: linux_distribution is removed in Python 3.7. Closing. -- nosy: +petr.viktorin resolution: -> out of date stage: -> resolved status: open -> closed ___ Python tracker

[issue32911] Doc strings no longer stored in body of AST

INADA Naoki added the comment: > B. Revert the feature now for 3.7.0, retargeting for 3.8, and produce a > 3.7.0b5 on a somewhat shorter cycle to allow downstream users to adapt to the > removal. Please note that it can't be reverted simply. The change was introduced

[issue33647] Make string.replace accept a dict instead of two arguments

Change by R. David Murray : -- versions: -Python 3.6, Python 3.7 ___ Python tracker ___

[issue33647] Make string.replace accept a dict instead of two arguments

R. David Murray added the comment: That is not kwargs, that's a passing a dict. Which is what you would want, since the strings you want to replace might not be valid identifiers, and so couldn't be passed as keyword arguments. I think I'm -0.5 on this. I don't think

[issue32374] Document that m_traverse for multi-phase initialized modules can be called with m_state=NULL

STINNER Victor added the comment: """ This particular kind of bad traverse is quite easy to write if an extension author doesn't read docs carefully, or has read an old version of them (before the fix here). And resulting errors aren't too obvious. So, there's code to

[issue32911] Doc strings no longer stored in body of AST

Matthias Bussonnier added the comment: Łukasz Langa wrote: > Inadasan, I think what we should do is to amend `ast.parse()` and `compile()` > docs that mode="exec" treats given code as a module, we should even give the > docstring handling as an example. That is

[issue33648] unused with_c_locale_warning option in configure should be removed

Eitan Adler added the comment: yeah, I was looking at that too. I think someone else modified related files but never ran a regen (or did so with different tooling than I). Even on master, my regen results in changes. -- ___

[issue33645] Importing bs4 fails with -3 option in Python 2.7.15

Change by Serhiy Storchaka : -- keywords: +patch pull_requests: +6753 stage: -> patch review ___ Python tracker ___

[issue33645] Importing bs4 fails with -3 option in Python 2.7.15

Serhiy Storchaka added the comment: Ah, I was fooled by the message about Beautiful Soup. It is not a message of an error, it is a part of source line, printed for a SyntaxError. Seems sources of Beautiful Soup intentionally contain a code invalid in Python 3.

[issue33648] unused with_c_locale_warning option in configure should be removed

Nick Coghlan added the comment: Huh, I thought I cleaned those out when I made the compilation unconditional. I guess not :) The actual fix looks good to me, but the autoconf regeneration looks unexpectedly noisy, so I've asked Benjamin Peterson to take a look at it.

[issue26826] Expose new copy_file_range() syscall in os module.

Giampaolo Rodola' added the comment: Check for availability in configure.ac appears to be broken. -- ___ Python tracker ___

[issue33645] Importing bs4 fails with -3 option in Python 2.7.15

Florian Schulze added the comment: Yes, it's third party code, but that worked up until Python 2.7.14 and only broke with 2.7.15, which is why I reported it here. The expected behaviour is not a SyntaxError, but a DeprecationWarning, which is what you get pre Python

[issue33648] unused with_c_locale_warning option in configure should be removed

Change by Eitan Adler : -- keywords: +patch pull_requests: +6752 stage: -> patch review ___ Python tracker ___

[issue33648] unused with_c_locale_warning option in configure should be removed

STINNER Victor added the comment: It's related to the PEP 538. Note: see also the PEP 540 (not directly related). -- nosy: +ncoghlan, vstinner ___ Python tracker

[issue33622] Fix errors handling in the garbage collector

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

[issue33648] unused with_c_locale_warning option in configure should be removed

New submission from Eitan Adler : There is an option for --with-c-locale-warning which was turned into a run-time option in eb81795d7d3a8c898fa89a376d63fc3bbfb9a081. The configuration should be cleaned up. -- messages: 317676 nosy: eitan.adler priority: normal

[issue33647] Make string.replace accept **kwargs instead of two arguments

Change by Eitan Adler : -- nosy: +eitan.adler ___ Python tracker ___ ___

[issue33645] Importing bs4 fails with -3 option in Python 2.7.15

Serhiy Storchaka added the comment: The Python interpreter itself does not have references to Beautiful Soup. This error is generated by third-party code. Report a bug on the Beautiful Soup bug tracker. -- nosy: +serhiy.storchaka resolution: -> third

[issue33646] os.fspath() bypasses __fspath__ for str subclasses

Serhiy Storchaka added the comment: It works as documented. https://docs.python.org/3/library/os.html#os.fspath https://www.python.org/dev/peps/pep-0519/ -- nosy: +serhiy.storchaka resolution: -> not a bug stage: -> resolved status: open -> closed

[issue33547] Relative imports do not replace local variables

Nick Coghlan added the comment: Not as a statement, but you can force it with importlib.import_module: $ python3 -c "import pkg; pkg.submodule = 1; import importlib; importlib.import_module('.submodule', 'pkg'); print(pkg.submodule)" pkg pkg.submodule We're getting

[issue33647] Make string.replace accept **kwargs instead of two arguments

New submission from Paal Pedersen : It would be really nice if the python core function string.replace is modifed to accept **kwargs instead of 2 arguments. then you could just do: kwargs = {'my': 'yours', 'string': 'car'} >>>'this is my string'.replace(kwargs) 'this is

[issue32911] Doc strings no longer stored in body of AST

Nick Coghlan added the comment: Ouch, I'd completely missed the fact that this would affect the parsing of bare strings to a simple AST (I was focused on functions and classes, as in Mark's original example). So even though I'm the author of

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

Change by Eitan Adler : -- nosy: +eitan.adler ___ Python tracker ___ ___

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

miss-islington added the comment: New changeset 20b797dafa149afcdd7b64e489eb8ea1386d8eb4 by Miss Islington (bot) in branch '3.6': bpo-33012: Add -Wno-cast-function-type for gcc 8. (GH-6757)

[issue33642] IDLE: Use variable number of lines in CodeContext

Cheryl Sabella added the comment: I know we usually don't change defaults, but I had tested this one and I think it works. After I changed it to 15 lines in master, I opened the installed version of IDLE 2.7. Although it showed 15 fixed lines, it made sense since

[issue32911] Doc strings no longer stored in body of AST

Change by Eitan Adler : -- nosy: +eitan.adler ___ Python tracker ___ ___

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

Change by miss-islington : -- pull_requests: +6751 ___ Python tracker ___

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

Serhiy Storchaka added the comment: New changeset ef91ddeae79497fac25545dd68ee55a5a3c60e8d by Serhiy Storchaka in branch '3.7': bpo-33012: Add -Wno-cast-function-type for gcc 8. (GH-6757)

[issue26826] Expose new copy_file_range() syscall in os module.

Marcos Dione added the comment: I'm really sorry, but I don't have time to continue with this (new daughter!). Can someone else pick it up? -- ___ Python tracker

[issue33646] os.fspath() bypasses __fspath__ for str subclasses

New submission from Joseph Martinot-Lagarde : os.fspath() returns its argument if it is a str. That means that it bypasses __fspath__ for str subclasses. This is the case for the library path.py for example. This is a corner case that was discovered while trying to fix

[issue32911] Doc strings no longer stored in body of AST

Min RK added the comment: In the A/B vote, I cast mine for B, for what it is worth, but it is not strongly held. >From the IPython side, I don't view our particular issue as a major regression >for users. The only affected case for us is interactively typed string

[issue33645] Importing bs4 fails with -3 option in Python 2.7.15

New submission from Florian Schulze : Since Python 2.7.15 import bs4 (BeautifulSoup4) fails when using the -3 option on the Python binary. ... 'You are trying to run the Python 2 version of Beautiful Soup under Python 3. This will not work.'<>'You need to convert

[issue24373] Use traverse & finalize in xxlimited and in PEP 489 tests

Serhiy Storchaka added the comment: tp_finalize handlers should return "void". See issue33644. -- nosy: +serhiy.storchaka ___ Python tracker

[issue33644] Fix signatures of tp_finalize handlers in testing code.

Change by Serhiy Storchaka : -- keywords: +patch pull_requests: +6750 stage: -> patch review ___ Python tracker ___

[issue33644] Fix signatures of tp_finalize handlers in testing code.

New submission from Serhiy Storchaka : tp_finalize handlers in Modules/_testmultiphase.c and Modules/_testmultiphase.c return int. This signature is incompatible with the signature of the tp_finalize slot. They should not return any value. -- components:

[issue26819] _ProactorReadPipeTransport pause_reading()/resume_reading() broken if called before any read is perfored

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

[issue26819] _ProactorReadPipeTransport pause_reading()/resume_reading() broken if called before any read is perfored

Andrew Svetlov added the comment: New changeset 749afe81ec0a4b92ad6b89a67c82f2c04f79c5ac by Andrew Svetlov (CtrlZvi) in branch '3.6': [3.6] bpo-26819: Prevent proactor double read on resume (GH-6921) (#7110)

[issue32374] Document that m_traverse for multi-phase initialized modules can be called with m_state=NULL

Petr Viktorin added the comment: > That's a weak test: if the script fails before calling > spec.loader.create_module(), the test also pass. If the function raises an > exception but don't crash, the test pass as well. That's a good argument, thanks. Marcel, would you mind

[issue33643] Mock functions with autospec STILL don't support assert_called_once, assert_called, assert_not_called

New submission from Piotr Dybowski : This is the same issue as: https://bugs.python.org/issue28380. It seems that fix has never been merged, -- components: Library (Lib) messages: 317659 nosy: dybi priority: normal severity: normal status: open title: Mock

[issue26819] _ProactorReadPipeTransport pause_reading()/resume_reading() broken if called before any read is perfored

Change by Zvi Effron : -- pull_requests: +6749 ___ Python tracker ___ ___

[issue33623] Fix possible SIGSGV when asyncio.Future is created in __del__

Serhiy Storchaka added the comment: It is okay if the fact that half of the objects are freed leads to raising exceptions. But an assertion failure is sign of bugs in Python core or the _asyncio module. PR 7080 removes one way of exposing these bugs, but bugs