Skip to content

requirements: Pin setuptools<81 to keep regtool working#100

Merged
DanielKellerM merged 2 commits into
pulp-platform:masterfrom
DanielKellerM:fix/pin-setuptools
May 19, 2026
Merged

requirements: Pin setuptools<81 to keep regtool working#100
DanielKellerM merged 2 commits into
pulp-platform:masterfrom
DanielKellerM:fix/pin-setuptools

Conversation

@DanielKellerM
Copy link
Copy Markdown
Collaborator

The vendored lowRISC regtool (register_interface/vendor/lowrisc_opentitan/util/regtool.py) imports pkg_resources, which was deprecated in setuptools 81.0 and removed in 82.0 (Feb 2026). Fresh installs of requirements.txt now break make idma_hw_all with:

ModuleNotFoundError: No module named 'pkg_resources'

Pin setuptools<81 as a stopgap until #73 (PeakRDL migration) lands or upstream regtool drops pkg_resources.

Resolves #94.

Copilot AI review requested due to automatic review settings May 16, 2026 10:01
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

The vendored lowRISC regtool (register_interface/vendor/lowrisc_opentitan/
util/regtool.py) imports pkg_resources, which was deprecated in setuptools
81.0 and removed in 82.0. Fresh installs now break with:

    ModuleNotFoundError: No module named 'pkg_resources'

Pin setuptools<81 as a stopgap until pulp-platform#73 (PeakRDL migration) lands or the
upstream regtool drops pkg_resources. Resolves pulp-platform#94.
@DanielKellerM DanielKellerM requested review from Copilot and removed request for thommythomaso May 16, 2026 10:04
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot wasn't able to review any files in this pull request.

@DanielKellerM DanielKellerM merged commit 62002b9 into pulp-platform:master May 19, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error while generating the frontend with setuptools >= 81.0.0

2 participants