Use shlex to tokenize the input string. Fixes #3 #6

mikebryant · 2016-05-06T11:02:53Z

Allow the use of quoted arguments, e.g.

Usage:
  test <arg1> <arg2>

with the input a "b c d"

If tokenization fails, fall back to the original behaviour

Allow the use of quoted arguments, e.g. ``` Usage: test <arg1> <arg2> ``` with the input `a "b c d"` If tokenization fails, fall back to the original behaviour

mboersma · 2016-06-09T22:03:17Z

This looks good to me. Anyone else in @docopt/docopt able to review this?

oprypin · 2016-06-09T22:10:46Z

app.py

@@ -27,6 +27,10 @@


 def run_docopt(doc, argv):
+    try:
+        argv = shlex.split(argv)


This works but I think it's best to do the splitting elsewhere, so that run_docopt can accept a list of arguments, as is usual.

Use shlex to tokenize the input string. Fixes docopt#3

2d7704f

Allow the use of quoted arguments, e.g. ``` Usage: test <arg1> <arg2> ``` with the input `a "b c d"` If tokenization fails, fall back to the original behaviour

oprypin reviewed Jun 9, 2016
View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Use shlex to tokenize the input string. Fixes #3 #6

Use shlex to tokenize the input string. Fixes #3 #6

mikebryant commented May 6, 2016

mboersma commented Jun 9, 2016

oprypin Jun 9, 2016

Use shlex to tokenize the input string. Fixes #3 #6

Are you sure you want to change the base?

Use shlex to tokenize the input string. Fixes #3 #6

Conversation

mikebryant commented May 6, 2016

mboersma commented Jun 9, 2016

oprypin Jun 9, 2016

Choose a reason for hiding this comment