Bobby R. Bruce has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/38476 )
Change subject: util,python: Updated maintainers.py to fix deprecation
warning
..
util,python: Updated maintainers.py to fix deprecation warning
The following warning was being thrown:
```
util/maint/lib/maintainers.py:120: YAMLLoadWarning: calling yaml.load()
without Loader=... is deprecated, as the default Loader is unsafe. Please
read https://msg.pyyaml.org/load for full details.
```
This is fixed by adding `Loader=yaml.SafeLoader` when calling
`yaml.load()`.
Change-Id: I3b79115379a45409967a8848175658ab3c13bfc7
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/38476
Reviewed-by: Jason Lowe-Power
Reviewed-by: Andreas Sandberg
Maintainer: Jason Lowe-Power
Tested-by: kokoro
---
M util/maint/lib/maintainers.py
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
Andreas Sandberg: Looks good to me, approved
kokoro: Regressions pass
diff --git a/util/maint/lib/maintainers.py b/util/maint/lib/maintainers.py
index 0b29b67..6dd8d26 100644
--- a/util/maint/lib/maintainers.py
+++ b/util/maint/lib/maintainers.py
@@ -106,7 +106,7 @@
@classmethod
def from_yaml(cls, yaml_str: str) -> "Maintainers":
-return cls(yaml.load(yaml_str))
+return cls(yaml.load(yaml_str, Loader=yaml.SafeLoader))
@classmethod
def _load_maintainers_file(cls,
@@ -117,9 +117,9 @@
if isinstance(path_or_file, str):
with open(path_or_file, 'r') as fin:
-return yaml.load(fin)
+return yaml.load(fin, Loader=yaml.SafeLoader)
else:
-return yaml.load(path_or_file)
+return yaml.load(path_or_file, Loader=yaml.SafeLoader)
@classmethod
def _parse_subsystem(cls, tag: str, ydict: Mapping[str, Any]) ->
Subsystem:
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/38476
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: I3b79115379a45409967a8848175658ab3c13bfc7
Gerrit-Change-Number: 38476
Gerrit-PatchSet: 3
Gerrit-Owner: Bobby R. Bruce
Gerrit-Reviewer: Andreas Sandberg
Gerrit-Reviewer: Bobby R. Bruce
Gerrit-Reviewer: Gabe Black
Gerrit-Reviewer: Hoa Nguyen
Gerrit-Reviewer: Jason Lowe-Power
Gerrit-Reviewer: Jason Lowe-Power
Gerrit-Reviewer: kokoro
Gerrit-MessageType: merged
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
