Bug 260997
| Summary: | AutoInstall.install_everything is unsafe | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Sam Sneddon [:gsnedders] <gsnedders> |
| Component: | Tools / Tests | Assignee: | Sam Sneddon [:gsnedders] <gsnedders> |
| Status: | RESOLVED FIXED | ||
| Severity: | Normal | CC: | jbedard, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Bug Depends on: | |||
| Bug Blocks: | 261004 | ||
Sam Sneddon [:gsnedders]
INTERNALERROR> File "/Volumes/gsnedders/projects/Safari/OpenSource/Tools/Scripts/libraries/resultsdbpy/.tox/py39/lib/python3.9/site-packages/webkitcorepy/autoinstall.py", line 642, in install_everything
INTERNALERROR> for packages in cls.packages.values():
INTERNALERROR> RuntimeError: dictionary changed size during iteration
I believe this is caused by:
AutoInstall.register(Package('twisted', Version(21, 2, 0), pypi_name='Twisted', implicit_deps=[
Package('incremental', Version(21, 3, 0)),
]))
…where the AutoInstaller will AutoInstall twisted, and then recurse into its implicit_deps, and it will then register incremental as it begins to install it—thus mutating the dict being iterated over.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Sam Sneddon [:gsnedders]
Pull request: https://github.com/WebKit/WebKit/pull/17317
EWS
Committed 267552@main (d94691e07a31): <https://commits.webkit.org/267552@main>
Reviewed commits have been landed. Closing PR #17317 and removing active labels.
Radar WebKit Bug Importer
<rdar://problem/114822346>