-
-
Notifications
You must be signed in to change notification settings - Fork 216
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Document upper limit for setuptools_scm and other upgrading issues #914
Comments
It's tricky to handle that For anything directly using it or maintained public plugins i directly reached out |
Our CI builds just broke with the new release and we can't really do anything about it since the issue occurs in one of our dependencies https://github.com/saltstack/salt/actions/runs/6251062159/job/16971960893?pr=64991#step:11:977 And the failure https://github.com/saltstack/salt/actions/runs/6251062159/job/16971960893?pr=64991#step:11:1285 |
FYI, we are forcefully installing from source(not wheel files) on that job. |
I'll take a look at either disabling the particular plugin or providing a shim I strongly recommend managing build dependents yourself if you have a custom pipeline |
We are. But they are pip installed nonetheless, and if the dependency doesn't pin their build dependencies, then we're stuck at this place. The link to the failure also shows another problem:
|
While I work on updating my packages I've made a PR for setuptools_scm_git_archive that should fix this: Changaco/setuptools_scm_git_archive#22 But that would require making a release for it. |
@djhoese i can provide a shim with a warning |
im scrapping the idea of a shim, but i'll block the plugin from the entrypoints as its in core now |
What does that mean "block the plugin"? Setuptools SCM just won't try to load it? Since we list it in our pyproject.toml as a build requirement this will still be an issue won't it? Or...I guess it won't be imported then so then it can't fail to import the removed |
if the entrypoint is skipped it wont import (it also needs stuff in other modules) |
i'll complete this tommorow moring as i need sleep before i can complete the release |
i'll yank 8.0.1 due the git-arcival issue and release a working and regression tested 8.0.2 tomorow |
I think 8.0.0 is the problem. That's when |
8.0.0 is already yanked
To clarify, you'll both yank, and release tomorrow, correct? Not yank today and release tomorrow? |
I'm experiencing this issue this afternoon when pip installing a project managed with poetry. It's a package written by my collaborator, and I have no idea where in that package setuptools is used if poetry is used already. Anyway, I'm seeing exactly the same error, and I wonder if there's a temporary fix that I can use for now? Thanks! |
@ziyuanzhao2000 Probably not, but hard to tell unless you link to a build log or copy/paste the error you're getting. |
Should I open a new issue for that or paste the link to my error log under this issue? Anyway, here you go: https://pastebin.com/s8M6Q39F Encountered this error when trying to install project using |
Just to be sure, what is your project and it's dependencies? I don't see anywhere in the log which dependencies are being installed when it fails. |
@djhoese project is here and you can see dependencies in the |
Oh great. It's vispy. That's my package. However, it looks like that is coming from napari and they have pinned the version of vispy to an old one (0.12.2). The newest release 0.14.0 fixes the setuptools_scm_git_archive dependency. You'll either need to check out napari's github repository and see if they have a planned fix/update or wait for setuptools_scm (this project) to release a workaround (hopefully tomorrow). |
…egacy-plugins fix #914: ignore the deprecated git archival plugin
I have some projects that don't get a lot of activity (or don't have enough time dedicated to maintenance) and as such were still using
setuptools_scm_git_archive
. The recent 8.0 release removed the.utils
module which was used bysetuptools_scm_git_archive
and is now resulting in:During pip install. Should projects using setuptools_scm add an upper limit to their setuptools_scm dependency to avoid issues like this in the future? Should setuptools_scm_git_archive have a patch release that doesn't include this import (if possible)? I'm actively going through removing the use of the git archive package from all of my packages that still used it, but it will take time and right now none of our users can install our packages.
The text was updated successfully, but these errors were encountered: