Skip to content

requirements: Tighten setuptools pin to <79#105

Merged
DanielKellerM merged 1 commit into
pulp-platform:develfrom
DanielKellerM:fix/setuptools-tighter
May 19, 2026
Merged

requirements: Tighten setuptools pin to <79#105
DanielKellerM merged 1 commit into
pulp-platform:develfrom
DanielKellerM:fix/setuptools-tighter

Conversation

@DanielKellerM
Copy link
Copy Markdown
Collaborator

Description

Follow-up to #100. The setuptools<81 pin we just landed is directionally correct but too loose: pip resolves to setuptools 79.0.1 (already in /home/gitlabci/.local/ on dolent runners) which still has the broken vendored jaraco/context.py:

# line 18 of setuptools 79.0.1's vendored jaraco/context.py:
from backports import tarfile

Anaconda's stub backports/__init__.py doesn't expose tarfile, so the import fails. Same failure as before the <81 pin.

Fix

Tighten the pin to setuptools<79. This forces pip to install setuptools 78.x, whose vendored jaraco doesn't have the backports.tarfile import.

Verification

Locally reproduced the failure on anaconda Python 3.11 and confirmed the import chain works with setuptools<79:

setuptools vendored jaraco/context.py has backports.tarfile import?
79.0.1 yes (broken)
78.0.0 no (works)
77.0.0 no (works)

Test plan

  • CI on master after merge: gitlab-ci/check should go green this time
  • If still failing: investigate whether dolent runner's /home/gitlabci/.local has additional cached envs

Targets devel per branch policy.

The previous <81 pin was directionally correct but too loose. pip
resolves to setuptools 79.0.1 which still has the broken vendored
jaraco/context.py:18 'from backports import tarfile' against anaconda's
stub backports namespace on the dolent GitLab CI runners.

setuptools 78.x is the latest version whose vendored jaraco doesn't
include the backports.tarfile import. Verified locally: pip
install 'setuptools<79' resolves to 78.x and the regtool import chain
works.
Copilot AI review requested due to automatic review settings May 19, 2026 14:32
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.

@DanielKellerM DanielKellerM merged commit 3c0cc3f into pulp-platform:devel May 19, 2026
6 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.

2 participants