bpo-25567: Add the support of bytes in quotes.

Doc/library/shlex.rst

@@ -39,9 +39,12 @@ The :mod:`shlex` module defines the following functions:
 
 .. function:: quote(s)
 
-   Return a shell-escaped version of the string *s*.  The returned value is a
-   string that can safely be used as one token in a shell command line, for
-   cases where you cannot use a list.
+   Return a shell-escaped version of the string *s* or the bytes object.  The
+   returned value is a string literal or bytes  that can safely be used as one
+   token in a shell command line, for cases where you cannot use a list.
+
+   .. versionchanged:: 3.8
+       The *s* parameter can be a bytes object.
 
    This idiom would be unsafe:
 

Lib/shlex.py

@@ -306,17 +306,23 @@ def split(s, comments=False, posix=True):
 
 
 _find_unsafe = re.compile(r'[^\w@%+=:,./-]', re.ASCII).search
+_find_unsafe_bytes = re.compile(b'[^\w@%+=:,./-]').search
 
 def quote(s):
     """Return a shell-escaped version of the string *s*."""
     if not s:
         return "''"
-    if _find_unsafe(s) is None:
-        return s
 
     # use single quotes, and put single quotes into double quotes
     # the string $'b is then quoted as '$'"'"'b'
-    return "'" + s.replace("'", "'\"'\"'") + "'"
+    if isinstance(s, bytes):
+        if _find_unsafe_bytes(s) is None:
+            return s
+        return b"'" + s.replace(b"'", b"'\"'\"'") + b"'"
+    else:
+        if _find_unsafe(s) is None:
+            return s
+        return "'" + s.replace("'", "'\"'\"'") + "'"
 
 
 def _print_tokens(lexer):

Lib/test/test_shlex.py

@@ -304,9 +304,14 @@ def testQuote(self):
         for u in unsafe:
             self.assertEqual(shlex.quote('test%sname' % u),
                              "'test%sname'" % u)
+            self.assertEqual(shlex.quote(b'test%sname' % u.encode('utf-8')),
+                             b"'test%sname'" % u.encode('utf-8'))
         for u in unsafe:
             self.assertEqual(shlex.quote("test%s'name'" % u),
                              "'test%s'\"'\"'name'\"'\"''" % u)
+            self.assertEqual(shlex.quote(b"test%s'name'" % u.encode('utf-8')),
+                             b"'test%s'\"'\"'name'\"'\"''" % u.encode('utf-8'))
+
 
 # Allow this test to be used with old shlex.py
 if not getattr(shlex, "split", None):

Misc/NEWS.d/next/Library/2018-12-04-16-15-06.bpo-25567.5krN3D.rst

@@ -0,0 +1 @@
+Add the bytes support the the :func:`shlex.quote` function.