Updated comment-matching regex to be Jython-compatible (#5959)

pypa · May 30, 2019 · 83d813c · 83d813c
2 parents 0cb8f60 + 6599f99
commit 83d813c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/news/3662.trivial b/news/3662.trivial
@@ -0,0 +1 @@
+Work around an issue with Jython's `re` implementation that resulted in a Java StackOverflowError.
diff --git a/src/pip/_internal/req/req_file.py b/src/pip/_internal/req/req_file.py
@@ -35,7 +35,7 @@
 __all__ = ['parse_requirements']
 
 SCHEME_RE = re.compile(r'^(http|https|file):', re.I)
-COMMENT_RE = re.compile(r'(^|\s)+#.*$')
+COMMENT_RE = re.compile(r'(^|\s+)#.*$')
 
 # Matches environment variable-style values in '${MY_VARIABLE_1}' with the
 # variable name consisting of only uppercase letters, digits or the '_'
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Work around an issue with Jython's `re` implementation that resulted in a Java StackOverflowError.