← Back to issue list

build(deps): constrain setuptools-scm due to cyclic dependency

View original Github issue

Metadata

Project
charmcraft
Number
#2602
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

Merged PR #2602 (commit f9c9ad06) adding a temporary setuptools-scm version constraint to work around the cyclic dependency in pypa/setuptools-scm#1302. The pin was later removed by #2615.

Resolution: closed resolved

Reason: The PR was approved by 2 reviewers with0 unresolved comments and merged to main as commit f9c9ad06 ('build(deps): constrain setuptools-scm due to cyclic dependency'), touching pyproject.toml and uv.lock to pin setuptools-scm around the upstream pypa/setuptools-scm#1302 build failure.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Workaround for https://github.com/pypa/setuptools-scm/issues/1302 Snap build not tested yet, but it works with `UV_PYTHON=3.12 UV_PYTHON_DOWNLOADS=never UV_NO_BINARY=1 uv sync --frozen --no-dev --no-editable --no-cache` Describe your changes. --- - [ ] I've followed the [contribution guidelines](https://github.com/canonical/charmcraft/blob/main/CONTRIBUTING.md). - [ ] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [ ] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #2602 (commit f9c9ad06) adding a temporary setuptools-scm version constraint to work around the cyclic dependency in pypa/setuptools-scm#1302. The pin was later removed by #2615.
stealth/space-bunny-alpha — closed resolved Merged in #2602 (f9c9ad06), adding a temporary setuptools-scm constraint for upstream #1302; #2615 later removed it after the cyclic-dependency bug was fixed.
qwen/qwen3.6-35b-a3b — — Merged to constrain setuptools-scm and resolve a cyclic dependency. Approved by two reviewers, it implements a workaround for upstream issue #1302 by pinning the package version. CI checks passed before integration.
qwen3.6-35b-a3b-mtp-q6 — — Merged a constraint for setuptools-scm to resolve a cyclic dependency issue. The workaround was successfully integrated despite non-blocking type-checking notifications.

Update history

No update history recorded yet.

Related work

  • Related To: setuptools-scm#1302 (confidence 95%)

    Upstream issue cited in the PR body as the cause of the cyclic dependency failure being worked around.

  • Superseded By: canonical/charmcraft#2615 (confidence 90%)

    Follow-up merged PR #2615 (commit e78c495b, 'build(deps): unpin setuptools-scm') removed the temporary constraint once the upstream cyclic-dependency bug was fixed; pyproject.toml at HEAD no longer contains setuptools-scm.

  • Related To: canonical/snapcraft#6123 (confidence 70%)

    Identical workaround applied in snapcraft for the same upstream bug; later reverted there by snapcraft#6131.

Related issues

Issue Project State Summary Similarity
#306 build(deps): constrain setuptools-scm due to cyclic dependency imagecraft merged Merged as fca3ff4, adding a temporary setuptools-scm constraint to work around pypa/setuptools-scm#1302's cyclic-dependency bug; #309 later removed the pin after the upstream fix.
94%
#1158 build(deps): constrain setuptools-scm due to cyclic dependency rockcraft merged Merged PR #1158 as commit 01ecfcd3, adding a setuptools-scm constraint to work around pypa/setuptools-scm#1302 and unblock dependency synchronization.
93%
#6123 build(deps): constrain setuptools-scm due to cyclic dependency snapcraft merged PR #6123 by lengau merged as commit 22682ed3c, adding a temporary setuptools-scm pin to work around pypa/setuptools-scm#1302 cyclic dependency. Later reverted by #6131 (d7872467d).
92%
#105 build(deps): constrain setuptools-scm due to cyclic dependency debcraft merged Merged in commit 30bdea8, temporarily constraining setuptools-scm to work around pypa/setuptools-scm#1302; debcraft#110 later removed the constraint.
90%
#1157 build: constrain setuptools-scm rockcraft merged Merged PR #1157 (3e5b3347) constrained setuptools-scm below 10.0.2, preventing the upstream build hang seen when building Pydantic.
80%
#309 build(deps): unpin setuptools-scm imagecraft merged Merged #309 in commit b1e78bf, removing the setuptools-scm pin after the upstream cyclic-dependency bug was fixed.
80%
#6130 build: constrain setuptools-scm<10.0.2 snapcraft closed Closed unmerged. The <10.0.2 constraint was made moot when setuptools-scm 10.0.5+ fixed pypa/setuptools-scm#1302 (#1330); PR #6131 (commit d7872467d) then removed the earlier pin from #6123 entirely.
78%
#1164 build(deps): unpin setuptools-scm rockcraft closed PR #1164 proposed removing the setuptools-scm workaround after the upstream bug was fixed, but closed unmerged with review pending and no recorded resolution; the constraint remains.
77%
#6131 build(deps): unpin setuptools-scm snapcraft merged Merged #6131 in commit d7872467d, removing the setuptools-scm pin after the upstream fix landed in setuptools-scm 10.0.5.
76%
#2615 build(deps): unpin setuptools-scm charmcraft merged Merged in canonical/charmcraft#2615 at e78c495b, removing the temporary setuptools-scm pin after the upstream cyclic-dependency bug was fixed.
75%