← Back to issue list

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

View original Github issue

Metadata

Project
snapcraft
Number
#6123
Type
pull request
State
merged
Author
lengau
Labels
—
Created
Updated
Closed

Current evaluation

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).

Resolution: closed resolved

Reason: The PR was approved and merged into main as commit 22682ed3c, applying the dependency constraint in pyproject.toml. The workaround was later undone by PR #6131 (commit d7872467d, 'build(deps): unpin setuptools-scm') once the upstream setuptools-scm issue no longer affected the build.

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` --- - [ ] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/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 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).
stealth/space-bunny-alpha — closed resolved Merged PR #6123 constrained setuptools-scm to work around pypa/setuptools-scm#1302's cyclic dependency. The temporary pin was later removed by PR #6131.
stealth/space-bunny-alpha — closed resolved Merged #6123 (22682ed3c) temporarily constrained setuptools-scm to avoid an upstream cyclic dependency; #6131 later removed the pin after setuptools-scm 10.0.5 fixed it.
qwen/qwen3.6-35b-a3b — — Merged to constrain setuptools-scm and resolve a cyclic dependency. Approved by two reviewers, passed CI, and applied with a six-line addition across two files.
qwen3.6-35b-a3b-mtp-q6 — — Merged a setuptools-scm version constraint to resolve a cyclic dependency issue. Verified with uv sync and successfully integrated into the project.

Update history

No update history recorded yet.

Related work

  • Superseded By: snapcraft#6131 (confidence 92%)

    Follow-up PR merged as d7872467d removing the setuptools-scm constraint added here, making this pin obsolete.

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

    Upstream cyclic dependency issue this pin was a workaround for.

Related issues

Issue Project State Summary Similarity
#2602 build(deps): constrain setuptools-scm due to cyclic dependency charmcraft merged 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.
92%
#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.
90%
#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.
89%
#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.
87%
#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.
79%
#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%
#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.
75%
#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.
75%
#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.
72%