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
extra_path kwarg to setup() undocumented #39968
Comments
I can't find documentation for extra_path anywhere.. but this is the extra_path: |
Logged In: YES extra_path also doesn't have a command-line equivalent. |
Unassign, I won't work on this. |
I've attached a documentation patch for this (for py3k) |
The patch has some words missing (“to put between”), but I can add them. I’m also going to include the example that was in Bob’s message, since the bit of doc in the table alone is not enough to understand clearly what this does IMO. I’m reassigning this to distutils2. distutils only gets bug fixes, and its doc is changed only when it says something that doesn’t work. This lets us put energy on distutils2. I’ll get to it this week-end or later. |
I don't understand why distutils won't even get documentation updates. Bob's report is about undocumented functionality that is used in real life and was the best way to install python distributions in a self-contained way before setuptools was invented. AFAIK The "options" argument to setup is also not documented an that one is also used (both py2exe and py2app mention this feature in their documentation). |
This strict freeze policy has been decided by Tarek. The less work there is on distutils’ side, the less synchronization we’ll have to do in distutils2. We do fix doc bugs, we can also fix markup or add links, but that’s it. Improving docs is for distutils2. Note that at first I wanted to improve distutils docs even though the code was frozen, but I now agree with Tarek that it would take up a lot of time that is better spent in distutils2. (Regarding the options parameter, a Montreal-Python contributor added a test for it and a TODO note about lack of documentation. See, things get better one step at a time :) I hope this answers your question without sounding too dictatorial. I understand your viewpoint but ultimately agree with the freeze. |
I've applied the patch on distutils2. This can now be closed. |
You shouldn’t have :) apiref.rst is very outdated in d2, because the setup function does not exist. The docs could be adjusted to document Distribution instead of setup, but we’re not even sure Distribution will stay. I think I will make a compromise here and apply the patch to distutils1 doc, so that the information is found on the official docs. |
Still, the extra_path argument exists and can be used, it's worth to That's also true that apiref.rst is outdated in d2, and will need a |
I added the missing words and reworded a few lines in the attached patch. That said, I find that the new doc is barely helpful; it describes what the code does, but does not explain why one would want to use it. Comments in the source suggest it’s a hack to support some corner case. This is still obscure to me after reading about it and looking at the code :( |
Given the progression in bpo-27919, I suggest this documentation effort can be dropped, but feel free to revive the conversation if you disagree. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: