Bobby Bruce has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/64851?usp=email )
Change subject: tests: Fix verifier '_iterable_regex' func for None regex
......................................................................
tests: Fix verifier '_iterable_regex' func for None regex
In the case where no regex was specified (i,e., `regex == None`), the
`_iterable_regex` function returned `None`. For the testlib to work this
function must return an iterable.
Without this patch it is not possible to compare two files without a
specifying a regex.
Change-Id: Ibc8a2f783c3b786dbdca6a4284850b594024f2f9
---
M tests/gem5/verifier.py
1 file changed, 18 insertions(+), 0 deletions(-)
diff --git a/tests/gem5/verifier.py b/tests/gem5/verifier.py
index ff90b6d..0df9a10 100644
--- a/tests/gem5/verifier.py
+++ b/tests/gem5/verifier.py
@@ -276,6 +276,8 @@
def _iterable_regex(regex):
+ if not regex:
+ return () # If no regex we return an empty tuple.
if isinstance(regex, _re_type) or isinstance(regex, str):
regex = (regex,)
return regex
--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/64851?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: Ibc8a2f783c3b786dbdca6a4284850b594024f2f9
Gerrit-Change-Number: 64851
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby Bruce <bbr...@ucdavis.edu>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org