-
-
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
DEP: move pkg_resources back inline in function (delayed import) #42006
DEP: move pkg_resources back inline in function (delayed import) #42006
Conversation
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.
while i understand this 'fixes' things it is misleading. we should just say setuptools is an actual dependency, which it is .
It is not. It has never been, until #41503 accidentally made it so |
nope it was always a dep just u declared -1 on this PE |
Of course it was always a "dependency", but there is a difference between "optional" and "required" dependencies. We have lots of optional dependencies that are not declared in |
Apart from the "required dependency or not" discussion (-> let's have that in #41815, since this PR doesn't actually address that), there is also an "import time" reason for merging this PR. Importing
Tested with
|
ok the delayed import is fine (as understand the total import issue), however we still need to declare this dependency. |
fine. can we also pls add to our build deps. |
@meeseeksdev backport 1.3.x |
…function (delayed import)
Something went wrong ... Please have a look at my logs. |
…delayed import) (#42037) Co-authored-by: Joris Van den Bossche <jorisvandenbossche@gmail.com>
xref #40169 (comment), fixup of #41503