Fix bug introduced by the previous patch by changing TimeRE.__seqToRe() to

brettcannon · brettcannon · commit ffa5cf9eae5f · 2004-10-06T22:48:58.000Z
accept any iterable instead of only a sliceable object.
diff --git a/Lib/_strptime.py b/Lib/_strptime.py
@@ -226,13 +226,12 @@ def __seqToRE(self, to_convert, directive):
         matching when 'abcdef' should have been the match).
 
         """
+        to_convert = sorted(to_convert, key=len, reverse=True)
         for value in to_convert:
             if value != '':
                 break
         else:
             return ''
-        to_convert = to_convert[:]
-        to_convert.sort(key=len, reverse=True)
         regex = '|'.join(re_escape(stuff) for stuff in to_convert)
         regex = '(?P<%s>%s' % (directive, regex)
         return '%s)' % regex
