Title: [282813] trunk/Tools
Revision
282813
Author
[email protected]
Date
2021-09-21 07:50:52 -0700 (Tue, 21 Sep 2021)

Log Message

[webkitcorepy] Fail to parse python required version when there's a space between operator and version
https://bugs.webkit.org/show_bug.cgi?id=230549

Patch by Carlos Garcia Campos <[email protected]> on 2021-09-21
Reviewed by Jonathan Bedard.

Add optional space between operator and version _expression_ in regular _expression_.

* Scripts/libraries/webkitcorepy/webkitcorepy/version.py:
(Version):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (282812 => 282813)


--- trunk/Tools/ChangeLog	2021-09-21 14:11:11 UTC (rev 282812)
+++ trunk/Tools/ChangeLog	2021-09-21 14:50:52 UTC (rev 282813)
@@ -1,3 +1,15 @@
+2021-09-21  Carlos Garcia Campos  <[email protected]>
+
+        [webkitcorepy] Fail to parse python required version when there's a space between operator and version
+        https://bugs.webkit.org/show_bug.cgi?id=230549
+
+        Reviewed by Jonathan Bedard.
+
+        Add optional space between operator and version _expression_ in regular _expression_.
+
+        * Scripts/libraries/webkitcorepy/webkitcorepy/version.py:
+        (Version):
+
 2021-09-20  Chris Dumez  <[email protected]>
 
         Reduce use of makeRefPtr() and use RefPtr { } directly

Modified: trunk/Tools/Scripts/libraries/webkitcorepy/webkitcorepy/version.py (282812 => 282813)


--- trunk/Tools/Scripts/libraries/webkitcorepy/webkitcorepy/version.py	2021-09-21 14:11:11 UTC (rev 282812)
+++ trunk/Tools/Scripts/libraries/webkitcorepy/webkitcorepy/version.py	2021-09-21 14:50:52 UTC (rev 282813)
@@ -26,7 +26,7 @@
 
 
 class Version(object):
-    MATCHES_RE = re.compile(r'(?P<operator>==|!=|>|<|>=|<=)?(?P<version>\d+(\.\d+)?(\.\d+)?(\.\*)?)')
+    MATCHES_RE = re.compile(r'(?P<operator>==|!=|>|<|>=|<=)? *(?P<version>\d+(\.\d+)?(\.\d+)?(\.\*)?)')
 
     @staticmethod
     def from_string(string):
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to