-
-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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
DEPS: clarifying setuptools's status as a dependency #41815
Comments
setuptools is needed and we should list it |
Setuptools (
Somewhat a separate issue from listing setuptools as dependency or not, but I think we should indeed look into replacing |
maybe open a new issue and we can close this one now that #41818 is merged. |
setup tools is an undeclared dependency now it should be an official runtime one |
There is a difference between "optional" and "required" dependencies. We have lots of optional dependencies that are not declared in It's true that we don't provide a proper error message about "pkg_resources" (setuptools) being required for using a plotting backend (as we do for other optional dependencies), but give a plain import error instead. This certainly gives the impression we assume the library is always present, but giving a more informative import error message is also something we can easily fix. |
removing the blocker label since #42006 fixed the import issue. |
is that now the only outstanding actionable item here? moving to 1.3.1 |
actually i think prob ok for 1.3.x to close this (as #42410) is going to remove the need for this in 3.8 |
we merged the update for 1.4 to remove the need for setuptools entirely, so not merging this to 1.3.x unless strong desire (as efffectively by not merging we are not changing 1.3 vs 1.2 at all w.r.t. requirements). |
Sounds good! |
In
install.rst
, we listsetuptools
as a runtime dependency. If that is the case, we don't need to vendorpackaging
#41199 since it's vendored bypkg_resources
, which is part ofsetuptools
. And we need to updatesetup.cfg
andREADME.md
to reflect this.If
setuptools
is not a runtime dependency, vendoringpackaging
is necessary. And, we need to list it as a test and optional dependency because we are usingpkg_resources
inpandas/plotting/_core.py
andpandas/tests/plotting/test_backend.py
#41503. An alternative ofpkg_resources
for accessing entry points isimportlib.metadata
, which is only available after Python 3.8. Andimportlib.metadata
is the suggested way to do this according to https://setuptools.readthedocs.io/en/latest/pkg_resources.html.The text was updated successfully, but these errors were encountered: