-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Note removal of dependency_links option #13461
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
Signed-off-by: Stephen Finucane <[email protected]>
Hi, thanks for submitting a PR to pip. However, I don't think we should be updating pip's changelog from over five years ago. The way you've updated this implies this is something that changed in 2019. Further, pip and setuptools are two separate applications, pip does not directly call |
That's my intention. pip 19.0 is the version that dropped the functionality that the option required so it makes sense to change the changelog for that version. While the changelog notes the removal of the CLI option, it doesn't note the impact on configuration files.
Oh, I know. But this is a historical note and that wasn't the case back then. It looks like pipenv also had support for the option (as a passthrough to setuptools) back then, but I don't see any sense in enumerating all package managers since most (all?) of them wrapped setuptools at the time. Again, I'm just trying to provide a small breadcrumb for users, but I'm okay with closing this if it doesn't suit. |
I'm fine with helping people understand what happened with - - Remove the deprecated ``--process-dependency-links`` option. (`#6060 <https://github.com/pypa/pip/issues/6060>`_)
+ - Remove the deprecated ``--process-dependency-links`` option. Affected users should consider using Direct URL dependency specifiers instead (`#6060 <https://github.com/pypa/pip/issues/6060>`_ I would link to some documentation explaining what a direct URL is, but alas, I'm not aware of any good user-friendly resources about direct URL requirements. There is the formal specification, but that's overly technical :( |
I'm inclined not to - from what I recall (I never used dependency links myself) they aren't a complete replacement, so you could easily end up with people getting more confused, rather than helping them. I agree with @notatallshaw here - this happened (in pip) 5 years ago, and I doubt anyone is still using dependency links by now. For people (like OP) interested in the history, the changelog is accurate in terms of what happened in pip. For more general information about the dependency links feature itself, interested researchers should be looking at the change history for the tool (setuptools) that provided it.
I'll add one point of clarification here. Pip (and probably most of the other tools you refer to) didn't "wrap" setuptools, in the sense of exposing setuptools functionality with a different interface. Rather, pip used setuptools to implement some of pip's functionality. The difference is subtle (and I'm almost certainly rewriting history to match how I view things now) but important, as it means that I don't think you should expect pip to document setuptools features (we never did, and since PEP 517 introduced the idea of alternative build backends it no longer even makes sense to do so). Thanks @stephenfin for your interest in improving pip, but I think this is something best left unaltered. |
In that case, I'll close the PR. Thanks for taking the time to file the PR even if we ultimately did not accept it. |
No problem. Thanks for the reviews nonetheless 🙏 |
Just provide a breadcrumb for users (like me) who wondered what happened to this feature.