[Notes] [Git][BuildStream/buildstream][chandan/fix-warning] 2 commits: tests/frontend/logging.py: Fix regex Deprecation Warning

[Chandan Singh <gitlab mg gitlab com>]

Title: GitLab

Chandan Singh pushed to branch chandan/fix-warning at BuildStream / buildstream

Commits:

• d960a337
  by Chandan Singh at 2018-12-14T19:29:18Z

  tests/frontend/logging.py: Fix regex Deprecation Warning
  
  Use raw strings for regex searches, which is the preferred way to do
  regular expressions in Python.
  
  Without this patch, currently we get the following warnings:
  
  ```
  tests/frontend/logging.py:44
    /builds/BuildStream/buildstream/dist/buildstream/tests/frontend/logging.py:44: DeprecationWarning: invalid escape sequence \[
      m = re.search("\[\d\d:\d\d:\d\d\]\[\]\[\] SUCCESS Checking sources", result.stderr)
  
  tests/frontend/logging.py:80
    /builds/BuildStream/buildstream/dist/buildstream/tests/frontend/logging.py:80: DeprecationWarning: invalid escape sequence \d
      m = re.search("\d\d:\d\d:\d\d,\d\d:\d\d:\d\d.\d{6},\d\d:\d\d:\d\d,,,SUCCESS,Checking sources", result.stderr)
  ```
  

• ef524b3d
  by Chandan Singh at 2018-12-14T19:29:25Z

  buildstream/utils.py: Fix regex Deprecation Warning
  
  Specify flags at the start of the _expression_ as per the recommendation
  of the standard library.
  
  Without this patch, we currently get the following warning:
  
  ```
  tests/examples/junctions.py::test_open_cross_junction_workspace
    /builds/BuildStream/buildstream/dist/buildstream/buildstream/utils.py:213: DeprecationWarning: Flags not at the start of the _expression_ '\\/[^/]*\\Z(?ms)'
      regexer = re.compile(_expression_)
  ```
  

2 changed files:

• buildstream/utils.py
• tests/frontend/logging.py

Changes:

buildstream/utils.py

---

...	...	@@ -1168,7 +1168,7 @@ def _call(*popenargs, terminate=False, **kwargs):
1168	1168	#
1169	1169	def _glob2re(pat):
1170	1170	i, n = 0, len(pat)
1171		- res = ''
	1171	+ res = '(?ms)'
1172	1172	while i < n:
1173	1173	c = pat[i]
1174	1174	i = i + 1
...	...	@@ -1205,7 +1205,7 @@ def _glob2re(pat):
1205	1205	res = '{}[{}]'.format(res, stuff)
1206	1206	else:
1207	1207	res = res + re.escape(c)
1208		- return res + r'\Z(?ms)'
	1208	+ return res + r'\Z'
1209	1209
1210	1210
1211	1211	# _deduplicate()

tests/frontend/logging.py

---

...	...	@@ -41,7 +41,7 @@ def test_default_logging(cli, tmpdir, datafiles):
41	41	result = cli.run(project=project, args=['fetch', element_name])
42	42	result.assert_success()
43	43
44		- m = re.search("\[\d\d:\d\d:\d\d\]\[\]\[\] SUCCESS Checking sources", result.stderr)
	44	+ m = re.search(r"\[\d\d:\d\d:\d\d\]\[\]\[\] SUCCESS Checking sources", result.stderr)
45	45	assert(m is not None)
46	46
47	47
...	...	@@ -77,7 +77,7 @@ def test_custom_logging(cli, tmpdir, datafiles):
77	77	result = cli.run(project=project, args=['fetch', element_name])
78	78	result.assert_success()
79	79
80		- m = re.search("\d\d:\d\d:\d\d,\d\d:\d\d:\d\d.\d{6},\d\d:\d\d:\d\d,,,SUCCESS,Checking sources", result.stderr)
	80	+ m = re.search(r"\d\d:\d\d:\d\d,\d\d:\d\d:\d\d.\d{6},\d\d:\d\d:\d\d,,,SUCCESS,Checking sources", result.stderr)
81	81	assert(m is not None)
82	82
83	83

—
View it on GitLab.
You're receiving this email because of your account on gitlab.com. If you'd like to receive fewer emails, you can adjust your notification settings.