Ryan Gambord has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/17888

Change subject: configs: Removed redudant exec-style import
......................................................................

configs: Removed redudant exec-style import

garnet_synth_traffic.py imports common.Options on line 40, so exec'ing
the Options.py file again seems redundant.

It also runs Options.py as a script rather than a module, which throws
ValueError: Attempted relative import in non-package due to the recent
change to python3 imports.

Change-Id: Id729a8dfa776af0d14312e765168aff6900eb727
Signed-off-by: Ryan Gambord <gambo...@oregonstate.edu>
---
M configs/example/garnet_synth_traffic.py
1 file changed, 0 insertions(+), 3 deletions(-)



diff --git a/configs/example/garnet_synth_traffic.py b/configs/example/garnet_synth_traffic.py
index 8396ddb..9878c23 100644
--- a/configs/example/garnet_synth_traffic.py
+++ b/configs/example/garnet_synth_traffic.py
@@ -88,9 +88,6 @@
 #
 Ruby.define_options(parser)

-exec(compile(open(os.path.join(config_root, "common", "Options.py")).read(),
-             os.path.join(config_root, "common", "Options.py"), 'exec'))
-
 (options, args) = parser.parse_args()

 if args:

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/17888
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Id729a8dfa776af0d14312e765168aff6900eb727
Gerrit-Change-Number: 17888
Gerrit-PatchSet: 1
Gerrit-Owner: Ryan Gambord <gambo...@oregonstate.edu>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to