[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Éric Araujo mer...@netwok.org added the comment: The module docstring (which duplicates the reST docs) still mentions list2cmdline. I’d vote for removal there too. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Changes by Eli Bendersky eli...@gmail.com: -- status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Roundup Robot devnull@devnull added the comment: New changeset e7a55e236b8b by Eli Bendersky in branch '3.1': Issue #11827: remove mention of list2cmdline in the doc of subprocess http://hg.python.org/cpython/rev/e7a55e236b8b -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Roundup Robot devnull@devnull added the comment: New changeset f38489a3394f by Eli Bendersky in branch '2.7': Issue #11827: remove mention of list2cmdline in the doc of subprocess http://hg.python.org/cpython/rev/f38489a3394f -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Changes by Eli Bendersky eli...@gmail.com: -- status: open - pending ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Eli Bendersky eli...@gmail.com added the comment: Patch committed to 3.3, 3.2, 3.1, 2.7 In case no objections arise, I will close this issue in a couple of days -- status: pending - open ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Éric Araujo mer...@netwok.org added the comment: LGTM. -- versions: +Python 2.7, Python 3.1, Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Ezio Melotti ezio.melo...@gmail.com added the comment: If a _ is added to list2cmdline the old name should be kept and deprecated. The function is also on 2.x and it's not deprecated there (and it's probably too late to deprecate it now), so we might have to wait a few versions before it will be possible to remove list2cmdline from 3.x. -- nosy: +ezio.melotti ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Changes by Éric Araujo mer...@netwok.org: -- nosy: +eric.araujo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Eli Bendersky eli...@gmail.com added the comment: Attaching a proposed patch for 3.2, focusing only on the documentation for the time being (I realize that deprecation is a loaded issue and should be probably handled in a centralized manner). The patch removes mention of list2cmdline, instead explaining its intention (using the docstring of list2cmdline quite literally). I realize this isn't an ideal approach, but IMHO it's much better than what we have now. Suggestions are most welcome. -- keywords: +patch Added file: http://bugs.python.org/file21634/issue11827.1.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
New submission from Eli Bendersky eli...@gmail.com: The documentation of subprocess.Popen mentions a function named list2cmdline(): On Windows: the Popen class uses CreateProcess() to execute the child program, which operates on strings. If args is a sequence, it will be converted to a string using the list2cmdline() method. Please note that not all MS Windows applications interpret the command line the same way: list2cmdline() is designed for applications using the same rules as the MS C runtime. I find this rather opaque and useless. list2cmdline() in subprocess is publicly accessible (doesn't begin with underscores) but it isn't documented. The solution can be one of the following: 1. Document list2cmdline in the docs of subprocess, making the reference to is useful 2. Don't mention list2cmdline and instead mention what it does with the command-line list -- assignee: docs@python components: Documentation messages: 133507 nosy: docs@python, eli.bendersky priority: normal severity: normal stage: needs patch status: open title: mention of list2cmdline() in docs of subprocess.Popen type: behavior versions: Python 3.2 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
R. David Murray rdmur...@bitdance.com added the comment: I vote for (2) (I presume 'it' in that sentence is 'subprocess'). list2cmdline shouldn't be a real public method, at least not without the issues surrounding being given careful design attention. -- nosy: +r.david.murray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Eli Bendersky eli...@gmail.com added the comment: I also prefer (2) since I see no reason for the user to use list2cmdline() directly, let alone from subprocess (had there been rationale for such a public function it should probably be in another module). As for 'it', I guess you can say it means 'subprocess' or 'list2cmdline', doesn't matter which. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
R. David Murray rdmur...@bitdance.com added the comment: Ah, right. I guess I was advocating that the docs be written from the perspective that list2cmdline doesn't exist as an identifiable entity. From the POV of the updated docs, it is just subprocess's behavior, and list2cmdline is an implementation detail. Which is what you were saying. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Ross Lagerwall rosslagerw...@gmail.com added the comment: #10838 has a bit of discussion about list2cmdline and being part of the public api (generally agreeing that it should be made private, I think). -- nosy: +rosslagerwall ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue11827] mention of list2cmdline() in docs of subprocess.Popen
Eli Bendersky eli...@gmail.com added the comment: Thanks for the pointer, Ross. So I propose to remove the mention of list2cmdline from the documentation of subprocess, explaining instead what it does to the path (since I think this should be publicly known, otherwise it's just black magic). I will prepare a patch for this and post it for review here before committing. Do you folks think we should also prepend an underscore to list2cmdline, thus settling the doubt about its public-ness once and for all? Other private functions in subprocess do have underscores prepended. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue11827 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
