-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Disable easy_install command and remove sandbox functionality. #2908
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
Conversation
e51d238
to
758589a
Compare
758589a
to
8d12d61
Compare
383ed7a
to
9116c7e
Compare
a85759c
to
93ce5a0
Compare
…e built. This expectation is discouraged if not deprecated and shouldn't be captured here.
f379532
to
791cdbb
Compare
791cdbb
to
1db0b12
Compare
I've filed pbr-2107732 to track an integration issue with the proposed work. |
Co-authored-by: Avasam <[email protected]>
1db0b12
to
54b9f78
Compare
976438b
to
16ebbf6
Compare
@jaraco this PR's title didn't set off alarm bells for me, but I glanced at the linked
Disabling Are you trying to remove both |
I appreciate the concern. Thanks for reaching out. I'm trying mainly to remove I was thinking that because |
16ebbf6
to
e5f80a1
Compare
In #4955, I'm considering a different tack for decoupling develop from easy_install. I've also created debt/remove-develop to track removing the develop command separately from the effort of disabling easy_install. |
e5f80a1
to
6370b0a
Compare
Tests still pass :D
The problem with that deprecation of all of direct It's possible to say "pin to and older version" and at some point that's going to be needed. But it'll still be disruptive. What I would recommend is decouple |
Thanks for this feedback. I agree that's a good plan and promises to be a lot less disruptive. I've continued to pursue #4955, which is reaching stability. I plan to merge that and this change at the same time as this change, even though they could be rolled back independently. |
Disable easy_install command and remove sandbox functionality.
Thanks @jaraco! I will follow up with PyTorch and check in other projects I'm familiar with for remaining |
This change broke |
Ref #917