[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Éric Araujo added the comment: > .. deprecated:: 1.6 This looks weird to me. -- ___ Python tracker ___ ___ Python-bugs-list mailing l

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Andrew Svetlov added the comment: I have applied all patches. Thanks, Chris. -- resolution: -> fixed stage: -> committed/rejected status: open -> closed type: -> enhancement ___ Python tracker __

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Roundup Robot added the comment: New changeset d9ca966cd116 by Andrew Svetlov in branch '2.7': Issue #14616: Document pipes.quote and mention this one in subprocess docs. http://hg.python.org/cpython/rev/d9ca966cd116 -- ___ Python tracker

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Roundup Robot added the comment: New changeset f9d11ca3ccd1 by Andrew Svetlov in branch '3.3': Issue #14616: Mention shlex.quote in subprocess docs. http://hg.python.org/cpython/rev/f9d11ca3ccd1 New changeset 72672cf5d850 by Andrew Svetlov in branch 'default': Merge issue #14616: Mention shlex.q

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Chris Rebert added the comment: Just to keep the ball rolling, in case it ends up being the solution ultimately chosen, here is a patch against 2.7 to document pipes.quote(). The text is yanked straight from shlex.quote(), the differences being: - qualify shlex.split() references - print functi

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Changes by Chris Rebert : Removed file: http://bugs.python.org/file27197/subprocess.rst-3.3.patch ___ Python tracker ___ ___ Python-bugs-list

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Chris Rebert added the comment: Here's a revised patch against the default branch (3.4 I presume?). -- Added file: http://bugs.python.org/file27760/subprocess.rst-default.diff ___ Python tracker ___

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Ezio Melotti added the comment: > I think patch for 3.3 should mention only shlex.quote. +1 I'm not sure what's the best thing to do about pipes.quote. Pointing to an undocumented function that is available in 3.3 in a different module doesn't sound like a really good idea. Maybe we could do

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Andrew Svetlov added the comment: I think patch for 3.3 should mention only shlex.quote. I don't care about 2.7 patch but it looks good to me. -- nosy: +asvetlov ___ Python tracker _

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Chris Rebert added the comment: No, the thought merely did not occur to me. I don't recall having seen such forward-looking notes in Python's documentation before. Should the pipes.quote() mention be axed from the 3.x patch? -- ___ Python tracker <

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Éric Araujo added the comment: You thought it was better not to mention that pipes.quote is semi-official and becomes public as shlex.quote in 3.3? -- ___ Python tracker ___ ___

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Changes by Chris Rebert : Removed file: http://bugs.python.org/file25570/subprocess_shlex_quote.patch ___ Python tracker ___ ___ Python-bugs-l

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Chris Rebert added the comment: Updated patches to mention pipes.quote(). -- Added file: http://bugs.python.org/file27197/subprocess.rst-3.3.patch ___ Python tracker ___

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Changes by Chris Rebert : Added file: http://bugs.python.org/file27196/subprocess.rst-2.7.patch ___ Python tracker ___ ___ Python-bugs-list ma

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Éric Araujo added the comment: Please also take my note about pipes.quote in my OP into account. -- ___ Python tracker ___ ___ Python

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Éric Araujo added the comment: Text LGTM; I haven’t looked at the position in the doc file though. Sandro, once again I’m adding you to nosy in the hope that you’ll have time to make a review and maybe a commit, with my thanks! -- nosy: +sandro.tosi _

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Chris Rebert added the comment: Patch to mention shlex.quote() in the `subprocess` module's "Frequently Used Arguments" Warning box. Could perhaps be a separate Note, but that could be clutter-y. -- keywords: +patch Added file: http://bugs.python.org/file25570/subprocess_shlex_quote.p

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Changes by Chris Rebert : -- nosy: +cvrebert ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

Changes by Tshepang Lekhonkhobe : -- nosy: +tshepang ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mai

[issue14616] subprocess docs should mention pipes.quote/shlex.quote

New submission from Éric Araujo : The warning at http://docs.python.org/library/subprocess#frequently-used-arguments should IMO recommend using shlex.quote. Even if it strongly advises against using shell=True, there are people who need or want to use it, so let’s give them a pointer to the