Mailman 3 June 2015 - docs

Re: [docs] Incorrect (misleading) statement in the execution model documentation (issue 24129)
by levkivskyi＠gmail.com June 21, 2015

June 21, 2015

Reviewers: eric.snow, Nick Coghlan, http://bugs.python.org/review/24129/diff/14851/Doc/reference/executionmodel… File Doc/reference/executionmodel.rst (right): http://bugs.python.org/review/24129/diff/14851/Doc/reference/executionmodel… Doc/reference/executionmodel.rst:148: Free variables follow the normal rules for name resolution, On 2015/06/19 21:57:02, eric.snow wrote: > It may be more clear to say: > > These references follow the normal rules for name resolution with one … [View More]

1 0

Re: [docs] Incorrect (misleading) statement in the execution model documentation (issue 24129)
by ncoghlan＠gmail.com June 20, 2015

June 20, 2015

This looks like a solid improvement to me, just a couple of minor tweaks suggested. http://bugs.python.org/review/24129/diff/15117/Doc/reference/executionmodel… File Doc/reference/executionmodel.rst (right): http://bugs.python.org/review/24129/diff/15117/Doc/reference/executionmodel… Doc/reference/executionmodel.rst:108: name refers to a local variable that has not been bound, an For the second sentence, I'd suggest: "If the current scope is a function scope, and the name refers to a local … [View More]

1 0

[issue13213] generator.throw() behavior
by Petri Lehtinen June 20, 2015

June 20, 2015

New submission from Petri Lehtinen <petri(a)digip.org>: The documentation of generator.throw() gives this signature: generator.throw(type[, value[, traceback]]) Looking at the code, it accepts the following arguments: g.throw(ExcType) --> raise ExcType() g.throw(ExcType, None) --> raise ExcType() g.throw(ExcType, None, tb) --> raise ExcType().with_traceback(tb) g.throw(ExcType, instace_of_ExcType) --> raise instance_of_ExcType g.throw(ExcType, … [View More]

4 3

[issue20344] subprocess.check_output() docs misrepresent what shell=True does
by Tobias Klausmann June 20, 2015

June 20, 2015

New submission from Tobias Klausmann: The subprocess docs state that the first argument can be either a string or an iterable that contains the program and arguments to run. It also points out that using shell=True allows for shell constructs. It does not mention a subtlety that is introduced by Python's use of sh -c (on Unix): Using a string that contains the full command line with args and shell=False breaks as expected: >>> subprocess.check_output("ls foo", shell=False) Traceback … [View More](most recent call last): File "<stdin>", line 1, in <module> File "/usr/lib64/python3.3/subprocess.py", line 576, in check_output with Popen(*popenargs, stdout=PIPE, **kwargs) as process: File "/usr/lib64/python3.3/subprocess.py", line 824, in __init__ restore_signals, start_new_session) File "/usr/lib64/python3.3/subprocess.py", line 1448, in _execute_child raise child_exception_type(errno_num, err_msg) FileNotFoundError: [Errno 2] No such file or directory: 'ls foo' Using shell=True instead works as expected (since foo does not exist, ls exits with non-0 and a different Exception is raised. This, too is to spec and exactly what the docs describe. >>> subprocess.check_output("ls foo", shell=True) ls: cannot access foo: No such file or directory Traceback (most recent call last): File "<stdin>", line 1, in <module> File "/usr/lib64/python3.3/subprocess.py", line 589, in check_output raise CalledProcessError(retcode, process.args, output=output) subprocess.CalledProcessError: Command 'ls foo' returned non-zero exit status 2 Using shell=False and making the command a list of command and args does the same thing: >>> subprocess.check_output(["ls", "foo"], shell=False) ls: cannot access foo: No such file or directory Traceback (most recent call last): File "<stdin>", line 1, in <module> File "/usr/lib64/python3.3/subprocess.py", line 589, in check_output raise CalledProcessError(retcode, process.args, output=output) subprocess.CalledProcessError: Command '['ls', 'foo']' returned non-zero exit status 2 But using an iterable _and_ requesting a shell results in something very broken: >>> subprocess.check_output(["ls", "foo"], shell=True) [contents of my homedir are returned] >>> Note that the argument "foo" completely disappears, apparently. strace() reveals that "foo" is never added to the call to "ls". Instead, it becomes $0 in the shell that ls runs in. This is exactly to spec (i.e. bash is not broken). "bash -c foo bar baz" will start a shell that sets $0 to "bar", $1 to "baz" and then executes "foo". Whereas "bash -c 'foo bar baz'" will run 'foo bar baz'. I think there are three possible fixes here: 1- Make check_output(list, shell=True) run something like "bash -c '%s'" % " ".join(list) 2- Change the docs to warn of the unintended consequences of combining lists with shell=True 3- Make check_output() throw an Exception if the first argument is a list and shell=True The first option would make it easiest for people to "just use it", but I fear the string manipulation may become the source of bugs with security implications in the future. The second option keeps the status quo, but people tend to not read docs, so it may not be as effective as one would like, especially since shell-True already has warnings and people tend to go "yeah I know about unverified input, but it's not a problem for me" and then ignore both warnings. Option 3 has the disadvantage that existing scripts that _expect_ the behavior may break. ---------- assignee: docs@python components: Documentation, Library (Lib) messages: 208811 nosy: docs@python, klausman priority: normal severity: normal status: open title: subprocess.check_output() docs misrepresent what shell=True does type: behavior versions: Python 3.3 _______________________________________ Python tracker <report(a)bugs.python.org> <http://bugs.python.org/issue20344> _______________________________________ [View Less]

4 11

[issue24460] urlencode() of dictionary not as expected
by Martin Panter June 20, 2015

June 20, 2015

Martin Panter added the comment: 1. urlencode(): I agree the documentation is unclear. But David Rueter’s suggestion does not help much. I think doseq=True is meant to also work for a mapping query (as in original post), and is not required in the sequence-of-tuples mode if each tuple has a single parameter value. Perhaps something like this could work instead: “When a sequence of two-element tuples is used as *query*, the first element of each tuple is a key and the second specifies one or … [View More]

1 0

Incorrect (misleading) statement in the execution model documentation (issue 24129)
by ericsnowcurrently＠gmail.com June 19, 2015

June 19, 2015

http://bugs.python.org/review/24129/diff/14851/Doc/reference/executionmodel… File Doc/reference/executionmodel.rst (right): http://bugs.python.org/review/24129/diff/14851/Doc/reference/executionmodel… Doc/reference/executionmodel.rst:148: Free variables follow the normal rules for name resolution, It may be more clear to say: These references follow the normal rules for name resolution with one exception. Unbound local variables are looked up in the global namespace. ... http://bugs.python.… [View More]

1 0

[issue24466] extend_path explanation in documentation is ambiguous
by alanf June 18, 2015

June 18, 2015

New submission from alanf: The description of pkgutil.extend_path in the doc (e.g., https://docs.python.org/2/library/pkgutil.html , https://docs.python.org/3/library/pkgutil.html ) is so ambiguous that I had to run a test to understand its behavior. The doc says: This will add to the package’s __path__ all subdirectories of directories on sys.path named after the package. It wasn't clear to me how this should be parsed. Using parentheses to group the clauses, there's this possibility : (1)… [View More]

1 0

make_server host parameter in underdocumented
by anatoly techtonik June 18, 2015

June 18, 2015

Hi, In https://docs.python.org/2/library/wsgiref.html#wsgiref.simple_server.make_s… example mentions host='', but doesn't clarify if that will make server listen on localhost or on all interfaces. -- anatoly t.

1 0

[issue23020] New matmul operator crashes modules compiled with CPython3.4
by Tal Einat June 18, 2015

June 18, 2015

Changes by Tal Einat <taleinat(a)gmail.com>: ---------- assignee: -> docs@python components: +Documentation nosy: +docs@python _______________________________________ Python tracker <report(a)bugs.python.org> <http://bugs.python.org/issue23020> _______________________________________

1 0