[compat] Work around kwargs bugs in old 2.6 Python releases (Fixes #3813)
authorPhilipp Hagemeister <phihag@phihag.de>
Sat, 15 Nov 2014 14:17:19 +0000 (15:17 +0100)
committerPhilipp Hagemeister <phihag@phihag.de>
Sat, 15 Nov 2014 14:17:19 +0000 (15:17 +0100)
youtube_dl/compat.py
youtube_dl/options.py

index b3752634a08a5982e52353be0ba272b6aba1f202..385924803f0fed16e62227a14c717486068690b2 100644 (file)
@@ -288,6 +288,14 @@ if sys.version_info < (3, 0) and sys.platform == 'win32':
 else:
     compat_getpass = getpass.getpass
 
+# Old 2.6 and 2.7 releases require kwargs to be bytes
+try:
+    (lambda x: x)(**{'x': 0})
+except TypeError:
+    def compat_kwargs(kwargs):
+        return dict((bytes(k), v) for k, v in kwargs.items())
+else:
+    compat_kwargs = lambda kwargs: kwargs
 
 __all__ = [
     'compat_HTTPError',
@@ -299,6 +307,7 @@ __all__ = [
     'compat_html_entities',
     'compat_html_parser',
     'compat_http_client',
+    'compat_kwargs',
     'compat_ord',
     'compat_parse_qs',
     'compat_print',
index c182abfdc8f211b7c7520547926ba09b40bf1071..16aa6ffca1c8f1e3199d0b2d570307ec5cf7be91 100644 (file)
@@ -8,6 +8,7 @@ import sys
 from .compat import (
     compat_expanduser,
     compat_getenv,
+    compat_kwargs,
 )
 from .utils import (
     get_term_width,
@@ -112,7 +113,7 @@ def parseOpts(overrideArguments=None):
         'conflict_handler': 'resolve',
     }
 
-    parser = optparse.OptionParser(**kw)
+    parser = optparse.OptionParser(**compat_kwargs(kw))
 
     general = optparse.OptionGroup(parser, 'General Options')
     general.add_option(