-
-
Notifications
You must be signed in to change notification settings - Fork 27
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
setuptools v66.0.0 #299
setuptools v66.0.0 #299
Conversation
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's possible that this may give some issues because of the removal of non PEP 440 compliant version numbers (see https://setuptools.pypa.io/en/stable/history.html#v66-0-0), but after a day there's no major complaints visible on the setuptools issue tracker yet. So let's give it a go.
@rgommers FWIW this did break a bunch of things downstream in conda-forge, @jaimergp and @jakirkham know the details, triggered by a conda-build issue of relying on Incidentally, a new conda release has been tagged this week, where this caught my eye. This makes this PR a higher risk change, but probably something you weren't aware of. So this is an FYI that while you were looking upstream in the packaging issue tracker, it makes sense to also look downstream if people are ready for it :) |
Thanks for flagging this @jezdez.
I'm not sure how that could have been done to be honest. Maybe we can have a policy on this repo that we should wait a week, or 2 weeks, before merging? That's the only thing I can think of.
I wasn't indeed. But there's many packages with lots of dependencies that use |
Honestly am not sure what Ralf should do additionally. Periodically In terms of the specific issue that occurred with Part of the reason we've developed the infrastructure we have (particularly around Getting to the point that I think you are making, how do we handle this going forward. We could...
Downstream testing could work, but it seems a bit too much. Users of conda-forge packages should be able to get the latest Getting more involved upstream could make sense. Though I think the Python world is moving away from everything depending on Really think we should move towards dropping |
That seems like a reasonable policy, not sure how easy it is to follow though.
Something something iceberg below the waterline. I'm glad we're working to move setuptools out of the dependency tree for conda-build. |
Yeah waiting 2 weeks is tricky. Honestly the problem the maintainers have there is unenviable. They are carrying a lot of technical debt that they are trying to figure out how to shed. |
Agreed, the removal was quite okay, as @jaimergp mentioned elsewhere, we should probably treat DeprecationWarnings as errors in conda and conda-build to earlier force us to move along.
That's awesome, and I appreciate that, and yet, I know with my Anaconda hat on, the way this is handled is somewhat different there, less automated, more people focused, which has its upsides and downsides.
Having spent my fair share with setuptools in the past I'm ready to call it what @rgommers already alluded to: if we can find ways for the conda ecosystem to rely on the newer, nimbler "pieces of a puzzle" libraries from PyPA and their related spec work, the better. conda/conda#12245 comes to mind as an interesting recent example, which may have lasting repercussions for the conda ecosystem.
Thanks for spelling this out very constructively, this is hugely appreciated, I couldn't agree more. |
+1 to all this. Let me make a concrete suggestion. Recently there have been discussions about bootstrapping, after various other pieces of the packaging puzzle broke because of switching backends and circular dependencies. It is now clear that there is one package, For packages with compiled code (are there any?), best to hold off moving for a little while longer. |
It is very likely that the current package version for this feedstock is out of date.
Checklist before merging this PR:
license_file
is packagedInformation about this PR:
@conda-forge-admin,
please add bot automerge
in the title and merge the resulting PR. This command will add our bot automerge feature to your feedstock.bot-rerun
label to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase@conda-forge-admin, please rerun bot
in a PR comment to have theconda-forge-admin
add it for you.Closes: #298
Dependency Analysis
Please note that this analysis is highly experimental. The aim here is to make maintenance easier by inspecting the package's dependencies. Importantly this analysis does not support optional dependencies, please double check those before making changes. If you do not want hinting of this kind ever please add
bot: inspection: false
to yourconda-forge.yml
. If you encounter issues with this feature please ping the bot teamconda-forge/bot
.Analysis by source code inspection shows a discrepancy between it and the the package's stated requirements in the meta.yaml.
Packages found by source code inspection but not in the meta.yaml:
This PR was created by the regro-cf-autotick-bot. The regro-cf-autotick-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by https://github.com/regro/autotick-bot/actions/runs/3927401495, please use this URL for debugging.