Skip to content
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

Revert "drop easy_install script and associated documentation" #1909

Merged
merged 2 commits into from
Nov 23, 2019

Conversation

benoit-pierre
Copy link
Member

This reverts commit 6e1838a.

@benoit-pierre
Copy link
Member Author

Also added changelog entries for this and #1830.

@jaraco
Copy link
Member

jaraco commented Nov 23, 2019

Thanks @benoit-pierre for putting this together. I wonder if we can avoid re-adding this functionality by instead moving it to another package (called easy_install), and having setuptools depend on that package. That decoupling would make it obvious what is deprecated and provide a path to backward compatibility (those requiring/expecting this functionality could retain it by installing easy_install separately).

My main reluctance for doing this is that by adding a dependency, that dependency becomes non-optional (pkg_resources will expect the easy_install package to be present and crash if it's not).

Let me put together a draft.

@pganssle
Copy link
Member

@jaraco I think that might overly complicate things. Even in the now-reverted change, all the old functionality was preserved, we just had removed one of the public interfaces. We can just add a new wrapper script that warns that it's deprecated and in say 1 year replace it with one that actually raises an exception, same way we've done for upload and register .

I think there will be very little ongoing support burden to ship a script that just raises an exception, so we should probably just do the lowest-effort thing that gets us to that endpoint without introducing any footguns for end users.

@jaraco
Copy link
Member

jaraco commented Nov 23, 2019

You make a good argument. I think I agree.

I did throw together the easy_install package. All it needs is a tag to cut a release, but I'll refrain from doing that if we're not going to use it.

@jaraco
Copy link
Member

jaraco commented Nov 23, 2019

In #1913, I drafted my concept, but right away it reveals issues (with bootstrapping), at least one of the foot guns. Let's proceed with this approach for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants