You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
36 lines
1.4 KiB
36 lines
1.4 KiB
From 36bcb170ba9097885902513640075eac2e6ce384 Mon Sep 17 00:00:00 2001
|
|
From: Calin Culianu <calin.culianu@gmail.com>
|
|
Date: Mon, 8 Nov 2021 18:15:59 -0600
|
|
Subject: [PATCH] Fixup for Python 3.10
|
|
|
|
Closes issue #109. Long story short: a few names from collection are
|
|
now moved to collection.abc exclusively starting in Python 3.10. The
|
|
only name this app uses from there that was moved is
|
|
`collections.Iterable`. Python versions starting from 3.3 support both
|
|
`collections.Iterable` and `collections.abc.Iterable` as the way to refer to
|
|
this class, which Python 3.10 being the first one to drop
|
|
`collections.Iterable`. So.. we just work around this API quirk
|
|
and always refer ot it as `collections.abc.Iterable`.
|
|
---
|
|
stem/control.py | 3 ++-
|
|
1 file changed, 2 insertions(+), 1 deletion(-)
|
|
|
|
--- a/stem/control.py
|
|
+++ b/stem/control.py
|
|
@@ -249,6 +249,7 @@ If you're fine with allowing your script
|
|
|
|
import calendar
|
|
import collections
|
|
+import collections.abc
|
|
import functools
|
|
import inspect
|
|
import io
|
|
@@ -2532,7 +2533,7 @@ class Controller(BaseController):
|
|
for param, value in params:
|
|
if isinstance(value, str):
|
|
query_comp.append('%s="%s"' % (param, value.strip()))
|
|
- elif isinstance(value, collections.Iterable):
|
|
+ elif isinstance(value, collections.abc.Iterable):
|
|
query_comp.extend(['%s="%s"' % (param, val.strip()) for val in value])
|
|
elif not value:
|
|
query_comp.append(param)
|