Jason R. Coombs added the comment:
Thanks Filipe for the summary. I was unsure the status.
Even if we grant that Debian is too intrusive in its patching of distutils,
it's Python and distutils that are making the change here, so it's not
unreasonable for Python to maintain feature parity
Change by Jason R. Coombs :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Jason R. Coombs added the comment:
New changeset 21d5897982698a461215203ab51f56ad05648001 by Miss Islington (bot)
in branch '3.10':
bpo-44784: Apply changes from importlib_metadata 4.6.3 (GH-27508) (#27510)
https://github.com/python/cpython/commit/21d5897982698a461215203ab51f56ad05648001
Change by Jason R. Coombs :
--
keywords: +patch
pull_requests: +26022
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27508
___
Python tracker
<https://bugs.python.org/issu
Jason R. Coombs added the comment:
Patch at https://github.com/python/importlib_metadata/pull/333.
--
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
All four of those tests had used `capture_warnings` in an attempt to capture
them, but it did not alter the warnings filter.
--
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
In addition to the two reported, there are two others:
- test_entry_points_dict_construction
- test_entry_points_by_index
that also fail. I've been able to replicate the issue in importlib_metadata, so
I'll fix it there first
Jason R. Coombs added the comment:
New changeset e63e6311aa258a5f3f49a7aed9fdde445fd384d6 by Jason R. Coombs in
branch 'main':
bpo-44771: Sync with importlib_resources 5.2.2, fixing refleak. (#27497)
https://github.com/python/cpython/commit/e63e6311aa258a5f3f49a7aed9fdde445fd384d6
Change by Jason R. Coombs :
--
pull_requests: +26015
pull_request: https://github.com/python/cpython/pull/27497
___
Python tracker
<https://bugs.python.org/issue44
Change by Jason R. Coombs :
--
nosy: +jaraco
nosy_count: 6.0 -> 7.0
pull_requests: +26014
pull_request: https://github.com/python/cpython/pull/27436
___
Python tracker
<https://bugs.python.org/issu
Jason R. Coombs added the comment:
Awesome. Thanks Łukasz.
--
___
Python tracker
<https://bugs.python.org/issue44777>
___
___
Python-bugs-list mailing list
Unsub
Change by Jason R. Coombs :
--
keywords: +patch
pull_requests: +25998
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27436
___
Python tracker
<https://bugs.python.org/issu
Change by Jason R. Coombs :
--
pull_requests: +25997
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27436
___
Python tracker
<https://bugs.python.org/issu
Change by Jason R. Coombs :
--
stage: patch review ->
___
Python tracker
<https://bugs.python.org/issue44777>
___
___
Python-bugs-list mailing list
Unsubscrib
Change by Jason R. Coombs :
--
pull_requests: -25996
___
Python tracker
<https://bugs.python.org/issue44777>
___
___
Python-bugs-list mailing list
Unsubscribe:
New submission from Jason R. Coombs :
In [this
comment](https://github.com/python/cpython/pull/27436#issuecomment-889815333),
I learned that it's possible to get repo clones into a bad state by:
- commit a text file to main (merge a PR)
- customize the newline handling for that file
Change by Jason R. Coombs :
--
keywords: +patch
pull_requests: +25996
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27436
___
Python tracker
<https://bugs.python.org/issu
New submission from Jason R. Coombs :
In [this
comment](https://github.com/python/cpython/pull/27436#issuecomment-889815333),
I learned that it's possible to break the buildbots in a way that's not fixable
with a simple code change.
The recommendation there was to contact the buildbot
Change by Jason R. Coombs :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Change by Jason R. Coombs :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Jason R. Coombs added the comment:
New changeset aaa83cdfab6817446285e631232f64b394ac6791 by Jason R. Coombs in
branch 'main':
bpo-44771: Apply changes from importlib_resources 5.2.1 (GH-27436)
https://github.com/python/cpython/commit/aaa83cdfab6817446285e631232f64b394ac6791
Jason R. Coombs added the comment:
In [this
comment](https://github.com/python/importlib_resources/pull/221#discussion_r679124389),
I did more analysis and assuaged my own concerns about the implementation.
Everything seems to be working as intended, thanks to the strong work by FFY00
Change by Jason R. Coombs :
--
nosy: -jaraco
___
Python tracker
<https://bugs.python.org/issue40263>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Jason R. Coombs :
--
pull_requests: -25966
___
Python tracker
<https://bugs.python.org/issue40263>
___
___
Python-bugs-list mailing list
Unsubscribe:
Jason R. Coombs added the comment:
On further consideration, the concern I had may not be manifest in the
implementation. It may be the _adapters.wrap_spec provides all the
compatibility that's needed. I'll see if I can concoct an idiomatic case, but I
may find that there's no concern
Jason R. Coombs added the comment:
I've put together a patch based on python/importlib_resources@3b24bd6307, and
interestingly, the tests pass... which is a little concerning because in
https://github.com/python/importlib_resources/pull/221#discussion_r659214483, I
describe a compatibility
Change by Jason R. Coombs :
--
nosy: +FFY00
___
Python tracker
<https://bugs.python.org/issue44771>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Jason R. Coombs :
--
nosy: +jaraco
nosy_count: 8.0 -> 9.0
pull_requests: +25966
pull_request: https://github.com/python/cpython/pull/27436
___
Python tracker
<https://bugs.python.org/issu
Change by Jason R. Coombs :
--
keywords: +patch
pull_requests: +25965
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/27436
___
Python tracker
<https://bugs.python.org/issu
New submission from Jason R. Coombs :
Importlib_resources 5.1 and 5.2 introduced the following changes (more details
at https://importlib-resources.readthedocs.io/en/latest/history.html#v5-2-1):
- Added ``simple`` module implementing adapters from a low-level resources
reader interface
Jason R. Coombs added the comment:
This issue is solved. Thanks Irit for taking the time to investigate the issue
and do the hard part of developing a test.
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
_
Jason R. Coombs added the comment:
New changeset 49b5e20fa8c091f9158ccd26f9a12fc1e91cfc93 by Jason R. Coombs in
branch '3.9':
[3.9] bpo-44461: Check early that a pdb target is valid for execution.
(GH-27227) (GH-27400)
https://github.com/python/cpython/commit
Change by Jason R. Coombs :
--
pull_requests: +25933
pull_request: https://github.com/python/cpython/pull/27400
___
Python tracker
<https://bugs.python.org/issue44
Change by Jason R. Coombs :
--
pull_requests: +25932
pull_request: https://github.com/python/cpython/pull/27399
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
New changeset ee03bad25e83b00ba5fc2a0265b48c6286e6b3f7 by Jason R. Coombs in
branch 'main':
bpo-44461: Check early that a pdb target is valid for execution. (#27227)
https://github.com/python/cpython/commit/ee03bad25e83b00ba5fc2a0265b48c6286e6b3f7
Change by Jason R. Coombs :
--
pull_requests: -25553
___
Python tracker
<https://bugs.python.org/issue44461>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Jason R. Coombs :
--
pull_requests: +25768
pull_request: https://github.com/python/cpython/pull/27227
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
Additional problems I noticed while working on the refactor:
- There is a lot of overlap in behavior between the implementations of
_run_script and _run_module (initializing private variables, setting
mainpyfile, resetting the __main__ namespace
Change by Jason R. Coombs :
--
pull_requests: +25724
pull_request: https://github.com/python/cpython/pull/27188
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
> My python version is 3.8.9 and zipp is at 3.5.0 (but 3.4.1 behaves the same
> for me).
It's not enough to have `zipp` 3.5.0. You need to use `zipp.Path` over
`zipfile.Path`.
--
___
Python tracker
Change by Jason R. Coombs :
--
resolution: -> duplicate
stage: -> resolved
status: open -> closed
superseder: -> Using zipfile.Path with several files prematurely closes zip
___
Python tracker
<https://bugs.python
Jason R. Coombs added the comment:
Changing the repro to:
```
import zipfile
try:
import zipp
except ImportError:
import zipfile as zipp
zip_file = zipfile.ZipFile('zipfile.zip')
name = zip_file.namelist()[0]
zipp.Path(zip_file)
zip_file.open(name)
```
I'm able now to test against
Jason R. Coombs added the comment:
Even simpler:
```
zip_file = zipfile.ZipFile('zipfile.zip')
name = zip_file.namelist()[0]
zipfile.Path(zip_file)
zip_file.open(name)
```
--
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
This also reproduces the failure:
```
zip_file = zipfile.ZipFile('zipfile.zip')
path = zipfile.Path(zip_file)
name = zip_file.namelist()[0]
del path
zip_file.open(name)
```
Removing `del path` bypasses the issue. Something about the destructor
Jason R. Coombs added the comment:
Even simpler:
```
import zipfile
zip_file = zipfile.ZipFile('zipfile.zip')
names = [each.name for each in zipfile.Path(zip_file).iterdir()]
zip_file.open(names[0])
```
--
___
Python tracker
<ht
Jason R. Coombs added the comment:
Here's a much simpler repro that avoids the class construction but triggers the
same error:
```
import zipfile
zip_file = zipfile.ZipFile('zipfile.zip')
names = [each.name for each in zipfile.Path(zip_file).iterdir()]
with zip_file.open(names[0]) as file
Jason R. Coombs added the comment:
I was able to replicate the error using the script as posted:
```
draft $ cat > issue44638.py
import zipfile
class TestClass:
def __init__(self, path):
self.zip_file = zipfile.ZipFile(path)
def iter_dir(self):
return [each.n
Jason R. Coombs added the comment:
Thanks Barry for filing this. I was planning to do it but happy for you to beat
me to it.
I had one question I'd been meaning to ask - with this change, can the code
paths that were deprecated in 3.10 be removed in 3.11 (SelectableGroups and
cast to dict
Jason R. Coombs added the comment:
I was able to export the configuration using these instructions:
https://docs.microsoft.com/en-us/visualstudio/install/import-export-installation-configurations?view=vs-2019
That produced this config:
{
"version": "1.0&
Change by Jason R. Coombs :
--
keywords: +patch
pull_requests: +25561
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/26992
___
Python tracker
<https://bugs.python.org/issu
New submission from Jason R. Coombs :
While investigating issue44461, I observed some complexities to the current
pdb.main implementation, some of which likely contributed to the bug being
present.
- variables are initialized to defaults
(https://github.com/python/cpython/blob
Change by Jason R. Coombs :
--
pull_requests: +25553
pull_request: https://github.com/python/cpython/pull/26992
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
Thanks Irit. Reviewing now...
--
assignee: -> jaraco
___
Python tracker
<https://bugs.python.org/issue44461>
___
___
Py
Change by Jason R. Coombs :
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Jason R. Coombs added the comment:
New changeset f4b31cdbc043449f3df7d291da67bcb3736be0db by Miss Islington (bot)
in branch '3.10':
bpo-44464: Remove special exclusion for flake8 in the deprecation warnings.
(GH-26807) (GH-26929)
https://github.com/python/cpython/commit
Jason R. Coombs added the comment:
New changeset 17b916737bd05a11ec24341990c4d8dbfee8f94b by Jason R. Coombs in
branch '3.9':
[3.9] bpo-37741: make importlib.metadata docs discoverable through a module
directive. (GH-25415) (GH-26500)
https://github.com/python/cpython/commit
Jason R. Coombs added the comment:
New changeset efe7d08d178a7c09bcca994f2068b019c8633d83 by Jason R. Coombs in
branch 'main':
bpo-44464: Remove special exclusion for flake8 in the deprecation warnings.
(#26807)
https://github.com/python/cpython/commit
Jason R. Coombs added the comment:
In https://github.com/python/importlib_metadata/issues/324, I painstakingly
created a library to perform robust performance monitoring of behaviors in the
library and specifically added tests that capture the performance of
entry_points against
Change by Jason Yundt :
--
nosy: +jayman
___
Python tracker
<https://bugs.python.org/issue44510>
___
___
Python-bugs-list mailing list
Unsubscribe:
Change by Jason R. Coombs :
--
keywords: +patch
pull_requests: +25388
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/26807
___
Python tracker
<https://bugs.python.org/issu
New submission from Jason R. Coombs :
importlib_metadata 4.5
(https://importlib-metadata.readthedocs.io/en/latest/history.html#v4-5-0)
removed the special exclusion for flake8 in the deprecation warning for
deprecated interfaces in entry_points. For the same motivations there, do the
same
Jason R. Coombs added the comment:
I submitted https://github.com/brechtm/rinohtype/pull/270 to address the issue
in rinohtype. Please feel free to reopen and address the questions posed above
if appropriate.
--
resolution: -> wont fix
stage: -> resolved
status: open -&g
Jason R. Coombs added the comment:
A small patch to rinoh seems to work around the issue:
```
diff --git a/src/rinoh/resource.py b/src/rinoh/resource.py
index 57143f9d..586a4bb7 100644
--- a/src/rinoh/resource.py
+++ b/src/rinoh/resource.py
@@ -132,7 +132,9 @@ class DynamicEntryPoint
Change by Jason R. Coombs :
--
keywords: -3.10regression
___
Python tracker
<https://bugs.python.org/issue44461>
___
___
Python-bugs-list mailing list
Unsub
Jason R. Coombs added the comment:
I was _unable_ to replicate the issue with a foo.py containing only `raise
ValueError`, even with `python -m pdb -m foo`.
--
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
I can replicate the issue with this command: `python -m pdb -m importlib`
(since `importlib` isn't executable, it triggers an error in runpy).
--
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
I encountered the same error on Python 3.9.
--
versions: +Python 3.9
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
It seems that even on Python 3.9 or 3.10b2, the
[DynamicRinohDistribution](https://github.com/brechtm/rinohtype/blob/e7f1c2c6066303d86cff4105be0829f512059be2/src/rinoh/resource.py#L132)
would return `None` for the `name` property (despite the [attempt
Change by Jason R. Coombs :
--
priority: normal -> deferred blocker
___
Python tracker
<https://bugs.python.org/issue44459>
___
___
Python-bugs-list mai
Jason R. Coombs added the comment:
Running the debugger, I can confirm that Distribution.name is None in this case:
```
~ $ python3.10 -m pip-run -q rinohtype -- -m pdb -m rinoh --help
WARNING: You are using pip version 21.1.1; however, version 21.1.2 is available.
You should consider
New submission from Jason R. Coombs :
While doing some debugging on issue44459 using Python 3.10b3, I was using Pdb
and after troubleshooting an exception and hitting 'q' to quit, I saw the
following:
```
(Pdb) q
Traceback (most recent call last):
File
"/Library/Frame
Jason R. Coombs added the comment:
I suspect the performance enhancements to distribution deduplication are
relevant here. Previously, distributions were deduplicated based on the
canonical distribution name. Now they're deduplicated on the normalized name.
And it seems that when attempting
Jason R. Coombs added the comment:
Big thanks Miro for the detailed work on this matter.
--
resolution: -> fixed
stage: patch review -> resolved
status: open -> closed
___
Python tracker
<https://bugs.python.or
Jason R. Coombs added the comment:
New changeset df1502e47fc1e0cf1e7d460ae04530c3e2e4a7c6 by Miro Hrončok in
branch 'main':
bpo-44451: Reset DeprecationWarning filters in
test_importlib.test_entry_points_by_index (GH-26784)
https://github.com/python/cpython/commit
Jason R. Coombs added the comment:
Other tests reset the filters? Good observation. I’d missed that detail. If
it’s easier to submit a PR to importlib_metadata, feel free to do that as I’ll
want the change in both places.
--
components: +Library (Lib) -Tests
stage: patch review
Jason R. Coombs added the comment:
The build error is tracked in issue43298.
--
___
Python tracker
<https://bugs.python.org/issue44246>
___
___
Python-bug
Jason R. Coombs added the comment:
In issue44246, I've encountered this error while attempting to build CPython in
a Docker image (built from
https://github.com/jaraco/jaraco.windows/blob/d2edad2e2af9d469189d7ac6a14a4ba6f6270348/Dockerfile).
Any suggestions on how to install the SDK
Jason R. Coombs added the comment:
I managed to put together a Dockerfile that seemingly has the build tools
installed
(https://github.com/jaraco/jaraco.windows/blob/d2edad2e2af9d469189d7ac6a14a4ba6f6270348/Dockerfile).
When I attempt to build CPython, however, it fails with this error
Jason R. Coombs added the comment:
In [this
Dockerfile](https://github.com/jaraco/jaraco.windows/blob/feature/vs-2/Dockerfile),
I've attempted to install Visual Studio, but without success. Docker fails to
build on [line
6](https://github.com/jaraco/jaraco.windows/blob
Jason R. Coombs added the comment:
As mentioned in msg394775, I'd like to decouple the performance concerns from
the original incompatibility. I recognize that performance regressions are in
their own way a form of incompatibility, but there have been a lot of changes
to entry points
Jason R. Coombs added the comment:
Thanks Steve for the feedback.
> If it's a warning, it could also be that the warning is being triggered
> somewhere else first. Since tests run in a random order, you'll want to look
> for how reliable the failure is.
It is a warning, but
Jason R. Coombs added the comment:
New changeset afb2eed72b32a35b4726ff35f92e4fbf54926046 by wim glenn in branch
'main':
bpo-37449: ensurepip uses importlib.resources.files() traversable APIs (#22659)
https://github.com/python/cpython/commit/afb2eed72b32a35b4726ff35f92e4fbf54926046
Jason R. Coombs added the comment:
The line where the failure occurs is the point where it's checking that the
warning was issued. The fact that a StopIteration is raised indicates that no
warnings were caught. I can think of a couple of scenarios where that could
happen:
- That warning
Change by Jason R. Coombs :
--
pull_requests: +25096
pull_request: https://github.com/python/cpython/pull/26500
___
Python tracker
<https://bugs.python.org/issue37
Jason R. Coombs added the comment:
New changeset d0991e2db3bb932e2411ee9dca54fd69ff2611c4 by Miss Islington (bot)
in branch '3.10':
bpo-44246: Remove note about access by index now that a compatibility shim is
offered. (GH-26472) (#26473)
https://github.com/python/cpython/commit
Jason R. Coombs added the comment:
New changeset 78d9a9b1904f0e1d9db1e941c19782f4f5a881d4 by Jason R. Coombs in
branch 'main':
bpo-44246: Remove note about access by index now that a compatibility shim is
offered. (GH-26472)
https://github.com/python/cpython/commit
Change by Jason R. Coombs :
--
pull_requests: +25067
pull_request: https://github.com/python/cpython/pull/26472
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
The compatibility concerns are addressed with PR 26468 and some of the
performance concerns may be mitigated with PR 26467. As I mentioned before, if
there are continuing performance concerns, please raise them separately
(preferably with python
Jason R. Coombs added the comment:
New changeset c34ed08d975fb7daa7b329f7c631647782290393 by Jason R. Coombs in
branch 'main':
bpo-44246: Restore compatibility in entry_points (GH-26468)
https://github.com/python/cpython/commit/c34ed08d975fb7daa7b329f7c631647782290393
Jason R. Coombs added the comment:
New changeset 410b70d39d9d77384f8b8597560f6731530149ca by Jason R. Coombs in
branch 'main':
bpo-44246: Entry points performance improvements. (GH-26467)
https://github.com/python/cpython/commit/410b70d39d9d77384f8b8597560f6731530149ca
Change by Jason R. Coombs :
--
pull_requests: +25063
pull_request: https://github.com/python/cpython/pull/26468
___
Python tracker
<https://bugs.python.org/issue44
Change by Jason R. Coombs :
--
pull_requests: +25062
pull_request: https://github.com/python/cpython/pull/26467
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
importlib_metadata 4.4 restores compatibility for the reported concerns. I'll
merge those into CPython later.
--
___
Python tracker
<https://bugs.python.org/issue44
Jason R. Coombs added the comment:
There are known performance concerns. I recommend to set those aside for now or
move them to a separate issue because
(a) The performance is theoretically better in the nominal case because it
avoids a sort/group operation.
(b) There are known performance
Jason R. Coombs added the comment:
Thanks Zac for your input.
> Just chiming in with a plea to slow down the rate of changes to
> importlib.metadata - I understand that you want to tidy up the API, but even
> deprecations cause substantial work downstream.
It would be difficult t
Jason R. Coombs added the comment:
> The standard library has clear rules regarding how previously working
> interfaces should be deprecated, and this changeset is violating those. At no
> point was documented that relying on the list/dict trait of the existing
> interface
Jason R. Coombs added the comment:
> there's the tuple subclass which pretends to be a dict.
There's no tuple subclass that's pretending to be a dict. It overrides
__getitem__ for convenience. It never claims to support Mapping.
> mypy complains about incorrect types in overrides fo
Jason R. Coombs added the comment:
New changeset 28f12c9f4f39d283d823d81e311d863526dfdb54 by Jason R. Coombs in
branch 'main':
bpo-44246: Update What's New for importlib.metadata. (#26408)
https://github.com/python/cpython/commit/28f12c9f4f39d283d823d81e311d863526dfdb54
Jason R. Coombs added the comment:
Although it feels the topic has shifted from the original concern (an
unintentional incompatibility) to the broader topic of the API change to
entry_points generally, I'm happy to address your comments:
> - I don't think they were discussed thoroug
Change by Jason R. Coombs :
--
keywords: +patch
pull_requests: +25001
stage: -> patch review
pull_request: https://github.com/python/cpython/pull/26408
___
Python tracker
<https://bugs.python.org/issu
201 - 300 of 2900 matches
Mail list logo
